From mboxrd@z Thu Jan 1 00:00:00 1970 From: Igor Opaniuk Date: Wed, 24 Jul 2019 15:19:31 +0300 Subject: [U-Boot] [PATCH v1] colibri_imx7: boot kernel in secure mode In-Reply-To: <5cea7bd07d31801ae20cd6841d4d75c2@agner.ch> References: <20190703135028.19271-1-igor.opaniuk@gmail.com> <5cea7bd07d31801ae20cd6841d4d75c2@agner.ch> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de Hi Stefan On Wed, Jul 24, 2019 at 2:32 PM Stefan Agner wrote: > > On 2019-07-03 15:50, Igor Opaniuk wrote: > > From: Igor Opaniuk > > > > NXP downstream kernel uses legacy method to enable other cores, > > which requires kernel to run in a security mode (althought upstream ker= nel > > uses PSCI for this). > > > > As we're using NXP kernel in our BSPs, lets enable this by default. > > I think I mentioned it internally, but just to be clear: I do not agree > with this patch. > > Upstream U-Boot should be able to boot upstream Linux. I'd rather prefer > to carry that patch in our downstream fork (or set the bootm_boot_mode > environment in our boot script). Totally agree with you here, BTW, seems that Stefano has applied by mistake to u-boot-imx fcc79eee14("colibri_imx7_emmc: enable CONFIG_ARMV7_BOOT_SEC_DEFAULT"), although I've asked to ignore it. Probably will send another patch that reverts this. > > -- > Stefan > > > > > Signed-off-by: Igor Opaniuk > > --- > > > > include/configs/colibri_imx7.h | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/include/configs/colibri_imx7.h b/include/configs/colibri_i= mx7.h > > index 40173b18fa..5408577661 100644 > > --- a/include/configs/colibri_imx7.h > > +++ b/include/configs/colibri_imx7.h > > @@ -164,7 +164,8 @@ > > "${board}/flash_blk.img && source ${loadaddr}\0" \ > > "setup=3Dsetenv setupargs " \ > > "console=3Dtty1 console=3D${console}" \ > > - ",${baudrate}n8 ${memargs} consoleblank=3D0\0" \ > > + ",${baudrate}n8 ${memargs} consoleblank=3D0; " \ > > + "setenv bootm_boot_mode sec\0" \ > > "setupdate=3Drun setsdupdate || run setusbupdate || run setethupd= ate\0" \ > > "setusbupdate=3Dusb start && setenv interface usb && " \ > > "fatload ${interface} 0:1 ${loadaddr} " \ --=20 Best regards - Freundliche Gr=C3=BCsse - Meilleures salutations Igor Opaniuk mailto: igor.opaniuk at gmail.com skype: igor.opanyuk +380 (93) 836 40 67 http://ua.linkedin.com/in/iopaniuk