[PATCH v2] ARM: lpc32xx: only run SoC init on LPC32xx hardware

[PATCH v2] ARM: lpc32xx: only run SoC init on LPC32xx hardware

[Karl Mehltretter]

lpc32xx_check_uid() and lpc32xx_pm_init() are arch_initcalls that poke
LPC32xx-only registers. Since the multiplatform conversion they also
run on other ARCH_MULTI_V5 boards where access faults e.g. on versatile:

  Unable to handle kernel paging request at virtual address f4004130
  PC is at lpc32xx_check_uid+0x2c/0x9c

Drop the arch_initcall() registrations and call both functions directly
from lpc3250_machine_init(), the machine's .init_machine hook.
The calls are placed in link order (common.c, pm.c, phy3250.c) to
keep their previous relative ordering.

Fixes: 75bf1bd7d2f9 ("ARM: lpc32xx: allow multiplatform build")
Suggested-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Karl Mehltretter <kmehltretter@gmail.com>
---
v2:
 - Rather than guarding the two arch_initcalls, drop the initcalls
   entirely and call both functions from lpc3250_machine_init(), which
   is already guarded properly. (Suggested by Arnd.)

 arch/arm/mach-lpc32xx/common.c  | 5 +----
 arch/arm/mach-lpc32xx/common.h  | 2 ++
 arch/arm/mach-lpc32xx/phy3250.c | 2 ++
 arch/arm/mach-lpc32xx/pm.c      | 5 +----
 4 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/arch/arm/mach-lpc32xx/common.c b/arch/arm/mach-lpc32xx/common.c
index 304ea61a0716..35ed3569c5a3 100644
--- a/arch/arm/mach-lpc32xx/common.c
+++ b/arch/arm/mach-lpc32xx/common.c
@@ -106,7 +106,7 @@ void __init lpc32xx_map_io(void)
 	iotable_init(lpc32xx_io_desc, ARRAY_SIZE(lpc32xx_io_desc));
 }
 
-static int __init lpc32xx_check_uid(void)
+void __init lpc32xx_check_uid(void)
 {
 	u32 uid[4];
 
@@ -119,7 +119,4 @@ static int __init lpc32xx_check_uid(void)
 		system_serial_low = uid[0];
 		system_serial_high = uid[1];
 	}
-
-	return 1;
 }
-arch_initcall(lpc32xx_check_uid);
diff --git a/arch/arm/mach-lpc32xx/common.h b/arch/arm/mach-lpc32xx/common.h
index 32f0ad217807..06b20bea324e 100644
--- a/arch/arm/mach-lpc32xx/common.h
+++ b/arch/arm/mach-lpc32xx/common.h
@@ -16,6 +16,8 @@
  * Other arch specific structures and functions
  */
 extern void __init lpc32xx_map_io(void);
