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

[Kuldeep Singh]

> Hi!
> 
> On 09.03.21 06:56, Kuldeep Singh wrote:
> > Hi Heiko,
> >
> >> -----Original Message-----
> >> From: Heiko Schocher <hs@xxxxxxx>
> >> Sent: Tuesday, March 9, 2021 11:01 AM
> >> To: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> >> Cc: Heiko Schocher <hs@xxxxxxx>; Ashish Kumar <ashish.kumar@xxxxxxx>;
> >> Mark Brown <broonie@xxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>;
> >> Yogesh Gaur <yogeshgaur.83@xxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx;
> >> linux- kernel@xxxxxxxxxxxxxxx; linux-spi@xxxxxxxxxxxxxxx
> >> 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@xxxxxxx>
> >> ---
> >>
> >> (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