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

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@gmail.com mailto:festevam@gmail.com> wrote:

Hi Vanessa,

On Wed, Aug 23, 2017 at 11:05 AM, Vanessa Maegima
<vanessa.maegima@nxp.com <mailto:vanessa.maegima@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