linux-kernel - Re: [PATCH v5 0/3] Add initialization of clock for StarFive JH7110 SoC

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: <51547f1a-4eb4-2ccd-1e06-8adb2b4b70bd@starfivetech.com>
Date:   Fri, 28 Jul 2023 16:47:27 +0800
From:   William Qiu <william.qiu@...rfivetech.com>
To:     <devicetree@...r.kernel.org>, <linux-spi@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <linux-riscv@...ts.infradead.org>
CC:     Mark Brown <broonie@...nel.org>, Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Emil Renner Berthing <kernel@...il.dk>,
        Ziv Xu <ziv.xu@...rfivetech.com>
Subject: Re: [PATCH v5 0/3] Add initialization of clock for StarFive JH7110
 SoC



On 2023/7/19 17:25, William Qiu wrote:
> Hi,
> 
> This patchset adds initial rudimentary support for the StarFive
> Quad SPI controller driver. And this driver will be used in
> StarFive's VisionFive 2 board. In 6.4, the QSPI_AHB and QSPI_APB
> clocks changed from the default ON state to the default OFF state,
> so these clocks need to be enabled in the driver.At the same time,
> dts patch is added to this series.
> 
> Changes v4->v5:
> - Rebased to v6.5rc2.
> - Changed the way to initialization the clocks.
> - Changed the layout for the SPI flash.
> 
> Changes v3->v4:
> - Added minItems for clocks.
> - Added clock names property.
> - Fixed formatting issues.
> 
> Changes v2->v3:
> - Rebased to v6.4rc6.
> - Renamed the clock names.
> - Changed the variable definition type.
> 
> Changes v1->v2:
> - Renamed the clock names.
> - Specified a different array of clocks.
> - Used clk_bulk_ APIs.
> 
> The patch series is based on v6.5rc2.
> 
> William Qiu (3):
>   dt-bindings: qspi: cdns,qspi-nor: Add clocks for StarFive JH7110 SoC
>   spi: cadence-quadspi: Add clock configuration for StarFive JH7110 QSPI
>   riscv: dts: starfive: Add QSPI controller node for StarFive JH7110 SoC
> 
>  .../bindings/spi/cdns,qspi-nor.yaml           | 12 +++-
>  .../jh7110-starfive-visionfive-2.dtsi         | 36 ++++++++++
>  arch/riscv/boot/dts/starfive/jh7110.dtsi      | 19 ++++++
>  drivers/spi/spi-cadence-quadspi.c             | 67 +++++++++++++++++++
>  4 files changed, 133 insertions(+), 1 deletion(-)
> 
> --
> 2.34.1
> 
Hi everyone,

Could you please help me review this patch series to see if there is
anything that needs to be modified?
Thanks for taking time to review this patch series.

Best Regards,
William