From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Wed, 30 Jan 2019 07:50:09 +0100 Subject: [U-Boot] [PATCH 16/20] imx8m: fix sd to mmc1 and emmc to mmc2 for i.MX8MM In-Reply-To: References: <20190128094807.26532-1-peng.fan@nxp.com> <20190128094807.26532-17-peng.fan@nxp.com> <20190129235759.3b561dfb@jawa> Message-ID: <20190130075009.01e71897@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de Hi Peng, > > -----Original Message----- > > From: Lukasz Majewski [mailto:lukma at denx.de] > > Sent: 2019年1月30日 6:58 > > To: Peng Fan > > Cc: sbabic at denx.de; Fabio Estevam ; > > dl-uboot-imx ; u-boot at lists.denx.de > > 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). > > 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 > > > --- > > > 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: > > lukma at denx.de 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: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: