Hi, Shawn Best Regards! Anson Huang > -----Original Message----- > From: Shawn Guo [mailto:shawnguo@xxxxxxxxxx] > Sent: 2019年2月28日 11:19 > To: Anson Huang <anson.huang@xxxxxxx> > Cc: robh+dt@xxxxxxxxxx; mark.rutland@xxxxxxx; s.hauer@xxxxxxxxxxxxxx; > kernel@xxxxxxxxxxxxxx; festevam@xxxxxxxxx; mturquette@xxxxxxxxxxxx; > sboyd@xxxxxxxxxx; Aisheng Dong <aisheng.dong@xxxxxxx>; Daniel Baluta > <daniel.baluta@xxxxxxx>; devicetree@xxxxxxxxxxxxxxx; linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux- > clk@xxxxxxxxxxxxxxx; dl-linux-imx <linux-imx@xxxxxxx> > Subject: Re: [PATCH V7 1/2] arm64: dts: freescale: imx8qxp: add cpu opp > table > > On Tue, Feb 26, 2019 at 05:17:31AM +0000, Anson Huang wrote: > > Add i.MX8QXP CPU opp table to support cpufreq. > > > > Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx> > > Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> > > Prefix 'arm64: dts: imx8qxp: ' would already be clear enough. I dropped > 'freescale' from there and applied patch. > > > --- > > No changes since V6. > > --- > > arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 30 > > ++++++++++++++++++++++++++++++ > > 1 file changed, 30 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > index 4c3dd95..41bf0ce 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > @@ -34,6 +34,9 @@ > > reg = <0x0 0x0>; > > enable-method = "psci"; > > next-level-cache = <&A35_L2>; > > + clocks = <&clk IMX_A35_CLK>; > > + operating-points-v2 = <&a35_0_opp_table>; > > + #cooling-cells = <2>; > > }; > > > > A35_1: cpu@1 { > > @@ -42,6 +45,9 @@ > > reg = <0x0 0x1>; > > enable-method = "psci"; > > next-level-cache = <&A35_L2>; > > + clocks = <&clk IMX_A35_CLK>; > > + operating-points-v2 = <&a35_0_opp_table>; > > + #cooling-cells = <2>; > > }; > > > > A35_2: cpu@2 { > > @@ -50,6 +56,9 @@ > > reg = <0x0 0x2>; > > enable-method = "psci"; > > next-level-cache = <&A35_L2>; > > + clocks = <&clk IMX_A35_CLK>; > > + operating-points-v2 = <&a35_0_opp_table>; > > + #cooling-cells = <2>; > > }; > > > > A35_3: cpu@3 { > > @@ -58,6 +67,9 @@ > > reg = <0x0 0x3>; > > enable-method = "psci"; > > next-level-cache = <&A35_L2>; > > + clocks = <&clk IMX_A35_CLK>; > > + operating-points-v2 = <&a35_0_opp_table>; > > + #cooling-cells = <2>; > > }; > > > > A35_L2: l2-cache0 { > > @@ -65,6 +77,24 @@ > > }; > > }; > > > > + a35_0_opp_table: opp-table { > > What does the '0' in the label mean? Looks like the '0' in the label is NOT necessary, we can just use 'a35_opp_table', do you want me resend the patch to remove '0'? Anson. > > Shawn > > > + compatible = "operating-points-v2"; > > + opp-shared; > > + > > + opp-900000000 { > > + opp-hz = /bits/ 64 <900000000>; > > + opp-microvolt = <1000000>; > > + clock-latency-ns = <150000>; > > + }; > > + > > + opp-1200000000 { > > + opp-hz = /bits/ 64 <1200000000>; > > + opp-microvolt = <1100000>; > > + clock-latency-ns = <150000>; > > + opp-suspend; > > + }; > > + }; > > + > > gic: interrupt-controller@51a00000 { > > compatible = "arm,gic-v3"; > > reg = <0x0 0x51a00000 0 0x10000>, /* GIC Dist */ > > -- > > 2.7.4 > > > > > > _______________________________________________ > > linux-arm-kernel mailing list > > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > > https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Flist > > s.infradead.org%2Fmailman%2Flistinfo%2Flinux-arm- > kernel&data=02%7C > > > 01%7Canson.huang%40nxp.com%7Cfe4ff92a639e4739c2a108d69d2b8e12%7 > C686ea1 > > > d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636869207728398808&sd > ata=oz%2 > > FPyAki6fkfQZqaSYkjWfo3m8S48sCzpDf2lxDIjKs%3D&reserved=0
