diff mbox series

Fix MMC numbering issue for Raspberry Pi 3

Message ID 20211027222615.18988-1-karels@FreeBSD.org
State Accepted
Commit 874e544e89a7f9f42796b23f8d43c56e203b6e69
Delegated to: Matthias Brugger
Headers show
Series Fix MMC numbering issue for Raspberry Pi 3 | expand

Commit Message

Mike Karels Oct. 27, 2021, 10:26 p.m. UTC
From: Mike Karels <karels@FreeBSD.org>

Using mmc.dtbo from rpi-firmware to switch the controller for the SD
card slot from sdhci to sdhost causes the numbering to change; the
SD card is then not recognized at boot.  Add to the range checked.
---
 include/configs/rpi.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Emmanuel Vadot Oct. 28, 2021, 7:44 a.m. UTC | #1
On Wed, 27 Oct 2021 22:26:15 +0000
karels@FreeBSD.org wrote:

> From: Mike Karels <karels@FreeBSD.org>
> 
> Using mmc.dtbo from rpi-firmware to switch the controller for the SD
> card slot from sdhci to sdhost causes the numbering to change; the
> SD card is then not recognized at boot.  Add to the range checked.
> ---
>  include/configs/rpi.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/include/configs/rpi.h b/include/configs/rpi.h
> index 55768a46da..4c5c1ac31f 100644
> --- a/include/configs/rpi.h
> +++ b/include/configs/rpi.h
> @@ -137,7 +137,8 @@
>  #if CONFIG_IS_ENABLED(CMD_MMC)
>  	#define BOOT_TARGET_MMC(func) \
>  		func(MMC, mmc, 0) \
> -		func(MMC, mmc, 1)
> +		func(MMC, mmc, 1) \
> +		func(MMC, mmc, 2)
>  #else
>  	#define BOOT_TARGET_MMC(func)
>  #endif
> -- 
> 2.32.0
> 

 Reviewed-by: Emmanuel Vadot <manu@FreeBSD.org>
Matthias Brugger Nov. 30, 2021, 3:09 p.m. UTC | #2
On Thu, Oct 28, 2021 at 09:44:09AM +0200, Emmanuel Vadot wrote:
> On Wed, 27 Oct 2021 22:26:15 +0000
> karels@FreeBSD.org wrote:
> 
> > From: Mike Karels <karels@FreeBSD.org>
> > 
> > Using mmc.dtbo from rpi-firmware to switch the controller for the SD
> > card slot from sdhci to sdhost causes the numbering to change; the
> > SD card is then not recognized at boot.  Add to the range checked.
> > ---
> >  include/configs/rpi.h | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/include/configs/rpi.h b/include/configs/rpi.h
> > index 55768a46da..4c5c1ac31f 100644
> > --- a/include/configs/rpi.h
> > +++ b/include/configs/rpi.h
> > @@ -137,7 +137,8 @@
> >  #if CONFIG_IS_ENABLED(CMD_MMC)
> >  	#define BOOT_TARGET_MMC(func) \
> >  		func(MMC, mmc, 0) \
> > -		func(MMC, mmc, 1)
> > +		func(MMC, mmc, 1) \
> > +		func(MMC, mmc, 2)
> >  #else
> >  	#define BOOT_TARGET_MMC(func)
> >  #endif
> > -- 
> > 2.32.0
> > 
> 
>  Reviewed-by: Emmanuel Vadot <manu@FreeBSD.org>

Applied to rpi-next

Thanks!

> 
> -- 
> Emmanuel Vadot <manu@bidouilliste.com> <manu@freebsd.org>
diff mbox series

Patch

diff --git a/include/configs/rpi.h b/include/configs/rpi.h
index 55768a46da..4c5c1ac31f 100644
--- a/include/configs/rpi.h
+++ b/include/configs/rpi.h
@@ -137,7 +137,8 @@ 
 #if CONFIG_IS_ENABLED(CMD_MMC)
 	#define BOOT_TARGET_MMC(func) \
 		func(MMC, mmc, 0) \
-		func(MMC, mmc, 1)
+		func(MMC, mmc, 1) \
+		func(MMC, mmc, 2)
 #else
 	#define BOOT_TARGET_MMC(func)
 #endif