Hi all,
The Stream Buffer for EtherAVB-IF (STBE) is an optional component, and
is not present on all R-Car Gen3 SoCs.
This patches series documents this in the DT bindings, and removes the
non-existing regions from the affected DTS files. Note that this region
is apparently not used by the RAVB driver.
The first patch is intended for either the net or DT tree.
The last two patches are intended for the renesas tree.
Compile-tested only.
Thanks for your comments!
Geert Uytterhoeven (3):
dt-bindings: net: renesas-ravb: Make stream buffer optional
arm64: dts: renesas: r8a77970: Remove non-existing STBE region
arm64: dts: renesas: r8a77995: Remove non-existing STBE region
Documentation/devicetree/bindings/net/renesas,ravb.txt | 5 ++++-
arch/arm64/boot/dts/renesas/r8a77970.dtsi | 2 +-
arch/arm64/boot/dts/renesas/r8a77995.dtsi | 2 +-
3 files changed, 6 insertions(+), 3 deletions(-)
--
2.7.4
Gr{oetje,eeting}s,
Geert
The Stream Buffer for EtherAVB-IF (STBE) is an optional component, and
is not present on all SoCs.
Document this in the DT bindings, including a list of SoCs that do have
it.
Fixes: 785ec87483d1e24a ("ravb: document R8A77970 bindings")
Fixes: f231c4178a655b09 ("dt-bindings: net: renesas-ravb: Add support for R8A77995 RAVB")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Documentation/devicetree/bindings/net/renesas,ravb.txt | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
@@ -26,7 +26,10 @@ Required properties: SoC-specific version corresponding to the platform first followed by the generic version.-- reg: offset and length of (1) the register block and (2) the stream buffer.+- reg: Offset and length of (1) the register block and (2) the stream buffer.+ The region for the register block is mandatory.+ The region for the stream buffer is optional, as it is only present on+ R-Car Gen2, RZ/G1, and R-Car H3 and M3-W. - interrupts: A list of interrupt-specifiers, one for each entry in interrupt-names. If interrupt-names is not present, an interrupt specifier
R-Car V3M does not have the Stream Buffer for EtherAVB-IF (STBE).
Note that the RAVB driver does not use this region.
Fixes: bea2ab136eaacec2 ("arm64: dts: renesas: r8a77970: add EtherAVB support")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
arch/arm64/boot/dts/renesas/r8a77970.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
From: Simon Horman <horms@verge.net.au> Date: 2018-02-06 13:17:13
On Tue, Feb 06, 2018 at 02:05:52PM +0100, Geert Uytterhoeven wrote:
The Stream Buffer for EtherAVB-IF (STBE) is an optional component, and
is not present on all SoCs.
Document this in the DT bindings, including a list of SoCs that do have
it.
Fixes: 785ec87483d1e24a ("ravb: document R8A77970 bindings")
Fixes: f231c4178a655b09 ("dt-bindings: net: renesas-ravb: Add support for R8A77995 RAVB")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
From: Sergei Shtylyov <hidden> Date: 2018-02-06 14:56:40
On 02/06/2018 04:05 PM, Geert Uytterhoeven wrote:
The Stream Buffer for EtherAVB-IF (STBE) is an optional component, and
is not present on all SoCs.
Document this in the DT bindings, including a list of SoCs that do have
it.
Fixes: 785ec87483d1e24a ("ravb: document R8A77970 bindings")
Fixes: f231c4178a655b09 ("dt-bindings: net: renesas-ravb: Add support for R8A77995 RAVB")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
@@ -26,7 +26,10 @@ Required properties: SoC-specific version corresponding to the platform first followed by the generic version.-- reg: offset and length of (1) the register block and (2) the stream buffer.+- reg: Offset and length of (1) the register block and (2) the stream buffer.+ The region for the register block is mandatory.+ The region for the stream buffer is optional, as it is only present on+ R-Car Gen2, RZ/G1, and R-Car H3 and M3-W.
Did you deliberately omit M3-N for now?
MBR, Sergei
Hi Sergei,
On Tue, Feb 6, 2018 at 3:56 PM, Sergei Shtylyov
[off-list ref] wrote:
On 02/06/2018 04:05 PM, Geert Uytterhoeven wrote:
quoted
The Stream Buffer for EtherAVB-IF (STBE) is an optional component, and
is not present on all SoCs.
Document this in the DT bindings, including a list of SoCs that do have
it.
Fixes: 785ec87483d1e24a ("ravb: document R8A77970 bindings")
Fixes: f231c4178a655b09 ("dt-bindings: net: renesas-ravb: Add support for R8A77995 RAVB")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
@@ -26,7 +26,10 @@ Required properties: SoC-specific version corresponding to the platform first followed by the generic version.-- reg: offset and length of (1) the register block and (2) the stream buffer.+- reg: Offset and length of (1) the register block and (2) the stream buffer.+ The region for the register block is mandatory.+ The region for the stream buffer is optional, as it is only present on+ R-Car Gen2, RZ/G1, and R-Car H3 and M3-W.
Did you deliberately omit M3-N for now?
Of course. It's not yet upstream.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
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
From: Simon Horman <horms@verge.net.au> Date: 2018-02-07 09:45:20
On Tue, Feb 06, 2018 at 02:05:53PM +0100, Geert Uytterhoeven wrote:
R-Car V3M does not have the Stream Buffer for EtherAVB-IF (STBE).
Note that the RAVB driver does not use this region.
Fixes: bea2ab136eaacec2 ("arm64: dts: renesas: r8a77970: add EtherAVB support")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
From: Rob Herring <robh@kernel.org> Date: 2018-02-09 22:55:02
On Tue, Feb 06, 2018 at 02:05:52PM +0100, Geert Uytterhoeven wrote:
The Stream Buffer for EtherAVB-IF (STBE) is an optional component, and
is not present on all SoCs.
Document this in the DT bindings, including a list of SoCs that do have
it.
Fixes: 785ec87483d1e24a ("ravb: document R8A77970 bindings")
Fixes: f231c4178a655b09 ("dt-bindings: net: renesas-ravb: Add support for R8A77995 RAVB")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Documentation/devicetree/bindings/net/renesas,ravb.txt | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)