linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] arm: fix imx_* defconfig
@ 2025-05-10  1:18 Denis Benato
  2025-05-10  1:18 ` [PATCH 1/3] ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP Denis Benato
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Denis Benato @ 2025-05-10  1:18 UTC (permalink / raw)
  To: Russell King
  Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	linux-arm-kernel, imx, linux-kernel, Denis Benato

Compiling the kernel using affected imx defconfigs lead
to a hard CPU crash that reaches a state where each register,
including PC ends up in 0xDEADBEEF.

That is because the SoC support depending on CONFIG_ARCH_MXC
gets disabled since CONFIG_ARCH_NXP is not defined while
said feature depends on it.

The default arm defconfig is unaffected as it defines
CONFIG_ARCH_NXP=y correctly as excepted.

These patches solve the crash causing my confusion:
https://lore.kernel.org/all/c84e8246-8104-4409-8d95-389d61bc07af@gmail.com

Denis Benato (3):
  ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP
  ARM: imx_v6_v7_defconfig: define CONFIG_ARCH_NXP
  ARM: imxrt_defconfig: define CONFIG_ARCH_NXP

 arch/arm/configs/imx_v4_v5_defconfig | 1 +
 arch/arm/configs/imx_v6_v7_defconfig | 1 +
 arch/arm/configs/imxrt_defconfig     | 1 +
 3 files changed, 3 insertions(+)

-- 
2.49.0



^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/3] ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP
  2025-05-10  1:18 [PATCH 0/3] arm: fix imx_* defconfig Denis Benato
@ 2025-05-10  1:18 ` Denis Benato
  2025-05-10 17:57   ` Fabio Estevam
  2025-05-10  1:18 ` [PATCH 2/3] ARM: imx_v6_v7_defconfig: " Denis Benato
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 7+ messages in thread
From: Denis Benato @ 2025-05-10  1:18 UTC (permalink / raw)
  To: Russell King
  Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	linux-arm-kernel, imx, linux-kernel, Denis Benato

Define CONFIG_ARCH_NXP so that CONFIG_ARCH_MXC won't be disabled
as it depends on CONFIG_ARCH_NXP.

Signed-off-by: Denis Benato <benato.denis96@gmail.com>
---
 arch/arm/configs/imx_v4_v5_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/imx_v4_v5_defconfig b/arch/arm/configs/imx_v4_v5_defconfig
index 875c8cdbada7..19e26b4a9fb7 100644
--- a/arch/arm/configs/imx_v4_v5_defconfig
+++ b/arch/arm/configs/imx_v4_v5_defconfig
@@ -11,6 +11,7 @@ CONFIG_PROFILING=y
 CONFIG_ARCH_MULTI_V4T=y
 CONFIG_ARCH_MULTI_V5=y
 # CONFIG_ARCH_MULTI_V7 is not set
+CONFIG_ARCH_NXP=y
 CONFIG_ARCH_MXC=y
 CONFIG_SOC_IMX1=y
 CONFIG_SOC_IMX25=y
-- 
2.49.0



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/3] ARM: imx_v6_v7_defconfig: define CONFIG_ARCH_NXP
  2025-05-10  1:18 [PATCH 0/3] arm: fix imx_* defconfig Denis Benato
  2025-05-10  1:18 ` [PATCH 1/3] ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP Denis Benato
@ 2025-05-10  1:18 ` Denis Benato
  2025-05-10  1:18 ` [PATCH 3/3] ARM: imxrt_defconfig: " Denis Benato
  2025-05-12  6:03 ` [PATCH 0/3] arm: fix imx_* defconfig Sascha Hauer
  3 siblings, 0 replies; 7+ messages in thread
From: Denis Benato @ 2025-05-10  1:18 UTC (permalink / raw)
  To: Russell King
  Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	linux-arm-kernel, imx, linux-kernel, Denis Benato

Define CONFIG_ARCH_NXP so that CONFIG_ARCH_MXC won't be disabled
as it depends on CONFIG_ARCH_NXP.

Signed-off-by: Denis Benato <benato.denis96@gmail.com>
---
 arch/arm/configs/imx_v6_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
