linux-kernel - Re: [PATCH v2 00/10] soc: sunxi: sram: Fixes and D1 support

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: <095cb5ca-3e63-50b0-f604-4c8c226f67e0@microchip.com>
Date:   Mon, 15 Aug 2022 06:58:00 +0000
From:   <Conor.Dooley@...rochip.com>
To:     <samuel@...lland.org>, <wens@...e.org>, <jernej.skrabec@...il.com>
CC:     <arnd@...db.de>, <hdegoede@...hat.com>, <icenowy@...c.io>,
        <krzysztof.kozlowski+dt@...aro.org>, <mripard@...nel.org>,
        <robh+dt@...nel.org>, <devicetree@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>, <linux-sunxi@...ts.linux.dev>
Subject: Re: [PATCH v2 00/10] soc: sunxi: sram: Fixes and D1 support

Hey Samuel,
On 15/08/2022 05:12, Samuel Holland wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> This series cleans up a few issues in the system controller driver, and
> then expands the exported regmap to support one of the pairs of LDOs
> built in to the D1 SoC.
> 
> Eventually, we will need to update the SRAM region claiming API so
> ownership can be swapped back and forth by consumer drivers. This will
> be necessary for uploading firmware to the R329/D1 DSPs, but it is not
> needed for initial bringup.

I tried this series on top of both 6.0-rc1 & next, but I saw a shed
load of errors from dtbs_check. I'll take a look at the rest of the
series later on, but just FYI. I know I have seen patches going into
different subsystems so I assume you're working on it. However, it
would be nice to let us what all the dependencies for this series are
dt-binding wise in the cover letter.
I have been trying to get riscv down to 0 warnings and would not be
in favour of accepting patches that go in the other direction.
(This patchset currently produces about 75 warnings)

Thanks,
Conor.

> 
> Changes in v2:
>   - New patch to first clean up the binding
> 
> Samuel Holland (10):
>    dt-bindings: sram: sunxi-sram: Clean up the compatible lists
>    dt-bindings: sram: sunxi-sram: Add D1 compatible string
>    soc: sunxi: sram: Actually claim SRAM regions
>    soc: sunxi: sram: Prevent the driver from being unbound
>    soc: sunxi: sram: Fix probe function ordering issues
>    soc: sunxi: sram: Fix debugfs info for A64 SRAM C
>    soc: sunxi: sram: Return void from the release function
>    soc: sunxi: sram: Save a pointer to the OF match data
>    soc: sunxi: sram: Export the LDO control register
>    soc: sunxi: sram: Add support for the D1 system control
> 
>   .../allwinner,sun4i-a10-system-control.yaml   | 82 ++++++++-----------
>   drivers/soc/sunxi/sunxi_sram.c                | 74 +++++++++--------
>   include/linux/soc/sunxi/sunxi_sram.h          |  2 +-
>   3 files changed, 73 insertions(+), 85 deletions(-)
> 
> --
> 2.35.1
>