[PATCH 2/7] ARM: dts: at91: sam9x60: Define pinctrl node with its label
Eugen Hristev
eugen.hristev at linaro.org
Tue Jan 28 12:14:12 CET 2025
On 1/27/25 12:09, Manikandan Muralidharan wrote:
> From: Balamanikandan Gunasundar <balamanikandan.gunasundar at microchip.com>
>
> Define the pinctrl nodes with its label and sort its child node
> alphabetically to align wit the Linux DT.Without this change
Typo: wit
> those pinctrl nodes are grouped under an additional 'pinctrl' child node
> that is not identified by the pinctrl driver when the GPIO banks
> are made as child nodes of pinctrl node.
Please align them as they are aligned in Linux, or, patch Linux DT first.
>
> Signed-off-by: Balamanikandan Gunasundar <balamanikandan.gunasundar at microchip.com>
> Signed-off-by: Manikandan Muralidharan <manikandan.m at microchip.com>
> ---
> arch/arm/dts/sam9x60ek.dts | 138 ++++++++++++++++++-------------------
> 1 file changed, 69 insertions(+), 69 deletions(-)
>
> diff --git a/arch/arm/dts/sam9x60ek.dts b/arch/arm/dts/sam9x60ek.dts
> index 74016f5e288..d8d7ad38b2b 100644
> --- a/arch/arm/dts/sam9x60ek.dts
> +++ b/arch/arm/dts/sam9x60ek.dts
> @@ -78,75 +78,6 @@
> };
> };
> };
> -
> - pinctrl {
> - nand {
> - pinctrl_nand_oe_we: nand-oe-we-0 {
> - atmel,pins =
> - <AT91_PIOD 0 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> - AT91_PIOD 1 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)>;
> - };
> -
> - pinctrl_nand_rb: nand-rb-0 {
> - atmel,pins =
> - <AT91_PIOD 5 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
> - };
> -
> - pinctrl_nand_cs: nand-cs-0 {
> - atmel,pins =
> - <AT91_PIOD 4 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
> - };
> - };
> -
> - ebi {
> - pinctrl_ebi_data_0_7: ebi-data-lsb-0 {
> - atmel,pins =
> - <AT91_PIOD 6 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> - AT91_PIOD 7 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> - AT91_PIOD 8 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> - AT91_PIOD 9 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> - AT91_PIOD 10 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> - AT91_PIOD 11 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> - AT91_PIOD 12 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> - AT91_PIOD 13 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)>;
> - };
> -
> - pinctrl_ebi_addr_nand: ebi-addr-0 {
> - atmel,pins =
> - <AT91_PIOD 2 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> - AT91_PIOD 3 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)>;
> - };
> - };
> -
> - pinctrl_qspi: qspi {
> - atmel,pins =
> - <AT91_PIOB 19 AT91_PERIPH_A AT91_PINCTRL_NONE
> - AT91_PIOB 20 AT91_PERIPH_A AT91_PINCTRL_NONE
> - AT91_PIOB 21 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> - AT91_PIOB 22 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> - AT91_PIOB 23 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> - AT91_PIOB 24 AT91_PERIPH_A AT91_PINCTRL_PULL_UP>;
> - };
> -
> - pinctrl_flx0: flx0_default {
> - atmel,pins =
> - <AT91_PIOA 0 AT91_PERIPH_A AT91_PINCTRL_NONE
> - AT91_PIOA 1 AT91_PERIPH_A AT91_PINCTRL_NONE>;
> - };
> -
> - pinctrl_onewire_tm_default: onewire_tm_default {
> - atmel,pins =
> - <AT91_PIOD 14 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
> - };
> -
> - usb1 {
> - pinctrl_usb_default: usb_default {
> - atmel,pins = <AT91_PIOD 15 AT91_PERIPH_GPIO AT91_PINCTRL_NONE
> - AT91_PIOD 16 AT91_PERIPH_GPIO AT91_PINCTRL_NONE>;
> - };
> - };
> -
> - };
> };
> };
> };
> @@ -221,6 +152,75 @@
> status = "okay";
> };
>
> +&pinctrl {
> + /* shared pinctrl settings */
> + ebi {
> + pinctrl_ebi_data_0_7: ebi-data-lsb-0 {
> + atmel,pins =
> + <AT91_PIOD 6 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> + AT91_PIOD 7 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> + AT91_PIOD 8 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> + AT91_PIOD 9 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> + AT91_PIOD 10 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> + AT91_PIOD 11 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> + AT91_PIOD 12 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> + AT91_PIOD 13 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)>;
> + };
> +
> + pinctrl_ebi_addr_nand: ebi-addr-0 {
> + atmel,pins =
> + <AT91_PIOD 2 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> + AT91_PIOD 3 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)>;
> + };
> + };
> +
> + nand {
> + pinctrl_nand_oe_we: nand-oe-we-0 {
> + atmel,pins =
> + <AT91_PIOD 0 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)
> + AT91_PIOD 1 AT91_PERIPH_A (AT91_PINCTRL_NONE | AT91_PINCTRL_SLEWRATE_DIS)>;
> + };
> +
> + pinctrl_nand_rb: nand-rb-0 {
> + atmel,pins =
> + <AT91_PIOD 5 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
> + };
> +
> + pinctrl_nand_cs: nand-cs-0 {
> + atmel,pins =
> + <AT91_PIOD 4 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
> + };
> + };
I see in Linux nand is before ebi or am I wrong ?
> +
> + pinctrl_flx0: flx0_default {
> + atmel,pins =
> + <AT91_PIOA 0 AT91_PERIPH_A AT91_PINCTRL_NONE
> + AT91_PIOA 1 AT91_PERIPH_A AT91_PINCTRL_NONE>;
> + };
> +
> + pinctrl_onewire_tm_default: onewire_tm_default {
> + atmel,pins =
> + <AT91_PIOD 14 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
> + };
> +
> + pinctrl_qspi: qspi {
> + atmel,pins =
> + <AT91_PIOB 19 AT91_PERIPH_A AT91_PINCTRL_NONE
> + AT91_PIOB 20 AT91_PERIPH_A AT91_PINCTRL_NONE
> + AT91_PIOB 21 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> + AT91_PIOB 22 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> + AT91_PIOB 23 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> + AT91_PIOB 24 AT91_PERIPH_A AT91_PINCTRL_PULL_UP>;
> + };
> +
> + usb1 {
> + pinctrl_usb_default: usb_default {
> + atmel,pins = <AT91_PIOD 15 AT91_PERIPH_GPIO AT91_PINCTRL_NONE
> + AT91_PIOD 16 AT91_PERIPH_GPIO AT91_PINCTRL_NONE>;
> + };
> + };
> +};
> +
> &usb1 {
> num-ports = <3>;
> atmel,vbus-gpio = <0
More information about the U-Boot
mailing list