From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 13 Apr 2016 17:59:44 -0400 Subject: [U-Boot] [PATCH] jetson-tk1: Set fdtfile environment variable In-Reply-To: <570E665F.405@wwwdotorg.org> References: <1460551699-30796-1-git-send-email-afaerber@suse.de> <570E41C0.1010904@suse.de> <570E665F.405@wwwdotorg.org> Message-ID: <20160413215944.GL13577@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, Apr 13, 2016 at 09:31:43AM -0600, Stephen Warren wrote: > On 04/13/2016 06:55 AM, Andreas F?rber wrote: > >Am 13.04.2016 um 14:48 schrieb Andreas F?rber: > >>The 4.5.0 kernel cannot cope with U-Boot's internal device tree, and the > >>distro boot commands are looking for $fdtfile, so provide it to avoid > >>having users supply a dumb boot.scr doing a setenv fdtfile ...; boot, > >>defeating the purpose of generic EFI boot. > >> > >>Cc: Stephen Warren > >>Cc: Alexander Graf > >>Signed-off-by: Andreas F?rber > >>--- > >> include/configs/jetson-tk1.h | 4 ++++ > >> 1 file changed, 4 insertions(+) > >> > >>diff --git a/include/configs/jetson-tk1.h b/include/configs/jetson-tk1.h > >>index 59dbb20..82a4be4 100644 > >>--- a/include/configs/jetson-tk1.h > >>+++ b/include/configs/jetson-tk1.h > >>@@ -63,6 +63,10 @@ > >> /* General networking support */ > >> #define CONFIG_CMD_DHCP > >> > >>+#define BOARD_EXTRA_ENV_SETTINGS \ > >>+ "fdtfile=tegra124-jetson-tk1.dtb\0" \ > >>+ "" > > > >Is there any more intelligent solution than doing this for each board? > > Yes, the distro boot scripts shouldn't be using $fdtfile > unconditionally since it's not guaranteed to be set. The model is > that boot scripts determine the FDT filename, and $fdtfile is an > optional override. > > It looks like the hard-coded use of $fdtfile was added into the EFI > path, which I didn't get to review, and which shouldn't be enabled > by default but unfortunately is. Bah. But the good news is we haven't done a release with EFI stuff yet, so we can fix it. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: