On Tue, Nov 29, 2016 at 07:26:55PM -0700, Simon Glass wrote: > Hi Marek, > > On 29 November 2016 at 18:28, Marek Vasut <ma...@denx.de> wrote: > > On 11/30/2016 01:32 AM, Simon Glass wrote: > >> Hi Marek, > >> > >> On 25 November 2016 at 15:32, Marek Vasut <ma...@denx.de> wrote: > >>> Add new configuration option CONFIG_MMC_TINY which strips away all > >>> memory allocation within the MMC code and code for handling multiple > >>> cards. This allows extremely space-constrained SPL code use the MMC > >>> framework. > >>> > >>> Signed-off-by: Marek Vasut <ma...@denx.de> > >>> Cc: Tom Rini <tr...@konsulko.com> > >>> Cc: Simon Glass <s...@chromium.org> > >>> --- > >>> V2: Switch the MMC_TINY option to Kconfig > >>> --- > >>> common/spl/spl_mmc.c | 6 +++++- > >>> drivers/mmc/Kconfig | 16 ++++++++++++++++ > >>> drivers/mmc/mmc.c | 31 ++++++++++++++++++++++++++++++- > >>> drivers/mmc/mmc_legacy.c | 32 ++++++++++++++++++++++++++++++++ > >>> include/mmc.h | 1 + > >>> 5 files changed, 84 insertions(+), 2 deletions(-) > >> > >> Reviewed-by: Simon Glass <s...@chromium.org> > >> > >> nit below. > >> > >> I assume this does not use DM? > > > > Nope, DM does not fit into those 14 kiB I have for SPL, no way. > > > >>> diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c > >>> index 43c1030..3cad998 100644 > >>> --- a/common/spl/spl_mmc.c > >>> +++ b/common/spl/spl_mmc.c > >>> @@ -306,7 +306,11 @@ static int spl_mmc_load_image(struct spl_image_info > >>> *spl_image, > >>> if (part == 7) > >>> part = 0; > >>> > >>> - err = blk_dselect_hwpart(mmc_get_blk_desc(mmc), > >>> part); > >>> + if (CONFIG_IS_ENABLED(MMC_TINY)) > >>> + err = mmc_switch_part(mmc, part); > >>> + else > >>> + err = > >>> blk_dselect_hwpart(mmc_get_blk_desc(mmc), part); > >>> + > >>> if (err) { > >>> #ifdef CONFIG_SPL_LIBCOMMON_SUPPORT > >>> puts("spl: mmc partition switch > >>> failed\n"); > >>> diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig > >>> index 24f4b28..aca438b8 100644 > >>> --- a/drivers/mmc/Kconfig > >>> +++ b/drivers/mmc/Kconfig > >>> @@ -26,6 +26,22 @@ config DM_MMC_OPS > >>> option will be removed as soon as all DM_MMC drivers use it, as > >>> it > >>> will the only supported behaviour. > >>> > >>> +config SPL_MMC_TINY > >>> + bool "Tiny MMC framework in SPL" > >>> + default n > >> > >> Not needed? > > > > Why not ? This makes the tinification configurable and is used on the > > CI20 (JZ4780 horrorchip). > > Well it should default to n anyway so I don't think you need that line.
To be clear, there's a lot of "default n" which isn't needed, but I hadn't publicly said anything about, but yes, we should stop adding that line as it is the normal default. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot