RE: [PATCH V4 2/2] clk: imx: scu: add cpu frequency scaling support

[Anson Huang]

Hi, Stephen

Best Regards!
Anson Huang

> -----Original Message-----
> From: Stephen Boyd [mailto:sboyd@xxxxxxxxxx]
> Sent: 2019å2æ22æ 4:48
> To: devicetree@xxxxxxxxxxxxxxx; festevam@xxxxxxxxx;
> kernel@xxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-
> clk@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; mark.rutland@xxxxxxx;
> mturquette@xxxxxxxxxxxx; robh+dt@xxxxxxxxxx; s.hauer@xxxxxxxxxxxxxx;
> shawnguo@xxxxxxxxxx; viresh.kumar@xxxxxxxxxx; Aisheng Dong
> <aisheng.dong@xxxxxxx>; Anson Huang <anson.huang@xxxxxxx>; Daniel
> Baluta <daniel.baluta@xxxxxxx>
> Cc: dl-linux-imx <linux-imx@xxxxxxx>
> Subject: Re: [PATCH V4 2/2] clk: imx: scu: add cpu frequency scaling support
> 
> Quoting Anson Huang (2019-02-13 17:54:08)
> > On NXP's i.MX SoCs with system controller inside, CPU frequency
> > scaling can ONLY be done by system controller firmware, and it can
> > ONLY be requested from secure mode, so Linux kernel has to call ARM
> > SMC to trap to ARM-Trusted-Firmware to request system controller
> > firmware to do CPU frequency scaling.
> >
> > This patch adds i.MX system controller CPU frequency scaling support,
> > it reuses cpufreq-dt driver and implement the CPU frequency scaling
> > inside SCU clock driver.
> >
> > Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx>
> > ---
> > Changes since V3:
> >         - use different clk ops for CPU clock to avoid runtime check in
> clk_set_rate;
> >         - use rsrc_id to determine whether it is CPU frequency change and also
> to get
> >           cluster ID for SMC call.
> 
> This directly conflicts with a patch to add get/set parent support from
> Aisheng Dong. Can you please work together and rebase the patch on top of
> that patch (which is in clk-next) and resend this?

I have resent the patch set based on top of clk-next and tested the function.
Please apply this resent patch, https://patchwork.kernel.org/patch/10825085/

Thanks,
Anson.