[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Zm+UTbGEhzbCgy1S@dragon>
Date: Mon, 17 Jun 2024 09:41:33 +0800
From: Shawn Guo <shawnguo2@...h.net>
To: Michael Walle <mwalle@...nel.org>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>, Li Yang <leoyang.li@....com>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>, Priit Laes <plaes@...es.org>,
Michael Grzeschik <m.grzeschik@...gutronix.de>,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Marco Felsch <m.felsch@...gutronix.de>, imx@...ts.linux.dev,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v2 09/13] ARM: dts: imx6qdl-kontron-samx6i: fix node names
On Thu, Jun 06, 2024 at 11:02:02AM +0200, Michael Walle wrote:
> Rename the nodes according to the schema. In particular, renamae the bit
s/renamae/rename
Shawn
> bang I2C controller to "i2c-N" and the mux nodes of the audmux to
> "mux-*". While at it, fix the typo "adu", which should have been "aud".
>
> Signed-off-by: Michael Walle <mwalle@...nel.org>
> ---
> .../boot/dts/nxp/imx/imx6qdl-kontron-samx6i.dtsi | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-kontron-samx6i.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-kontron-samx6i.dtsi
> index d36743e47552..85e5adac6ee9 100644
> --- a/arch/arm/boot/dts/nxp/imx/imx6qdl-kontron-samx6i.dtsi
> +++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-kontron-samx6i.dtsi
> @@ -149,7 +149,7 @@ lcd_backlight: lcd-backlight {
> status = "disabled";
> };
>
> - i2c_intern: i2c-gpio-intern {
> + i2c_intern: i2c-0 {
> compatible = "i2c-gpio";
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_i2c_gpio_intern>;
> @@ -160,7 +160,7 @@ i2c_intern: i2c-gpio-intern {
> #size-cells = <0>;
> };
>
> - i2c_lcd: i2c-gpio-lcd {
> + i2c_lcd: i2c-1 {
> compatible = "i2c-gpio";
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_i2c_gpio_lcd>;
> @@ -172,7 +172,7 @@ i2c_lcd: i2c-gpio-lcd {
> status = "disabled";
> };
>
> - i2c_cam: i2c-gpio-cam {
> + i2c_cam: i2c-2 {
> compatible = "i2c-gpio";
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_i2c_gpio_cam>;
> @@ -190,7 +190,7 @@ &audmux {
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_audmux>;
>
> - audmux_ssi1 {
> + mux-ssi1 {
> fsl,audmux-port = <MX51_AUDMUX_PORT1_SSI0>;
> fsl,port-config = <
> (IMX_AUDMUX_V2_PTCR_TFSEL(MX51_AUDMUX_PORT3) |
> @@ -202,7 +202,7 @@ IMX_AUDMUX_V2_PDCR_RXDSEL(MX51_AUDMUX_PORT3)
> >;
> };
>
> - audmux_adu3 {
> + mux-aud3 {
> fsl,audmux-port = <MX51_AUDMUX_PORT3>;
> fsl,port-config = <
> IMX_AUDMUX_V2_PTCR_SYN
> @@ -210,7 +210,7 @@ IMX_AUDMUX_V2_PDCR_RXDSEL(MX51_AUDMUX_PORT1_SSI0)
> >;
> };
>
> - audmux_ssi2 {
> + mux-ssi2 {
> fsl,audmux-port = <MX51_AUDMUX_PORT2_SSI1>;
> fsl,port-config = <
> (IMX_AUDMUX_V2_PTCR_TFSEL(MX51_AUDMUX_PORT4) |
> @@ -222,7 +222,7 @@ IMX_AUDMUX_V2_PDCR_RXDSEL(MX51_AUDMUX_PORT4)
> >;
> };
>
> - audmux_adu4 {
> + mux-aud4 {
> fsl,audmux-port = <MX51_AUDMUX_PORT4>;
> fsl,port-config = <
> IMX_AUDMUX_V2_PTCR_SYN
> --
> 2.39.2
>
Powered by blists - more mailing lists