Re: [PATCH v2] arm64: dts: Hi3660: Fix up psci state id

2017-12-28 Thread Leo Yan
On Thu, Dec 28, 2017 at 09:14:03PM +, Wei Xu wrote: > Hi Leo, [...] > Sorry, since this patch is still under discussion, > I will drop it firstly. > Thanks! Thanks, Wei. > Best Regards, > Wei

Re: [PATCH v2] arm64: dts: Hi3660: Fix up psci state id

2017-12-28 Thread Wei Xu
Hi Leo, On 2017/12/22 9:37, Wei Xu wrote: > Hi Leo, > > On 2017/12/12 9:12, Leo Yan wrote: >> Thanks a lot for Vincent Guittot careful work to find bug for 'CPU_NAP' >> idle state. From ftrace log we can observe CA73 CPUs can be easily >> waken up from 'CPU_NAP' state but the 'waken up' CPUs doe

Re: [PATCH v2] arm64: dts: Hi3660: Fix up psci state id

2017-12-27 Thread Vincent Guittot
Hi Leo On 25 December 2017 at 03:22, Leo Yan wrote: > Hi Vincent, > > [ + John, Kevin Wang ] > > On Fri, Dec 22, 2017 at 03:22:51PM +0100, Vincent Guittot wrote: >> Hi Leo, >> >> Sorry for jumping late in the discussion but should we also remove >> the NAP state from the property cpu-idle-states

Re: [PATCH v2] arm64: dts: Hi3660: Fix up psci state id

2017-12-24 Thread Leo Yan
Hi Vincent, [ + John, Kevin Wang ] On Fri, Dec 22, 2017 at 03:22:51PM +0100, Vincent Guittot wrote: > Hi Leo, > > Sorry for jumping late in the discussion but should we also remove > the NAP state from the property cpu-idle-states of the CPUs because > this state not supported by the platform a

Re: [PATCH v2] arm64: dts: Hi3660: Fix up psci state id

2017-12-22 Thread Vincent Guittot
Hi Leo, Sorry for jumping late in the discussion but should we also remove the NAP state from the property cpu-idle-states of the CPUs because this state not supported by the platform at least for now and may be not in a near future ? Then, I have another question regarding the update of the psc

Re: [PATCH v2] arm64: dts: Hi3660: Fix up psci state id

2017-12-22 Thread Wei Xu
Hi Leo, On 2017/12/12 9:12, Leo Yan wrote: > Thanks a lot for Vincent Guittot careful work to find bug for 'CPU_NAP' > idle state. From ftrace log we can observe CA73 CPUs can be easily > waken up from 'CPU_NAP' state but the 'waken up' CPUs doesn't handle > anything and sleep again; so there hav

[PATCH v2] arm64: dts: Hi3660: Fix up psci state id

2017-12-12 Thread Leo Yan
Thanks a lot for Vincent Guittot careful work to find bug for 'CPU_NAP' idle state. From ftrace log we can observe CA73 CPUs can be easily waken up from 'CPU_NAP' state but the 'waken up' CPUs doesn't handle anything and sleep again; so there have tons of trace events for CA73 CPUs entering and ex