[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <910a5806-9a08-adf4-4fba-d5ec2f5807ff@metux.net>
Date: Mon, 17 Jun 2019 22:21:06 +0200
From: "Enrico Weigelt, metux IT consult" <lkml@...ux.net>
To: Markus Elfring <Markus.Elfring@....de>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
"Rafael J. Wysocki" <rafael@...nel.org>
Cc: cocci@...teme.lip6.fr, kernel-janitors@...r.kernel.org,
linux-kernel@...r.kernel.org,
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
Bartosz Golaszewski <bgolaszewski@...libre.com>,
Gilles Muller <Gilles.Muller@...6.fr>,
Himanshu Jha <himanshujha199640@...il.com>,
Linus Walleij <linus.walleij@...aro.org>,
Masahiro Yamada <yamada.masahiro@...ionext.com>,
Michal Marek <michal.lkml@...kovi.net>,
Nicolas Palix <nicolas.palix@...g.fr>
Subject: Re: [PATCH] drivers: Inline code in devm_platform_ioremap_resource()
from two functions
On 14.06.19 11:22, Markus Elfring wrote:
> From: Markus Elfring <elfring@...rs.sourceforge.net>
> Date: Fri, 14 Jun 2019 11:05:33 +0200
>
> Two function calls were combined in this function implementation.
> Inline corresponding code so that extra error checks can be avoided here.
What exactly is the purpose of this ?
Looks like a notable code duplication ... I thought we usually try to
reduce this, instead of introducing new ones.
--mtx
--
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
info@...ux.net -- +49-151-27565287
Powered by blists - more mailing lists