linux-kernel - RE: [EXT] [PATCH v2 2/4] dt-bindings: spi: add compatible entry for imx8mp in FlexSPI controller

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: <DB6PR0402MB275899B5183869C7D471118AE0929@DB6PR0402MB2758.eurprd04.prod.outlook.com>
Date:   Tue, 9 Mar 2021 06:29:40 +0000
From:   Kuldeep Singh <kuldeep.singh@....com>
To:     "hs@...x.de" <hs@...x.de>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        Mark Brown <broonie@...nel.org>
CC:     Ashish Kumar <ashish.kumar@....com>,
        Rob Herring <robh+dt@...nel.org>,
        Yogesh Gaur <yogeshgaur.83@...il.com>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-spi@...r.kernel.org" <linux-spi@...r.kernel.org>
Subject: RE: [EXT] [PATCH v2 2/4] dt-bindings: spi: add compatible entry for
 imx8mp in FlexSPI controller

> Hi!
> 
> On 09.03.21 06:56, Kuldeep Singh wrote:
> > Hi Heiko,
> >
> >> -----Original Message-----
> >> From: Heiko Schocher <hs@...x.de>
> >> Sent: Tuesday, March 9, 2021 11:01 AM
> >> To: linux-arm-kernel@...ts.infradead.org
> >> Cc: Heiko Schocher <hs@...x.de>; Ashish Kumar <ashish.kumar@....com>;
> >> Mark Brown <broonie@...nel.org>; Rob Herring <robh+dt@...nel.org>;
> >> Yogesh Gaur <yogeshgaur.83@...il.com>; devicetree@...r.kernel.org;
> >> linux- kernel@...r.kernel.org; linux-spi@...r.kernel.org
> >> Subject: [EXT] [PATCH v2 2/4] dt-bindings: spi: add compatible entry
> >> for imx8mp in FlexSPI controller
> >>
> >> Caution: EXT Email
> >>
> >> add compatible entry "nxp,imx8mp-fspi" in NXP FlexSPI controller
> >>
> >> Signed-off-by: Heiko Schocher <hs@...x.de>
> >> ---
> >>
> >> (no changes since v1)
> >>
> >>  Documentation/devicetree/bindings/spi/spi-nxp-fspi.txt | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/Documentation/devicetree/bindings/spi/spi-nxp-fspi.txt
> >> b/Documentation/devicetree/bindings/spi/spi-nxp-fspi.txt
> >> index 7ac60d9fe3571..fd5f081f6d91b 100644
> >> --- a/Documentation/devicetree/bindings/spi/spi-nxp-fspi.txt
> >> +++ b/Documentation/devicetree/bindings/spi/spi-nxp-fspi.txt
> >> @@ -4,6 +4,7 @@ Required properties:
> >>    - compatible : Should be "nxp,lx2160a-fspi"
> >>                             "nxp,imx8qxp-fspi"
> >>                             "nxp,imx8mm-fspi"
> >> +                           "nxp,imx8mp-fspi"
> >
> > It seems the changes are not on top of tree. Please see[1] for latest changes
> including imx8dxl entry.
> > Snippet below:
> >
> > Required properties:
> >   - compatible : Should be "nxp,lx2160a-fspi"
> >                           "nxp,imx8qxp-fspi"
> >                           "nxp,imx8mm-fspi"
> >                           "nxp,imx8dxl-fspi"
> 
> My patches are based on mainline..
> *   144c79ef33536 - (origin/master, origin/HEAD) Merge tag 'perf-tools-fixes-for-
> v5.12-2020-03-07'
> of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux (vor 32 Stunden)
> <Linus Torvalds>
> 
> Should I post this patch rebased on [1] or may all patches rebased on [1] ?

Hi Heiko,

As per cover letter, I can identify that first two are driver/Documentation patches and other two are device-tree patches.
Device-tree patches(3/4 and 4/4) will go via Shawn tree (git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git)
And initial two driver patches(1/4 and 2/4) will go via Mark's tree (git://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git)

One approach I can think of is to send driver patches first and later send device-tree patches mentioning dependency on first two patches.
Maybe Mark can comment here if he has any other opinion.

Thanks
Kuldeep