From: "Sébastien Szymanski" <sebastien.szymanski@armadeus.com>
To: Stefan Wahren <stefan.wahren@i2se.com>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Shawn Guo <shawnguo@kernel.org>,
Sascha Hauer <s.hauer@pengutronix.de>,
Fabio Estevam <festevam@gmail.com>
Cc: linux-imx@nxp.com, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/9] ARM: dts: imx28-apf28: Convert to use label references
Date: Mon, 13 Feb 2023 09:22:34 +0100 [thread overview]
Message-ID: <58919223-cd4e-9a54-52ab-90f86552d4e2@armadeus.com> (raw)
In-Reply-To: <20230201203338.9525-2-stefan.wahren@i2se.com>
Hello Stefan,
On 2/1/23 21:33, Stefan Wahren wrote:
> These Armadeus board files still use node name and unit address
> to reference parts from the imx28.dtsi . This causes a lot of
> redundancy. So use label references in order to make it easier
> to maintain.
>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
> Cc: <support@armadeus.com>
> ---
> arch/arm/boot/dts/imx28-apf28.dts | 96 ++++-----
> arch/arm/boot/dts/imx28-apf28dev.dts | 312 +++++++++++++--------------
> 2 files changed, 196 insertions(+), 212 deletions(-)
>
> diff --git a/arch/arm/boot/dts/imx28-apf28.dts b/arch/arm/boot/dts/imx28-apf28.dts
> index 14a92fe59770..98672932e41b 100644
> --- a/arch/arm/boot/dts/imx28-apf28.dts
> +++ b/arch/arm/boot/dts/imx28-apf28.dts
> @@ -14,67 +14,59 @@ memory@40000000 {
> device_type = "memory";
> reg = <0x40000000 0x08000000>;
> };
> +};
>
> - apb@80000000 {
> - apbh@80000000 {
> - nand-controller@8000c000 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&gpmi_pins_a &gpmi_status_cfg>;
> - status = "okay";
> -
> - partition@0 {
> - label = "u-boot";
> - reg = <0x0 0x300000>;
> - };
> +&duart {
> + pinctrl-names = "default";
> + pinctrl-0 = <&duart_pins_a>;
> + status = "okay";
> +};
>
> - partition@300000 {
> - label = "env";
> - reg = <0x300000 0x80000>;
> - };
> +&gpmi {
> + pinctrl-names = "default";
> + pinctrl-0 = <&gpmi_pins_a &gpmi_status_cfg>;
> + status = "okay";
>
> - partition@380000 {
> - label = "env2";
> - reg = <0x380000 0x80000>;
> - };
> + partition@0 {
> + label = "u-boot";
> + reg = <0x0 0x300000>;
> + };
>
> - partition@400000 {
> - label = "dtb";
> - reg = <0x400000 0x80000>;
> - };
> + partition@300000 {
> + label = "env";
> + reg = <0x300000 0x80000>;
> + };
>
> - partition@480000 {
> - label = "splash";
> - reg = <0x480000 0x80000>;
> - };
> + partition@380000 {
> + label = "env2";
> + reg = <0x380000 0x80000>;
> + };
>
> - partition@500000 {
> - label = "kernel";
> - reg = <0x500000 0x800000>;
> - };
> + partition@400000 {
> + label = "dtb";
> + reg = <0x400000 0x80000>;
> + };
>
> - partition@d00000 {
> - label = "rootfs";
> - reg = <0xd00000 0xf300000>;
> - };
> - };
> - };
> + partition@480000 {
> + label = "splash";
> + reg = <0x480000 0x80000>;
> + };
>
> - apbx@80040000 {
> - duart: serial@80074000 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&duart_pins_a>;
> - status = "okay";
> - };
> - };
> + partition@500000 {
> + label = "kernel";
> + reg = <0x500000 0x800000>;
> };
>
> - ahb@80080000 {
> - mac0: ethernet@800f0000 {
> - phy-mode = "rmii";
> - pinctrl-names = "default";
> - pinctrl-0 = <&mac0_pins_a>;
> - phy-reset-gpios = <&gpio4 13 GPIO_ACTIVE_LOW>;
> - status = "okay";
> - };
> + partition@d00000 {
> + label = "rootfs";
> + reg = <0xd00000 0xf300000>;
> };
> };
> +
> +&mac0 {
> + phy-mode = "rmii";
> + pinctrl-names = "default";
> + pinctrl-0 = <&mac0_pins_a>;
> + phy-reset-gpios = <&gpio4 13 GPIO_ACTIVE_LOW>;
> + status = "okay";
> +};
> diff --git a/arch/arm/boot/dts/imx28-apf28dev.dts b/arch/arm/boot/dts/imx28-apf28dev.dts
> index 1b253b47006c..47e2397193ad 100644
> --- a/arch/arm/boot/dts/imx28-apf28dev.dts
> +++ b/arch/arm/boot/dts/imx28-apf28dev.dts
> @@ -10,166 +10,6 @@ / {
> model = "Armadeus Systems APF28Dev docking/development board";
> compatible = "armadeus,imx28-apf28dev", "armadeus,imx28-apf28", "fsl,imx28";
>
> - apb@80000000 {
> - apbh@80000000 {
> - ssp0: spi@80010000 {
> - compatible = "fsl,imx28-mmc";
> - pinctrl-names = "default";
> - pinctrl-0 = <&mmc0_4bit_pins_a
> - &mmc0_cd_cfg &mmc0_sck_cfg>;
> - bus-width = <4>;
> - status = "okay";
> - };
> -
> - ssp2: spi@80014000 {
> - compatible = "fsl,imx28-spi";
> - pinctrl-names = "default";
> - pinctrl-0 = <&spi2_pins_a>;
> - status = "okay";
> - };
> -
> - pinctrl@80018000 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&hog_pins_apf28dev>;
> -
> - hog_pins_apf28dev: hog@0 {
> - reg = <0>;
> - fsl,pinmux-ids = <
> - MX28_PAD_LCD_D16__GPIO_1_16
> - MX28_PAD_LCD_D17__GPIO_1_17
> - MX28_PAD_LCD_D18__GPIO_1_18
> - MX28_PAD_LCD_D19__GPIO_1_19
> - MX28_PAD_LCD_D20__GPIO_1_20
> - MX28_PAD_LCD_D21__GPIO_1_21
> - MX28_PAD_LCD_D22__GPIO_1_22
> - MX28_PAD_GPMI_CE1N__GPIO_0_17
> - >;
> - fsl,drive-strength = <MXS_DRIVE_4mA>;
> - fsl,voltage = <MXS_VOLTAGE_HIGH>;
> - fsl,pull-up = <MXS_PULL_DISABLE>;
> - };
> -
> - lcdif_pins_apf28dev: lcdif-apf28dev@0 {
> - reg = <0>;
> - fsl,pinmux-ids = <
> - MX28_PAD_LCD_RD_E__LCD_VSYNC
> - MX28_PAD_LCD_WR_RWN__LCD_HSYNC
> - MX28_PAD_LCD_RS__LCD_DOTCLK
> - MX28_PAD_LCD_CS__LCD_ENABLE
> - >;
> - fsl,drive-strength = <MXS_DRIVE_4mA>;
> - fsl,voltage = <MXS_VOLTAGE_HIGH>;
> - fsl,pull-up = <MXS_PULL_DISABLE>;
> - };
> -
> - usb0_otg_apf28dev: otg-apf28dev@0 {
> - reg = <0>;
> - fsl,pinmux-ids = <
> - MX28_PAD_LCD_D23__GPIO_1_23
> - >;
> - fsl,drive-strength = <MXS_DRIVE_4mA>;
> - fsl,voltage = <MXS_VOLTAGE_HIGH>;
> - fsl,pull-up = <MXS_PULL_DISABLE>;
> - };
> - };
> -
> - lcdif@80030000 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&lcdif_16bit_pins_a
> - &lcdif_pins_apf28dev>;
> - display = <&display0>;
> - status = "okay";
> -
> - display0: display0 {
> - bits-per-pixel = <16>;
> - bus-width = <16>;
> -
> - display-timings {
> - native-mode = <&timing0>;
> - timing0: timing0 {
> - clock-frequency = <33000033>;
> - hactive = <800>;
> - vactive = <480>;
> - hback-porch = <96>;
> - hfront-porch = <96>;
> - vback-porch = <20>;
> - vfront-porch = <21>;
> - hsync-len = <64>;
> - vsync-len = <4>;
> - hsync-active = <1>;
> - vsync-active = <1>;
> - de-active = <1>;
> - pixelclk-active = <0>;
> - };
> - };
> - };
> - };
> -
> - can0: can@80032000 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&can0_pins_a>;
> - xceiver-supply = <®_can0_vcc>;
> - status = "okay";
> - };
> - };
> -
> - apbx@80040000 {
> - lradc@80050000 {
> - fsl,lradc-touchscreen-wires = <4>;
> - status = "okay";
> - };
> -
> - i2c0: i2c@80058000 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&i2c0_pins_a>;
> - status = "okay";
> - };
> -
> - pwm: pwm@80064000 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&pwm3_pins_a &pwm4_pins_a>;
> - status = "okay";
> - };
> -
> - auart0: serial@8006a000 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&auart0_pins_a>;
> - uart-has-rtscts;
> - status = "okay";
> - };
> -
> - usbphy0: usbphy@8007c000 {
> - status = "okay";
> - };
> -
> - usbphy1: usbphy@8007e000 {
> - status = "okay";
> - };
> - };
> - };
> -
> - ahb@80080000 {
> - usb0: usb@80080000 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&usb0_otg_apf28dev
> - &usb0_id_pins_b>;
> - vbus-supply = <®_usb0_vbus>;
> - status = "okay";
> - };
> -
> - usb1: usb@80090000 {
> - status = "okay";
> - };
> -
> - mac1: ethernet@800f4000 {
> - phy-mode = "rmii";
> - pinctrl-names = "default";
> - pinctrl-0 = <&mac1_pins_a>;
> - phy-reset-gpios = <&gpio1 29 GPIO_ACTIVE_LOW>;
> - status = "okay";
> - };
> - };
> -
> regulators {
> compatible = "simple-bus";
> #address-cells = <1>;
> @@ -223,3 +63,155 @@ user-button {
> };
> };
> };
> +
> +&auart0 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&auart0_pins_a>;
> + uart-has-rtscts;
> + status = "okay";
> +};
> +
> +&can0 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&can0_pins_a>;
> + xceiver-supply = <®_can0_vcc>;
> + status = "okay";
> +};
> +
> +&lcdif {
> + pinctrl-names = "default";
> + pinctrl-0 = <&lcdif_16bit_pins_a
> + &lcdif_pins_apf28dev>;
> + display = <&display0>;
> + status = "okay";
> +
> + display0: display0 {
> + bits-per-pixel = <16>;
> + bus-width = <16>;
> +
> + display-timings {
> + native-mode = <&timing0>;
> + timing0: timing0 {
> + clock-frequency = <33000033>;
> + hactive = <800>;
> + vactive = <480>;
> + hback-porch = <96>;
> + hfront-porch = <96>;
> + vback-porch = <20>;
> + vfront-porch = <21>;
> + hsync-len = <64>;
> + vsync-len = <4>;
> + hsync-active = <1>;
> + vsync-active = <1>;
> + de-active = <1>;
> + pixelclk-active = <0>;
> + };
> + };
> + };
> +};
> +
> +&lradc {
> + fsl,lradc-touchscreen-wires = <4>;
> + status = "okay";
> +};
> +
> +&i2c0 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&i2c0_pins_a>;
> + status = "okay";
> +};
This i2c0 node should be before the lcdif node to have the nodes in
alphabetical order.
Regards,
> +
> +&mac1 {
> + phy-mode = "rmii";
> + pinctrl-names = "default";
> + pinctrl-0 = <&mac1_pins_a>;
> + phy-reset-gpios = <&gpio1 29 GPIO_ACTIVE_LOW>;
> + status = "okay";
> +};
> +
> +&pinctrl {
> + pinctrl-names = "default";
> + pinctrl-0 = <&hog_pins_apf28dev>;
> +
> + hog_pins_apf28dev: hog@0 {
> + reg = <0>;
> + fsl,pinmux-ids = <
> + MX28_PAD_LCD_D16__GPIO_1_16
> + MX28_PAD_LCD_D17__GPIO_1_17
> + MX28_PAD_LCD_D18__GPIO_1_18
> + MX28_PAD_LCD_D19__GPIO_1_19
> + MX28_PAD_LCD_D20__GPIO_1_20
> + MX28_PAD_LCD_D21__GPIO_1_21
> + MX28_PAD_LCD_D22__GPIO_1_22
> + MX28_PAD_GPMI_CE1N__GPIO_0_17
> + >;
> + fsl,drive-strength = <MXS_DRIVE_4mA>;
> + fsl,voltage = <MXS_VOLTAGE_HIGH>;
> + fsl,pull-up = <MXS_PULL_DISABLE>;
> + };
> +
> + lcdif_pins_apf28dev: lcdif-apf28dev@0 {
> + reg = <0>;
> + fsl,pinmux-ids = <
> + MX28_PAD_LCD_RD_E__LCD_VSYNC
> + MX28_PAD_LCD_WR_RWN__LCD_HSYNC
> + MX28_PAD_LCD_RS__LCD_DOTCLK
> + MX28_PAD_LCD_CS__LCD_ENABLE
> + >;
> + fsl,drive-strength = <MXS_DRIVE_4mA>;
> + fsl,voltage = <MXS_VOLTAGE_HIGH>;
> + fsl,pull-up = <MXS_PULL_DISABLE>;
> + };
> +
> + usb0_otg_apf28dev: otg-apf28dev@0 {
> + reg = <0>;
> + fsl,pinmux-ids = <
> + MX28_PAD_LCD_D23__GPIO_1_23
> + >;
> + fsl,drive-strength = <MXS_DRIVE_4mA>;
> + fsl,voltage = <MXS_VOLTAGE_HIGH>;
> + fsl,pull-up = <MXS_PULL_DISABLE>;
> + };
> +};
The pinctrl node is usually placed at the end of the file.
Thank you !
Regards,
> +
> +&pwm {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pwm3_pins_a &pwm4_pins_a>;
> + status = "okay";
> +};
> +
> +&ssp0 {
> + compatible = "fsl,imx28-mmc";
> + pinctrl-names = "default";
> + pinctrl-0 = <&mmc0_4bit_pins_a
> + &mmc0_cd_cfg &mmc0_sck_cfg>;
> + bus-width = <4>;
> + status = "okay";
> +};
> +
> +&ssp2 {
> + compatible = "fsl,imx28-spi";
> + pinctrl-names = "default";
> + pinctrl-0 = <&spi2_pins_a>;
> + status = "okay";
> +};
> +
> +&usb0 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&usb0_otg_apf28dev
> + &usb0_id_pins_b>;
> + vbus-supply = <®_usb0_vbus>;
> + status = "okay";
> +};
> +
> +&usb1 {
> + status = "okay";
> +};
> +
> +&usbphy0 {
> + status = "okay";
> +};
> +
> +&usbphy1 {
> + status = "okay";
> +};
--
Sébastien Szymanski, Armadeus Systems
Software engineer
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-02-13 8:27 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-01 20:33 [PATCH 0/9] ARM: dts: imx28: Clean up older DTS files Stefan Wahren
2023-02-01 20:33 ` [PATCH 1/9] ARM: dts: imx28-apf28: Convert to use label references Stefan Wahren
2023-02-13 8:22 ` Sébastien Szymanski [this message]
2023-02-13 14:50 ` Stefan Wahren
2023-02-01 20:33 ` [PATCH 2/9] ARM: dts: imx28-m28/sps1: " Stefan Wahren
2023-02-01 21:03 ` Marek Vasut
2023-02-01 20:33 ` [PATCH 3/9] ARM: dts: imx28-apx4devkit: " Stefan Wahren
2023-02-01 20:33 ` [PATCH 4/9] ARM: dts: imx28-cfa10036: " Stefan Wahren
2023-02-01 20:33 ` [PATCH 5/9] ARM: dts: imx28-duckbill: " Stefan Wahren
2023-02-01 20:33 ` [PATCH 6/9] ARM: dts: imx28-duckbill-2: Include base board Stefan Wahren
2023-02-01 20:33 ` [PATCH 7/9] ARM: dts: imx28-evk: Convert to use label references Stefan Wahren
2023-02-01 20:33 ` [PATCH 8/9] ARM: dts: imx28-ts4600: " Stefan Wahren
2023-02-01 20:33 ` [PATCH 9/9] ARM: dts: imx28-tx28: add SPDX-License-Identifier Stefan Wahren
2023-02-02 6:16 ` Lothar Waßmann
2023-02-03 9:44 ` [PATCH 0/9] ARM: dts: imx28: Clean up older DTS files Sébastien Szymanski
2023-02-03 10:16 ` Krzysztof Kozlowski
2023-02-03 13:13 ` Sascha Hauer
2023-03-06 2:58 ` Shawn Guo
2023-03-07 9:18 ` Krzysztof Kozlowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=58919223-cd4e-9a54-52ab-90f86552d4e2@armadeus.com \
--to=sebastien.szymanski@armadeus.com \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=stefan.wahren@i2se.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).