> -----Original Message-----
> From: Lukasz Majewski [mailto:[email protected]]
> Sent: 2019年1月30日 14:50
> To: Peng Fan <[email protected]>
> Cc: [email protected]; Fabio Estevam <[email protected]>;
> dl-uboot-imx <[email protected]>; [email protected]
> Subject: Re: [U-Boot] [PATCH 16/20] imx8m: fix sd to mmc1 and emmc to
> mmc2 for i.MX8MM
> 
> Hi Peng,
> 
> > > -----Original Message-----
> > > From: Lukasz Majewski [mailto:[email protected]]
> > > Sent: 2019年1月30日 6:58
> > > To: Peng Fan <[email protected]>
> > > Cc: [email protected]; Fabio Estevam <[email protected]>;
> > > dl-uboot-imx <[email protected]>; [email protected]
> > > Subject: Re: [U-Boot] [PATCH 16/20] imx8m: fix sd to mmc1 and emmc
> > > to mmc2 for i.MX8MM
> > >
> > > Hi Peng,
> > >
> > > > Since the SD is usdhc2 and eMMC is usdhc3,
> > >
> > > Is this true on all IMX8M boards? Or is it only on the development
> > > kit you do have?
> >
> > This is a hack for board, needs to be fixed in next version.
> >
> > >
> > > My point is that this shall be setup by DTS aliases or maybe by
> > > Kconfig option.
> >
> > Could you share more information?
> 
> For example:
> 
> / {
>       model = "K+P iMX6Q";
>       compatible = "kp,imx6-kp", "fsl,imx6";
> 
>       aliases {
>               mmc0 = &usdhc2;
>               mmc1 = &usdhc4;
>               usb1 = &usbh1;
>       };
> 
>       chosen {
>               stdout-path = &uart1;
>       };
> 
> 
> And the "aliases" set the order in which you get the devices (like mmc0,
> mmc1 above).

Then we need SPL to have Device tree support, right?

Thanks,
Peng.

> 
> >
> > Thanks,
> > Peng.
> >
> > >
> > > > this cause mapping problem
> > > > for spl_boot_device. So far hard coded them to correct MMC index,
> > > > so that SD and eMMC boot can work.
> > > >
> > > > Signed-off-by: Peng Fan <[email protected]>
> > > > ---
> > > >  arch/arm/mach-imx/spl.c | 9 +++++++++
> > > >  1 file changed, 9 insertions(+)
> > > >
> > > > diff --git a/arch/arm/mach-imx/spl.c b/arch/arm/mach-imx/spl.c
> > > > index ebd8ff9290..0048832be8 100644
> > > > --- a/arch/arm/mach-imx/spl.c
> > > > +++ b/arch/arm/mach-imx/spl.c
> > > > @@ -147,9 +147,18 @@ u32 spl_boot_device(void)
> > > >         case SD1_BOOT:
> > > >         case MMC1_BOOT:
> > > >                 return BOOT_DEVICE_MMC1;
> > > > +#if defined(CONFIG_IMX8MM)
> > > > +       case SD2_BOOT:
> > > > +       case MMC2_BOOT:
> > > > +               return BOOT_DEVICE_MMC1;
> > > > +       case SD3_BOOT:
> > > > +       case MMC3_BOOT:
> > > > +               return BOOT_DEVICE_MMC2;
> > > > +#else
> > > >         case SD2_BOOT:
> > > >         case MMC2_BOOT:
> > > >                 return BOOT_DEVICE_MMC2;
> > > > +#endif
> > > >  #endif
> > > >         case NAND_BOOT:
> > > >                 return BOOT_DEVICE_NAND;
> > >
> > >
> > >
> > >
> > > Best regards,
> > >
> > > Lukasz Majewski
> > >
> > > --
> > >
> > > DENX Software Engineering GmbH,      Managing Director: Wolfgang
> > > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell,
> > > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email:
> > > [email protected]
> 
> 
> 
> 
> Best regards,
> 
> Lukasz Majewski
> 
> --
> 
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email:
> [email protected]
_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to