Re: [RFT PATCH v3] ARM: omap1: enable real software node lookup of GPIOs on Nokia 770
From: Bartosz Golaszewski <brgl@kernel.org>
Date: 2026-03-27 17:23:42
Also in:
linux-omap, lkml
On Fri, Mar 27, 2026 at 5:59 PM Aaro Koskinen [off-list ref] wrote:
Hi, On Fri, Mar 27, 2026 at 03:22:12PM +0100, Bartosz Golaszewski wrote:quoted
Hmm, I'm wondering if there's a race with consumers already requesting the GPIOs after the controller device is registered but before the software node is added. I'll send a version with software nodes being registered first, then passes as firmware nodes to the platform device API before the device is registered.It crashes early, I was able to get an UART log from OSK (another 16xx board): [ 1.001525] Register r12 information: 2-page vmalloc region starting at 0xc2808000 allocated at kernel_clone+0xa4/0x20c [ 1.013092] Process swapper/0 (pid: 1, stack limit = 0x(ptrval)) [ 1.019500] Stack: (0xc2809ed0 to 0xc280a000) [ 1.024230] 9ec0: c072d000 c0529474 c06b3aa0 c050a3cc [ 1.032958] 9ee0: c072d000 c085c000 00000002 c052582c c050a324 c072d000 00000000 c0503160 [ 1.041687] 9f00: 00002710 00000000 c04da8f8 c0060900 c2809f64 ffffffff 00010000 946f70b5 [ 1.050384] 9f20: 00000062 c0816120 00000002 c052582c c0525848 c072d000 c04da8f8 c0060a18 [ 1.059112] 9f40: c2809f64 c2809f64 00000000 946f70b5 00000062 c0816120 00000002 c052582c [ 1.067810] 9f60: c052584c c072d000 c04da8f8 c050352c 00000002 00000002 00000000 c0502400 [ 1.076507] 9f80: c2809f7c 00000000 c03f86f4 00000000 00000000 00000000 00000000 00000000 [ 1.085205] 9fa0: 00000000 c03f8704 00000000 c000850c 00000000 00000000 00000000 00000000 [ 1.093902] 9fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 1.102600] 9fe0: 00000000 00000000 00000000 00000000 00000013 00000000 00000000 00000000 [ 1.111206] Call trace: [ 1.111328] software_node_to_swnode from device_add_software_node+0x20/0x80 [ 1.121704] device_add_software_node from omap16xx_gpio_init+0xa8/0xe4 [ 1.128997] omap16xx_gpio_init from do_one_initcall+0x68/0x1f4 [ 1.135620] do_one_initcall from kernel_init_freeable+0x1ec/0x240 [ 1.142517] kernel_init_freeable from kernel_init+0x10/0x108 [ 1.148864] kernel_init from ret_from_fork+0x14/0x28 [ 1.154357] Exception stack(0xc2809fb0 to 0xc2809ff8) [ 1.159820] 9fa0: 00000000 00000000 00000000 00000000 [ 1.168518] 9fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 1.177185] 9fe0: 00000000 00000000 00000000 00000000 00000013 00000000 [ 1.184295] Code: e3500000 012fff1e e59f3034 e5932000 (e5923000) [ 1.191040] ---[ end trace 0000000000000000 ]--- [ 1.196350] Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b [ 1.204559] ---[ end Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b ]---
Thanks. This makes sense. Both omap16xx_gpio_init() and software_node_init() run as postcore_initcall() so if the order is not right, it will fail. Cc'ing Andy who's a reviewer for software nodes. Andy: is there any reason to run software_node_init() as a postcore initcall? It only allocates the kset, can we move it to core_initcall() by any chance? Bart