mbox series

[0/6] Update compat strings for SD/MMC nodes on RZ/{G2L (family), G3S, V2M} SoCs

Message ID 20240422213006.505576-1-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive)
Headers show
Series Update compat strings for SD/MMC nodes on RZ/{G2L (family), G3S, V2M} SoCs | expand

Message

Lad, Prabhakar April 22, 2024, 9:30 p.m. UTC
From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Hi All,

- RZ/G2UL and RZ/Five ("r9a07g043")
- RZ/G2L(C) ("r9a07g044")
- RZ/V2L ("r9a07g054")
- RZ/G3S ("r9a08g045")
- RZ/V2M ("r9a09g011")

The SD/MMC Interface in the above listed SoCs is not identical to that of
R-Car Gen3. These SoCs have HS400 disabled and use fixed address mode.
Therefore, we need to apply fixed_addr_mode and hs400_disabled quirks.
'renesas,rzg2l-sdhi' is introduced as a generic compatible string for the
above SoCs where fixed_addr_mode and hs400_disabled quirks will be applied.

Cheers,
Prabhakar

Lad Prabhakar (6):
  dt-bindings: mmc: renesas,sdhi: Drop 'items' keyword
  dt-bindings: mmc: renesas,sdhi: Document RZ/G2L family compatibility
  mmc: renesas_sdhi: Add compatible string for RZ/G2L family, RZ/G3S,
    and RZ/V2M SoCs
  arm64: dts: renesas: r9a09g011: Update fallback string for SDHI nodes
  arm64: dts: renesas: rzg2l-family: Update fallback string for SDHI
    nodes
  arm64: dts: renesas: r9a08g045: Update fallback string for SDHI nodes

 .../devicetree/bindings/mmc/renesas,sdhi.yaml | 38 ++++++++-----------
 arch/arm64/boot/dts/renesas/r9a07g043.dtsi    |  4 +-
 arch/arm64/boot/dts/renesas/r9a07g044.dtsi    |  4 +-
 arch/arm64/boot/dts/renesas/r9a07g054.dtsi    |  4 +-
 arch/arm64/boot/dts/renesas/r9a08g045.dtsi    |  6 +--
 arch/arm64/boot/dts/renesas/r9a09g011.dtsi    |  6 +--
 drivers/mmc/host/renesas_sdhi_internal_dmac.c |  8 ++--
 7 files changed, 31 insertions(+), 39 deletions(-)

Comments

Geert Uytterhoeven April 23, 2024, 6:57 a.m. UTC | #1
Hi Prabhakar,

On Mon, Apr 22, 2024 at 11:30 PM Prabhakar <prabhakar.csengg@gmail.com> wrote:
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> - RZ/G2UL and RZ/Five ("r9a07g043")
> - RZ/G2L(C) ("r9a07g044")
> - RZ/V2L ("r9a07g054")
> - RZ/G3S ("r9a08g045")
> - RZ/V2M ("r9a09g011")
>
> The SD/MMC Interface in the above listed SoCs is not identical to that of
> R-Car Gen3. These SoCs have HS400 disabled and use fixed address mode.
> Therefore, we need to apply fixed_addr_mode and hs400_disabled quirks.
> 'renesas,rzg2l-sdhi' is introduced as a generic compatible string for the
> above SoCs where fixed_addr_mode and hs400_disabled quirks will be applied.

Thanks for your series!

> Lad Prabhakar (6):
>   dt-bindings: mmc: renesas,sdhi: Drop 'items' keyword
>   dt-bindings: mmc: renesas,sdhi: Document RZ/G2L family compatibility
>   mmc: renesas_sdhi: Add compatible string for RZ/G2L family, RZ/G3S,
>     and RZ/V2M SoCs
>   arm64: dts: renesas: r9a09g011: Update fallback string for SDHI nodes
>   arm64: dts: renesas: rzg2l-family: Update fallback string for SDHI
>     nodes
>   arm64: dts: renesas: r9a08g045: Update fallback string for SDHI nodes

