[U-Boot] [PATCH] pico-imx7d: Add support for secure boot

[Stefano Babic]

Hi Vanessa,

On 23/08/2017 18:16, Vanessa Ayumi Maegima wrote:
> Hi Fabio,
> 
> On Wed, Aug 23, 2017 at 11:11 AM, Fabio Estevam <festevam at gmail.com
> <mailto:festevam at gmail.com>> wrote:
> 
>     Hi Vanessa,
> 
>     On Wed, Aug 23, 2017 at 11:05 AM, Vanessa Maegima
>     <vanessa.maegima at nxp.com <mailto:vanessa.maegima at nxp.com>> wrote:
>     > Add a new defconfig to enable secure boot on pico-imx7d.
> 
>     Please explain in the commit log why you are adding this new defconfig.
> 
>     Anyway, I understant that this could be useful if someone tries to
>     boot this board with NXP 4.1.15 kernel.
> 
>     Not sure it is worth the effort of adding a new defconfig only for
>     this purpose as mainline kernel can be used with existing U-Boot
>     defconfig.
> 
> 
> ​I made this patch to enable booting with linux-fslc 4.1 kernel,
> but if you think it is not worth maintaining a new defconfig we
> can skip it.​

I agree that adding a new defconfig just for it seems overkilling (we
have to maintain, etc.). But you can maybe add documentation in the
README file, adding a section for NXP 4.1 kernel, and explaining how to
setup U-Boot with secure boot (with "make menuconfig, etc.).

Best regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================