From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julia Lawall Date: Sat, 08 Jun 2019 17:26:35 +0000 Subject: Re: Coccinelle: api: add devm_platform_ioremap_resource script Message-Id: List-Id: References: <20190406061112.31620-1-himanshujha199640@gmail.com> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Markus Elfring Cc: Nicolas Palix , Michal Marek , Greg Kroah-Hartman , Linus Walleij , kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, Bartosz Golaszewski , Andy Shevchenko , cocci@systeme.lip6.fr 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