Hi Peng, > > -----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?
Yes, this would be the easiest way to support this feature. If your board doesn't have any footprint constraints (and boottime) then IMHO it would be worth to switch to full (including SPL) conversion to DM/DTS. Other option would be to provide such configuration via Kconfig or define spl_boot_device(void) in board's file. > > 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] 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]
pgplpv2EZK3n3.pgp
Description: OpenPGP digital signature
_______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
