[U-Boot] [PATCH] evb_rk3399: revert CONFIG_SYS_MMC_ENV_DEV to 0

Philipp Tomsich philipp.tomsich at theobroma-systems.com
Fri Dec 14 10:13:09 UTC 2018


+Kever

Max,

> On 14.12.2018, at 09:21, Max Kellermann <max at blarg.de> wrote:
> 
> On 2018/12/13 22:59, Philipp Tomsich <philipp.tomsich at theobroma-systems.com> wrote:
>> On 04.12.2018, at 12:00, Max Kellermann <max.kellermann at gmail.com> wrote:
>>> 
>>> This was changed to 1 in commit 0717dde057e, but a few months later,
>>> commit 5f9411af37b swapped the order of eMMC and SD card by assigning
>>> indexed aliases to `&sdhci` and `&sdmmc`.
>> 
>> If this is a straight revert, I’d appreciate it if the was marked as such by
>> creating the commit using a 'git revert’ and then adding additional background
>> to explain why the revert was done.
> 
> Technically, it is a straight revert, but I replaced git's boilerplate
> text with a descriptive commit message, without omitting any
> information from the boilerplate.  If you prefer to leave the full
> boilerplate text in, I can re-add it and resend the patch.
> 
> However, semantically, it is not a revert, because it does not revert
> the semantic effect of commit 0717dde057e; quite opposite, it restores
> the effect of commit 0717dde057e after it was broken by commit
> 5f9411af37b.
> 
>> However, we have a bigger issue here: the original commit has been in the
>> tree since 2016 and I am reluctant to change such fundamental naming without
>> a rationale.  This is particularily in knowledge of the on-list discussion that we’ve
>> had with Kever recently regarding his request to change the search order for booting.
> 
> The naming was (accidently?) changed in commit 5f9411af37b, which
> effectively undid the effect of commit 0717dde057e, and my patch only
> repairs that.
> 
>> Could you explain why you think that this should be changed?
> 
> The goal of commit 0717dde057e was to boot from eMMC by default, and
> at the time, eMMC was "mmc1".  Later, commit 5f9411af37b swapped the
> order of "mmc", making eMMC="mmc0".  This broke commit 0717dde057e.
> 
> This is how it looks like on my AIO-3399J:
> 
> => mmc list
> dwmmc at fe320000: 1
> sdhci at fe330000: 0 (eMMC)
> 
> As you see, CONFIG_SYS_MMC_ENV_DEV=1 will boot from SD card, not from
> eMMC.
> 
> I submitted the patch because it looked like the side effect of commit
> 5f9411af37b was an accident, so I attempted fixed it.

Unfortunately this seems less like a side-effect of that change than of the fact
that there’s a variety of RK3399 boards out there that don’t have dedicated 
board-support in U-Boot and the EVB-defaults may be wrong.
The RK3399-EVB board-suupport does not provide any support to either identify
what board it’s on or to allow overriding these device-order.

@Kever: Could you please confirm whether the CONFIG_SYS_MMC_ENV_DEV
is appropriate for the EVB itself or if it should be changed for the EVB’s config
globally?

Thanks,
Philipp.


More information about the U-Boot mailing list