[PATCH 1/1] pico-imx7d: add baseboard SD card boot detect
Szőke Kálmán Benjamin
egyszeregy at freemail.hu
Sun Nov 19 22:08:00 CET 2023
Hi,All previouse stage/patches was a "blind alley" solution because finally i found, many code parts are ready from freesacle sources. So needed to refactor the full codes, which seems now a totally new code/solution for it.So, I don't see the point in describing them in a changelog, useless to share logs about crappy attempts. Also i do not know how can be the git configured to use custom PATCH prefix, it was generated by git automatically.(void)(devno); is need because devno arguments is an unused variable, need to use this solution to suppress any potential warnings.I am ready with this patch, i will not change anything, i am done. If you feel something is missing in the descriptions or in code feel free to change it in text of my patch source or in a new patch, you can do it, if you are a maintainer. Now it's your turn to finalize it if something need more.-------- Eredeti levél --------Feladó: Fabio Estevam <festevam at gmail.com>Dátum: 2023 november 15 18:31:21Tárgy: Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detectCímzett: egyszeregy at freemail.huHi Benjamin, You should have indicated that this is v7. The subject should be: [PATCH v7] pico-imx7d: add baseboard SD card boot detect No need for adding 1/1 for a single patch. On Tue, Nov 7, 2023 at 7:16 PM <egyszeregy at freemail.hu> wrote: > > From: Benjamin Szőke <egyszeregy at freemail.hu> > > Technexion PICO-IMX7 SoM is supporting USDHC3 (eMMC or micro SD on SoM) > and USDHC1 (SD on carrier board) to use on any carrier board like > PICO-NYMPH. Based on the U-Boot version from Technexion it adds > baseboard SD card boot detect to able to boot from selected USDHC1 > or USDHC3 boot devices. > > - Add baseboard SD card boot detect from Technexion's U-Boot. > - Implement board_mmc_get_env_dev() for pico-imx7d SoM. > - Implement mmc_map_to_kernel_blk() for pico-imx7d SoM. > - Add board_late_mmc_env_init() to use from common freescale > functions to able to provide "mmcdev" and "mmcroot" variables > in environment of U-boot. > - Add "mmc1" alias to use for usdhc1 in imx7d-pico-pi-u-boot.dtsi. > - In default environment set "mmcautodetect=yes" to use boot detection. These comments above should be placed below the --- line. > Signed-off-by: Benjamin Szőke <egyszeregy at freemail.hu> > --- Here you should place the changelog: Changes since v6: - bla bla bla > +#if CONFIG_IS_ENABLED(FSL_ESDHC_IMX) > +#if CONFIG_IS_ENABLED(ENV_IS_IN_MMC) || CONFIG_IS_ENABLED(ENV_IS_NOWHERE) > +int board_mmc_get_env_dev(int devno) > +{ > + int dev_env = 0; > + (void)(devno); Why? Just drop this ' (void)(devno);' line. I tested it successfully, so: Tested-by: Fabio Estevam <festevam at gmail.com>
More information about the U-Boot
mailing list