The DTS patches have a hard dependency on the driver changes, right?
So they cannot be applied in parallel.

Gr{oetje,eeting}s,

                        Geert
Lad, Prabhakar April 23, 2024, 7:14 a.m. UTC | #2
Hi Geert,

On Tue, Apr 23, 2024 at 7:57 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> Hi Prabhakar,
>
> On Mon, Apr 22, 2024 at 11:30 PM Prabhakar <prabhakar.csengg@gmail.com> wrote:
> > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > - RZ/G2UL and RZ/Five ("r9a07g043")
> > - RZ/G2L(C) ("r9a07g044")
> > - RZ/V2L ("r9a07g054")
> > - RZ/G3S ("r9a08g045")
> > - RZ/V2M ("r9a09g011")
> >
> > The SD/MMC Interface in the above listed SoCs is not identical to that of
> > R-Car Gen3. These SoCs have HS400 disabled and use fixed address mode.
> > Therefore, we need to apply fixed_addr_mode and hs400_disabled quirks.
> > 'renesas,rzg2l-sdhi' is introduced as a generic compatible string for the
> > above SoCs where fixed_addr_mode and hs400_disabled quirks will be applied.
>
> Thanks for your series!
>
> > Lad Prabhakar (6):
> >   dt-bindings: mmc: renesas,sdhi: Drop 'items' keyword
> >   dt-bindings: mmc: renesas,sdhi: Document RZ/G2L family compatibility
> >   mmc: renesas_sdhi: Add compatible string for RZ/G2L family, RZ/G3S,
> >     and RZ/V2M SoCs
> >   arm64: dts: renesas: r9a09g011: Update fallback string for SDHI nodes
> >   arm64: dts: renesas: rzg2l-family: Update fallback string for SDHI
> >     nodes
> >   arm64: dts: renesas: r9a08g045: Update fallback string for SDHI nodes
>
> The DTS patches have a hard dependency on the driver changes, right?
> So they cannot be applied in parallel.
>
Agreed, I sent them together to make the DT bot happy for the dtbs
checks. While sending a v2 Ill split them up.

Cheers,
Prabhakar
Geert Uytterhoeven May 30, 2024, 1:43 p.m. UTC | #3
On Tue, Apr 23, 2024 at 8:57 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> On Mon, Apr 22, 2024 at 11:30 PM Prabhakar <prabhakar.csengg@gmail.com> wrote:
> > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > - RZ/G2UL and RZ/Five ("r9a07g043")
> > - RZ/G2L(C) ("r9a07g044")
> > - RZ/V2L ("r9a07g054")
> > - RZ/G3S ("r9a08g045")
> > - RZ/V2M ("r9a09g011")
> >
> > The SD/MMC Interface in the above listed SoCs is not identical to that of
> > R-Car Gen3. These SoCs have HS400 disabled and use fixed address mode.
> > Therefore, we need to apply fixed_addr_mode and hs400_disabled quirks.
> > 'renesas,rzg2l-sdhi' is introduced as a generic compatible string for the
> > above SoCs where fixed_addr_mode and hs400_disabled quirks will be applied.
>
> Thanks for your series!
>
> > Lad Prabhakar (6):
> >   dt-bindings: mmc: renesas,sdhi: Drop 'items' keyword
> >   dt-bindings: mmc: renesas,sdhi: Document RZ/G2L family compatibility
> >   mmc: renesas_sdhi: Add compatible string for RZ/G2L family, RZ/G3S,
> >     and RZ/V2M SoCs
> >   arm64: dts: renesas: r9a09g011: Update fallback string for SDHI nodes
> >   arm64: dts: renesas: rzg2l-family: Update fallback string for SDHI
> >     nodes
> >   arm64: dts: renesas: r9a08g045: Update fallback string for SDHI nodes
>
> The DTS patches have a hard dependency on the driver changes, right?
> So they cannot be applied in parallel.

As this dependency is not i v6.10-rc1, I will queue the DTS patches
in renesas-devel for v6.11.

Gr{oetje,eeting}s,

                        Geert