+extern void __init lpc32xx_check_uid(void);
+extern void __init lpc32xx_pm_init(void);
 extern void __init lpc32xx_serial_init(void);
 
 /*
diff --git a/arch/arm/mach-lpc32xx/phy3250.c b/arch/arm/mach-lpc32xx/phy3250.c
index 66701bf43248..ddc6333ca55d 100644
--- a/arch/arm/mach-lpc32xx/phy3250.c
+++ b/arch/arm/mach-lpc32xx/phy3250.c
@@ -71,6 +71,8 @@ static const struct of_dev_auxdata lpc32xx_auxdata_lookup[] __initconst = {
 
 static void __init lpc3250_machine_init(void)
 {
+	lpc32xx_check_uid();
+	lpc32xx_pm_init();
 	lpc32xx_serial_init();
 
 	of_platform_default_populate(NULL, lpc32xx_auxdata_lookup, NULL);
diff --git a/arch/arm/mach-lpc32xx/pm.c b/arch/arm/mach-lpc32xx/pm.c
index 2572bd89a5e8..9b5c5e1462ed 100644
--- a/arch/arm/mach-lpc32xx/pm.c
+++ b/arch/arm/mach-lpc32xx/pm.c
@@ -120,7 +120,7 @@ static const struct platform_suspend_ops lpc32xx_pm_ops = {
 #define EMC_DYN_MEM_CTRL_OFS 0x20
 #define EMC_SRMMC           (1 << 3)
 #define EMC_CTRL_REG io_p2v(LPC32XX_EMC_BASE + EMC_DYN_MEM_CTRL_OFS)
-static int __init lpc32xx_pm_init(void)
+void __init lpc32xx_pm_init(void)
 {
 	/*
 	 * Setup SDRAM self-refresh clock to automatically disable o
@@ -129,7 +129,4 @@ static int __init lpc32xx_pm_init(void)
 	__raw_writel(__raw_readl(EMC_CTRL_REG) | EMC_SRMMC, EMC_CTRL_REG);
 
 	suspend_set_ops(&lpc32xx_pm_ops);
-
-	return 0;
 }
-arch_initcall(lpc32xx_pm_init);
-- 
2.39.5 (Apple Git-154)

Re: [PATCH v2] ARM: lpc32xx: only run SoC init on LPC32xx hardware

[Arnd Bergmann]

On Mon, Jul 13, 2026, at 21:25, Karl Mehltretter wrote:
> lpc32xx_check_uid() and lpc32xx_pm_init() are arch_initcalls that poke
> LPC32xx-only registers. Since the multiplatform conversion they also
> run on other ARCH_MULTI_V5 boards where access faults e.g. on versatile:
>
>   Unable to handle kernel paging request at virtual address f4004130
>   PC is at lpc32xx_check_uid+0x2c/0x9c
>
> Drop the arch_initcall() registrations and call both functions directly
> from lpc3250_machine_init(), the machine's .init_machine hook.
> The calls are placed in link order (common.c, pm.c, phy3250.c) to
> keep their previous relative ordering.
>
> Fixes: 75bf1bd7d2f9 ("ARM: lpc32xx: allow multiplatform build")
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Karl Mehltretter <kmehltretter@gmail.com>

Looks good to me,

Reviewed-by: Arnd Bergmann <arnd@arndb.de>

Vladimir, I assume you'll pick this up and forward it to
soc@lists.linux.dev. Let us know if you prefer Karl to send it
there directly and save you the extra pull request.

      Arnd

Re: [PATCH v2] ARM: lpc32xx: only run SoC init on LPC32xx hardware

[Vladimir Zapolskiy]

On 7/14/26 09:12, Arnd Bergmann wrote:
> On Mon, Jul 13, 2026, at 21:25, Karl Mehltretter wrote:
>> lpc32xx_check_uid() and lpc32xx_pm_init() are arch_initcalls that poke
>> LPC32xx-only registers. Since the multiplatform conversion they also
>> run on other ARCH_MULTI_V5 boards where access faults e.g. on versatile:
>>
>>    Unable to handle kernel paging request at virtual address f4004130
>>    PC is at lpc32xx_check_uid+0x2c/0x9c
>>
>> Drop the arch_initcall() registrations and call both functions directly
>> from lpc3250_machine_init(), the machine's .init_machine hook.
>> The calls are placed in link order (common.c, pm.c, phy3250.c) to
>> keep their previous relative ordering.
>>
>> Fixes: 75bf1bd7d2f9 ("ARM: lpc32xx: allow multiplatform build")
>> Suggested-by: Arnd Bergmann <arnd@arndb.de>
>> Signed-off-by: Karl Mehltretter <kmehltretter@gmail.com>
> 
> Looks good to me,
> 
> Reviewed-by: Arnd Bergmann <arnd@arndb.de>
> 
> Vladimir, I assume you'll pick this up and forward it to
> soc@lists.linux.dev. Let us know if you prefer Karl to send it
> there directly and save you the extra pull request.

Will do it next week, if it's not too late.

Many thanks for the change and for the review!

-- 
Best wishes,
Vladimir

Re: [PATCH v2] ARM: lpc32xx: only run SoC init on LPC32xx hardware

[Vladimir Zapolskiy]

On 7/13/26 22:25, Karl Mehltretter wrote:
> lpc32xx_check_uid() and lpc32xx_pm_init() are arch_initcalls that poke
> LPC32xx-only registers. Since the multiplatform conversion they also
> run on other ARCH_MULTI_V5 boards where access faults e.g. on versatile:
> 
>    Unable to handle kernel paging request at virtual address f4004130
>    PC is at lpc32xx_check_uid+0x2c/0x9c
> 
> Drop the arch_initcall() registrations and call both functions directly
> from lpc3250_machine_init(), the machine's .init_machine hook.
> The calls are placed in link order (common.c, pm.c, phy3250.c) to
> keep their previous relative ordering.
> 
> Fixes: 75bf1bd7d2f9 ("ARM: lpc32xx: allow multiplatform build")
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Karl Mehltretter <kmehltretter@gmail.com>
> ---
> v2:
>   - Rather than guarding the two arch_initcalls, drop the initcalls
>     entirely and call both functions from lpc3250_machine_init(), which
>     is already guarded properly. (Suggested by Arnd.)
> 
>   arch/arm/mach-lpc32xx/common.c  | 5 +----
>   arch/arm/mach-lpc32xx/common.h  | 2 ++
>   arch/arm/mach-lpc32xx/phy3250.c | 2 ++
>   arch/arm/mach-lpc32xx/pm.c      | 5 +----
>   4 files changed, 6 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm/mach-lpc32xx/common.c b/arch/arm/mach-lpc32xx/common.c
> index 304ea61a0716..35ed3569c5a3 100644
> --- a/arch/arm/mach-lpc32xx/common.c
> +++ b/arch/arm/mach-lpc32xx/common.c
> @@ -106,7 +106,7 @@ void __init lpc32xx_map_io(void)
>   	iotable_init(lpc32xx_io_desc, ARRAY_SIZE(lpc32xx_io_desc));
>   }
>   
> -static int __init lpc32xx_check_uid(void)
> +void __init lpc32xx_check_uid(void)
>   {
>   	u32 uid[4];
>   
> @@ -119,7 +119,4 @@ static int __init lpc32xx_check_uid(void)
>   		system_serial_low = uid[0];
>   		system_serial_high = uid[1];
>   	}
> -
> -	return 1;
>   }
> -arch_initcall(lpc32xx_check_uid);
> diff --git a/arch/arm/mach-lpc32xx/common.h b/arch/arm/mach-lpc32xx/common.h
> index 32f0ad217807..06b20bea324e 100644
> --- a/arch/arm/mach-lpc32xx/common.h
> +++ b/arch/arm/mach-lpc32xx/common.h
> @@ -16,6 +16,8 @@
>    * Other arch specific structures and functions
>    */
>   extern void __init lpc32xx_map_io(void);
> +extern void __init lpc32xx_check_uid(void);
> +extern void __init lpc32xx_pm_init(void);
>   extern void __init lpc32xx_serial_init(void);
>   
>   /*
> diff --git a/arch/arm/mach-lpc32xx/phy3250.c b/arch/arm/mach-lpc32xx/phy3250.c
> index 66701bf43248..ddc6333ca55d 100644
> --- a/arch/arm/mach-lpc32xx/phy3250.c
> +++ b/arch/arm/mach-lpc32xx/phy3250.c
> @@ -71,6 +71,8 @@ static const struct of_dev_auxdata lpc32xx_auxdata_lookup[] __initconst = {
>   
>   static void __init lpc3250_machine_init(void)
>   {
> +	lpc32xx_check_uid();
> +	lpc32xx_pm_init();
>   	lpc32xx_serial_init();
>   
>   	of_platform_default_populate(NULL, lpc32xx_auxdata_lookup, NULL);
> diff --git a/arch/arm/mach-lpc32xx/pm.c b/arch/arm/mach-lpc32xx/pm.c
> index 2572bd89a5e8..9b5c5e1462ed 100644
> --- a/arch/arm/mach-lpc32xx/pm.c
> +++ b/arch/arm/mach-lpc32xx/pm.c
> @@ -120,7 +120,7 @@ static const struct platform_suspend_ops lpc32xx_pm_ops = {
>   #define EMC_DYN_MEM_CTRL_OFS 0x20
>   #define EMC_SRMMC           (1 << 3)
>   #define EMC_CTRL_REG io_p2v(LPC32XX_EMC_BASE + EMC_DYN_MEM_CTRL_OFS)
> -static int __init lpc32xx_pm_init(void)
> +void __init lpc32xx_pm_init(void)
>   {
>   	/*
>   	 * Setup SDRAM self-refresh clock to automatically disable o
> @@ -129,7 +129,4 @@ static int __init lpc32xx_pm_init(void)
>   	__raw_writel(__raw_readl(EMC_CTRL_REG) | EMC_SRMMC, EMC_CTRL_REG);
>   
>   	suspend_set_ops(&lpc32xx_pm_ops);
> -
> -	return 0;
>   }
> -arch_initcall(lpc32xx_pm_init);

Reviewed-by: Vladimir Zapolskiy <vz@kernel.org>

-- 
Best wishes,
Vladimir