Re: [PATCH] dts: arm: imx6: remove mmc1 alias

[Ahmad Fatoum <a.fatoum@pengutronix.de>]

Hello Fabian,

On 10/21/25 4:42 PM, Fabian Pfitzner wrote:
> Setting the mmc1 alias to usdhc4 in the dts overwrites the mmc1 alias set in
> the included imx6dl.dtsi file. This causes mmc1 and mmc3 pointing to the same address.
> 
> This creates the device name as /dev/mmc1* instead of the expected /dev/mmc3* name.
> As the barebox update handler is configured for mmc3, updating is not
> possible anymore.
> 
> This patch solves this by removing the mmc1 alias from the dts.

I suspect this may be specific to your out-of-tree patches as there is
no imx6s-btx53lb000.dts in upstream barebox.

> 
> Signed-off-by: Fabian Pfitzner <f.pfitzner@pengutronix.de>
> ---
>  dts/src/arm/nxp/imx/imx6s-btx53lb000.dts | 4 ----

In any case, we don't accept patches against dts/. Patches against dts/
need to be submitted through Linux and are then imported into barebox
along with the regular device tree syncs as described in the barebox
documentation.

What's possible to patch in barebox is adding overrides in arch/arm/dts.

Cheers,
Ahmad

>  1 file changed, 4 deletions(-)
> 
> diff --git a/dts/src/arm/nxp/imx/imx6s-btx53lb000.dts b/dts/src/arm/nxp/imx/imx6s-btx53lb000.dts
> index f8141832cd..c395bffe06 100644
> --- a/dts/src/arm/nxp/imx/imx6s-btx53lb000.dts
> +++ b/dts/src/arm/nxp/imx/imx6s-btx53lb000.dts
> @@ -9,10 +9,6 @@ / {
>  	model = "Waygate BTX53LB000v2";
>  	compatible = "karo,imx6dl-tx6dl", "fsl,imx6dl";
>  
> -	aliases {
> -		mmc1 = &usdhc4;
> -	};
> -
>  	backlight: backlight {
>  		compatible = "pwm-backlight";
>  		pinctrl-0 = <&pinctrl_backlight>;

-- 
Pengutronix e.K.                  |                             |
Steuerwalder Str. 21              | http://www.pengutronix.de/  |
31137 Hildesheim, Germany         | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686  | Fax:   +49-5121-206917-5555 |