* [PATCH 0/2] RZ/G2UL SMARC EVK trivial changes @ 2022-09-15 16:52 Prabhakar 2022-09-15 16:52 ` [PATCH 1/2] arm64: dts: renesas: rzg2ul-smarc: Include SoM DTSI into board DTS Prabhakar 2022-09-15 16:52 ` [PATCH 2/2] arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux Prabhakar 0 siblings, 2 replies; 9+ messages in thread From: Prabhakar @ 2022-09-15 16:52 UTC (permalink / raw) To: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski Cc: linux-renesas-soc, devicetree, linux-kernel, Prabhakar, Biju Das, Lad Prabhakar From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Hi All, This patch series includes changes to RZ/G2UL SMARC SoM and carrier board DTS/I so that it can be re-used by RZ/Five SMARC EVK. It does not have any functional changes. Cheers, Prabhakar Lad Prabhakar (2): arm64: dts: renesas: rzg2ul-smarc: Include SoM DTSI into board DTS arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux arch/arm64/boot/dts/renesas/r9a07g043u11-smarc.dts | 12 ++++++++++++ .../boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi | 13 +++++++++++++ arch/arm64/boot/dts/renesas/rzg2ul-smarc-som.dtsi | 7 +++++++ arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi | 11 ----------- 4 files changed, 32 insertions(+), 11 deletions(-) -- 2.25.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/2] arm64: dts: renesas: rzg2ul-smarc: Include SoM DTSI into board DTS 2022-09-15 16:52 [PATCH 0/2] RZ/G2UL SMARC EVK trivial changes Prabhakar @ 2022-09-15 16:52 ` Prabhakar 2022-09-18 9:20 ` Geert Uytterhoeven 2022-09-15 16:52 ` [PATCH 2/2] arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux Prabhakar 1 sibling, 1 reply; 9+ messages in thread From: Prabhakar @ 2022-09-15 16:52 UTC (permalink / raw) To: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski Cc: linux-renesas-soc, devicetree, linux-kernel, Prabhakar, Biju Das, Lad Prabhakar From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Move including the rzg2ul-smarc-som.dtsi from the carrier board rzg2ul-smarc.dtsi to the actual RZ/G2UL SMARC EVK board dts r9a07g043u11-smarc.dts. Also move the SW_SW0_DEV_SEL and SW_ET0_EN_N macros to board dts as they are used by SoM and carrier board DTS/I. This is in preparation of re-using the SoM and carrier board DTSIs for RZ/Five SMARC EVK. Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> --- arch/arm64/boot/dts/renesas/r9a07g043u11-smarc.dts | 12 ++++++++++++ arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi | 11 ----------- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git a/arch/arm64/boot/dts/renesas/r9a07g043u11-smarc.dts b/arch/arm64/boot/dts/renesas/r9a07g043u11-smarc.dts index 121e55282d18..059885a01ede 100644 --- a/arch/arm64/boot/dts/renesas/r9a07g043u11-smarc.dts +++ b/arch/arm64/boot/dts/renesas/r9a07g043u11-smarc.dts @@ -6,7 +6,19 @@ */ /dts-v1/; + +/* + * DIP-Switch SW1 setting + * 1 : High; 0: Low + * SW1-2 : SW_SD0_DEV_SEL (0: uSD; 1: eMMC) + * SW1-3 : SW_ET0_EN_N (0: ETHER0; 1: CAN0, CAN1, SSI1, RSPI1) + * Please change below macros according to SW1 setting on the SoM + */ +#define SW_SW0_DEV_SEL 1 +#define SW_ET0_EN_N 1 + #include "r9a07g043.dtsi" +#include "rzg2ul-smarc-som.dtsi" #include "rzg2ul-smarc.dtsi" / { diff --git a/arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi index f9835c12023e..2a1331ed1a5c 100644 --- a/arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi +++ b/arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi @@ -5,17 +5,6 @@ * Copyright (C) 2022 Renesas Electronics Corp. */ -/* - * DIP-Switch SW1 setting - * 1 : High; 0: Low - * SW1-2 : SW_SD0_DEV_SEL (0: uSD; 1: eMMC) - * SW1-3 : SW_ET0_EN_N (0: ETHER0; 1: CAN0, CAN1, SSI1, RSPI1) - * Please change below macros according to SW1 setting - */ -#define SW_SW0_DEV_SEL 1 -#define SW_ET0_EN_N 1 - -#include "rzg2ul-smarc-som.dtsi" #include "rzg2ul-smarc-pinfunction.dtsi" #include "rz-smarc-common.dtsi" -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] arm64: dts: renesas: rzg2ul-smarc: Include SoM DTSI into board DTS 2022-09-15 16:52 ` [PATCH 1/2] arm64: dts: renesas: rzg2ul-smarc: Include SoM DTSI into board DTS Prabhakar @ 2022-09-18 9:20 ` Geert Uytterhoeven 2022-09-19 8:28 ` Lad, Prabhakar 0 siblings, 1 reply; 9+ messages in thread From: Geert Uytterhoeven @ 2022-09-18 9:20 UTC (permalink / raw) To: Prabhakar Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, linux-renesas-soc, devicetree, linux-kernel, Biju Das, Lad Prabhakar Hi Prabhakar, On Thu, Sep 15, 2022 at 6:53 PM Prabhakar <prabhakar.csengg@gmail.com> wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > Move including the rzg2ul-smarc-som.dtsi from the carrier board > rzg2ul-smarc.dtsi to the actual RZ/G2UL SMARC EVK board dts > r9a07g043u11-smarc.dts. Also move the SW_SW0_DEV_SEL and > SW_ET0_EN_N macros to board dts as they are used by SoM and carrier > board DTS/I. > > This is in preparation of re-using the SoM and carrier board DTSIs > for RZ/Five SMARC EVK. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> i.e. will queue in renesas-devel for v6.1. For consistency, you may want to do the same with rzg2lc-smarc-som.dtsi. 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] 9+ messages in thread
* Re: [PATCH 1/2] arm64: dts: renesas: rzg2ul-smarc: Include SoM DTSI into board DTS 2022-09-18 9:20 ` Geert Uytterhoeven @ 2022-09-19 8:28 ` Lad, Prabhakar 0 siblings, 0 replies; 9+ messages in thread From: Lad, Prabhakar @ 2022-09-19 8:28 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, linux-renesas-soc, devicetree, linux-kernel, Biju Das, Lad Prabhakar Hi Geert, Thank you for the review. On Sun, Sep 18, 2022 at 10:21 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > Hi Prabhakar, > > On Thu, Sep 15, 2022 at 6:53 PM Prabhakar <prabhakar.csengg@gmail.com> wrote: > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > Move including the rzg2ul-smarc-som.dtsi from the carrier board > > rzg2ul-smarc.dtsi to the actual RZ/G2UL SMARC EVK board dts > > r9a07g043u11-smarc.dts. Also move the SW_SW0_DEV_SEL and > > SW_ET0_EN_N macros to board dts as they are used by SoM and carrier > > board DTS/I. > > > > This is in preparation of re-using the SoM and carrier board DTSIs > > for RZ/Five SMARC EVK. > > > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > i.e. will queue in renesas-devel for v6.1. > > For consistency, you may want to do the same with rzg2lc-smarc-som.dtsi. > Agreed, will do. Cheers, Prabhakar ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/2] arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux 2022-09-15 16:52 [PATCH 0/2] RZ/G2UL SMARC EVK trivial changes Prabhakar 2022-09-15 16:52 ` [PATCH 1/2] arm64: dts: renesas: rzg2ul-smarc: Include SoM DTSI into board DTS Prabhakar @ 2022-09-15 16:52 ` Prabhakar 2022-10-25 8:13 ` Geert Uytterhoeven 1 sibling, 1 reply; 9+ messages in thread From: Prabhakar @ 2022-09-15 16:52 UTC (permalink / raw) To: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski Cc: linux-renesas-soc, devicetree, linux-kernel, Prabhakar, Biju Das, Lad Prabhakar From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> In preparation to re-use the RZ/G2UL SMARC SoM and carrier DTS/I with the RZ/Five add /omit-if-no-ref/ keyword to pinmux entries as the support for RZ/Five SMARC EVK will be gradually added. Once we have full blown support for RZ/Five SMARC EVK we can get rid of the /omit-if-no-ref/ keyword. Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> --- .../boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi | 13 +++++++++++++ arch/arm64/boot/dts/renesas/rzg2ul-smarc-som.dtsi | 7 +++++++ 2 files changed, 20 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi b/arch/arm64/boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi index bd8bc858c28c..2b400100472e 100644 --- a/arch/arm64/boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi +++ b/arch/arm64/boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi @@ -12,12 +12,14 @@ &pinctrl { pinctrl-0 = <&sound_clk_pins>; pinctrl-names = "default"; + /omit-if-no-ref/ can0_pins: can0 { pinmux = <RZG2L_PORT_PINMUX(1, 1, 3)>, /* TX */ <RZG2L_PORT_PINMUX(1, 2, 3)>; /* RX */ }; #if (SW_ET0_EN_N) + /omit-if-no-ref/ can0-stb-hog { gpio-hog; gpios = <RZG2L_GPIO(2, 2) GPIO_ACTIVE_HIGH>; @@ -26,12 +28,14 @@ can0-stb-hog { }; #endif + /omit-if-no-ref/ can1_pins: can1 { pinmux = <RZG2L_PORT_PINMUX(2, 0, 3)>, /* TX */ <RZG2L_PORT_PINMUX(2, 1, 3)>; /* RX */ }; #if (SW_ET0_EN_N) + /omit-if-no-ref/ can1-stb-hog { gpio-hog; gpios = <RZG2L_GPIO(2, 3) GPIO_ACTIVE_HIGH>; @@ -40,11 +44,13 @@ can1-stb-hog { }; #endif + /omit-if-no-ref/ i2c0_pins: i2c0 { pins = "RIIC0_SDA", "RIIC0_SCL"; input-enable; }; + /omit-if-no-ref/ i2c1_pins: i2c1 { pins = "RIIC1_SDA", "RIIC1_SCL"; input-enable; @@ -55,6 +61,7 @@ scif0_pins: scif0 { <RZG2L_PORT_PINMUX(6, 3, 6)>; /* RxD */ }; + /omit-if-no-ref/ sd1-pwr-en-hog { gpio-hog; gpios = <RZG2L_GPIO(0, 3) GPIO_ACTIVE_HIGH>; @@ -62,6 +69,7 @@ sd1-pwr-en-hog { line-name = "sd1_pwr_en"; }; + /omit-if-no-ref/ sdhi1_pins: sd1 { sd1_data { pins = "SD1_DATA0", "SD1_DATA1", "SD1_DATA2", "SD1_DATA3"; @@ -78,6 +86,7 @@ sd1_mux { }; }; + /omit-if-no-ref/ sdhi1_pins_uhs: sd1_uhs { sd1_data_uhs { pins = "SD1_DATA0", "SD1_DATA1", "SD1_DATA2", "SD1_DATA3"; @@ -94,11 +103,13 @@ sd1_mux_uhs { }; }; + /omit-if-no-ref/ sound_clk_pins: sound_clk { pins = "AUDIO_CLK1", "AUDIO_CLK2"; input-enable; }; + /omit-if-no-ref/ ssi1_pins: ssi1 { pinmux = <RZG2L_PORT_PINMUX(3, 0, 2)>, /* BCK */ <RZG2L_PORT_PINMUX(3, 1, 2)>, /* RCK */ @@ -106,12 +117,14 @@ ssi1_pins: ssi1 { <RZG2L_PORT_PINMUX(3, 3, 2)>; /* RXD */ }; + /omit-if-no-ref/ usb0_pins: usb0 { pinmux = <RZG2L_PORT_PINMUX(5, 0, 1)>, /* VBUS */ <RZG2L_PORT_PINMUX(5, 2, 1)>, /* OVC */ <RZG2L_PORT_PINMUX(5, 3, 1)>; /* OTG_ID */ }; + /omit-if-no-ref/ usb1_pins: usb1 { pinmux = <RZG2L_PORT_PINMUX(5, 4, 5)>, /* OVC */ <RZG2L_PORT_PINMUX(6, 0, 1)>; /* VBUS */ diff --git a/arch/arm64/boot/dts/renesas/rzg2ul-smarc-som.dtsi b/arch/arm64/boot/dts/renesas/rzg2ul-smarc-som.dtsi index 2a0feb53f0dc..a0a41a062f55 100644 --- a/arch/arm64/boot/dts/renesas/rzg2ul-smarc-som.dtsi +++ b/arch/arm64/boot/dts/renesas/rzg2ul-smarc-som.dtsi @@ -132,10 +132,12 @@ &ostm2 { }; &pinctrl { + /omit-if-no-ref/ adc_pins: adc { pinmux = <RZG2L_PORT_PINMUX(6, 2, 1)>; /* ADC_TRG */ }; + /omit-if-no-ref/ eth0_pins: eth0 { pinmux = <RZG2L_PORT_PINMUX(4, 5, 1)>, /* ET0_LINKSTA */ <RZG2L_PORT_PINMUX(4, 3, 1)>, /* ET0_MDC */ @@ -154,6 +156,7 @@ eth0_pins: eth0 { <RZG2L_PORT_PINMUX(4, 1, 1)>; /* ET0_RXD3 */ }; + /omit-if-no-ref/ eth1_pins: eth1 { pinmux = <RZG2L_PORT_PINMUX(10, 4, 1)>, /* ET1_LINKSTA */ <RZG2L_PORT_PINMUX(10, 2, 1)>, /* ET1_MDC */ @@ -172,6 +175,7 @@ eth1_pins: eth1 { <RZG2L_PORT_PINMUX(10, 0, 1)>; /* ET1_RXD3 */ }; + /omit-if-no-ref/ sdhi0_emmc_pins: sd0emmc { sd0_emmc_data { pins = "SD0_DATA0", "SD0_DATA1", "SD0_DATA2", "SD0_DATA3", @@ -190,6 +194,7 @@ sd0_emmc_rst { }; }; + /omit-if-no-ref/ sdhi0_pins: sd0 { sd0_data { pins = "SD0_DATA0", "SD0_DATA1", "SD0_DATA2", "SD0_DATA3"; @@ -206,6 +211,7 @@ sd0_mux { }; }; + /omit-if-no-ref/ sdhi0_pins_uhs: sd0_uhs { sd0_data_uhs { pins = "SD0_DATA0", "SD0_DATA1", "SD0_DATA2", "SD0_DATA3"; @@ -222,6 +228,7 @@ sd0_mux_uhs { }; }; + /omit-if-no-ref/ spi1_pins: rspi1 { pinmux = <RZG2L_PORT_PINMUX(4, 0, 2)>, /* CK */ <RZG2L_PORT_PINMUX(4, 1, 2)>, /* MOSI */ -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux 2022-09-15 16:52 ` [PATCH 2/2] arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux Prabhakar @ 2022-10-25 8:13 ` Geert Uytterhoeven 2022-10-25 22:38 ` Lad, Prabhakar 0 siblings, 1 reply; 9+ messages in thread From: Geert Uytterhoeven @ 2022-10-25 8:13 UTC (permalink / raw) To: Prabhakar Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, linux-renesas-soc, devicetree, linux-kernel, Biju Das, Lad Prabhakar Hi Prabhakar, On Thu, Sep 15, 2022 at 6:53 PM Prabhakar <prabhakar.csengg@gmail.com> wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > In preparation to re-use the RZ/G2UL SMARC SoM and carrier DTS/I with the > RZ/Five add /omit-if-no-ref/ keyword to pinmux entries as the support for > RZ/Five SMARC EVK will be gradually added. > > Once we have full blown support for RZ/Five SMARC EVK we can get rid of > the /omit-if-no-ref/ keyword. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Thanks for your patch! I finally had a deeper look at this... Why do you want to disable these nodes? While they are indeed not used yet on RZ/Five, they are valid hardware descriptions for the RZ/Five SMARC EVK, and their presence doesn't harm anything. I do see a valid use case for marking pin control subnodes with /omit-if-no-ref/: you can provide all possible configurations as a convenience for the user, so the user no longer has to look up the numeric parameters of the RZG2L_PORT_PINMUX() macros. But IMHO those would belong in the SoC-specific .dtsi, not in a board .dtsi. See e.g. the massive use of /omit-if-no-ref/ in sunxi and rockchip .dtsi files. Am I missing something? > --- a/arch/arm64/boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi > +++ b/arch/arm64/boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi > @@ -12,12 +12,14 @@ &pinctrl { > pinctrl-0 = <&sound_clk_pins>; ^^^^^^^^^^^^^^ > pinctrl-names = "default"; > > + /omit-if-no-ref/ > can0_pins: can0 { > pinmux = <RZG2L_PORT_PINMUX(1, 1, 3)>, /* TX */ > <RZG2L_PORT_PINMUX(1, 2, 3)>; /* RX */ > }; > > #if (SW_ET0_EN_N) > + /omit-if-no-ref/ > can0-stb-hog { > gpio-hog; > gpios = <RZG2L_GPIO(2, 2) GPIO_ACTIVE_HIGH>; > @@ -94,11 +103,13 @@ sd1_mux_uhs { > }; > }; > > + /omit-if-no-ref/ > sound_clk_pins: sound_clk { FTR, this one is always referenced. > pins = "AUDIO_CLK1", "AUDIO_CLK2"; > input-enable; > }; 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] 9+ messages in thread
* Re: [PATCH 2/2] arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux 2022-10-25 8:13 ` Geert Uytterhoeven @ 2022-10-25 22:38 ` Lad, Prabhakar 2022-10-26 7:28 ` Geert Uytterhoeven 0 siblings, 1 reply; 9+ messages in thread From: Lad, Prabhakar @ 2022-10-25 22:38 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, linux-renesas-soc, devicetree, linux-kernel, Biju Das, Lad Prabhakar Hi Geert, Thank you for the review. On Tue, Oct 25, 2022 at 9:13 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > Hi Prabhakar, > > On Thu, Sep 15, 2022 at 6:53 PM Prabhakar <prabhakar.csengg@gmail.com> wrote: > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > In preparation to re-use the RZ/G2UL SMARC SoM and carrier DTS/I with the > > RZ/Five add /omit-if-no-ref/ keyword to pinmux entries as the support for > > RZ/Five SMARC EVK will be gradually added. > > > > Once we have full blown support for RZ/Five SMARC EVK we can get rid of > > the /omit-if-no-ref/ keyword. > > > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > Thanks for your patch! > > I finally had a deeper look at this... > > Why do you want to disable these nodes? While they are indeed not > used yet on RZ/Five, they are valid hardware descriptions for the > RZ/Five SMARC EVK, and their presence doesn't harm anything. > > I do see a valid use case for marking pin control subnodes with > /omit-if-no-ref/: you can provide all possible configurations as a > convenience for the user, so the user no longer has to look up the > numeric parameters of the RZG2L_PORT_PINMUX() macros. > But IMHO those would belong in the SoC-specific .dtsi, not in a > board .dtsi. See e.g. the massive use of /omit-if-no-ref/ in sunxi > and rockchip .dtsi files. > > Am I missing something? > My intention was to keep the DTB as minimal as possible so that it includes just the required pinmuxes which were enabled on the RZ/Five. For example [0], [1] we do delete the pinctrl for the nodes which are marked as disabled. Do you think we should drop it? But now that things are falling in place for RZ/Five we can ignore this patch. [0] arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi [1] arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi Cheers, Prabhakar > > --- a/arch/arm64/boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi > > +++ b/arch/arm64/boot/dts/renesas/rzg2ul-smarc-pinfunction.dtsi > > @@ -12,12 +12,14 @@ &pinctrl { > > pinctrl-0 = <&sound_clk_pins>; > ^^^^^^^^^^^^^^ > > pinctrl-names = "default"; > > > > + /omit-if-no-ref/ > > can0_pins: can0 { > > pinmux = <RZG2L_PORT_PINMUX(1, 1, 3)>, /* TX */ > > <RZG2L_PORT_PINMUX(1, 2, 3)>; /* RX */ > > }; > > > > #if (SW_ET0_EN_N) > > + /omit-if-no-ref/ > > can0-stb-hog { > > gpio-hog; > > gpios = <RZG2L_GPIO(2, 2) GPIO_ACTIVE_HIGH>; > > > @@ -94,11 +103,13 @@ sd1_mux_uhs { > > }; > > }; > > > > + /omit-if-no-ref/ > > sound_clk_pins: sound_clk { > > FTR, this one is always referenced. > > > pins = "AUDIO_CLK1", "AUDIO_CLK2"; > > input-enable; > > }; > > 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] 9+ messages in thread
* Re: [PATCH 2/2] arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux 2022-10-25 22:38 ` Lad, Prabhakar @ 2022-10-26 7:28 ` Geert Uytterhoeven 2022-10-26 8:46 ` Lad, Prabhakar 0 siblings, 1 reply; 9+ messages in thread From: Geert Uytterhoeven @ 2022-10-26 7:28 UTC (permalink / raw) To: Lad, Prabhakar Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, linux-renesas-soc, devicetree, linux-kernel, Biju Das, Lad Prabhakar Hi Prabhakar, On Wed, Oct 26, 2022 at 12:39 AM Lad, Prabhakar <prabhakar.csengg@gmail.com> wrote: > On Tue, Oct 25, 2022 at 9:13 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > On Thu, Sep 15, 2022 at 6:53 PM Prabhakar <prabhakar.csengg@gmail.com> wrote: > > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > > > In preparation to re-use the RZ/G2UL SMARC SoM and carrier DTS/I with the > > > RZ/Five add /omit-if-no-ref/ keyword to pinmux entries as the support for > > > RZ/Five SMARC EVK will be gradually added. > > > > > > Once we have full blown support for RZ/Five SMARC EVK we can get rid of > > > the /omit-if-no-ref/ keyword. > > > > > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > Thanks for your patch! > > > > I finally had a deeper look at this... > > > > Why do you want to disable these nodes? While they are indeed not > > used yet on RZ/Five, they are valid hardware descriptions for the > > RZ/Five SMARC EVK, and their presence doesn't harm anything. > > > > I do see a valid use case for marking pin control subnodes with > > /omit-if-no-ref/: you can provide all possible configurations as a > > convenience for the user, so the user no longer has to look up the > > numeric parameters of the RZG2L_PORT_PINMUX() macros. > > But IMHO those would belong in the SoC-specific .dtsi, not in a > > board .dtsi. See e.g. the massive use of /omit-if-no-ref/ in sunxi > > and rockchip .dtsi files. > > > > Am I missing something? > > > My intention was to keep the DTB as minimal as possible so that it > includes just the required pinmuxes which were enabled on the RZ/Five. > For example [0], [1] we do delete the pinctrl for the nodes which are > marked as disabled. Do you think we should drop it? You mean /delete-property/ pinctrl-0; /delete-property/ pinctrl-names; ? These do not delete pinctrl subnodes, but pinctrl properties in disabled device nodes pointing to pinctrl subnodes. The actual pinctrl subnodes are still present. > But now that things are falling in place for RZ/Five we can ignore this patch. Agreed. > [0] arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi > [1] arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi 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] 9+ messages in thread
* Re: [PATCH 2/2] arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux 2022-10-26 7:28 ` Geert Uytterhoeven @ 2022-10-26 8:46 ` Lad, Prabhakar 0 siblings, 0 replies; 9+ messages in thread From: Lad, Prabhakar @ 2022-10-26 8:46 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, linux-renesas-soc, devicetree, linux-kernel, Biju Das, Lad Prabhakar Hi Geert, On Wed, Oct 26, 2022 at 8:29 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > Hi Prabhakar, > > On Wed, Oct 26, 2022 at 12:39 AM Lad, Prabhakar > <prabhakar.csengg@gmail.com> wrote: > > On Tue, Oct 25, 2022 at 9:13 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > > On Thu, Sep 15, 2022 at 6:53 PM Prabhakar <prabhakar.csengg@gmail.com> wrote: > > > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > > > > > In preparation to re-use the RZ/G2UL SMARC SoM and carrier DTS/I with the > > > > RZ/Five add /omit-if-no-ref/ keyword to pinmux entries as the support for > > > > RZ/Five SMARC EVK will be gradually added. > > > > > > > > Once we have full blown support for RZ/Five SMARC EVK we can get rid of > > > > the /omit-if-no-ref/ keyword. > > > > > > > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > > > Thanks for your patch! > > > > > > I finally had a deeper look at this... > > > > > > Why do you want to disable these nodes? While they are indeed not > > > used yet on RZ/Five, they are valid hardware descriptions for the > > > RZ/Five SMARC EVK, and their presence doesn't harm anything. > > > > > > I do see a valid use case for marking pin control subnodes with > > > /omit-if-no-ref/: you can provide all possible configurations as a > > > convenience for the user, so the user no longer has to look up the > > > numeric parameters of the RZG2L_PORT_PINMUX() macros. > > > But IMHO those would belong in the SoC-specific .dtsi, not in a > > > board .dtsi. See e.g. the massive use of /omit-if-no-ref/ in sunxi > > > and rockchip .dtsi files. > > > > > > Am I missing something? > > > > > My intention was to keep the DTB as minimal as possible so that it > > includes just the required pinmuxes which were enabled on the RZ/Five. > > For example [0], [1] we do delete the pinctrl for the nodes which are > > marked as disabled. Do you think we should drop it? > > You mean > > /delete-property/ pinctrl-0; > /delete-property/ pinctrl-names; > > ? > These do not delete pinctrl subnodes, but pinctrl properties in disabled > device nodes pointing to pinctrl subnodes. The actual pinctrl subnodes > are still present. > Ahh right the pinctrl subnodes will still remain. Cheers, Prabhakar ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-10-26 8:47 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-09-15 16:52 [PATCH 0/2] RZ/G2UL SMARC EVK trivial changes Prabhakar 2022-09-15 16:52 ` [PATCH 1/2] arm64: dts: renesas: rzg2ul-smarc: Include SoM DTSI into board DTS Prabhakar 2022-09-18 9:20 ` Geert Uytterhoeven 2022-09-19 8:28 ` Lad, Prabhakar 2022-09-15 16:52 ` [PATCH 2/2] arm64: dts: renesas: rzg2ul-smarc: Add /omit-if-no-ref/ to pinmux Prabhakar 2022-10-25 8:13 ` Geert Uytterhoeven 2022-10-25 22:38 ` Lad, Prabhakar 2022-10-26 7:28 ` Geert Uytterhoeven 2022-10-26 8:46 ` Lad, Prabhakar
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).