linux-kernel - Re: [PATCH v3 0/2] Add Qualcomm MSM8939 GCC binding and driver

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]

Date:   Fri, 08 May 2020 12:06:23 +0200
From:   Vincent Knecht <vincent.knecht@...loo.org>
To:     Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
        Bryan O'Donoghue <pure.logic@...us-software.ie>,
        agross@...nel.org, bjorn.andersson@...aro.org,
        mturquette@...libre.com, sboyd@...nel.org, robh+dt@...nel.org
Cc:     linux-arm-msm@...r.kernel.org, linux-clk@...r.kernel.org,
        linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
        shawn.guo@...aro.org, p.zabel@...gutronix.de,
        Konrad Dybcio <konradybcio@...il.com>
Subject: Re: [PATCH v3 0/2] Add Qualcomm MSM8939 GCC binding and driver

Le jeudi 07 mai 2020 à 17:06 +0100, Bryan O'Donoghue a écrit :
> On 07/05/2020 16:57, Bryan O'Donoghue wrote:
> > On 23/04/2020 11:34, Bryan O'Donoghue wrote:
> > > V3:
> > > This update removes the old clock name arrays which I forgot to prune in
> > > the previous V2.
> > > 
> > > git diff bod/clk-next+msm8939 bod/clk-next+msm8939-v2.1
> > 
> > I should have mentioned.
> > 
> > If you want to test this you'll need to do the following to your 
> > msm8939.dtsi
> > 
> > diff --git a/arch/arm64/boot/dts/qcom/msm8939.dtsi 
> > b/arch/arm64/boot/dts/qcom/msm8939.dtsi
> > index 996425a70014..2d54ad7f99b6 100644
> > --- a/arch/arm64/boot/dts/qcom/msm8939.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/msm8939.dtsi
> > @@ -673,6 +673,10 @@ gcc: clock-controller@...0000 {
> >                          reg = <0x1800000 0x80000>;
> > +                       clock-names = "xo",
> > +                                     "sleep_clk";
> > +                       clocks = <&xo_board>,
> > +                                <&sleep_clk>;
> >                  };
> > 
> > "xo" and "sleep_clk" names can be resolved.
> > 
> > ---
> > bod
> 
> and this...
> 
> clocks {
>      xo_board: xo_board {
>          compatible = "fixed-clock";
>          #clock-cells = <0>;
>          clock-frequency = <19200000>;
>      };
> 
>      sleep_clk: sleep_clk {
>          compatible = "fixed-clock";
>          #clock-cells = <0>;
>          clock-frequency = <32768>;
>      };
> };
> 

Thank you, works for me over v5.7-rc2 with an Alcatel Idol 3 (5.5").

Tested-by: Vincent Knecht <vincent.knecht@...loo.org>