linux-kernel - Re: [PATCH] arm64: dts: ns2: Fix spi-cpol and spi-cpha property

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]

Message-ID: <20220302130734.GA2026@9a2d8922b8f1>
Date:   Wed, 2 Mar 2022 18:37:34 +0530
From:   Kuldeep Singh <singh.kuldeep87k@...il.com>
To:     Florian Fainelli <f.fainelli@...il.com>
Cc:     Scott Branden <sbranden@...adcom.com>, Ray Jui <rjui@...adcom.com>,
        devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: ns2: Fix spi-cpol and spi-cpha property

On Mon, Feb 28, 2022 at 01:31:50PM -0800, Florian Fainelli wrote:
> 
> 
> On 2/28/2022 3:09 AM, Kuldeep Singh wrote:
> > Broadcom ns2 platform has spi-cpol and spi-cpho properties set
> > incorrectly. As per spi-slave-peripheral-prop.yaml, these properties are
> > of flag or boolean type and not integer type. Fix the values.
> > 
> > CC: Ray Jui <rjui@...adcom.com>
> > CC: Scott Branden <sbranden@...adcom.com>
> > CC: Florian Fainelli <f.fainelli@...il.com>
> > Fixes: d69dbd9f41a7c (arm64: dts: Add ARM PL022 SPI DT nodes for NS2)
> > Signed-off-by: Kuldeep Singh <singh.kuldeep87k@...il.com>
> > ---
> > Hi,
> > 
> > This patch is on top of git://github.com/broadcom/cygnus-linux.git,
> > master branch which is not updated since 4.14 kernel.
> > Hope the reference is correct. Thanks!
> 
> The reference is not correct, but it applied cleanly anyway. The correct
> reference would have been the github.com/Broadcom/stblinux
> refs/heads/devicetree-arm64/next

Thanks Florian for mentioning.
I took a quick look at MAINTAINERS file before sending patch and found
cygnus-linux repo as base reference. It seems the correct reference is
not updated in MAINTAINERS file.

> 
> Applied, thanks!

Thanks!

-- 
Best Regards
Kuldeep