linux-kernel - Re: [PATCH v6 00/15] Multiple addition and improvement to ipq8064 gcc

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:   Thu, 3 Mar 2022 08:49:36 +0000
From:   Jonathan McDowell <noodles@...th.li>
To:     Ansuel Smith <ansuelsmth@...il.com>
Cc:     Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        Taniya Das <tdas@...eaurora.org>,
        linux-arm-msm@...r.kernel.org, linux-clk@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6 00/15] Multiple addition and improvement to ipq8064 gcc

On Sat, Feb 26, 2022 at 02:52:20PM +0100, Ansuel Smith wrote:
> This is an attempt in making the ipq8064 SoC actually usable. Currently
> many feature are missing for this SoC and devs user off-the-tree patches
> to make it work (example patch for missing clock, patch for cpufreq
> driver, patch to add missing node in the dts)
> 
> I notice there was some work in modernizing the gcc driver for other
> qcom target but this wasn't done for ipq806x. This does exactly this, we
> drop any parent_names stuff and we switch to the parent_data way. We
> also drop the pxo and cxo source clk from gcc driver and we refer to the
> dts for it.
> 
> This also add all the missing feature for the nss cores and the
> cryptoengine in them. It does also introduce the required flags to make
> the RPM actually work and NOT reject any command. There was an attempt
> in declaring these clock as core clock in the dts but this ends up in no
> serial as the kernel makes these clock not accessible. We just want to
> make the kernel NOT disable them if unused nothing more.
> 
> At the end we update the ipq8064 dtsi to add the pxo and cxo tag and
> declare them in gcc and also fix a problem with tsens probe.

FWIW, series tested on my RB3011. No regressions, tsens driver now
correctly loads.

Tested-by: Jonathan McDowell <noodles@...th.li>

> v6:
> - Add more info about the changed define
> - Fixed wrong definition for pxo/cxo parent map
> v5:
> - Drop patch removing fixed clk
> - Use name in parent_data to keep compatibility with old dtb
> - Fix error in the documentation commits
> - Keep old copyright for gcc documentation
> - Fix an error with the rcg floor ops
> - Set nss clk based on the device compatible
> v4:
> - Drop drivers in all the patches.
> - Introduce floor ops for sdc
> - gcc.yaml to gcc-other.yaml
> - gcc-common.yaml to gcc.yaml
> v3:
> - Rework Documentation with Rob suggestions
> v2:
> - Fix error from Rob bot.
> - Add additional commits to make qcom,gcc.yaml a template
> - Squash parent_hws patch with the modernize patch
> - Create gcc_pxo instead of using long define.
> 
> Ansuel Smith (15):
>   dt-bindings: clock: split qcom,gcc.yaml to common and specific schema
>   dt-bindings: clock: simplify qcom,gcc-apq8064 Documentation
>   dt-bindings: clock: document qcom,gcc-ipq8064 binding
>   clk: qcom: gcc-ipq806x: fix wrong naming for gcc_pxo_pll8_pll0
>   clk: qcom: gcc-ipq806x: convert parent_names to parent_data
>   clk: qcom: gcc-ipq806x: use ARRAY_SIZE for num_parents
>   clk: qcom: gcc-ipq806x: add additional freq nss cores
>   clk: qcom: gcc-ipq806x: add unusued flag for critical clock
>   clk: qcom: clk-rcg: add clk_rcg_floor_ops ops
>   clk: qcom: gcc-ipq806x: add additional freq for sdc table
>   dt-bindings: clock: add ipq8064 ce5 clk define
>   clk: qcom: gcc-ipq806x: add CryptoEngine clocks
>   dt-bindings: reset: add ipq8064 ce5 resets
>   clk: qcom: gcc-ipq806x: add CryptoEngine resets
>   ARM: dts: qcom: add syscon and cxo/pxo clock to gcc node for ipq8064
> 
>  .../bindings/clock/qcom,gcc-apq8064.yaml      |  29 +-
>  .../bindings/clock/qcom,gcc-ipq8064.yaml      |  76 ++
>  .../bindings/clock/qcom,gcc-other.yaml        |  70 ++
>  .../devicetree/bindings/clock/qcom,gcc.yaml   |  59 +-
>  arch/arm/boot/dts/qcom-ipq8064.dtsi           |   8 +-
>  drivers/clk/qcom/clk-rcg.c                    |  24 +
>  drivers/clk/qcom/clk-rcg.h                    |   1 +
>  drivers/clk/qcom/gcc-ipq806x.c                | 649 +++++++++++++-----
>  include/dt-bindings/clock/qcom,gcc-ipq806x.h  |   5 +-
>  include/dt-bindings/reset/qcom,gcc-ipq806x.h  |   5 +
>  10 files changed, 685 insertions(+), 241 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/clock/qcom,gcc-ipq8064.yaml
>  create mode 100644 Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml
> 
> -- 
> 2.34.1

J.

-- 
Web [            Even the Evening Herald slags me off.             ]
site: https:// [                                          ]      Made by
www.earth.li/~noodles/  [                      ]         HuggieTag 0.0.24