From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrice CHOTARD Date: Mon, 11 May 2020 14:01:38 +0000 Subject: [Uboot-stm32] [PATCH] ARM: dts: stm32mp1: DT alignment with Linux 5.7-rc2 In-Reply-To: <20200430155234.1.If8f63efaa47eb0830ce241e3a62b2aa420e0883a@changeid> References: <20200430155234.1.If8f63efaa47eb0830ce241e3a62b2aa420e0883a@changeid> Message-ID: <6a244f0b-daaf-017d-1132-2b694cf09994@st.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Patrick On 4/30/20 3:52 PM, Patrick Delaunay wrote: > DT alignment with Linux 5.7-rc2, including the kernel commits > > 431c89e6f323 ARM: dts: stm32: use correct vqmmc regu for eMMC on stm32mp1 ED1/EV1 boards > 79e965053872 ARM: dts: stm32: add disable-wp property for SD-card on STM32MP1 boards > 877db62ea516 ARM: dts: stm32: add cd-gpios properties for SD-cards on STM32MP1 boards > 7519e95ba5f8 ARM: dts: stm32: Do clean up in stmpic nodes on stm32mp15 boards > f68e2dbc591a ARM: dts: stm32: Rename stmfx joystick pins on stm32mp157c-ev1 > d6210da4f8bf ARM: dts: stm32: add cpu clock-frequency property on stm32mp15x > b65b6fc56925 ARM: dts: stm32: add wakeup-source in all I2C nodes of stm32mp157c > 1c1cf5996cfb ARM: dts: stm32: add i2c4 sleep pinctrl on stm32mp157c-ed1 > bef15fc0fad9 ARM: dts: stm32: add i2c2/i2c5 sleep pinctrl on stm32mp157c-ev1 > b7fc0a87b9ac ARM: dts: stm32: add i2c4 sleep pinctrl on stm32mp15xx-dkx > a5e557655285 ARM: dts: stm32: set i2c4 bus freq to 400KHz on stm32mp15 DK boards > 8bc631b650a6 ARM: dts: stm32: set i2c4 bus freq to 400KHz on stm32mp157c-ed1 > fccd6a577bb3 ARM: dts: stm32: Correct stmfx node name on stm32mp157c-ev1 board > cc775a83db65 ARM: dts: stm32: add resets property on all DMA nodes on stm32mp151 > c5fae093511b ARM: dts: stm32: enable USB OTG Dual Role on stm32mp157c-ev1 > 9879e2165758 ARM: dts: stm32: add USB OTG pinctrl to stm32mp15 > 82ac8a81f985 ARM: dts: stm32: add USB OTG full support on stm32mp151 > 8714b26e2863 ARM: dts: stm32: remove useless properties in stm32mp157a-avenger96 stmpic node > a7959919709e ARM: dts: stm32: Add UART8 pins A pinmux entry on stm32mp1 > 4d7c53a684da ARM: dts: stm32: Add USART3 pins A pinmux entry on stm32mp1 > 80ab128332ee ARM: dts: stm32: Add SAI2A pins B pinmux entry on stm32mp1 > ab7f98c0c546 ARM: dts: stm32: Add Ethernet0 RMII pins A pinmux entry on stm32mp1 > > Signed-off-by: Patrick Delaunay > --- > Hi, > > Dependency with correction of GPIO support in SPL: > > [v2,09/12] gpio: stm32: support gpio ops in SPL > http://patchwork.ozlabs.org/project/uboot/patch/20200422142834.v2.9.I355ddbc804eba6047ea147d830be57a5b9c4a87e at changeid/ > > Patrick > > > arch/arm/dts/stm32mp15-pinctrl.dtsi | 92 +++++++++++++++++++++++++++++ > arch/arm/dts/stm32mp151.dtsi | 13 +++- > arch/arm/dts/stm32mp153.dtsi | 1 + > arch/arm/dts/stm32mp157c-ed1.dts | 12 ++-- > arch/arm/dts/stm32mp157c-ev1.dts | 13 ++-- > arch/arm/dts/stm32mp15xx-dhcom.dtsi | 3 +- > arch/arm/dts/stm32mp15xx-dhcor.dtsi | 8 --- > arch/arm/dts/stm32mp15xx-dkx.dtsi | 10 ++-- > 8 files changed, 126 insertions(+), 26 deletions(-) > > diff --git a/arch/arm/dts/stm32mp15-pinctrl.dtsi b/arch/arm/dts/stm32mp15-pinctrl.dtsi > index 29acdc4afd..8d00391978 100644 > --- a/arch/arm/dts/stm32mp15-pinctrl.dtsi > +++ b/arch/arm/dts/stm32mp15-pinctrl.dtsi > @@ -213,6 +213,40 @@ > }; > }; > > + ethernet0_rmii_pins_a: rmii-0 { > + pins1 { > + pinmux = , /* ETH1_RMII_TXD0 */ > + , /* ETH1_RMII_TXD1 */ > + , /* ETH1_RMII_TX_EN */ > + , /* ETH1_RMII_REF_CLK */ > + , /* ETH1_MDIO */ > + ; /* ETH1_MDC */ > + bias-disable; > + drive-push-pull; > + slew-rate = <2>; > + }; > + pins2 { > + pinmux = , /* ETH1_RMII_RXD0 */ > + , /* ETH1_RMII_RXD1 */ > + ; /* ETH1_RMII_CRS_DV */ > + bias-disable; > + }; > + }; > + > + ethernet0_rmii_pins_sleep_a: rmii-sleep-0 { > + pins1 { > + pinmux = , /* ETH1_RMII_TXD0 */ > + , /* ETH1_RMII_TXD1 */ > + , /* ETH1_RMII_TX_EN */ > + , /* ETH1_MDIO */ > + , /* ETH1_MDC */ > + , /* ETH1_RMII_RXD0 */ > + , /* ETH1_RMII_RXD1 */ > + , /* ETH1_RMII_REF_CLK */ > + ; /* ETH1_RMII_CRS_DV */ > + }; > + }; > + > fmc_pins_a: fmc-0 { > pins1 { > pinmux = , /* FMC_NOE */ > @@ -736,6 +770,25 @@ > }; > }; > > + sai2a_pins_b: sai2a-2 { > + pins1 { > + pinmux = , /* SAI2_SD_A */ > + , /* SAI2_FS_A */ > + ; /* SAI2_SCK_A */ > + slew-rate = <0>; > + drive-push-pull; > + bias-disable; > + }; > + }; > + > + sai2a_sleep_pins_b: sai2a-sleep-3 { > + pins { > + pinmux = , /* SAI2_SD_A */ > + , /* SAI2_FS_A */ > + ; /* SAI2_SCK_A */ > + }; > + }; > + > sai2b_pins_a: sai2b-0 { > pins1 { > pinmux = , /* SAI2_SCK_B */ > @@ -1118,6 +1171,19 @@ > }; > }; > > + usart3_pins_a: usart3-0 { > + pins1 { > + pinmux = ; /* USART3_TX */ > + bias-disable; > + drive-push-pull; > + slew-rate = <0>; > + }; > + pins2 { > + pinmux = ; /* USART3_RX */ > + bias-disable; > + }; > + }; > + > uart4_pins_a: uart4-0 { > pins1 { > pinmux = ; /* UART4_TX */ > @@ -1158,6 +1224,32 @@ > bias-disable; > }; > }; > + > + uart8_pins_a: uart8-0 { > + pins1 { > + pinmux = ; /* UART8_TX */ > + bias-disable; > + drive-push-pull; > + slew-rate = <0>; > + }; > + pins2 { > + pinmux = ; /* UART8_RX */ > + bias-disable; > + }; > + }; > + > + usbotg_hs_pins_a: usbotg-hs-0 { > + pins { > + pinmux = ; /* OTG_ID */ > + }; > + }; > + > + usbotg_fs_dp_dm_pins_a: usbotg-fs-dp-dm-0 { > + pins { > + pinmux = , /* OTG_FS_DM */ > + ; /* OTG_FS_DP */ > + }; > + }; > }; > > &pinctrl_z { > diff --git a/arch/arm/dts/stm32mp151.dtsi b/arch/arm/dts/stm32mp151.dtsi > index f185639a46..75d2c0d296 100644 > --- a/arch/arm/dts/stm32mp151.dtsi > +++ b/arch/arm/dts/stm32mp151.dtsi > @@ -17,6 +17,7 @@ > > cpu0: cpu at 0 { > compatible = "arm,cortex-a7"; > + clock-frequency = <650000000>; > device_type = "cpu"; > reg = <0>; > }; > @@ -490,6 +491,7 @@ > resets = <&rcc I2C1_R>; > #address-cells = <1>; > #size-cells = <0>; > + wakeup-source; > status = "disabled"; > }; > > @@ -503,6 +505,7 @@ > resets = <&rcc I2C2_R>; > #address-cells = <1>; > #size-cells = <0>; > + wakeup-source; > status = "disabled"; > }; > > @@ -516,6 +519,7 @@ > resets = <&rcc I2C3_R>; > #address-cells = <1>; > #size-cells = <0>; > + wakeup-source; > status = "disabled"; > }; > > @@ -529,6 +533,7 @@ > resets = <&rcc I2C5_R>; > #address-cells = <1>; > #size-cells = <0>; > + wakeup-source; > status = "disabled"; > }; > > @@ -966,6 +971,7 @@ > , > ; > clocks = <&rcc DMA1>; > + resets = <&rcc DMA1_R>; > #dma-cells = <4>; > st,mem2mem; > dma-requests = <8>; > @@ -983,6 +989,7 @@ > , > ; > clocks = <&rcc DMA2>; > + resets = <&rcc DMA2_R>; > #dma-cells = <4>; > st,mem2mem; > dma-requests = <8>; > @@ -996,6 +1003,7 @@ > dma-masters = <&dma1 &dma2>; > dma-channels = <16>; > clocks = <&rcc DMAMUX>; > + resets = <&rcc DMAMUX_R>; > }; > > adc: adc at 48003000 { > @@ -1051,7 +1059,7 @@ > }; > > usbotg_hs: usb-otg at 49000000 { > - compatible = "snps,dwc2"; > + compatible = "st,stm32mp15-hsotg", "snps,dwc2"; > reg = <0x49000000 0x10000>; > clocks = <&rcc USBO_K>; > clock-names = "otg"; > @@ -1296,6 +1304,7 @@ > reg = <0x58000000 0x1000>; > interrupts = ; > clocks = <&rcc MDMA>; > + resets = <&rcc MDMA_R>; > #dma-cells = <5>; > dma-channels = <32>; > dma-requests = <48>; > @@ -1495,6 +1504,7 @@ > resets = <&rcc I2C4_R>; > #address-cells = <1>; > #size-cells = <0>; > + wakeup-source; > status = "disabled"; > }; > > @@ -1530,6 +1540,7 @@ > resets = <&rcc I2C6_R>; > #address-cells = <1>; > #size-cells = <0>; > + wakeup-source; > status = "disabled"; > }; > > diff --git a/arch/arm/dts/stm32mp153.dtsi b/arch/arm/dts/stm32mp153.dtsi > index 2d759fc601..6d9ab08667 100644 > --- a/arch/arm/dts/stm32mp153.dtsi > +++ b/arch/arm/dts/stm32mp153.dtsi > @@ -10,6 +10,7 @@ > cpus { > cpu1: cpu at 1 { > compatible = "arm,cortex-a7"; > + clock-frequency = <650000000>; > device_type = "cpu"; > reg = <1>; > }; > diff --git a/arch/arm/dts/stm32mp157c-ed1.dts b/arch/arm/dts/stm32mp157c-ed1.dts > index 05d5326405..4fb71100f5 100644 > --- a/arch/arm/dts/stm32mp157c-ed1.dts > +++ b/arch/arm/dts/stm32mp157c-ed1.dts > @@ -135,10 +135,12 @@ > }; > > &i2c4 { > - pinctrl-names = "default"; > + pinctrl-names = "default", "sleep"; > pinctrl-0 = <&i2c4_pins_a>; > + pinctrl-1 = <&i2c4_pins_sleep_a>; > i2c-scl-rising-time-ns = <185>; > i2c-scl-falling-time-ns = <20>; > + clock-frequency = <400000>; > status = "okay"; > /* spare dmas for other usage */ > /delete-property/dmas; > @@ -223,8 +225,6 @@ > > vdd_usb: ldo4 { > regulator-name = "vdd_usb"; > - regulator-min-microvolt = <3300000>; > - regulator-max-microvolt = <3300000>; > interrupts = ; > }; > > @@ -246,7 +246,6 @@ > vref_ddr: vref_ddr { > regulator-name = "vref_ddr"; > regulator-always-on; > - regulator-over-current-protection; > }; > > bst_out: boost { > @@ -318,7 +317,8 @@ > pinctrl-0 = <&sdmmc1_b4_pins_a &sdmmc1_dir_pins_a>; > pinctrl-1 = <&sdmmc1_b4_od_pins_a &sdmmc1_dir_pins_a>; > pinctrl-2 = <&sdmmc1_b4_sleep_pins_a &sdmmc1_dir_sleep_pins_a>; > - broken-cd; > + cd-gpios = <&gpiog 1 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>; > + disable-wp; > st,sig-dir; > st,neg-edge; > st,use-ckin; > @@ -339,7 +339,7 @@ > st,neg-edge; > bus-width = <8>; > vmmc-supply = <&v3v3>; > - vqmmc-supply = <&v3v3>; > + vqmmc-supply = <&vdd>; > mmc-ddr-3_3v; > status = "okay"; > }; > diff --git a/arch/arm/dts/stm32mp157c-ev1.dts b/arch/arm/dts/stm32mp157c-ev1.dts > index 228e35e168..8a4c7ff31a 100644 > --- a/arch/arm/dts/stm32mp157c-ev1.dts > +++ b/arch/arm/dts/stm32mp157c-ev1.dts > @@ -174,8 +174,9 @@ > }; > > &i2c2 { > - pinctrl-names = "default"; > + pinctrl-names = "default", "sleep"; > pinctrl-0 = <&i2c2_pins_a>; > + pinctrl-1 = <&i2c2_pins_sleep_a>; > i2c-scl-rising-time-ns = <185>; > i2c-scl-falling-time-ns = <20>; > status = "okay"; > @@ -210,7 +211,7 @@ > interrupt-parent = <&gpioi>; > vdd-supply = <&v3v3>; > > - stmfx_pinctrl: stmfx-pin-controller { > + stmfx_pinctrl: pinctrl { > compatible = "st,stmfx-0300-pinctrl"; > gpio-controller; > #gpio-cells = <2>; > @@ -218,7 +219,7 @@ > #interrupt-cells = <2>; > gpio-ranges = <&stmfx_pinctrl 0 0 24>; > > - joystick_pins: joystick { > + joystick_pins: joystick-pins { > pins = "gpio0", "gpio1", "gpio2", "gpio3", "gpio4"; > bias-pull-down; > }; > @@ -227,8 +228,9 @@ > }; > > &i2c5 { > - pinctrl-names = "default"; > + pinctrl-names = "default", "sleep"; > pinctrl-0 = <&i2c5_pins_a>; > + pinctrl-1 = <&i2c5_pins_sleep_a>; > i2c-scl-rising-time-ns = <185>; > i2c-scl-falling-time-ns = <20>; > status = "okay"; > @@ -353,7 +355,8 @@ > }; > > &usbotg_hs { > - dr_mode = "peripheral"; > + pinctrl-0 = <&usbotg_hs_pins_a>; > + pinctrl-names = "default"; > phys = <&usbphyc_port1 0>; > phy-names = "usb2-phy"; > status = "okay"; > diff --git a/arch/arm/dts/stm32mp15xx-dhcom.dtsi b/arch/arm/dts/stm32mp15xx-dhcom.dtsi > index d8a255b9c6..b3f4cb4515 100644 > --- a/arch/arm/dts/stm32mp15xx-dhcom.dtsi > +++ b/arch/arm/dts/stm32mp15xx-dhcom.dtsi > @@ -269,7 +269,8 @@ > pinctrl-0 = <&sdmmc1_b4_pins_a &sdmmc1_dir_pins_a>; > pinctrl-1 = <&sdmmc1_b4_od_pins_a &sdmmc1_dir_pins_a>; > pinctrl-2 = <&sdmmc1_b4_sleep_pins_a &sdmmc1_dir_sleep_pins_a>; > - broken-cd; > + cd-gpios = <&gpioi 8 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>; > + disable-wp; > st,sig-dir; > st,neg-edge; > st,use-ckin; > diff --git a/arch/arm/dts/stm32mp15xx-dhcor.dtsi b/arch/arm/dts/stm32mp15xx-dhcor.dtsi > index 97d370e119..ce17997142 100644 > --- a/arch/arm/dts/stm32mp15xx-dhcor.dtsi > +++ b/arch/arm/dts/stm32mp15xx-dhcor.dtsi > @@ -51,10 +51,6 @@ > #interrupt-cells = <2>; > status = "okay"; > > - st,main-control-register = <0x04>; > - st,vin-control-register = <0xc0>; > - st,usb-control-register = <0x30>; > - > regulators { > compatible = "st,stpmic1-regulators"; > > @@ -89,7 +85,6 @@ > regulator-min-microvolt = <2900000>; > regulator-max-microvolt = <2900000>; > regulator-always-on; > - st,mask_reset; > regulator-initial-mode = <0>; > regulator-over-current-protection; > }; > @@ -129,8 +124,6 @@ > > vdd_usb: ldo4 { > regulator-name = "vdd_usb"; > - regulator-min-microvolt = <3300000>; > - regulator-max-microvolt = <3300000>; > interrupts = ; > interrupt-parent = <&pmic>; > }; > @@ -156,7 +149,6 @@ > vref_ddr: vref_ddr { > regulator-name = "vref_ddr"; > regulator-always-on; > - regulator-over-current-protection; > }; > > bst_out: boost { > diff --git a/arch/arm/dts/stm32mp15xx-dkx.dtsi b/arch/arm/dts/stm32mp15xx-dkx.dtsi > index 9ce10a52f0..812e370ee4 100644 > --- a/arch/arm/dts/stm32mp15xx-dkx.dtsi > +++ b/arch/arm/dts/stm32mp15xx-dkx.dtsi > @@ -221,10 +221,12 @@ > }; > > &i2c4 { > - pinctrl-names = "default"; > + pinctrl-names = "default", "sleep"; > pinctrl-0 = <&i2c4_pins_a>; > + pinctrl-1 = <&i2c4_pins_sleep_a>; > i2c-scl-rising-time-ns = <185>; > i2c-scl-falling-time-ns = <20>; > + clock-frequency = <400000>; > status = "okay"; > /* spare dmas for other usage */ > /delete-property/dmas; > @@ -327,8 +329,6 @@ > > vdd_usb: ldo4 { > regulator-name = "vdd_usb"; > - regulator-min-microvolt = <3300000>; > - regulator-max-microvolt = <3300000>; > interrupts = ; > }; > > @@ -351,7 +351,6 @@ > vref_ddr: vref_ddr { > regulator-name = "vref_ddr"; > regulator-always-on; > - regulator-over-current-protection; > }; > > bst_out: boost { > @@ -502,7 +501,8 @@ > pinctrl-0 = <&sdmmc1_b4_pins_a>; > pinctrl-1 = <&sdmmc1_b4_od_pins_a>; > pinctrl-2 = <&sdmmc1_b4_sleep_pins_a>; > - broken-cd; > + cd-gpios = <&gpiob 7 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>; > + disable-wp; > st,neg-edge; > bus-width = <4>; > vmmc-supply = <&v3v3>; Reviewed-by: Patrice Chotard