* [PATCH] dts: arm: imx6: remove mmc1 alias
@ 2025-10-21 14:42 Fabian Pfitzner
2025-10-21 15:03 ` Ahmad Fatoum
0 siblings, 1 reply; 2+ messages in thread
From: Fabian Pfitzner @ 2025-10-21 14:42 UTC (permalink / raw)
To: barebox; +Cc: Fabian Pfitzner
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.
Signed-off-by: Fabian Pfitzner <f.pfitzner@pengutronix.de>
---
dts/src/arm/nxp/imx/imx6s-btx53lb000.dts | 4 ----
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>;
--
2.47.3
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] dts: arm: imx6: remove mmc1 alias
2025-10-21 14:42 [PATCH] dts: arm: imx6: remove mmc1 alias Fabian Pfitzner
@ 2025-10-21 15:03 ` Ahmad Fatoum
0 siblings, 0 replies; 2+ messages in thread
From: Ahmad Fatoum @ 2025-10-21 15:03 UTC (permalink / raw)
To: Fabian Pfitzner, barebox
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 |
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-10-21 15:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-10-21 14:42 [PATCH] dts: arm: imx6: remove mmc1 alias Fabian Pfitzner
2025-10-21 15:03 ` Ahmad Fatoum
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox