[PATCH] ARM: dts: stm32: Fix sdmmc2 pins again

Alexandre Torgue alexandre.torgue at st.com
Fri Aug 28 04:59:36 EDT 2020


Hi Marek

On 7/31/20 1:27 AM, Marek Vasut wrote:
> The AV96 uses sdmmc2_d47_pins_c and sdmmc2_d47_sleep_pins_c, which
> differ from sdmmc2_d47_pins_b and sdmmc2_d47_sleep_pins_b in one
> pin, SDMMC2_D5, which is PA15 in the former and PA9 in the later.
> The PA15 is correct on AV96, so fix this. This error is likely a
> result of rebasing across the stm32mp1 DT pinctrl rework.
> 
> Fixes: 611325f68102 ("ARM: dts: stm32: Add eMMC attached to SDMMC2 on AV96")
> Signed-off-by: Marek Vasut <marex at denx.de>
> Cc: Alexandre Torgue <alexandre.torgue at st.com>
> Cc: Maxime Coquelin <mcoquelin.stm32 at gmail.com>
> Cc: Patrice Chotard <patrice.chotard at st.com>
> Cc: Patrick Delaunay <patrick.delaunay at st.com>
> Cc: linux-stm32 at st-md-mailman.stormreply.com
> To: linux-arm-kernel at lists.infradead.org
> ---
>   arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi
> index 930202742a3f..905cd7bb98cf 100644
> --- a/arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi
> +++ b/arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi
> @@ -295,9 +295,9 @@ &sdmmc1 {
>   
>   &sdmmc2 {
>   	pinctrl-names = "default", "opendrain", "sleep";
> -	pinctrl-0 = <&sdmmc2_b4_pins_a &sdmmc2_d47_pins_b>;
> -	pinctrl-1 = <&sdmmc2_b4_od_pins_a &sdmmc2_d47_pins_b>;
> -	pinctrl-2 = <&sdmmc2_b4_sleep_pins_a &sdmmc2_d47_sleep_pins_b>;
> +	pinctrl-0 = <&sdmmc2_b4_pins_a &sdmmc2_d47_pins_c>;
> +	pinctrl-1 = <&sdmmc2_b4_od_pins_a &sdmmc2_d47_pins_c>;
> +	pinctrl-2 = <&sdmmc2_b4_sleep_pins_a &sdmmc2_d47_sleep_pins_c>;
>   	bus-width = <8>;
>   	mmc-ddr-1_8v;
>   	no-sd;
> 

Applied on stm32-next. I modified a bit the commit title to mention 
which board is targeted.

Thanks.
Alex



More information about the linux-arm-kernel mailing list