Re: [PATCH v5 07/14] ARM: mvebu: Extend the pmsu registers

2014-03-26 Thread Thomas Petazzoni
Dear Sebastian Hesselbarth, On Wed, 26 Mar 2014 13:04:55 +0100, Sebastian Hesselbarth wrote: > On 03/25/2014 11:48 PM, Gregory CLEMENT wrote: > > The initial binding for PMSU were wrong. It didn't take into account > > all the registers from the PMSU and moreover it referred to registers > > which

Re: [PATCH v5 07/14] ARM: mvebu: Extend the pmsu registers

2014-03-26 Thread Gregory CLEMENT
On 26/03/2014 13:04, Sebastian Hesselbarth wrote: > On 03/25/2014 11:48 PM, Gregory CLEMENT wrote: >> The initial binding for PMSU were wrong. It didn't take into account >> all the registers from the PMSU and moreover it referred to registers >> which are not part of PMSU. >> >> The Power Manageme

Re: [PATCH v5 07/14] ARM: mvebu: Extend the pmsu registers

2014-03-26 Thread Sebastian Hesselbarth
On 03/25/2014 11:48 PM, Gregory CLEMENT wrote: The initial binding for PMSU were wrong. It didn't take into account all the registers from the PMSU and moreover it referred to registers which are not part of PMSU. The Power Management Unit Service block also controls the Coherency Fabric subsyst

Re: [PATCH v5 07/14] ARM: mvebu: Extend the pmsu registers

2014-03-26 Thread Sebastian Hesselbarth
On 03/26/2014 10:33 AM, Gregory CLEMENT wrote: On 26/03/2014 01:30, Jason Cooper wrote: On Tue, Mar 25, 2014 at 11:48:18PM +0100, Gregory CLEMENT wrote: The initial binding for PMSU were wrong. It didn't take into account all the registers from the PMSU and moreover it referred to registers whi

Re: [PATCH v5 07/14] ARM: mvebu: Extend the pmsu registers

2014-03-26 Thread Gregory CLEMENT
Hi Jason, On 26/03/2014 01:30, Jason Cooper wrote: > On Tue, Mar 25, 2014 at 11:48:18PM +0100, Gregory CLEMENT wrote: >> The initial binding for PMSU were wrong. It didn't take into account >> all the registers from the PMSU and moreover it referred to registers >> which are not part of PMSU. >> >

Re: [PATCH v5 07/14] ARM: mvebu: Extend the pmsu registers

2014-03-25 Thread Jason Cooper
On Tue, Mar 25, 2014 at 11:48:18PM +0100, Gregory CLEMENT wrote: > The initial binding for PMSU were wrong. It didn't take into account > all the registers from the PMSU and moreover it referred to registers > which are not part of PMSU. > > The Power Management Unit Service block also controls th

[PATCH v5 07/14] ARM: mvebu: Extend the pmsu registers

2014-03-25 Thread Gregory CLEMENT
The initial binding for PMSU were wrong. It didn't take into account all the registers from the PMSU and moreover it referred to registers which are not part of PMSU. The Power Management Unit Service block also controls the Coherency Fabric subsystem. These registers are needed for the CPU idle i