From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Sat, 10 Feb 2018 11:25:11 +0100 Subject: [U-Boot] [PATCH v2] spl: eMMC/SD: Provide one __weak spl_boot_mode() function In-Reply-To: <173841a1-c54b-225d-4557-a5ed6ec757c8@denx.de> References: <20180203072952.5459-1-lukma@denx.de> <20180209235315.GF9245@largo.jsg.id.au> <20180210014516.21d534ea@jawa> <20180210032149.GG9245@largo.jsg.id.au> <20180210105726.419b0f94@jawa> <173841a1-c54b-225d-4557-a5ed6ec757c8@denx.de> Message-ID: <20180210112511.2d24ec70@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sat, 10 Feb 2018 11:00:12 +0100 Marek Vasut wrote: > On 02/10/2018 10:57 AM, Lukasz Majewski wrote: > > Hi Jonathan, > > > >> On Sat, Feb 10, 2018 at 01:45:16AM +0100, Lukasz Majewski wrote: > >>> Hi Jonathan, > >>> > >>>> On Sat, Feb 03, 2018 at 11:00:35AM -0200, Fabio Estevam > >>>> wrote: > >>>>> On Sat, Feb 3, 2018 at 5:29 AM, Lukasz Majewski > >>>>> wrote: > >>>>>> The goal of this patch is to clean up the code related to > >>>>>> choosing SPL MMC boot mode. > >>>>>> > >>>>>> The spl_boot_mode() now is called only in spl_mmc_load_image() > >>>>>> function, which is only compiled in if CONFIG_SPL_MMC_SUPPORT > >>>>>> is enabled. > >>>>>> > >>>>>> To achieve the goal, all per mach/arch implementations > >>>>>> eligible for unification has been replaced with one __weak > >>>>>> implementation. > >>>>>> > >>>>>> Signed-off-by: Lukasz Majewski > >>>>>> Reviewed-by: Marek Vasut > >>>>>> Reviewed-by: Stefano Babic > >>>>>> Acked-by: Michal Simek (For > >>>>>> ZynqMP) > >>>>> > >>>>> Nice cleanup: > >>>>> > >>>>> Reviewed-by: Fabio Estevam > >>>> > >>>> This has broken booting via mmc with mx6cuboxi for me. > >>>> > >>>> SPL loops on > >>>> > >>>> U-Boot SPL 2018.03-rc1-00212-g48914fc119 (Feb 10 2018 - 11:04:33 > >>>> +1300) Trying to boot from MMC1 > >>>> Failed to mount ext2 filesystem... > >>>> spl_load_image_ext: ext4fs mount err - 0 > >>> > >>> Could you check what is the status of following defines in > >>> your .config file: > >>> > >>> CONFIG_SPL_FAT_SUPPORT > >>> CONFIG_SUPPORT_EMMC_BOOT > >> > >> "# CONFIG_SPL_FAT_SUPPORT is not set" > >> > >> CONFIG_SUPPORT_EMMC_BOOT is not in the .config at all > >> > >> CONFIG_SPL_EXT_SUPPORT previously for imx6 would result in > >> MMCSD_MODE_RAW but now it results in MMCSD_MODE_FS. > >> > >>> > >>> And if by any chance your don't have: > >>> CONFIG_SPL_EXT_SUPPORT defined? > >> > >> CONFIG_SPL_EXT_SUPPORT=y > > > > Could you for test comment out the above define? > > If you look at the logic in spl_mmc.c , you'll see this will work. But > what about users who have EXT enabled and want to use it as fallback > after loading from RAW failed ? :) > The code now in __weak function has been took directly from socfpga port (as looked as clean and simple). Apparently, this port did not provide such fallback facility :-) It seems to me that the code from imx function would need to be used instead. 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-10 Fax: (+49)-8142-66989-80 Email: wd 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: