From mboxrd@z Thu Jan 1 00:00:00 1970 From: Philipp Tomsich Date: Wed, 28 Nov 2018 10:15:45 +0100 Subject: [U-Boot] [RESENT PATCH v3] rockchip: update emmc/sd index for distro boot order In-Reply-To: <20181128020602.18058-1-kever.yang@rock-chips.com> References: <20181128020602.18058-1-kever.yang@rock-chips.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de Kever, > On 28.11.2018, at 03:06, Kever Yang wrote: > > 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. You never addressed the review comment from Klaus (from the review in May): > Also prioritising SD card over eMMC does not make any sense to me. At least on > RK3399 and RK3368 the default ROM boot order is first eMMC then SD card. So > starting U-Boot from eMMC and then loading the Kernel from SD-card doesn’t sound > right for me. This will change default behaviour and may break things for users in the field. Before we can move forward, we really need to establish a consensus on this and how users will be affected. While this doesn’t matter much for our boards, as we have logic to rewrite the default boot during boot-up anyway, I expect a lot of trouble for mainline users with their own boards... > Signed-off-by: Kever Yang > --- > > 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 >