linux-kernel - Aw: Re: [PATCH v1] arm64: dts: rockchip: Add sata2 node to rk356x

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] [day] [month] [year] [list]

Date:   Sun, 27 Feb 2022 09:13:42 +0100
From:   Frank Wunderlich <frank-w@...lic-files.de>
To:     Peter Geis <pgwipeout@...il.com>
Cc:     Heiko Stübner <heiko@...ech.de>,
        "open list:ARM/Rockchip SoC..." <linux-rockchip@...ts.infradead.org>,
        Frank Wunderlich <linux@...web.de>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com>,
        Michael Riesch <michael.riesch@...fvision.net>,
        devicetree <devicetree@...r.kernel.org>,
        arm-mail-list <linux-arm-kernel@...ts.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Aw: Re: [PATCH v1] arm64: dts: rockchip: Add sata2 node to rk356x

Hi

> Gesendet: Samstag, 26. Februar 2022 um 19:15 Uhr
> Von: "Peter Geis" <pgwipeout@...il.com>
> On Sat, Feb 26, 2022 at 1:08 PM Heiko Stübner <heiko@...ech.de> wrote:
> >
> > Hi Frank,
> >
> > Am Samstag, 26. Februar 2022, 14:57:24 CET schrieb Frank Wunderlich:
> > > From: Frank Wunderlich <frank-w@...lic-files.de>
> > >
> > > RK356x supports up to 3 sata controllers which were compatible with the
> > > existing snps,dwc-ahci binding.
> > >
> > > My board has only sata2 connected to combphy2 so only add this one.
> >
> > how far does the added node diverge from the vendor kernel?
> >
> > If it's pretty much similar between both, we can assume the other nodes
> > should work pretty well as well and therefore should all of them at once
> > and hope for the best?
> 
> There's essentially zero divergence (minus the change due to combophy
> changing), and likely won't be until the ahci-platform.txt is
> converted to yaml.
> 
> I have tested both SATA1 and SATA2 successfully on the rk3566.
> I don't have any rk3568 boards that are operational yet to test SATA0.

Like Peter says, the sata-node(s) is basicly same as downstream, only combphy-label has changed.
Sata1 from Peters tree [1] seems to have only this change.
I'll send an followup containing all 3, but had only tested sata2.

already sent a bindings-patch for the ahci-platform.txt [2]

regards Frank

[1] https://gitlab.com/pgwipeout/linux-next/-/blob/main/arch/arm64/boot/dts/rockchip/rk356x.dtsi#L238
[2] https://lore.kernel.org/lkml/20220226204404.109867-1-linux@fw-web.de/T/