index 062c1eb8dd60..22edc6d17520 100644
--- a/arch/arm/configs/imx_v6_v7_defconfig
+++ b/arch/arm/configs/imx_v6_v7_defconfig
@@ -13,6 +13,7 @@ CONFIG_BLK_DEV_INITRD=y
 CONFIG_EXPERT=y
 CONFIG_PERF_EVENTS=y
 CONFIG_ARCH_MULTI_V6=y
+CONFIG_ARCH_NXP=y
 CONFIG_ARCH_MXC=y
 CONFIG_SOC_IMX31=y
 CONFIG_SOC_IMX35=y
-- 
2.49.0



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 3/3] ARM: imxrt_defconfig: define CONFIG_ARCH_NXP
  2025-05-10  1:18 [PATCH 0/3] arm: fix imx_* defconfig Denis Benato
  2025-05-10  1:18 ` [PATCH 1/3] ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP Denis Benato
  2025-05-10  1:18 ` [PATCH 2/3] ARM: imx_v6_v7_defconfig: " Denis Benato
@ 2025-05-10  1:18 ` Denis Benato
  2025-05-12  6:03 ` [PATCH 0/3] arm: fix imx_* defconfig Sascha Hauer
  3 siblings, 0 replies; 7+ messages in thread
From: Denis Benato @ 2025-05-10  1:18 UTC (permalink / raw)
  To: Russell King
  Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	linux-arm-kernel, imx, linux-kernel, Denis Benato

Define CONFIG_ARCH_NXP so that CONFIG_ARCH_MXC won't be disabled
as it depends on CONFIG_ARCH_NXP.

Signed-off-by: Denis Benato <benato.denis96@gmail.com>
---
 arch/arm/configs/imxrt_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/imxrt_defconfig b/arch/arm/configs/imxrt_defconfig
index 52dba3762996..7cc1555c0231 100644
--- a/arch/arm/configs/imxrt_defconfig
+++ b/arch/arm/configs/imxrt_defconfig
@@ -2,6 +2,7 @@
 CONFIG_BPF_SYSCALL=y
 CONFIG_SCHED_AUTOGROUP=y
 # CONFIG_MMU is not set
+CONFIG_ARCH_NXP=y
 CONFIG_ARCH_MXC=y
 CONFIG_SOC_IMXRT=y
 CONFIG_SET_MEM_PARAM=y
-- 
2.49.0



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/3] ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP
  2025-05-10  1:18 ` [PATCH 1/3] ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP Denis Benato
@ 2025-05-10 17:57   ` Fabio Estevam
  0 siblings, 0 replies; 7+ messages in thread
From: Fabio Estevam @ 2025-05-10 17:57 UTC (permalink / raw)
  To: Denis Benato
  Cc: Russell King, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
	linux-arm-kernel, imx, linux-kernel

On Fri, May 9, 2025 at 10:18 PM Denis Benato <benato.denis96@gmail.com> wrote:
>
> Define CONFIG_ARCH_NXP so that CONFIG_ARCH_MXC won't be disabled
> as it depends on CONFIG_ARCH_NXP.

This is not correct.

$ make imx_v4_v5_defconfig
#
# configuration written to .config
#
$ cat .config | grep CONFIG_ARCH_MXC
CONFIG_ARCH_MXC=y

CONFIG_ARCH_NXP is an arm64 option.


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 0/3] arm: fix imx_* defconfig
  2025-05-10  1:18 [PATCH 0/3] arm: fix imx_* defconfig Denis Benato
                   ` (2 preceding siblings ...)
  2025-05-10  1:18 ` [PATCH 3/3] ARM: imxrt_defconfig: " Denis Benato
@ 2025-05-12  6:03 ` Sascha Hauer
  2025-05-12 18:09   ` Denis Benato
  3 siblings, 1 reply; 7+ messages in thread
From: Sascha Hauer @ 2025-05-12  6:03 UTC (permalink / raw)
  To: Denis Benato
  Cc: Russell King, Shawn Guo, Pengutronix Kernel Team, Fabio Estevam,
	linux-arm-kernel, imx, linux-kernel

Hi Denis,

