* [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree @ 2025-03-24 14:50 Thomas Bonnefille 2025-03-24 17:28 ` Rob Herring (Arm) 2025-04-11 9:21 ` Geert Uytterhoeven 0 siblings, 2 replies; 8+ messages in thread From: Thomas Bonnefille @ 2025-03-24 14:50 UTC (permalink / raw) To: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: Thomas Petazzoni, Miquèl Raynal, linux-renesas-soc, devicetree, linux-kernel, Clément Léger, Thomas Bonnefille, Wolfram Sang, Niklas Söderlund From: Clément Léger <clement.leger@bootlin.com> The EB board (Expansion board) supports both RZ/N1D and RZ-N1S. Since this configuration targets only the RZ/N1D, it is named r9a06g032-rzn1d400-eb. It adds support for the 2 additional switch ports (port C and D) that are available on that board. Signed-off-by: Clément Léger <clement.leger@bootlin.com> [Thomas: move the DTS to the Renesas directory, declare the PHY LEDs] Signed-off-by: Thomas Bonnefille <thomas.bonnefille@bootlin.com> Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> Tested-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> --- This short series adds support for the RZ/N1 Expansion Board. This board is a carrier board on which a daughter board (either RZ/N1D or RZ/N1S) can be plugged. The device-tree that is added by this series enables the use to the 2 external switch ports that are present on this board. --- V4: - Drop trailing whitespaces V3: - Drop bindings commit as it was applied to master - Move Makefile modification to arch/arm/boot/dts/renesas/Makefile - Declare LEDs in PHY. - Use the driver default LED configuration as there was no reason to use a different one. V2: - Add "renesas,rzn1d400-db" in list of compatibles for EB board - Replace '_' with '-' in eth pins node name - Split some long lines in dts --- arch/arm/boot/dts/renesas/Makefile | 1 + .../arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dts | 120 +++++++++++++++++++++ 2 files changed, 121 insertions(+) diff --git a/arch/arm/boot/dts/renesas/Makefile b/arch/arm/boot/dts/renesas/Makefile index 833a02447ecf7a02bd2efe70fae15213ede9a6de..947c7fe0280337a3aa6e9a0257f406694892239c 100644 --- a/arch/arm/boot/dts/renesas/Makefile +++ b/arch/arm/boot/dts/renesas/Makefile @@ -30,4 +30,5 @@ dtb-$(CONFIG_ARCH_RENESAS) += \ r8a7794-alt.dtb \ r8a7794-silk.dtb \ r9a06g032-rzn1d400-db.dtb \ + r9a06g032-rzn1d400-eb.dtb \ sh73a0-kzm9g.dtb diff --git a/arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dts b/arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dts new file mode 100644 index 0000000000000000000000000000000000000000..1aac06b0e423345bb6d9b88f86cfee99c11def46 --- /dev/null +++ b/arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dts @@ -0,0 +1,120 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Device Tree Source for the RZN1D-EB Board + * + * Copyright (C) 2023 Schneider-Electric + * + */ + +#include "r9a06g032-rzn1d400-db.dts" + +/ { + model = "RZN1D-EB Board"; + compatible = "renesas,rzn1d400-eb", "renesas,rzn1d400-db", + "renesas,r9a06g032"; +}; + +&mii_conv2 { + renesas,miic-input = <MIIC_SWITCH_PORTD>; + status = "okay"; +}; + +&mii_conv3 { + renesas,miic-input = <MIIC_SWITCH_PORTC>; + status = "okay"; +}; + +&pinctrl{ + pins_eth1: pins-eth1 { + pinmux = <RZN1_PINMUX(12, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(13, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(14, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(15, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(16, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(17, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(18, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(19, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(20, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(21, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(22, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(23, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>; + drive-strength = <6>; + bias-disable; + }; + + pins_eth2: pins-eth2 { + pinmux = <RZN1_PINMUX(24, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(25, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(26, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(27, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(28, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(29, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(30, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(31, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(32, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(33, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(34, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, + <RZN1_PINMUX(35, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>; + drive-strength = <6>; + bias-disable; + }; +}; + +&switch { + pinctrl-names = "default"; + pinctrl-0 = <&pins_eth1>, <&pins_eth2>, <&pins_eth3>, <&pins_eth4>, + <&pins_mdio1>; + + mdio { + /* CN15 and CN16 switches must be configured in MDIO2 mode */ + switch0phy1: ethernet-phy@1 { + reg = <1>; + leds { + #address-cells = <1>; + #size-cells = <0>; + + led@0 { + reg = <0>; + }; + led@1 { + reg = <1>; + }; + led@2 { + reg = <2>; + }; + }; + }; + + switch0phy10: ethernet-phy@10 { + reg = <10>; + leds { + #address-cells = <1>; + #size-cells = <0>; + + led@0 { + reg = <0>; + }; + led@1 { + reg = <1>; + }; + led@2 { + reg = <2>; + }; + }; + }; + }; +}; + +&switch_port2 { + label = "lan2"; + phy-mode = "rgmii-id"; + phy-handle = <&switch0phy10>; + status = "okay"; +}; + +&switch_port3 { + label = "lan3"; + phy-mode = "rgmii-id"; + phy-handle = <&switch0phy1>; + status = "okay"; +}; --- base-commit: 4701f33a10702d5fc577c32434eb62adde0a1ae1 change-id: 20250127-rzn1d400-eb-3fc1479a13e6 Best regards, -- Thomas Bonnefille <thomas.bonnefille@bootlin.com> ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree 2025-03-24 14:50 [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree Thomas Bonnefille @ 2025-03-24 17:28 ` Rob Herring (Arm) 2025-03-24 18:36 ` Wolfram Sang 2025-04-11 9:21 ` Geert Uytterhoeven 1 sibling, 1 reply; 8+ messages in thread From: Rob Herring (Arm) @ 2025-03-24 17:28 UTC (permalink / raw) To: Thomas Bonnefille Cc: Thomas Petazzoni, Magnus Damm, Clément Léger, devicetree, Conor Dooley, linux-kernel, linux-renesas-soc, Krzysztof Kozlowski, Wolfram Sang, Niklas Söderlund, Miquèl Raynal, Geert Uytterhoeven On Mon, 24 Mar 2025 15:50:44 +0100, Thomas Bonnefille wrote: > From: Clément Léger <clement.leger@bootlin.com> > > The EB board (Expansion board) supports both RZ/N1D and RZ-N1S. Since this > configuration targets only the RZ/N1D, it is named r9a06g032-rzn1d400-eb. > It adds support for the 2 additional switch ports (port C and D) that are > available on that board. > > Signed-off-by: Clément Léger <clement.leger@bootlin.com> > [Thomas: move the DTS to the Renesas directory, declare the PHY LEDs] > Signed-off-by: Thomas Bonnefille <thomas.bonnefille@bootlin.com> > Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > Tested-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > This short series adds support for the RZ/N1 Expansion Board. This board > is a carrier board on which a daughter board (either RZ/N1D or RZ/N1S) > can be plugged. The device-tree that is added by this series enables the > use to the 2 external switch ports that are present on this board. > --- > V4: > - Drop trailing whitespaces > > V3: > - Drop bindings commit as it was applied to master > - Move Makefile modification to arch/arm/boot/dts/renesas/Makefile > - Declare LEDs in PHY. > - Use the driver default LED configuration as there was no reason to > use a different one. > > V2: > - Add "renesas,rzn1d400-db" in list of compatibles for EB board > - Replace '_' with '-' in eth pins node name > - Split some long lines in dts > --- > arch/arm/boot/dts/renesas/Makefile | 1 + > .../arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dts | 120 +++++++++++++++++++++ > 2 files changed, 121 insertions(+) > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade New warnings running 'make CHECK_DTBS=y for arch/arm/boot/dts/renesas/' for 20250324-rzn1d400-eb-v4-1-d7ebbbad1918@bootlin.com: arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: cpus: cpu@1:enable-method:0: 'spin-table' was expected from schema $id: http://devicetree.org/schemas/cpus.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@40060000: compatible: 'oneOf' conditional failed, one must be fixed: ['renesas,r9a06g032-uart', 'renesas,rzn1-uart', 'snps,dw-apb-uart'] is too long 'renesas,r9a06g032-uart' is not one of ['brcm,bcm11351-dw-apb-uart', 'brcm,bcm21664-dw-apb-uart', 'rockchip,px30-uart', 'rockchip,rk1808-uart', 'rockchip,rk3036-uart', 'rockchip,rk3066-uart', 'rockchip,rk3128-uart', 'rockchip,rk3188-uart', 'rockchip,rk3288-uart', 'rockchip,rk3308-uart', 'rockchip,rk3328-uart', 'rockchip,rk3368-uart', 'rockchip,rk3399-uart', 'rockchip,rk3528-uart', 'rockchip,rk3568-uart', 'rockchip,rk3576-uart', 'rockchip,rk3588-uart', 'rockchip,rv1108-uart', 'rockchip,rv1126-uart', 'sophgo,sg2044-uart', 'starfive,jh7100-hsuart', 'starfive,jh7100-uart', 'starfive,jh7110-uart'] 'snps,dw-apb-uart' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@40060000: Unevaluated properties are not allowed ('compatible' was unexpected) from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@40061000: compatible: 'oneOf' conditional failed, one must be fixed: ['renesas,r9a06g032-uart', 'renesas,rzn1-uart', 'snps,dw-apb-uart'] is too long 'renesas,r9a06g032-uart' is not one of ['brcm,bcm11351-dw-apb-uart', 'brcm,bcm21664-dw-apb-uart', 'rockchip,px30-uart', 'rockchip,rk1808-uart', 'rockchip,rk3036-uart', 'rockchip,rk3066-uart', 'rockchip,rk3128-uart', 'rockchip,rk3188-uart', 'rockchip,rk3288-uart', 'rockchip,rk3308-uart', 'rockchip,rk3328-uart', 'rockchip,rk3368-uart', 'rockchip,rk3399-uart', 'rockchip,rk3528-uart', 'rockchip,rk3568-uart', 'rockchip,rk3576-uart', 'rockchip,rk3588-uart', 'rockchip,rv1108-uart', 'rockchip,rv1126-uart', 'sophgo,sg2044-uart', 'starfive,jh7100-hsuart', 'starfive,jh7100-uart', 'starfive,jh7110-uart'] 'snps,dw-apb-uart' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@40062000: compatible: 'oneOf' conditional failed, one must be fixed: ['renesas,r9a06g032-uart', 'renesas,rzn1-uart', 'snps,dw-apb-uart'] is too long 'renesas,r9a06g032-uart' is not one of ['brcm,bcm11351-dw-apb-uart', 'brcm,bcm21664-dw-apb-uart', 'rockchip,px30-uart', 'rockchip,rk1808-uart', 'rockchip,rk3036-uart', 'rockchip,rk3066-uart', 'rockchip,rk3128-uart', 'rockchip,rk3188-uart', 'rockchip,rk3288-uart', 'rockchip,rk3308-uart', 'rockchip,rk3328-uart', 'rockchip,rk3368-uart', 'rockchip,rk3399-uart', 'rockchip,rk3528-uart', 'rockchip,rk3568-uart', 'rockchip,rk3576-uart', 'rockchip,rk3588-uart', 'rockchip,rv1108-uart', 'rockchip,rv1126-uart', 'sophgo,sg2044-uart', 'starfive,jh7100-hsuart', 'starfive,jh7100-uart', 'starfive,jh7110-uart'] 'snps,dw-apb-uart' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50000000: dma-names:0: 'tx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50000000: dma-names:1: 'rx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50001000: dma-names:0: 'tx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50001000: dma-names:1: 'rx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50002000: dma-names:0: 'tx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50002000: dma-names:1: 'rx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50003000: dma-names:0: 'tx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50003000: dma-names:1: 'rx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50004000: dma-names:0: 'tx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50004000: dma-names:1: 'rx' was expected from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: switch@44050000: Unevaluated properties are not allowed ('#address-cells', '#size-cells' were unexpected) from schema $id: http://devicetree.org/schemas/net/dsa/renesas,rzn1-a5psw.yaml# ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree 2025-03-24 17:28 ` Rob Herring (Arm) @ 2025-03-24 18:36 ` Wolfram Sang 0 siblings, 0 replies; 8+ messages in thread From: Wolfram Sang @ 2025-03-24 18:36 UTC (permalink / raw) To: Rob Herring (Arm) Cc: Thomas Bonnefille, Thomas Petazzoni, Magnus Damm, Clément Léger, devicetree, Conor Dooley, linux-kernel, linux-renesas-soc, Krzysztof Kozlowski, Niklas Söderlund, Miquèl Raynal, Geert Uytterhoeven [-- Attachment #1: Type: text/plain, Size: 5655 bytes --] Hi all, > My bot found new DTB warnings on the .dts files added or changed in this > series. These warnings did exist before the patch and come from the underlying SoC-dtsi. > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: cpus: cpu@1:enable-method:0: 'spin-table' was expected > from schema $id: http://devicetree.org/schemas/cpus.yaml# This needs to be worked on (indipendently of this patch). > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@40060000: compatible: 'oneOf' conditional failed, one must be fixed: > ['renesas,r9a06g032-uart', 'renesas,rzn1-uart', 'snps,dw-apb-uart'] is too long > 'renesas,r9a06g032-uart' is not one of ['brcm,bcm11351-dw-apb-uart', 'brcm,bcm21664-dw-apb-uart', 'rockchip,px30-uart', 'rockchip,rk1808-uart', 'rockchip,rk3036-uart', 'rockchip,rk3066-uart', 'rockchip,rk3128-uart', 'rockchip,rk3188-uart', 'rockchip,rk3288-uart', 'rockchip,rk3308-uart', 'rockchip,rk3328-uart', 'rockchip,rk3368-uart', 'rockchip,rk3399-uart', 'rockchip,rk3528-uart', 'rockchip,rk3568-uart', 'rockchip,rk3576-uart', 'rockchip,rk3588-uart', 'rockchip,rv1108-uart', 'rockchip,rv1126-uart', 'sophgo,sg2044-uart', 'starfive,jh7100-hsuart', 'starfive,jh7100-uart', 'starfive,jh7110-uart'] > 'snps,dw-apb-uart' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@40060000: Unevaluated properties are not allowed ('compatible' was unexpected) > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@40061000: compatible: 'oneOf' conditional failed, one must be fixed: > ['renesas,r9a06g032-uart', 'renesas,rzn1-uart', 'snps,dw-apb-uart'] is too long > 'renesas,r9a06g032-uart' is not one of ['brcm,bcm11351-dw-apb-uart', 'brcm,bcm21664-dw-apb-uart', 'rockchip,px30-uart', 'rockchip,rk1808-uart', 'rockchip,rk3036-uart', 'rockchip,rk3066-uart', 'rockchip,rk3128-uart', 'rockchip,rk3188-uart', 'rockchip,rk3288-uart', 'rockchip,rk3308-uart', 'rockchip,rk3328-uart', 'rockchip,rk3368-uart', 'rockchip,rk3399-uart', 'rockchip,rk3528-uart', 'rockchip,rk3568-uart', 'rockchip,rk3576-uart', 'rockchip,rk3588-uart', 'rockchip,rv1108-uart', 'rockchip,rv1126-uart', 'sophgo,sg2044-uart', 'starfive,jh7100-hsuart', 'starfive,jh7100-uart', 'starfive,jh7110-uart'] > 'snps,dw-apb-uart' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@40062000: compatible: 'oneOf' conditional failed, one must be fixed: > ['renesas,r9a06g032-uart', 'renesas,rzn1-uart', 'snps,dw-apb-uart'] is too long > 'renesas,r9a06g032-uart' is not one of ['brcm,bcm11351-dw-apb-uart', 'brcm,bcm21664-dw-apb-uart', 'rockchip,px30-uart', 'rockchip,rk1808-uart', 'rockchip,rk3036-uart', 'rockchip,rk3066-uart', 'rockchip,rk3128-uart', 'rockchip,rk3188-uart', 'rockchip,rk3288-uart', 'rockchip,rk3308-uart', 'rockchip,rk3328-uart', 'rockchip,rk3368-uart', 'rockchip,rk3399-uart', 'rockchip,rk3528-uart', 'rockchip,rk3568-uart', 'rockchip,rk3576-uart', 'rockchip,rk3588-uart', 'rockchip,rv1108-uart', 'rockchip,rv1126-uart', 'sophgo,sg2044-uart', 'starfive,jh7100-hsuart', 'starfive,jh7100-uart', 'starfive,jh7110-uart'] > 'snps,dw-apb-uart' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50000000: dma-names:0: 'tx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50000000: dma-names:1: 'rx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50001000: dma-names:0: 'tx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50001000: dma-names:1: 'rx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50002000: dma-names:0: 'tx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50002000: dma-names:1: 'rx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50003000: dma-names:0: 'tx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50003000: dma-names:1: 'rx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50004000: dma-names:0: 'tx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: serial@50004000: dma-names:1: 'rx' was expected > from schema $id: http://devicetree.org/schemas/serial/snps-dw-apb-uart.yaml# For these, patches exist and are on the way. > arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dtb: switch@44050000: Unevaluated properties are not allowed ('#address-cells', '#size-cells' were unexpected) > from schema $id: http://devicetree.org/schemas/net/dsa/renesas,rzn1-a5psw.yaml# This still needs fixing. I don't think these are blockers for the patch here. Happy hacking, Wolfram [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree 2025-03-24 14:50 [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree Thomas Bonnefille 2025-03-24 17:28 ` Rob Herring (Arm) @ 2025-04-11 9:21 ` Geert Uytterhoeven 2025-04-11 9:53 ` Wolfram Sang 2025-04-11 10:22 ` Geert Uytterhoeven 1 sibling, 2 replies; 8+ messages in thread From: Geert Uytterhoeven @ 2025-04-11 9:21 UTC (permalink / raw) To: Thomas Bonnefille Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Thomas Petazzoni, Miquèl Raynal, linux-renesas-soc, devicetree, linux-kernel, Clément Léger, Wolfram Sang, Niklas Söderlund Hi Thomas, On Mon, 24 Mar 2025 at 15:51, Thomas Bonnefille <thomas.bonnefille@bootlin.com> wrote: > From: Clément Léger <clement.leger@bootlin.com> > > The EB board (Expansion board) supports both RZ/N1D and RZ-N1S. Since this > configuration targets only the RZ/N1D, it is named r9a06g032-rzn1d400-eb. > It adds support for the 2 additional switch ports (port C and D) that are > available on that board. > > Signed-off-by: Clément Léger <clement.leger@bootlin.com> > [Thomas: move the DTS to the Renesas directory, declare the PHY LEDs] > Signed-off-by: Thomas Bonnefille <thomas.bonnefille@bootlin.com> > Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > Tested-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > This short series adds support for the RZ/N1 Expansion Board. This board > is a carrier board on which a daughter board (either RZ/N1D or RZ/N1S) > can be plugged. The device-tree that is added by this series enables the > use to the 2 external switch ports that are present on this board. > --- > V4: > - Drop trailing whitespaces > > V3: > - Drop bindings commit as it was applied to master > - Move Makefile modification to arch/arm/boot/dts/renesas/Makefile > - Declare LEDs in PHY. > - Use the driver default LED configuration as there was no reason to > use a different one. Thanks for the update! > --- /dev/null > +++ b/arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dts > @@ -0,0 +1,120 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Device Tree Source for the RZN1D-EB Board > + * > + * Copyright (C) 2023 Schneider-Electric > + * > + */ > + > +#include "r9a06g032-rzn1d400-db.dts" > + > +/ { > + model = "RZN1D-EB Board"; > + compatible = "renesas,rzn1d400-eb", "renesas,rzn1d400-db", > + "renesas,r9a06g032"; > +}; > + > +&mii_conv2 { > + renesas,miic-input = <MIIC_SWITCH_PORTD>; > + status = "okay"; > +}; > + > +&mii_conv3 { > + renesas,miic-input = <MIIC_SWITCH_PORTC>; > + status = "okay"; > +}; > + > +&pinctrl{ Missing space. > + pins_eth1: pins-eth1 { > + pinmux = <RZN1_PINMUX(12, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(13, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(14, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(15, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(16, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(17, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(18, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(19, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(20, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(21, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(22, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(23, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>; > + drive-strength = <6>; > + bias-disable; > + }; > + > + pins_eth2: pins-eth2 { > + pinmux = <RZN1_PINMUX(24, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(25, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(26, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(27, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(28, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(29, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(30, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(31, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(32, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(33, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(34, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>, > + <RZN1_PINMUX(35, RZN1_FUNC_CLK_ETH_MII_RGMII_RMII)>; > + drive-strength = <6>; > + bias-disable; > + }; > +}; > + > +&switch { > + pinctrl-names = "default"; (from v2) No need to specify pinctrl-names, as it is inherited from r9a06g032-rzn1d400-db.dts. > + pinctrl-0 = <&pins_eth1>, <&pins_eth2>, <&pins_eth3>, <&pins_eth4>, > + <&pins_mdio1>; > + > + mdio { > + /* CN15 and CN16 switches must be configured in MDIO2 mode */ > + switch0phy1: ethernet-phy@1 { > + reg = <1>; > + leds { > + #address-cells = <1>; > + #size-cells = <0>; > + > + led@0 { > + reg = <0>; color = <LED_COLOR_ID_GREEN>; > + }; > + led@1 { > + reg = <1>; color = <LED_COLOR_ID_ORANGE>; > + }; The above should also have one of: function = LED_FUNCTION_LAN; function = LED_FUNCTION_SPEED_LAN; I don't know the LED function mapping. > + led@2 { > + reg = <2>; > + }; LED2/_INT is used as an interrupt pin, not as an LED. > + }; > + }; > + > + switch0phy10: ethernet-phy@10 { Same comments for this one. 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree 2025-04-11 9:21 ` Geert Uytterhoeven @ 2025-04-11 9:53 ` Wolfram Sang 2025-04-11 10:07 ` Geert Uytterhoeven 2025-04-11 10:22 ` Geert Uytterhoeven 1 sibling, 1 reply; 8+ messages in thread From: Wolfram Sang @ 2025-04-11 9:53 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Thomas Bonnefille, Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Thomas Petazzoni, Miquèl Raynal, linux-renesas-soc, devicetree, linux-kernel, Clément Léger, Niklas Söderlund [-- Attachment #1: Type: text/plain, Size: 1265 bytes --] > > + pinctrl-0 = <&pins_eth1>, <&pins_eth2>, <&pins_eth3>, <&pins_eth4>, > > + <&pins_mdio1>; > > + > > + mdio { > > + /* CN15 and CN16 switches must be configured in MDIO2 mode */ > > + switch0phy1: ethernet-phy@1 { > > + reg = <1>; > > + leds { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + led@0 { > > + reg = <0>; > > color = <LED_COLOR_ID_GREEN>; > > > + }; > > + led@1 { > > + reg = <1>; > > color = <LED_COLOR_ID_ORANGE>; > > > + }; > > The above should also have one of: > > function = LED_FUNCTION_LAN; > function = LED_FUNCTION_SPEED_LAN; > > I don't know the LED function mapping. I have an incremental fix for the LEDs to this patch. Thomas cannot really do it because he doesn't have the board. I was waiting with my patch until this patch is upstream, but I better send it out now, so you can squash it into this one? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree 2025-04-11 9:53 ` Wolfram Sang @ 2025-04-11 10:07 ` Geert Uytterhoeven 0 siblings, 0 replies; 8+ messages in thread From: Geert Uytterhoeven @ 2025-04-11 10:07 UTC (permalink / raw) To: Wolfram Sang, Geert Uytterhoeven, Thomas Bonnefille, Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Thomas Petazzoni, Miquèl Raynal, linux-renesas-soc, devicetree, linux-kernel, Clément Léger, Niklas Söderlund Hi Wolfram, On Fri, 11 Apr 2025 at 11:53, Wolfram Sang <wsa+renesas@sang-engineering.com> wrote: > > > + pinctrl-0 = <&pins_eth1>, <&pins_eth2>, <&pins_eth3>, <&pins_eth4>, > > > + <&pins_mdio1>; > > > + > > > + mdio { > > > + /* CN15 and CN16 switches must be configured in MDIO2 mode */ > > > + switch0phy1: ethernet-phy@1 { > > > + reg = <1>; > > > + leds { > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + > > > + led@0 { > > > + reg = <0>; > > > > color = <LED_COLOR_ID_GREEN>; > > > > > + }; > > > + led@1 { > > > + reg = <1>; > > > > color = <LED_COLOR_ID_ORANGE>; > > > > > + }; > > > > The above should also have one of: > > > > function = LED_FUNCTION_LAN; > > function = LED_FUNCTION_SPEED_LAN; > > > > I don't know the LED function mapping. > > I have an incremental fix for the LEDs to this patch. Thomas cannot > really do it because he doesn't have the board. I was waiting with my > patch until this patch is upstream, but I better send it out now, so you > can squash it into this one? Yes please.This patch is becoming too much of a dependency. Thanks! 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree 2025-04-11 9:21 ` Geert Uytterhoeven 2025-04-11 9:53 ` Wolfram Sang @ 2025-04-11 10:22 ` Geert Uytterhoeven 2025-04-11 10:38 ` Wolfram Sang 1 sibling, 1 reply; 8+ messages in thread From: Geert Uytterhoeven @ 2025-04-11 10:22 UTC (permalink / raw) To: Thomas Bonnefille Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Thomas Petazzoni, Miquèl Raynal, linux-renesas-soc, devicetree, linux-kernel, Clément Léger, Wolfram Sang, Niklas Söderlund On Fri, 11 Apr 2025 at 11:21, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > On Mon, 24 Mar 2025 at 15:51, Thomas Bonnefille > <thomas.bonnefille@bootlin.com> wrote: > > From: Clément Léger <clement.leger@bootlin.com> > > > > The EB board (Expansion board) supports both RZ/N1D and RZ-N1S. Since this > > configuration targets only the RZ/N1D, it is named r9a06g032-rzn1d400-eb. > > It adds support for the 2 additional switch ports (port C and D) that are > > available on that board. > > > > Signed-off-by: Clément Léger <clement.leger@bootlin.com> > > [Thomas: move the DTS to the Renesas directory, declare the PHY LEDs] > > Signed-off-by: Thomas Bonnefille <thomas.bonnefille@bootlin.com> > > Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > > Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > > Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > > Tested-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > > --- /dev/null > > +++ b/arch/arm/boot/dts/renesas/r9a06g032-rzn1d400-eb.dts > > +&pinctrl{ > > Missing space. > > > +&switch { > > + pinctrl-names = "default"; > > (from v2) No need to specify pinctrl-names, as it is inherited from > r9a06g032-rzn1d400-db.dts. I will fix these while folding in "[PATCH] ARM: dts: renesas: r9a06g032-rzn1d400-eb: correct LAN LED nodes" and queuing in renesas-devel for v6.16. 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree 2025-04-11 10:22 ` Geert Uytterhoeven @ 2025-04-11 10:38 ` Wolfram Sang 0 siblings, 0 replies; 8+ messages in thread From: Wolfram Sang @ 2025-04-11 10:38 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Thomas Bonnefille, Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Thomas Petazzoni, Miquèl Raynal, linux-renesas-soc, devicetree, linux-kernel, Clément Léger, Niklas Söderlund [-- Attachment #1: Type: text/plain, Size: 172 bytes --] > I will fix these while folding in "[PATCH] ARM: dts: renesas: > r9a06g032-rzn1d400-eb: correct LAN LED nodes" and queuing in > renesas-devel for v6.16. Thanks, Geert! [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2025-04-11 10:38 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2025-03-24 14:50 [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree Thomas Bonnefille 2025-03-24 17:28 ` Rob Herring (Arm) 2025-03-24 18:36 ` Wolfram Sang 2025-04-11 9:21 ` Geert Uytterhoeven 2025-04-11 9:53 ` Wolfram Sang 2025-04-11 10:07 ` Geert Uytterhoeven 2025-04-11 10:22 ` Geert Uytterhoeven 2025-04-11 10:38 ` Wolfram Sang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox