[v4,2/3] warp7: secure_defconfig: Remove secure_defconfig

Message ID 20180424173413.11068-3-bryan.odonoghue@linaro.org
State New
Headers show
Series
  • WaRP7 unify secure and non-secure defconfigs
Related show

Commit Message

Bryan O'Donoghue April 24, 2018, 5:34 p.m.
This patch removes warp7_secure_defconfig. A previous patch set
CONFIG_ARMV7_BOOT_SEC_DEFAULT=y on the unsecure WaRP7 config. Fabio asked
if I could confirm that the NXP and upstream kernels will boot on the WaRP7
with CONFIG_ARMV7_BOOT_SEC_DEFAULT=y. I can confirm that this is the case,
so there's no need to support the secure defconfig - drop it now.

Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Suggested-by: Fabio Estevam <fabio.estevam@nxp.com>
Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>
---
 board/warp7/MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)

Comments

Fabio Estevam April 24, 2018, 5:37 p.m. | #1
Hi Bryan,

On Tue, Apr 24, 2018 at 2:34 PM, Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
> This patch removes warp7_secure_defconfig. A previous patch set
> CONFIG_ARMV7_BOOT_SEC_DEFAULT=y on the unsecure WaRP7 config. Fabio asked
> if I could confirm that the NXP and upstream kernels will boot on the WaRP7
> with CONFIG_ARMV7_BOOT_SEC_DEFAULT=y. I can confirm that this is the case,
> so there's no need to support the secure defconfig - drop it now.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> Suggested-by: Fabio Estevam <fabio.estevam@nxp.com>
> Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>
> ---
>  board/warp7/MAINTAINERS | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/board/warp7/MAINTAINERS b/board/warp7/MAINTAINERS
> index 0fc9746606..1d3ee29222 100644
> --- a/board/warp7/MAINTAINERS
> +++ b/board/warp7/MAINTAINERS
> @@ -4,4 +4,3 @@ S:      Maintained
>  F:     board/warp7/
>  F:     include/configs/warp7.h
>  F:     configs/warp7_defconfig
> -F:     configs/warp7_secure_defconfig

In v4 you missed to actually remove configs/warp7_secure_defconfig :-)
Bryan O'Donoghue April 24, 2018, 5:40 p.m. | #2
On 24/04/18 18:37, Fabio Estevam wrote:
> Hi Bryan,
> 
> On Tue, Apr 24, 2018 at 2:34 PM, Bryan O'Donoghue
> <bryan.odonoghue@linaro.org> wrote:
>> This patch removes warp7_secure_defconfig. A previous patch set
>> CONFIG_ARMV7_BOOT_SEC_DEFAULT=y on the unsecure WaRP7 config. Fabio asked
>> if I could confirm that the NXP and upstream kernels will boot on the WaRP7
>> with CONFIG_ARMV7_BOOT_SEC_DEFAULT=y. I can confirm that this is the case,
>> so there's no need to support the secure defconfig - drop it now.
>>
>> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
>> Suggested-by: Fabio Estevam <fabio.estevam@nxp.com>
>> Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>
>> ---
>>   board/warp7/MAINTAINERS | 1 -
>>   1 file changed, 1 deletion(-)
>>
>> diff --git a/board/warp7/MAINTAINERS b/board/warp7/MAINTAINERS
>> index 0fc9746606..1d3ee29222 100644
>> --- a/board/warp7/MAINTAINERS
>> +++ b/board/warp7/MAINTAINERS
>> @@ -4,4 +4,3 @@ S:      Maintained
>>   F:     board/warp7/
>>   F:     include/configs/warp7.h
>>   F:     configs/warp7_defconfig
>> -F:     configs/warp7_secure_defconfig
> 
> In v4 you missed to actually remove configs/warp7_secure_defconfig :-)
> 
ah damnit i had a merge conflict ... I remember now

Patch

diff --git a/board/warp7/MAINTAINERS b/board/warp7/MAINTAINERS
index 0fc9746606..1d3ee29222 100644
--- a/board/warp7/MAINTAINERS
+++ b/board/warp7/MAINTAINERS
@@ -4,4 +4,3 @@  S:	Maintained
 F:	board/warp7/
 F:	include/configs/warp7.h
 F:	configs/warp7_defconfig
-F:	configs/warp7_secure_defconfig