Re: [PATCH 1/1] pseries/hotplug-memory.c: Replace nested ifs by switch-case

2019-08-10 Thread Michael Ellerman
On Thu, 2019-08-01 at 22:52:51 UTC, Leonardo Bras wrote: > I noticed these nested ifs can be easily replaced by switch-cases, > which can improve readability. > > Signed-off-by: Leonardo Bras > Reviewed-by: David Hildenbrand Series applied to powerpc next, thanks. https://git.kernel.org/powerp

Re: [PATCH 1/1] pseries/hotplug-memory.c: Replace nested ifs by switch-case

2019-08-02 Thread Leonardo Bras
On Fri, 2019-08-02 at 22:26 +1000, Michael Ellerman wrote: > Leonardo Bras writes: > > I noticed these nested ifs can be easily replaced by switch-cases, > > which can improve readability. > > > > Signed-off-by: Leonardo Bras > > --- > > .../platforms/pseries/hotplug-memory.c| 26 ++

Re: [PATCH 1/1] pseries/hotplug-memory.c: Replace nested ifs by switch-case

2019-08-02 Thread Michael Ellerman
Leonardo Bras writes: > I noticed these nested ifs can be easily replaced by switch-cases, > which can improve readability. > > Signed-off-by: Leonardo Bras > --- > .../platforms/pseries/hotplug-memory.c| 26 +-- > 1 file changed, 18 insertions(+), 8 deletions(-) Thanks,

Re: [PATCH 1/1] pseries/hotplug-memory.c: Replace nested ifs by switch-case

2019-08-02 Thread David Hildenbrand
On 02.08.19 00:52, Leonardo Bras wrote: > I noticed these nested ifs can be easily replaced by switch-cases, > which can improve readability. > > Signed-off-by: Leonardo Bras > --- > .../platforms/pseries/hotplug-memory.c| 26 +-- > 1 file changed, 18 insertions(+), 8 del

[PATCH 1/1] pseries/hotplug-memory.c: Replace nested ifs by switch-case

2019-08-01 Thread Leonardo Bras
I noticed these nested ifs can be easily replaced by switch-cases, which can improve readability. Signed-off-by: Leonardo Bras --- .../platforms/pseries/hotplug-memory.c| 26 +-- 1 file changed, 18 insertions(+), 8 deletions(-) diff --git a/arch/powerpc/platforms/pseries