Re: [PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-13 Thread Vipin K Parashar
On 05/11/2015 12:19 PM, Michael Ellerman wrote: On Thu, 2015-05-07 at 15:00 +0530, Vipin K Parashar wrote: This patch adds support for FSP EPOW (Early Power Off Warning) and DPO (Delayed Power Off) events support for PowerNV platform. EPOW events are generated by SPCN/FSP due to various

Re: [PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-12 Thread Triggering
Accidentally hit the reply button instead of reply all. Re sending. On 12-May-2015, at 11:55, trigg mr.triggtr...@gmail.com wrote: Hi Stewart, On Tue, May 12, 2015 at 4:01 AM, Stewart Smith stew...@linux.vnet.ibm.com wrote: trigg mr.triggtr...@gmail.com writes: ---

Re: [PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-11 Thread Michael Ellerman
On Thu, 2015-05-07 at 15:00 +0530, Vipin K Parashar wrote: This patch adds support for FSP EPOW (Early Power Off Warning) and DPO (Delayed Power Off) events support for PowerNV platform. EPOW events are generated by SPCN/FSP due to various critical system conditions that need system shutdown.

Re: [PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-11 Thread Vipin K Parashar
Hi Joel, Thanks for review. My comments below. On 05/08/2015 06:56 AM, Joel Stanley wrote: Hello Vipin, On Thu, May 7, 2015 at 7:00 PM, Vipin K Parashar vi...@linux.vnet.ibm.com wrote: This patch adds support for FSP EPOW (Early Power Off Warning) and DPO (Delayed Power Off) events

Re: [PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-11 Thread Vipin K Parashar
On 05/11/2015 12:19 PM, Michael Ellerman wrote: On Thu, 2015-05-07 at 15:00 +0530, Vipin K Parashar wrote: This patch adds support for FSP EPOW (Early Power Off Warning) and DPO (Delayed Power Off) events support for PowerNV platform. EPOW events are generated by SPCN/FSP due to various

Re: [PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-11 Thread Vipin K Parashar
On 05/11/2015 02:31 PM, Vipin K Parashar wrote: On 05/11/2015 12:19 PM, Michael Ellerman wrote: On Thu, 2015-05-07 at 15:00 +0530, Vipin K Parashar wrote: This patch adds support for FSP EPOW (Early Power Off Warning) and DPO (Delayed Power Off) events support for PowerNV platform. EPOW

Re: [PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-11 Thread Stewart Smith
trigg mr.triggtr...@gmail.com writes: --- a/arch/powerpc/include/asm/opal-api.h +++ b/arch/powerpc/include/asm/opal-api.h @@ -730,6 +730,36 @@ struct opal_i2c_request { __be64 buffer_ra; /* Buffer real address */ }; +/* + * EPOW status sharing (OPAL and the host) +

Re: [PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-08 Thread trigg
Hi Vipin, These comments are in addition to what Joel has said in his review. On Thu, May 7, 2015 at 3:00 PM, Vipin K Parashar vi...@linux.vnet.ibm.com wrote: This patch adds support for FSP EPOW (Early Power Off Warning) and DPO (Delayed Power Off) events support for PowerNV platform. EPOW

[PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-07 Thread Vipin K Parashar
This patch adds support for FSP EPOW (Early Power Off Warning) and DPO (Delayed Power Off) events support for PowerNV platform. EPOW events are generated by SPCN/FSP due to various critical system conditions that need system shutdown. Few examples of these conditions are high ambient temperature

Re: [PATCH v2 1/2] powerpc/powernv: Add poweroff (EPOW, DPO) events support for PowerNV platform

2015-05-07 Thread Joel Stanley
Hello Vipin, On Thu, May 7, 2015 at 7:00 PM, Vipin K Parashar vi...@linux.vnet.ibm.com wrote: This patch adds support for FSP EPOW (Early Power Off Warning) and DPO (Delayed Power Off) events support for PowerNV platform. I reviewed this patch for the changes it made to the existing poweroff