From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peng Fan Date: Sat, 27 Apr 2019 00:11:33 +0000 Subject: [U-Boot] [REGRESSION] [PATCH] imx: 8qxp_mek: fix fdt_file and console In-Reply-To: References: <20190415121352.10138-1-peng.fan@nxp.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de > Subject: [REGRESSION] [PATCH] imx: 8qxp_mek: fix fdt_file and console > > Hi Peng and Stefano > > On Mon, 2019-04-15 at 12:00 +0000, Peng Fan wrote: > > Fix fdt_file and console to boot upstream Linux Kernel. > > > > Upstream linux use imx8qxp-mek.dtb, and pass lpuart32 to earlycon will > > not work for i.MX8QXP, only need to pass earlycon, check > > drivers/tty/serial/earlycon.c, " /* Just 'earlycon' is a valid param > > for devicetree and ACPI SPCR. > > */ " > > > > Signed-off-by: Peng Fan > > --- > > include/configs/imx8qxp_mek.h | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/include/configs/imx8qxp_mek.h > > b/include/configs/imx8qxp_mek.h index 1cff18e05e..5c83505ff0 100644 > > --- a/include/configs/imx8qxp_mek.h > > +++ b/include/configs/imx8qxp_mek.h > > @@ -65,11 +65,11 @@ > > "script=boot.scr\0" \ > > "image=Image\0" \ > > "panel=NULL\0" \ > > - "console=ttyLP0,${baudrate} > > earlycon=lpuart32,0x5a060000,${baudrate}\0" \ > > + "console=ttyLP0,${baudrate} earlycon\0" \ > > I don't think using ${baudrate} in console= like that actually works. > At least for me this lead to the following: You mean include/env_default.h this not work? Regards, Peng. > > root at colibri-imx8qxp:~# cat /proc/cmdline console=ttyLP3,${baudrate} > earlycon,115200 root=PARTUUID=5e6e3605-02 rootwait > mmcautodetect=yes > > E.g. ${baudrate} is never actually evaluated unless one does an explicit > assignment aka setenv something something. > > > "fdt_addr=0x83000000\0" \ > > "fdt_high=0xffffffffffffffff\0" \ > > "boot_fdt=try\0" \ > > - "fdt_file=fsl-imx8qxp-mek.dtb\0" \ > > + "fdt_file=imx8qxp-mek.dtb\0" \ > > "initrd_addr=0x83800000\0" \ > > "initrd_high=0xffffffffffffffff\0" \ > > "mmcdev="__stringify(CONFIG_SYS_MMC_ENV_DEV)"\0" \ > > -- > > 2.16.4 > > > > _______________________________________________ > > U-Boot mailing list > > U-Boot at lists.denx.de > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flist > > > s.denx.de%2Flistinfo%2Fu-boot&data=02%7C01%7Cpeng.fan%40nxp.co > m%7C > > > 0e6a5db6025549efd94708d6ca76ec09%7C686ea1d3bc2b4c6fa92cd99c5c30 > 1635%7C > > > 0%7C0%7C636919009450074747&sdata=g8pl8UzcwoFAlfZAHDOeBb1s4 > zcH%2BzJ > > qFkGukoHAfnQ%3D&reserved=0 > > Cheers > > Marcel