> -----Original Message----- > From: Martin Kepplinger <martink@xxxxxxxxx> > Sent: Sunday, June 23, 2019 7:21 PM > To: Anson Huang <anson.huang@xxxxxxx>; daniel.lezcano@xxxxxxxxxx; > tglx@xxxxxxxxxxxxx; robh+dt@xxxxxxxxxx; mark.rutland@xxxxxxx; > shawnguo@xxxxxxxxxx; s.hauer@xxxxxxxxxxxxxx; kernel@xxxxxxxxxxxxxx; > festevam@xxxxxxxxx; l.stach@xxxxxxxxxxxxxx; Abel Vesa > <abel.vesa@xxxxxxx>; ccaione@xxxxxxxxxxxx; angus@xxxxxxxx; > andrew.smirnov@xxxxxxxxx; agx@xxxxxxxxxxx; linux- > kernel@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx > Cc: dl-linux-imx <linux-imx@xxxxxxx> > Subject: Re: [PATCH 3/3] arm64: dts: imx8mq: Add system counter node > > On 22.06.19 16:16, Martin Kepplinger wrote: > > On 21.06.19 10:28, Anson.Huang@xxxxxxx wrote: > >> From: Anson Huang <Anson.Huang@xxxxxxx> > >> > >> Add i.MX8MQ system counter node to enable timer-imx-sysctr broadcast > >> timer driver. > > > > with these changes and TIMER_IMX_SYS_CTR selected, I don't see cpuidle > > working yet (which is what I want to achieve on imx8mq). Might there > > be a system counter clock definition or anything else missing? > > > > To be clear about what I tried running: Abel's wakeup-workaround (with the > corresponding ATF changes): > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flkml.o > rg%2Flkml%2F2019%2F6%2F10%2F350&data=02%7C01%7CAnson.Huan > g%40nxp.com%7C445690743187422a2d2b08d6f7ccd665%7C686ea1d3bc2b4 > c6fa92cd99c5c301635%7C0%7C0%7C636968856477185012&sdata=k1Aj > CJ5SGUYQE7VnzciihRTKgf1yi4bDaBqCCv9DzpU%3D&reserved=0 plus > your patches here (although you don't define a system counter clock, like you > do in your imx8mm patches). > > In any case, I might try to enable cpuidle totally wrong still :) and I'd be happy > for hints and test your changes (no matter how fit they are to be merged > right now). Could be something else than this patch series, i.MX8MQ uses platform driver model for clock driver, to enable system counter driver, something needs to be changed in system counter driver, and no need to have system counter clock definition in i.MX8MQ clock tree, while i.MX8MM uses OF_CLK as clock driver initialization, so we need system counter clock definition. My understanding is, even without this patch series, cpu-idle should be able to work, but The last CPU could be always powered ON to ack as broadcast timer, if with system counter enabled, all CPUs can be powered OFF when entering state #1 idle independently, as system counter can be broadcast timer. Correct me if anything wrong, maybe latest upstream kernel has something different about this part. Thanks, Anson. > > thanks, > martin
