linux-kernel - RE: [PATCH] ARM: dts: imx7ulp: Add cpu clock-frequency property

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <DB3PR0402MB39164DF380E6B13558E758E7F5420@DB3PR0402MB3916.eurprd04.prod.outlook.com>
Date:   Tue, 3 Dec 2019 01:09:22 +0000
From:   Anson Huang <anson.huang@....com>
To:     Shawn Guo <shawnguo@...nel.org>
CC:     "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "mark.rutland@....com" <mark.rutland@....com>,
        "s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        "festevam@...il.com" <festevam@...il.com>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        dl-linux-imx <linux-imx@....com>
Subject: RE: [PATCH] ARM: dts: imx7ulp: Add cpu clock-frequency property


> Subject: Re: [PATCH] ARM: dts: imx7ulp: Add cpu clock-frequency property
> 
> On Tue, Nov 05, 2019 at 09:49:38AM +0800, Anson Huang wrote:
> > Add "clock-frequency" property to avoid below warning on i.MX7ULP:
> >
> > [    0.011762] /cpus/cpu@0 missing clock-frequency property
> >
> > Signed-off-by: Anson Huang <Anson.Huang@....com>
> > ---
> >  arch/arm/boot/dts/imx7ulp.dtsi | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/arm/boot/dts/imx7ulp.dtsi
> > b/arch/arm/boot/dts/imx7ulp.dtsi index d37a192..87b2237 100644
> > --- a/arch/arm/boot/dts/imx7ulp.dtsi
> > +++ b/arch/arm/boot/dts/imx7ulp.dtsi
> > @@ -41,6 +41,7 @@
> >  			compatible = "arm,cortex-a7";
> >  			device_type = "cpu";
> >  			reg = <0>;
> > +			clock-frequency = <500210526>;
> 
> I cannot find the binding doc for this property.  What is the definition of it,
> the maximum frequency that the cpu could possibly run at?


The code is as below, maybe the property is missing from the beginning of this code,
this property should mean the current frequency of CPU running at I think:

arch/arm/kernel/topology.c

122                 rate = of_get_property(cn, "clock-frequency", &len);
123                 if (!rate || len != 4) {
124                         pr_err("%pOF missing clock-frequency property\n", cn);
125                         continue;
126                 }

Thanks,
Anson