diff mbox series

[14/20] dt-bindings: spi: renesas,sh-msiof: Add r8a774e1 support

Message ID 1594811350-14066-15-git-send-email-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive)
State Accepted
Delegated to: Geert Uytterhoeven
Headers show
Series Add support for [H]SCIF/TMU/CMT/THS/SDHI/MSIOF/CAN[FD]/I2C/IIC/RWDT on R8A774E1 | expand

Commit Message

Lad Prabhakar July 15, 2020, 11:09 a.m. UTC
Document RZ/G2H (R8A774E1) SoC bindings.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Reviewed-by: Marian-Cristian Rotariu <marian-cristian.rotariu.rb@bp.renesas.com>
---
 Documentation/devicetree/bindings/spi/renesas,sh-msiof.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Geert Uytterhoeven July 16, 2020, 2:29 p.m. UTC | #1
On Wed, Jul 15, 2020 at 1:10 PM Lad Prabhakar
<prabhakar.mahadev-lad.rj@bp.renesas.com> wrote:
> Document RZ/G2H (R8A774E1) SoC bindings.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Reviewed-by: Marian-Cristian Rotariu <marian-cristian.rotariu.rb@bp.renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert
Mark Brown July 17, 2020, 11:59 a.m. UTC | #2
On Wed, Jul 15, 2020 at 12:09:04PM +0100, Lad Prabhakar wrote:
> Document RZ/G2H (R8A774E1) SoC bindings.

Please in future could you split things like this up into per subsystem
serieses?  That's a more normal approach and avoids the huge threads and
CC lists.
Prabhakar July 17, 2020, 12:15 p.m. UTC | #3
Hi Mark,

On Fri, Jul 17, 2020 at 12:59 PM Mark Brown <broonie@kernel.org> wrote:
>
> On Wed, Jul 15, 2020 at 12:09:04PM +0100, Lad Prabhakar wrote:
> > Document RZ/G2H (R8A774E1) SoC bindings.
>
> Please in future could you split things like this up into per subsystem
> serieses?  That's a more normal approach and avoids the huge threads and
> CC lists.

Sorry for doing this, In future I shall keep that in mind. (Wanted to
get in most patches for RZ/G2H in V5.9 window)

Cheers,
--Prabhakar
Mark Brown July 17, 2020, 12:22 p.m. UTC | #4
On Fri, Jul 17, 2020 at 01:15:13PM +0100, Lad, Prabhakar wrote:
> On Fri, Jul 17, 2020 at 12:59 PM Mark Brown <broonie@kernel.org> wrote:

> > On Wed, Jul 15, 2020 at 12:09:04PM +0100, Lad Prabhakar wrote:
> > > Document RZ/G2H (R8A774E1) SoC bindings.

> > Please in future could you split things like this up into per subsystem
> > serieses?  That's a more normal approach and avoids the huge threads and
> > CC lists.

> Sorry for doing this, In future I shall keep that in mind. (Wanted to
> get in most patches for RZ/G2H in V5.9 window)

If anything sending things as a big series touching lots of subsystems
can slow things down as people figure out dependencies and who's going
to actually apply things.
Rob Herring July 21, 2020, 2:51 a.m. UTC | #5
On Fri, Jul 17, 2020 at 01:22:09PM +0100, Mark Brown wrote:
> On Fri, Jul 17, 2020 at 01:15:13PM +0100, Lad, Prabhakar wrote:
> > On Fri, Jul 17, 2020 at 12:59 PM Mark Brown <broonie@kernel.org> wrote:
> 
> > > On Wed, Jul 15, 2020 at 12:09:04PM +0100, Lad Prabhakar wrote:
> > > > Document RZ/G2H (R8A774E1) SoC bindings.
> 
> > > Please in future could you split things like this up into per subsystem
> > > serieses?  That's a more normal approach and avoids the huge threads and
> > > CC lists.
> 
> > Sorry for doing this, In future I shall keep that in mind. (Wanted to
> > get in most patches for RZ/G2H in V5.9 window)
> 
> If anything sending things as a big series touching lots of subsystems
> can slow things down as people figure out dependencies and who's going
> to actually apply things.

I'd be fine with one patch adding all the compatibles for a new SoC 
where there are no other changes and take that via the soc or DT tree, 
but the Renesas folks need to figure out any cross tree dependencies 
with other Renesas changes. That's a bit harder right now with schema 
conversions though.

Rob
Rob Herring July 21, 2020, 2:51 a.m. UTC | #6
On Wed, 15 Jul 2020 12:09:04 +0100, Lad Prabhakar wrote:
> Document RZ/G2H (R8A774E1) SoC bindings.
> 
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Reviewed-by: Marian-Cristian Rotariu <marian-cristian.rotariu.rb@bp.renesas.com>
> ---
>  Documentation/devicetree/bindings/spi/renesas,sh-msiof.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring <robh@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/spi/renesas,sh-msiof.yaml b/Documentation/devicetree/bindings/spi/renesas,sh-msiof.yaml
index ef27a789c08f..9f7b118adcaf 100644
--- a/Documentation/devicetree/bindings/spi/renesas,sh-msiof.yaml
+++ b/Documentation/devicetree/bindings/spi/renesas,sh-msiof.yaml
@@ -38,6 +38,7 @@  properties:
               - renesas,msiof-r8a774a1      # RZ/G2M
               - renesas,msiof-r8a774b1      # RZ/G2N
               - renesas,msiof-r8a774c0      # RZ/G2E
+              - renesas,msiof-r8a774e1      # RZ/G2H
               - renesas,msiof-r8a7795       # R-Car H3
               - renesas,msiof-r8a7796       # R-Car M3-W
               - renesas,msiof-r8a77965      # R-Car M3-N