[PATCH] ARM: dts: sunxi: Add MMC alias for consistent enumeration

[PATCH] ARM: dts: sunxi: Add MMC alias for consistent enumeration

[Stefan Gloor]

Add explicit alias for MMC devices, so that (e)MMC and micro SD cards
are enumerated consistently. This should fix spurious boot failures when
specifying a hard-coded root partition, e.g., mmcblk0p2.

Link: https://github.com/linux-sunxi/meta-sunxi/pull/431
Signed-off-by: Stefan Gloor <code@stefan-gloor.ch>
---
 arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts b/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts
index a1b89b2a2999..9cd1eb690134 100644
--- a/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts
+++ b/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts
@@ -60,6 +60,8 @@ aliases {
 		serial2 = &uart7;
 		spi0 = &spi1;
 		spi1 = &spi2;
+		mmc0 = &mmc0;
+		mmc1 = &mmc3;
 	};
 
 	chosen {
-- 
2.45.2

Re: [PATCH] ARM: dts: sunxi: Add MMC alias for consistent enumeration

[Chen-Yu Tsai]

On Thu, Dec 19, 2024 at 6:58 PM Stefan Gloor <code@stefan-gloor.ch> wrote:
>
> Add explicit alias for MMC devices, so that (e)MMC and micro SD cards
> are enumerated consistently. This should fix spurious boot failures when
> specifying a hard-coded root partition, e.g., mmcblk0p2.

IIRC we have chosen against adding explicit MMC aliases for the sunxi
platforms. Please try using UUID= or PARTUUID= for root partition
references.

ChenYu

> Link: https://github.com/linux-sunxi/meta-sunxi/pull/431
> Signed-off-by: Stefan Gloor <code@stefan-gloor.ch>
> ---
>  arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts b/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts
> index a1b89b2a2999..9cd1eb690134 100644
> --- a/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts
> +++ b/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts
> @@ -60,6 +60,8 @@ aliases {
>                 serial2 = &uart7;
>                 spi0 = &spi1;
>                 spi1 = &spi2;
> +               mmc0 = &mmc0;
> +               mmc1 = &mmc3;
>         };
>
>         chosen {
> --
> 2.45.2
>
>

Re: [PATCH] ARM: dts: sunxi: Add MMC alias for consistent enumeration

[Andre Przywara]

On Wed, 1 Jan 2025 16:58:28 +0800
Chen-Yu Tsai <wens@csie.org> wrote:

Hi,

> On Thu, Dec 19, 2024 at 6:58 PM Stefan Gloor <code@stefan-gloor.ch> wrote:
> >
> > Add explicit alias for MMC devices, so that (e)MMC and micro SD cards
> > are enumerated consistently. This should fix spurious boot failures when
> > specifying a hard-coded root partition, e.g., mmcblk0p2.  
> 
> IIRC we have chosen against adding explicit MMC aliases for the sunxi
> platforms. Please try using UUID= or PARTUUID= for root partition
> references.

Yes, and LABEL= would be another way.

Alternatively you could use U-Boot's DT when booting the board. U-Boot
will add those aliases during build time (because it needs them for its
own purposes), and by using $fdtcontroladdr (instead of $fdt_addr_r,
on the bootz command line) you get this for free, and even save yourself
from loading some DT.
We sync the DTs regularly from the kernel to U-Boot now, and at the
moment they are identical between the kernel and U-Boot, so you don't
lose anything.

Cheers,
Andre

> > Link: https://github.com/linux-sunxi/meta-sunxi/pull/431
> > Signed-off-by: Stefan Gloor <code@stefan-gloor.ch>
> > ---
> >  arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts b/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts
> > index a1b89b2a2999..9cd1eb690134 100644
> > --- a/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts
> > +++ b/arch/arm/boot/dts/allwinner/sun7i-a20-olinuxino-micro.dts
> > @@ -60,6 +60,8 @@ aliases {
> >                 serial2 = &uart7;
> >                 spi0 = &spi1;
> >                 spi1 = &spi2;
> > +               mmc0 = &mmc0;
> > +               mmc1 = &mmc3;
> >         };
> >
> >         chosen {
> > --
> > 2.45.2
> >
> >  
> 

Re: [PATCH] ARM: dts: sunxi: Add MMC alias for consistent enumeration

[Stefan Gloor]

Hi Chen-Yu and Andre, thank you for your input.


On 01.01.25 09:58, Chen-Yu Tsai wrote:

> IIRC we have chosen against adding explicit MMC aliases for the sunxi
> platforms. Please try using UUID= or PARTUUID= for root partition
> references.

I see, no problem. Out of curiosity, what is the reason for this 
decision? I couldn't find related discussions on the mailing list.

Personally, I like the UUID= or PARTUUID= method and I do have a working 
Yocto setup that uses PARTUUID to find the rootfs. It integrates nicely 
with the automatic disk/uuid creation and U-boot using extlinux.conf 
[1]. I do however understand that this requires major changes to the 
meta-sunxi layer and the maintainers don't like breaking a working 
system with U-boot scripts. So I went with the easier way of adding 
aliases. For future reference to anyone, this patch has now been added 
downstream to meta-sunxi [2].

[1] https://github.com/linux-sunxi/meta-sunxi/pull/431

[2] https://github.com/linux-sunxi/meta-sunxi/pull/432


On 02.01.25 00:28, Andre Przywara wrote:
> Yes, and LABEL= would be another way.
>
> Alternatively you could use U-Boot's DT when booting the board. U-Boot
> will add those aliases during build time (because it needs them for its
> own purposes), and by using $fdtcontroladdr (instead of $fdt_addr_r,
> on the bootz command line) you get this for free, and even save yourself
> from loading some DT.
> We sync the DTs regularly from the kernel to U-Boot now, and at the
> moment they are identical between the kernel and U-Boot, so you don't
> lose anything.

Thanks Andre for the helpful tips. I'll remember this for future reference.

Best,

Stefan

Re: [PATCH] ARM: dts: sunxi: Add MMC alias for consistent enumeration

[Andre Przywara]

On Sun, 5 Jan 2025 13:47:55 +0100
Stefan Gloor <code@stefan-gloor.ch> wrote:

Hi,

> Hi Chen-Yu and Andre, thank you for your input.
> 
> On 01.01.25 09:58, Chen-Yu Tsai wrote:
> 
> > IIRC we have chosen against adding explicit MMC aliases for the sunxi
> > platforms. Please try using UUID= or PARTUUID= for root partition
> > references.  
> 
> I see, no problem. Out of curiosity, what is the reason for this 
> decision? I couldn't find related discussions on the mailing list.

There were quite some discussions a while ago, I remember Maxime being
involved back then. But I am too lazy to dig those out now, maybe I
find some pointers later.

There are arguments for and against this, and it's not crystal clear.
Some platforms (Rockchip?) decided to allow them, but we said no.
One key argument was that the kernel never promised any kind of
consistent numbering, whether it's /dev/mmcblk0 or /dev/mmcblk1 is
entirely up to the kernel to decide. And we don't have this connection
for other devices, for instance USB disks. The plug and play nature of
those makes it obvious that we cannot guarantee this anyway, and I have
seen systems where a USB stick at boot became /dev/sda, and the first
SATA disk was enumerated as /dev/sdb.
Which means the reliance on numbering if ill-fated, and we need (and
have!) better and more sustainable methods anyway, so why not use them
everywhere? The mainline kernel tends to push against sketchy solutions
to very particular problems, because they often come back and bite us
later, and then nobody of the original people are around anymore to fix
them.

I see your point of "but it's such an easy solution", and there is an
argument that the hardware dictates some kind of ordering (SDC0 vs.
SDC3), but I don't think it outweighs the argument above, and lures
people into relying on things that the kernel doesn't want to promise.
 
> Personally, I like the UUID= or PARTUUID= method and I do have a working 
> Yocto setup that uses PARTUUID to find the rootfs. It integrates nicely 
> with the automatic disk/uuid creation and U-boot using extlinux.conf 
> [1]. I do however understand that this requires major changes to the 
> meta-sunxi layer and the maintainers don't like breaking a working 
> system with U-boot scripts. So I went with the easier way of adding 
> aliases. For future reference to anyone, this patch has now been added 
> downstream to meta-sunxi [2].

That's a shame, and I really wish Yocto people wouldn't just take
patches that have not been reviewed or even NAK'ed upstream. Those
meta layers becoming a pool of unreviewed or rejected patches is
really a bad side effect of Yocto making it easy to apply patches.

Another point: this is fixing the problem for just this particular
board. What about the other V3s boards, and in extension the other
Allwinner boards? I count 240 DTs for 32-bit boards alone in the repo,
so a more generic solutions sounds much more reasonable than applying
band-aid to one board.

Cheers,
Andre

> [1] https://github.com/linux-sunxi/meta-sunxi/pull/431
> 
> [2] https://github.com/linux-sunxi/meta-sunxi/pull/432
> 
> 
> On 02.01.25 00:28, Andre Przywara wrote:
> > Yes, and LABEL= would be another way.
> >
> > Alternatively you could use U-Boot's DT when booting the board. U-Boot
> > will add those aliases during build time (because it needs them for its
> > own purposes), and by using $fdtcontroladdr (instead of $fdt_addr_r,
> > on the bootz command line) you get this for free, and even save yourself
> > from loading some DT.
> > We sync the DTs regularly from the kernel to U-Boot now, and at the
> > moment they are identical between the kernel and U-Boot, so you don't
> > lose anything.  
> 
> Thanks Andre for the helpful tips. I'll remember this for future reference.
> 
> Best,
> 
> Stefan
> 
>