[U-Boot] [RESENT PATCH v3] rockchip: update emmc/sd index for distro boot order

Kever Yang kever.yang at rock-chips.com
Thu Jan 24 08:21:31 UTC 2019


The comment is what we want to do, so it's not correct to fix the
comment only.

    "First try to boot from SD() , then eMMC()"


Thanks,
- Kever
On 01/23/2019 07:00 PM, Philipp Tomsich wrote:
> My gut feeling would be to fix the comment only.
>
>> On 23.01.2019, at 03:14, Kever Yang <kever.yang at rock-chips.com
>> <mailto:kever.yang at rock-chips.com>> wrote:
>>
>> Hi Philipp, Simon,
>>
>>    Is it possible for this patch to be merged?
>>    Right now, two board owner does not  like this patch but both of
>> you says this patch do not affect your board, and most of other
>> boards are maintained by me, and I have share why I need this patch
>> and why this patch not affect so much very clearly in previous mail.
>>     Please evaluate it again and close this topic, at least the
>> original code comment "First try to boot from SD (index 0), then eMMC
>> (index 1) " is not correct, something must be fixed.
>>
>> Thanks,
>> - Kever
>>
>> Kever Yang <kever.yang at rock-chips.com
>> <mailto:kever.yang at rock-chips.com>> 于2018年11月28日周三 上午10:06写道:
>>
>>     According to the emmc/sdcard index in dts alias, emmc is always 0 and
>>     sdcard index is 1, let's update to using correct mmc number for
>>     distro
>>     boot order in common header.
>>
>>     SD card suppost to have higher priority so that people can boot into
>>     the firmware in SD card, this is very convenient for developer
>>     try with
>>     distro img from SUSE, Fedora and etc. Developer only need to 'dd' the
>>     Distro image(which id download from OS vendor release) into SD
>>     card without
>>     any modify and then we can boot it up directly.
>>
>>     Signed-off-by: Kever Yang <kever.yang at rock-chips.com
>>     <mailto:kever.yang at rock-chips.com>>
>>     ---
>>
>>     Changes in v3:
>>     - update the commit message
>>     Series-changes: 2
>>     - update the commit message
>>
>>      include/configs/rockchip-common.h | 6 +++---
>>      1 file changed, 3 insertions(+), 3 deletions(-)
>>
>>     diff --git a/include/configs/rockchip-common.h
>>     b/include/configs/rockchip-common.h
>>     index 68e1105a4b..8a72613e52 100644
>>     --- a/include/configs/rockchip-common.h
>>     +++ b/include/configs/rockchip-common.h
>>     @@ -11,11 +11,11 @@
>>
>>      #ifndef CONFIG_SPL_BUILD
>>
>>     -/* First try to boot from SD (index 0), then eMMC (index 1) */
>>     +/* First try to boot from SD (index 1), then eMMC (index 0) */
>>      #if CONFIG_IS_ENABLED(CMD_MMC)
>>             #define BOOT_TARGET_MMC(func) \
>>     -               func(MMC, mmc, 0) \
>>     -               func(MMC, mmc, 1)
>>     +               func(MMC, mmc, 1) \
>>     +               func(MMC, mmc, 0)
>>      #else
>>             #define BOOT_TARGET_MMC(func)
>>      #endif
>>     -- 
>>     2.18.0
>>
>



More information about the U-Boot mailing list