2022-10-12 14:49:27

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH] dt-bindings: memory: renesas,rpc-if: Document R-Car V4H support

From: Hai Pham <[email protected]>

Document support for the SPI Multi I/O Bus Controller (RPC-IF) in the
R-Car V4H SoC.

Signed-off-by: Hai Pham <[email protected]>
Signed-off-by: Geert Uytterhoeven <[email protected]>
---
.../bindings/memory-controllers/renesas,rpc-if.yaml | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/memory-controllers/renesas,rpc-if.yaml b/Documentation/devicetree/bindings/memory-controllers/renesas,rpc-if.yaml
index 645249ea21d19cd9..30a403b1b79a2c73 100644
--- a/Documentation/devicetree/bindings/memory-controllers/renesas,rpc-if.yaml
+++ b/Documentation/devicetree/bindings/memory-controllers/renesas,rpc-if.yaml
@@ -42,6 +42,11 @@ properties:
- renesas,r8a779a0-rpc-if # R-Car V3U
- const: renesas,rcar-gen3-rpc-if # a generic R-Car gen3 or RZ/G2{E,H,M,N} device

+ - items:
+ - enum:
+ - renesas,r8a779g0-rpc-if # R-Car V4H
+ - const: renesas,rcar-gen4-rpc-if # a generic R-Car gen4 device
+
- items:
- enum:
- renesas,r9a07g043-rpc-if # RZ/G2UL
--
2.25.1


2022-10-12 16:51:26

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: memory: renesas,rpc-if: Document R-Car V4H support

On Wed, 12 Oct 2022 15:51:46 +0200, Geert Uytterhoeven wrote:
> From: Hai Pham <[email protected]>
>
> Document support for the SPI Multi I/O Bus Controller (RPC-IF) in the
> R-Car V4H SoC.
>
> Signed-off-by: Hai Pham <[email protected]>
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> ---
> .../bindings/memory-controllers/renesas,rpc-if.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>

Reviewed-by: Rob Herring <[email protected]>

2022-10-12 19:16:15

by Wolfram Sang

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: memory: renesas,rpc-if: Document R-Car V4H support

On Wed, Oct 12, 2022 at 03:51:46PM +0200, Geert Uytterhoeven wrote:
> From: Hai Pham <[email protected]>
>
> Document support for the SPI Multi I/O Bus Controller (RPC-IF) in the
> R-Car V4H SoC.
>
> Signed-off-by: Hai Pham <[email protected]>
> Signed-off-by: Geert Uytterhoeven <[email protected]>

Reviewed-by: Wolfram Sang <[email protected]>

What about moving V3U to the new Gen4 section?


Attachments:
(No filename) (444.00 B)
signature.asc (849.00 B)
Download all attachments

2022-10-13 07:14:22

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: memory: renesas,rpc-if: Document R-Car V4H support

Hi Wolfram,

On Wed, Oct 12, 2022 at 9:04 PM Wolfram Sang
<[email protected]> wrote:
> On Wed, Oct 12, 2022 at 03:51:46PM +0200, Geert Uytterhoeven wrote:
> > From: Hai Pham <[email protected]>
> >
> > Document support for the SPI Multi I/O Bus Controller (RPC-IF) in the
> > R-Car V4H SoC.
> >
> > Signed-off-by: Hai Pham <[email protected]>
> > Signed-off-by: Geert Uytterhoeven <[email protected]>
>
> Reviewed-by: Wolfram Sang <[email protected]>

Thanks!

> What about moving V3U to the new Gen4 section?

I actually wrote such a patch a while back.
However, I didn't send it after discovering the R-Car V3U Series
User’s Manual Rev. 0.50 does not mention the fourth Strobe Timing
Adjustment bit (STRTIM) in the RPC-IF PHY Control Register (PHYCNT),
which is present on R-Car S4-8 and V4H, but not on R-Car Gen3.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

2022-10-13 08:08:41

by Wolfram Sang

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: memory: renesas,rpc-if: Document R-Car V4H support


> I actually wrote such a patch a while back.
> However, I didn't send it after discovering the R-Car V3U Series
> User’s Manual Rev. 0.50 does not mention the fourth Strobe Timing
> Adjustment bit (STRTIM) in the RPC-IF PHY Control Register (PHYCNT),
> which is present on R-Car S4-8 and V4H, but not on R-Car Gen3.

I see. Thanks for the heads up! Maybe this is worth a comment because it
is an exception to the usual behaviour?


Attachments:
(No filename) (445.00 B)
signature.asc (849.00 B)
Download all attachments

2022-10-13 08:43:55

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: memory: renesas,rpc-if: Document R-Car V4H support

Hi Wolfram,

On Thu, Oct 13, 2022 at 10:00 AM Wolfram Sang
<[email protected]> wrote:
> > I actually wrote such a patch a while back.
> > However, I didn't send it after discovering the R-Car V3U Series
> > User’s Manual Rev. 0.50 does not mention the fourth Strobe Timing
> > Adjustment bit (STRTIM) in the RPC-IF PHY Control Register (PHYCNT),
> > which is present on R-Car S4-8 and V4H, but not on R-Car Gen3.
>
> I see. Thanks for the heads up! Maybe this is worth a comment because it
> is an exception to the usual behaviour?

You mean an exception to the exception to the usual behaviour that
R-Car <foo>3 is part of the R-Car Gen3 family, making it the normal
rule? ;-)

We're actually not 100% sure the bit is present or not. Rev. 0.50 is
still an early revision of the R-Car V3U documentation, so it might lack
some updates. I tried changing all bits of the PHYCNT register, but
unfortunately all bits except for bit 31 stick on all SoCs I tried, so
we can't use this method to determine which bits exist and which don't.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

2022-10-18 15:50:48

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: memory: renesas,rpc-if: Document R-Car V4H support

On Wed, 12 Oct 2022 15:51:46 +0200, Geert Uytterhoeven wrote:
> From: Hai Pham <[email protected]>
>
> Document support for the SPI Multi I/O Bus Controller (RPC-IF) in the
> R-Car V4H SoC.
>
>

Applied, thanks!

[1/1] dt-bindings: memory: renesas,rpc-if: Document R-Car V4H support
https://git.kernel.org/krzk/linux-mem-ctrl/c/fad9489abf932ff01ca2b9a9bf41a11311bdc8d5

Best regards,
--
Krzysztof Kozlowski <[email protected]>