Re: [PATCH 4/8] PCI/hotplug/rpa: Code cleanup

2014-11-25 Thread Gavin Shan
On Wed, Nov 26, 2014 at 10:02:31AM +1100, Benjamin Herrenschmidt wrote: >On Tue, 2014-11-25 at 09:49 +1100, Gavin Shan wrote: >> The patch applies code cleanup to RPA modules to address following >> issues and it shouldn't affect the logic: >> >>* Coding style issue: removed unnecessary "break

Re: [PATCH 4/8] PCI/hotplug/rpa: Code cleanup

2014-11-25 Thread Benjamin Herrenschmidt
On Tue, 2014-11-25 at 09:49 +1100, Gavin Shan wrote: > The patch applies code cleanup to RPA modules to address following > issues and it shouldn't affect the logic: > >* Coding style issue: removed unnecessary "break" for default case > in switch statement and added default case; removed

[PATCH 4/8] PCI/hotplug/rpa: Code cleanup

2014-11-24 Thread Gavin Shan
The patch applies code cleanup to RPA modules to address following issues and it shouldn't affect the logic: * Coding style issue: removed unnecessary "break" for default case in switch statement and added default case; removed unnecessary braces or parenthese for if or return stateme