From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9D5F4C28CC5 for ; Sat, 8 Jun 2019 17:27:03 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 71E462146E for ; Sat, 8 Jun 2019 17:27:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727287AbfFHR1C (ORCPT ); Sat, 8 Jun 2019 13:27:02 -0400 Received: from mail2-relais-roc.national.inria.fr ([192.134.164.83]:64960 "EHLO mail2-relais-roc.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727202AbfFHR1C (ORCPT ); Sat, 8 Jun 2019 13:27:02 -0400 X-IronPort-AV: E=Sophos;i="5.63,568,1557180000"; d="scan'208";a="386580694" Received: from unknown (HELO hadrien) ([89.191.33.34]) by mail2-relais-roc.national.inria.fr with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 Jun 2019 19:26:59 +0200 Date: Sat, 8 Jun 2019 19:26:35 +0200 (CEST) From: Julia Lawall X-X-Sender: jll@hadrien To: Markus Elfring cc: Himanshu Jha , cocci@systeme.lip6.fr, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, Andy Shevchenko , Bartosz Golaszewski , Gilles Muller , Greg Kroah-Hartman , Linus Walleij , Masahiro Yamada , Michal Marek , Nicolas Palix Subject: Re: Coccinelle: api: add devm_platform_ioremap_resource script In-Reply-To: Message-ID: References: <20190406061112.31620-1-himanshujha199640@gmail.com> User-Agent: Alpine 2.21 (DEB 202 2017-01-01) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 8 Jun 2019, Markus Elfring wrote: > > +- e1 = devm_ioremap_resource(arg4, id); > > ++ e1 = devm_platform_ioremap_resource(arg1, arg3); > > Can the following specification variant matter for the shown SmPL > change approach? > > + e1 = > +- devm_ioremap_resource(arg4, id > ++ devm_platform_ioremap_resource(arg1, arg3 > + ); In the latter case, the original formatting of e1 will be preserved. But there is not usually any interesting formatting on the left side of an assignment (ie typically no newlines or comments). I can see no purpose to factorizing the right parenthesis. julia