Re: [PATCH 3/5] powerpc/powernv: Cleanup on PNV_EEH_STATE_ENABLED

2014-02-22 Thread Gavin Shan
On Sat, Feb 22, 2014 at 06:56:49AM +1100, Benjamin Herrenschmidt wrote: On Fri, 2014-02-21 at 19:53 +0800, Gavin Shan wrote: The flag PNV_EEH_STATE_ENABLED is put into pnv_phb::eeh_state, which is protected by CONFIG_EEH. We needn't that. Instead, we can have pnv_phb::flags and maintain all

[PATCH 3/5] powerpc/powernv: Cleanup on PNV_EEH_STATE_ENABLED

2014-02-21 Thread Gavin Shan
The flag PNV_EEH_STATE_ENABLED is put into pnv_phb::eeh_state, which is protected by CONFIG_EEH. We needn't that. Instead, we can have pnv_phb::flags and maintain all flags there, which is the purpose of the patch. Signed-off-by: Gavin Shan sha...@linux.vnet.ibm.com ---

Re: [PATCH 3/5] powerpc/powernv: Cleanup on PNV_EEH_STATE_ENABLED

2014-02-21 Thread Benjamin Herrenschmidt
On Fri, 2014-02-21 at 19:53 +0800, Gavin Shan wrote: The flag PNV_EEH_STATE_ENABLED is put into pnv_phb::eeh_state, which is protected by CONFIG_EEH. We needn't that. Instead, we can have pnv_phb::flags and maintain all flags there, which is the purpose of the patch. Can you explain a bit