Re: [PATCH] drivers: Inline code in devm_platform_ioremap_resource() from two functions

2019-06-17 Thread Enrico Weigelt, metux IT consult
On 14.06.19 11:22, Markus Elfring wrote: > From: Markus Elfring > 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

Re: [PATCH] drivers: Inline code in devm_platform_ioremap_resource() from two functions

2019-06-14 Thread Greg Kroah-Hartman
On Fri, Jun 14, 2019 at 11:22:40AM +0200, Markus Elfring wrote: > From: Markus Elfring > 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. > > Signed-off-by:

Re: [Cocci] [PATCH] drivers: Inline code in devm_platform_ioremap_resource() from two functions

2019-06-14 Thread Julia Lawall
On Fri, 14 Jun 2019, Markus Elfring wrote: > From: Markus Elfring > 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. I don't see any point to this at all.

[PATCH] drivers: Inline code in devm_platform_ioremap_resource() from two functions

2019-06-14 Thread Markus Elfring
From: Markus Elfring 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. Signed-off-by: Markus Elfring --- drivers/base/platform.c | 39