On Sat, May 10, 2025 at 03:18:03AM +0200, Denis Benato wrote:
> Compiling the kernel using affected imx defconfigs lead
> to a hard CPU crash that reaches a state where each register,
> including PC ends up in 0xDEADBEEF.
> 
> That is because the SoC support depending on CONFIG_ARCH_MXC
> gets disabled since CONFIG_ARCH_NXP is not defined while
> said feature depends on it.
> 
> The default arm defconfig is unaffected as it defines
> CONFIG_ARCH_NXP=y correctly as excepted.
> 
> These patches solve the crash causing my confusion:
> https://lore.kernel.org/all/c84e8246-8104-4409-8d95-389d61bc07af@gmail.com

In this mail you say you want to start a ARMv8 i.MX8MM board. The
configs you are changing are for old 32bit ARMv4-7 SoCs.
arch/arm64/configs/defconfig is for you.

Sascha

> 
> Denis Benato (3):
>   ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP
>   ARM: imx_v6_v7_defconfig: define CONFIG_ARCH_NXP
>   ARM: imxrt_defconfig: define CONFIG_ARCH_NXP
> 
>  arch/arm/configs/imx_v4_v5_defconfig | 1 +
>  arch/arm/configs/imx_v6_v7_defconfig | 1 +
>  arch/arm/configs/imxrt_defconfig     | 1 +
>  3 files changed, 3 insertions(+)
> 
> -- 
> 2.49.0
> 
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 0/3] arm: fix imx_* defconfig
  2025-05-12  6:03 ` [PATCH 0/3] arm: fix imx_* defconfig Sascha Hauer
@ 2025-05-12 18:09   ` Denis Benato
  0 siblings, 0 replies; 7+ messages in thread
From: Denis Benato @ 2025-05-12 18:09 UTC (permalink / raw)
  To: Sascha Hauer
  Cc: Russell King, Shawn Guo, Pengutronix Kernel Team, Fabio Estevam,
	linux-arm-kernel, imx, linux-kernel


On 5/12/25 08:03, Sascha Hauer wrote:
> Hi Denis,
>
> On Sat, May 10, 2025 at 03:18:03AM +0200, Denis Benato wrote:
>> Compiling the kernel using affected imx defconfigs lead
>> to a hard CPU crash that reaches a state where each register,
>> including PC ends up in 0xDEADBEEF.
>>
>> That is because the SoC support depending on CONFIG_ARCH_MXC
>> gets disabled since CONFIG_ARCH_NXP is not defined while
>> said feature depends on it.
>>
>> The default arm defconfig is unaffected as it defines
>> CONFIG_ARCH_NXP=y correctly as excepted.
>>
>> These patches solve the crash causing my confusion:
>> https://lore.kernel.org/all/c84e8246-8104-4409-8d95-389d61bc07af@gmail.com
> In this mail you say you want to start a ARMv8 i.MX8MM board. The
> configs you are changing are for old 32bit ARMv4-7 SoCs.
> arch/arm64/configs/defconfig is for you.

Hello and thanks for both of your answers.


Yes, I have made the mistake of using that defconfig for arm while building an arm64 kernel.


I am sorry and thank you both for correcting me.

Please, ignore these patches.

Denis B.

> Sascha
>
>> Denis Benato (3):
>>   ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP
>>   ARM: imx_v6_v7_defconfig: define CONFIG_ARCH_NXP
>>   ARM: imxrt_defconfig: define CONFIG_ARCH_NXP
>>
>>  arch/arm/configs/imx_v4_v5_defconfig | 1 +
>>  arch/arm/configs/imx_v6_v7_defconfig | 1 +
>>  arch/arm/configs/imxrt_defconfig     | 1 +
>>  3 files changed, 3 insertions(+)
>>
>> -- 
>> 2.49.0
>>
>>


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2025-05-12 18:29 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-10  1:18 [PATCH 0/3] arm: fix imx_* defconfig Denis Benato
2025-05-10  1:18 ` [PATCH 1/3] ARM: imx_v4_v5_defconfig: define CONFIG_ARCH_NXP Denis Benato
2025-05-10 17:57   ` Fabio Estevam
2025-05-10  1:18 ` [PATCH 2/3] ARM: imx_v6_v7_defconfig: " Denis Benato
2025-05-10  1:18 ` [PATCH 3/3] ARM: imxrt_defconfig: " Denis Benato
2025-05-12  6:03 ` [PATCH 0/3] arm: fix imx_* defconfig Sascha Hauer
2025-05-12 18:09   ` Denis Benato

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).