diff mbox

[U-Boot,1/2] mx7: Place MX7_SEC option in Kconfig

Message ID 1465562773-12369-1-git-send-email-fabio.estevam@nxp.com
State Accepted
Commit 303977430983f163c30a8937de737d1d6abe78ba
Delegated to: Stefano Babic
Headers show

Commit Message

Fabio Estevam June 10, 2016, 12:46 p.m. UTC
MX7_SEC is an existing configuration option that allows booting the
kernel in secure mode.

Place this option in Kconfig, so that boards can select this option
in their defconfig files.

Selecting this option is necessary when booting a kernel provided by
NXP, such as 3.14_GA and 4.1.15_GA.

Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
 arch/arm/imx-common/Kconfig | 9 +++++++++
 1 file changed, 9 insertions(+)

Comments

Fabio Estevam June 17, 2016, 11:59 a.m. UTC | #1
Hi Stefano,

On Fri, Jun 10, 2016 at 9:46 AM, Fabio Estevam <fabio.estevam@nxp.com> wrote:
> MX7_SEC is an existing configuration option that allows booting the
> kernel in secure mode.
>
> Place this option in Kconfig, so that boards can select this option
> in their defconfig files.
>
> Selecting this option is necessary when booting a kernel provided by
> NXP, such as 3.14_GA and 4.1.15_GA.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>

Any comments about this one?

Also, it would be nice if you could create a 'next' branch so that non
bug fixes could be collected there.

Thanks
Stefano Babic June 17, 2016, 12:50 p.m. UTC | #2
Hi Fabio,

On 17/06/2016 13:59, Fabio Estevam wrote:
> Hi Stefano,
> 
> On Fri, Jun 10, 2016 at 9:46 AM, Fabio Estevam <fabio.estevam@nxp.com> wrote:
>> MX7_SEC is an existing configuration option that allows booting the
>> kernel in secure mode.
>>
>> Place this option in Kconfig, so that boards can select this option
>> in their defconfig files.
>>
>> Selecting this option is necessary when booting a kernel provided by
>> NXP, such as 3.14_GA and 4.1.15_GA.
>>
>> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
> 
> Any comments about this one?
> 

It is in my queue for applying it.

> Also, it would be nice if you could create a 'next' branch so that non
> bug fixes could be collected there.

Sure - I am still busy, but I plan to do this in the week-end, and also
applying the bug fixes to -master.

Regards,
Stefano
Michael Nazzareno Trimarchi June 17, 2016, 12:56 p.m. UTC | #3
Hi

On Fri, Jun 17, 2016 at 2:50 PM, Stefano Babic <sbabic@denx.de> wrote:
> Hi Fabio,
>
> On 17/06/2016 13:59, Fabio Estevam wrote:
>> Hi Stefano,
>>
>> On Fri, Jun 10, 2016 at 9:46 AM, Fabio Estevam <fabio.estevam@nxp.com> wrote:
>>> MX7_SEC is an existing configuration option that allows booting the
>>> kernel in secure mode.
>>>
>>> Place this option in Kconfig, so that boards can select this option
>>> in their defconfig files.
>>>
>>> Selecting this option is necessary when booting a kernel provided by
>>> NXP, such as 3.14_GA and 4.1.15_GA.
>>>
>>> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
>>
>> Any comments about this one?
>>
>
> It is in my queue for applying it.
>
>> Also, it would be nice if you could create a 'next' branch so that non
>> bug fixes could be collected there.
>
> Sure - I am still busy, but I plan to do this in the week-end, and also
> applying the bug fixes to -master.
>

Good, I was having this problem on my booting

Michael

> 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@denx.de
> =====================================================================
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
Fabio Estevam June 17, 2016, 1:34 p.m. UTC | #4
On Fri, Jun 17, 2016 at 9:50 AM, Stefano Babic <sbabic@denx.de> wrote:

> Sure - I am still busy, but I plan to do this in the week-end, and also
> applying the bug fixes to -master.

Excellent, thanks!
Fabio Estevam June 17, 2016, 2:54 p.m. UTC | #5
On Fri, Jun 17, 2016 at 9:56 AM, Michael Trimarchi
<michael@amarulasolutions.com> wrote:

> Good, I was having this problem on my booting

Great, in this case providing a Tested-by tag would be nice :-)
Michael Nazzareno Trimarchi June 17, 2016, 2:55 p.m. UTC | #6
Hi


On Fri, Jun 17, 2016 at 4:54 PM, Fabio Estevam <festevam@gmail.com> wrote:
> On Fri, Jun 17, 2016 at 9:56 AM, Michael Trimarchi
> <michael@amarulasolutions.com> wrote:
>
>> Good, I was having this problem on my booting
>
> Great, in this case providing a Tested-by tag would be nice :-)

Yes I will do. Different design ;) but way tonight ;)

Michael
Michael Nazzareno Trimarchi June 17, 2016, 6:07 p.m. UTC | #7
Hi Stefano

On Fri, Jun 17, 2016 at 4:55 PM, Michael Trimarchi
<michael@amarulasolutions.com> wrote:
> Hi
>
>
> On Fri, Jun 17, 2016 at 4:54 PM, Fabio Estevam <festevam@gmail.com> wrote:
>> On Fri, Jun 17, 2016 at 9:56 AM, Michael Trimarchi
>> <michael@amarulasolutions.com> wrote:
>>
>>> Good, I was having this problem on my booting
>>
>> Great, in this case providing a Tested-by tag would be nice :-)
>
> Yes I will do. Different design ;) but way tonight ;)
>

Tested-by: Michael Trimarchi <michael@amarulasolutions.com>

> Michael
>
>
> --
> | Michael Nazzareno Trimarchi                     Amarula Solutions BV |
> | COO  -  Founder                                      Cruquiuskade 47 |
> | +31(0)851119172                                 Amsterdam 1018 AM NL |
> |                  [`as] http://www.amarulasolutions.com               |
Stefano Babic June 18, 2016, 8:56 a.m. UTC | #8
On 17/06/2016 14:56, Michael Trimarchi wrote:
> Hi
> 
> On Fri, Jun 17, 2016 at 2:50 PM, Stefano Babic <sbabic@denx.de> wrote:
>> Hi Fabio,
>>
>> On 17/06/2016 13:59, Fabio Estevam wrote:
>>> Hi Stefano,
>>>
>>> On Fri, Jun 10, 2016 at 9:46 AM, Fabio Estevam <fabio.estevam@nxp.com> wrote:
>>>> MX7_SEC is an existing configuration option that allows booting the
>>>> kernel in secure mode.
>>>>
>>>> Place this option in Kconfig, so that boards can select this option
>>>> in their defconfig files.
>>>>
>>>> Selecting this option is necessary when booting a kernel provided by
>>>> NXP, such as 3.14_GA and 4.1.15_GA.
>>>>
>>>> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
>>>
>>> Any comments about this one?


Applied to -next, thanks !

Best regards,
Stefano Babic
diff mbox

Patch

diff --git a/arch/arm/imx-common/Kconfig b/arch/arm/imx-common/Kconfig
index 1b7da5a..c6b38bc 100644
--- a/arch/arm/imx-common/Kconfig
+++ b/arch/arm/imx-common/Kconfig
@@ -17,3 +17,12 @@  config IMX_BOOTAUX
 	depends on ARCH_MX7 || ARCH_MX6
 	help
 	  bootaux [addr] to boot auxiliary core.
+
+config MX7_SEC
+	bool "Support booting MX7 in secure mode"
+	depends on ARCH_MX7
+	help
+	  NXP kernel requires to boot MX7 in secure mode. Select
+	  this options if you want to boot a NXP kernel. In order
+	  to boot a mainline kernel this option needs to be
+	  unselected.