1. Patchew
  2. linux
  3. View series

[PATCH v2 0/3] gpio: shared: another set of small fixes

Bartosz Golaszewski posted 3 patches 1 month ago 
drivers/gpio/gpiolib-shared.c | 34 ++++++++++++++++------------------
1 file changed, 16 insertions(+), 18 deletions(-)
[PATCH v2 0/3] gpio: shared: another set of small fixes
Posted by Bartosz Golaszewski 1 month ago 
Here are three more patches fixing issues reported with shared GPIO
management.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
---
Changes in v2:
- Fix mutex deadlock
- Fix lockdep false-positives
- Add a patch reducing the number of memory allocations
- Link to v1: https://lore.kernel.org/r/20260105-gpio-shared-fixes-v1-0-76d6ff0afcd8@oss.qualcomm.com

---
Bartosz Golaszewski (3):
      gpio: shared: assign the correct firmware node for reset-gpio use-case
      gpio: shared: fix a race condition
      gpio: shared: don't allocate the lookup table until we really need it

 drivers/gpio/gpiolib-shared.c | 34 ++++++++++++++++------------------
 1 file changed, 16 insertions(+), 18 deletions(-)
---
base-commit: 19fb766a1e5ed5943a62fc671c09d45352a81b1d
change-id: 20260105-gpio-shared-fixes-40a8ec3b6b25

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
Re: [PATCH v2 0/3] gpio: shared: another set of small fixes
Posted by Cosmin Tanislav 4 weeks, 1 day ago 
On 1/6/26 11:34 AM, Bartosz Golaszewski wrote:
> Here are three more patches fixing issues reported with shared GPIO
> management.
> 

I'm still getting some intermittent failures on today's next (which
contains this series).

This does not seem to happen on every boot, so it might be some race.

[    2.169655] Mem abort info:
[    2.169663]   ESR = 0x0000000096000004
[    2.169670]   EC = 0x25: DABT (current EL), IL = 32 bits
[    2.176773] ehci-platform 92040100.usb: irq 64, io mem 0x92040100
[    2.186626]   SET = 0, FnV = 0
[    2.204264] renesas_sdhi_internal_dmac 92090000.mmc: mmc1 base at 
0x0000000092090000, max clock rate 200 MHz
[    2.204308] ehci-platform 92040100.usb: USB 2.0 started, EHCI 1.10
[    2.208714] hub 2-0:1.0: USB hub found
[    2.214345]   EA = 0, S1PTW = 0
[    2.222949] hub 2-0:1.0: 1 port detected
[    2.231502]   FSC = 0x04: level 0 translation fault
[    2.239779] hub 1-0:1.0: USB hub found
[    2.248022] Data abort info:
[    2.256579] hub 1-0:1.0: 1 port detected
[    2.264724]   ISV = 0, ISS = 0x00000004, ISS2 = 0x00000000
[    2.487291]   CM = 0, WnR = 0, TnD = 0, TagAccess = 0
[    2.493148]   GCS = 0, Overlay = 0, DirtyBit = 0, Xs = 0
[    2.499300] [0000000000000000] user address but active_mm is swapper
[    2.506556] Internal error: Oops: 0000000096000004 [#1]  SMP
[    2.513092] Modules linked in:
[    2.516610] CPU: 3 UID: 0 PID: 48 Comm: kworker/u16:2 Not tainted 
6.19.0-rc4-next-20260108+ #246 PREEMPT
[    2.527564] Hardware name: Renesas RZ/T2H EVK Board based on 
r9a09g077m44 (DT)
[    2.535903] Workqueue: async async_run_entry_fn
[    2.541141] pstate: 80400005 (Nzcv daif +PAN -UAO -TCO -DIT -SSBS 
BTYPE=--)
[    2.549076] pc : gpiod_direction_input_nonotify+0x18/0x22c
[    2.555408] lr : gpiod_configure_flags+0x1c4/0x380
[    2.560835] sp : ffff8000830e38c0
[    2.564530] x29: ffff8000830e38c0 x28: ffff80008243894f x27: 
ffff0001808f6410
[    2.572773] x26: ffff800081e77160 x25: 0000000000000000 x24: 
0000000000000000
[    2.580914] x23: 0000000000000000 x22: ffff800081e77160 x21: 
ffff80008243892f
[    2.589054] x20: 0000000000000000 x19: 0000000000000000 x18: 
000000000000000a
[    2.597101] x17: ffff80007fffffff x16: ffff000180054480 x15: 
0000000000000000
[    2.605345] x14: 0000000000000000 x13: ffff800080ed7600 x12: 
0000000000000000
[    2.613485] x11: 0000000000000000 x10: 00007fff01adbb48 x9 : 
0000000000000005
[    2.621626] x8 : 0101010101010101 x7 : fffffdffc6009b00 x6 : 
051573657480ffff
[    2.629869] x5 : 0000000000000138 x4 : 0000000000000000 x3 : 
0000000000000001
[    2.638113] x2 : 0000000000000000 x1 : 0000000000000000 x0 : 
0000000000000000
[    2.646254] Call trace:
[    2.648968]  gpiod_direction_input_nonotify+0x18/0x22c (P)
[    2.655299]  gpiod_configure_flags+0x1c4/0x380
[    2.660428]  gpiod_find_and_request+0x14c/0x524
[    2.665650]  gpiod_get_index+0x58/0x7c
[    2.669874]  devm_gpiod_get_index+0x20/0x90
[    2.674695]  mmc_gpiod_request_ro+0x30/0xb0
[    2.679525]  mmc_of_parse+0x100/0x688
[    2.683749]  tmio_mmc_host_alloc+0x80/0x110
[    2.688571]  renesas_sdhi_probe+0x150/0x760
[    2.693392]  renesas_sdhi_internal_dmac_probe+0x64/0x80
[    2.699357]  platform_probe+0x5c/0xa4
[    2.703581]  really_probe+0xbc/0x2ac
[    2.707703]  __driver_probe_device+0x78/0x118
[    2.712728]  driver_probe_device+0x3c/0x178
[    2.717550]  __device_attach_driver+0xb8/0x140
[    2.722678]  bus_for_each_drv+0x88/0xe8
[    2.727099]  __device_attach_async_helper+0xb0/0xd4
[    2.732526]  async_run_entry_fn+0x34/0xe0
[    2.737048]  process_one_work+0x150/0x290
[    2.741673]  worker_thread+0x18c/0x300
[    2.745915]  kthread+0x118/0x124
[    2.749635]  ret_from_fork+0x10/0x20
[    2.753762] Code: 910003fd a90153f3 aa0003f4 a9025bf5 (f9400013)
[    2.760790] ---[ end trace 0000000000000000 ]---

After running ./scripts/decode_stacktrace.sh

[    2.169655] Mem abort info:
[    2.169663]   ESR = 0x0000000096000004
[    2.169670]   EC = 0x25: DABT (current EL), IL = 32 bits
[    2.176773] ehci-platform 92040100.usb: irq 64, io mem 0x92040100
[    2.186626]   SET = 0, FnV = 0
[    2.204264] renesas_sdhi_internal_dmac 92090000.mmc: mmc1 base at 
0x0000000092090000, max clock rate 200 MHz
[    2.204308] ehci-platform 92040100.usb: USB 2.0 started, EHCI 1.10
[    2.208714] hub 2-0:1.0: USB hub found
[    2.214345]   EA = 0, S1PTW = 0
[    2.222949] hub 2-0:1.0: 1 port detected
[    2.231502]   FSC = 0x04: level 0 translation fault
[    2.239779] hub 1-0:1.0: USB hub found
[    2.248022] Data abort info:
[    2.256579] hub 1-0:1.0: 1 port detected
[    2.264724]   ISV = 0, ISS = 0x00000004, ISS2 = 0x00000000
[    2.487291]   CM = 0, WnR = 0, TnD = 0, TagAccess = 0
[    2.493148]   GCS = 0, Overlay = 0, DirtyBit = 0, Xs = 0
[    2.499300] [0000000000000000] user address but active_mm is swapper
[    2.506556] Internal error: Oops: 0000000096000004 [#1]  SMP
[    2.513092] Modules linked in:
[    2.527564] Hardware name: Renesas RZ/T2H EVK Board based on 
r9a09g077m44 (DT)
[    2.535903] Workqueue: async async_run_entry_fn
[    2.541141] pstate: 80400005 (Nzcv daif +PAN -UAO -TCO -DIT -SSBS 
BTYPE=--)
[    2.549076] pc : gpiod_direction_input_nonotify 
(drivers/gpio/gpiolib.h:231 (discriminator 1) 
drivers/gpio/gpiolib.c:2863 (discriminator 1))
[    2.555408] lr : gpiod_configure_flags (drivers/gpio/gpiolib.c:4956)
[    2.560835] sp : ffff8000830e38c0
[    2.564530] x29: ffff8000830e38c0 x28: ffff80008243894f x27: 
ffff0001808f6410
[    2.572773] x26: ffff800081e77160 x25: 0000000000000000 x24: 
0000000000000000
[    2.580914] x23: 0000000000000000 x22: ffff800081e77160 x21: 
ffff80008243892f
[    2.589054] x20: 0000000000000000 x19: 0000000000000000 x18: 
000000000000000a
[    2.597101] x17: ffff80007fffffff x16: ffff000180054480 x15: 
0000000000000000
[    2.605345] x14: 0000000000000000 x13: ffff800080ed7600 x12: 
0000000000000000
[    2.613485] x11: 0000000000000000 x10: 00007fff01adbb48 x9 : 
0000000000000005
[    2.621626] x8 : 0101010101010101 x7 : fffffdffc6009b00 x6 : 
051573657480ffff
[    2.629869] x5 : 0000000000000138 x4 : 0000000000000000 x3 : 
0000000000000001
[    2.638113] x2 : 0000000000000000 x1 : 0000000000000000 x0 : 
0000000000000000
[    2.646254] Call trace:
[    2.648968]  gpiod_direction_input_nonotify 
(drivers/gpio/gpiolib.h:231 (discriminator 1) 
drivers/gpio/gpiolib.c:2863 (discriminator 1)) (P)
[    2.655299]  gpiod_configure_flags (drivers/gpio/gpiolib.c:4956)
[    2.660428]  gpiod_find_and_request (drivers/gpio/gpiolib.c:4774)
[    2.665650]  gpiod_get_index (drivers/gpio/gpiolib.c:4986)
[    2.669874]  devm_gpiod_get_index (drivers/gpio/gpiolib-devres.c:102)
[    2.674695]  mmc_gpiod_request_ro (drivers/mmc/core/slot-gpio.c:255)
[    2.679525]  mmc_of_parse (drivers/mmc/core/host.c:348)
[    2.683749]  tmio_mmc_host_alloc (drivers/mmc/host/tmio_mmc_core.c:1149)
[    2.688571]  renesas_sdhi_probe 
(drivers/mmc/host/renesas_sdhi_core.c:1119)
[    2.693392]  renesas_sdhi_internal_dmac_probe 
(drivers/mmc/host/renesas_sdhi_internal_dmac.c:599)
[    2.699357]  platform_probe (drivers/base/platform.c:1449 
(discriminator 1))
[    2.703581]  really_probe (drivers/base/dd.c:581 drivers/base/dd.c:659)
[    2.707703]  __driver_probe_device (drivers/base/dd.c:801)
[    2.712728]  driver_probe_device (drivers/base/dd.c:831)
[    2.717550]  __device_attach_driver (drivers/base/dd.c:960)
[    2.722678]  bus_for_each_drv (drivers/base/bus.c:500)
[    2.727099]  __device_attach_async_helper 
(include/linux/pm_runtime.h:447 drivers/base/dd.c:991)
[    2.732526]  async_run_entry_fn (kernel/async.c:136 (discriminator 3))
[    2.737048]  process_one_work (kernel/workqueue.c:3290 (discriminator 1))
[    2.741673]  worker_thread (kernel/workqueue.c:3356 (discriminator 2) 
kernel/workqueue.c:3443 (discriminator 2))
[    2.745915]  kthread (kernel/kthread.c:467)
[    2.749635]  ret_from_fork (arch/arm64/kernel/entry.S:861)
[    2.753762] Code: 910003fd a90153f3 aa0003f4 a9025bf5 (f9400013)

> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
> ---
> Changes in v2:
> - Fix mutex deadlock
> - Fix lockdep false-positives
> - Add a patch reducing the number of memory allocations
> - Link to v1: https://lore.kernel.org/r/20260105-gpio-shared-fixes-v1-0-76d6ff0afcd8@oss.qualcomm.com
> 
> ---
> Bartosz Golaszewski (3):
>        gpio: shared: assign the correct firmware node for reset-gpio use-case
>        gpio: shared: fix a race condition
>        gpio: shared: don't allocate the lookup table until we really need it
> 
>   drivers/gpio/gpiolib-shared.c | 34 ++++++++++++++++------------------
>   1 file changed, 16 insertions(+), 18 deletions(-)
> ---
> base-commit: 19fb766a1e5ed5943a62fc671c09d45352a81b1d
> change-id: 20260105-gpio-shared-fixes-40a8ec3b6b25
> 
> Best regards,
Re: [PATCH v2 0/3] gpio: shared: another set of small fixes
Posted by Marek Szyprowski 4 weeks, 1 day ago 
On 08.01.2026 13:13, Cosmin Tanislav wrote:
> On 1/6/26 11:34 AM, Bartosz Golaszewski wrote:
>> Here are three more patches fixing issues reported with shared GPIO
>> management.
>>
>
> I'm still getting some intermittent failures on today's next (which
> contains this series).

Please check this fix:

https://lore.kernel.org/all/20260108102314.18816-1-bartosz.golaszewski@oss.qualcomm.com/

Looks exactly like the issue I've observed and finally debugged 
yesterday on TM2e board.


>
> This does not seem to happen on every boot, so it might be some race.
>
> [    2.169655] Mem abort info:
> [    2.169663]   ESR = 0x0000000096000004
> [    2.169670]   EC = 0x25: DABT (current EL), IL = 32 bits
> [    2.176773] ehci-platform 92040100.usb: irq 64, io mem 0x92040100
> [    2.186626]   SET = 0, FnV = 0
> [    2.204264] renesas_sdhi_internal_dmac 92090000.mmc: mmc1 base at 
> 0x0000000092090000, max clock rate 200 MHz
> [    2.204308] ehci-platform 92040100.usb: USB 2.0 started, EHCI 1.10
> [    2.208714] hub 2-0:1.0: USB hub found
> [    2.214345]   EA = 0, S1PTW = 0
> [    2.222949] hub 2-0:1.0: 1 port detected
> [    2.231502]   FSC = 0x04: level 0 translation fault
> [    2.239779] hub 1-0:1.0: USB hub found
> [    2.248022] Data abort info:
> [    2.256579] hub 1-0:1.0: 1 port detected
> [    2.264724]   ISV = 0, ISS = 0x00000004, ISS2 = 0x00000000
> [    2.487291]   CM = 0, WnR = 0, TnD = 0, TagAccess = 0
> [    2.493148]   GCS = 0, Overlay = 0, DirtyBit = 0, Xs = 0
> [    2.499300] [0000000000000000] user address but active_mm is swapper
> [    2.506556] Internal error: Oops: 0000000096000004 [#1]  SMP
> [    2.513092] Modules linked in:
> [    2.516610] CPU: 3 UID: 0 PID: 48 Comm: kworker/u16:2 Not tainted 
> 6.19.0-rc4-next-20260108+ #246 PREEMPT
> [    2.527564] Hardware name: Renesas RZ/T2H EVK Board based on 
> r9a09g077m44 (DT)
> [    2.535903] Workqueue: async async_run_entry_fn
> [    2.541141] pstate: 80400005 (Nzcv daif +PAN -UAO -TCO -DIT -SSBS 
> BTYPE=--)
> [    2.549076] pc : gpiod_direction_input_nonotify+0x18/0x22c
> [    2.555408] lr : gpiod_configure_flags+0x1c4/0x380
> [    2.560835] sp : ffff8000830e38c0
> [    2.564530] x29: ffff8000830e38c0 x28: ffff80008243894f x27: 
> ffff0001808f6410
> [    2.572773] x26: ffff800081e77160 x25: 0000000000000000 x24: 
> 0000000000000000
> [    2.580914] x23: 0000000000000000 x22: ffff800081e77160 x21: 
> ffff80008243892f
> [    2.589054] x20: 0000000000000000 x19: 0000000000000000 x18: 
> 000000000000000a
> [    2.597101] x17: ffff80007fffffff x16: ffff000180054480 x15: 
> 0000000000000000
> [    2.605345] x14: 0000000000000000 x13: ffff800080ed7600 x12: 
> 0000000000000000
> [    2.613485] x11: 0000000000000000 x10: 00007fff01adbb48 x9 : 
> 0000000000000005
> [    2.621626] x8 : 0101010101010101 x7 : fffffdffc6009b00 x6 : 
> 051573657480ffff
> [    2.629869] x5 : 0000000000000138 x4 : 0000000000000000 x3 : 
> 0000000000000001
> [    2.638113] x2 : 0000000000000000 x1 : 0000000000000000 x0 : 
> 0000000000000000
> [    2.646254] Call trace:
> [    2.648968]  gpiod_direction_input_nonotify+0x18/0x22c (P)
> [    2.655299]  gpiod_configure_flags+0x1c4/0x380
> [    2.660428]  gpiod_find_and_request+0x14c/0x524
> [    2.665650]  gpiod_get_index+0x58/0x7c
> [    2.669874]  devm_gpiod_get_index+0x20/0x90
> [    2.674695]  mmc_gpiod_request_ro+0x30/0xb0
> [    2.679525]  mmc_of_parse+0x100/0x688
> [    2.683749]  tmio_mmc_host_alloc+0x80/0x110
> [    2.688571]  renesas_sdhi_probe+0x150/0x760
> [    2.693392]  renesas_sdhi_internal_dmac_probe+0x64/0x80
> [    2.699357]  platform_probe+0x5c/0xa4
> [    2.703581]  really_probe+0xbc/0x2ac
> [    2.707703]  __driver_probe_device+0x78/0x118
> [    2.712728]  driver_probe_device+0x3c/0x178
> [    2.717550]  __device_attach_driver+0xb8/0x140
> [    2.722678]  bus_for_each_drv+0x88/0xe8
> [    2.727099]  __device_attach_async_helper+0xb0/0xd4
> [    2.732526]  async_run_entry_fn+0x34/0xe0
> [    2.737048]  process_one_work+0x150/0x290
> [    2.741673]  worker_thread+0x18c/0x300
> [    2.745915]  kthread+0x118/0x124
> [    2.749635]  ret_from_fork+0x10/0x20
> [    2.753762] Code: 910003fd a90153f3 aa0003f4 a9025bf5 (f9400013)
> [    2.760790] ---[ end trace 0000000000000000 ]---
>
> After running ./scripts/decode_stacktrace.sh
>
> [    2.169655] Mem abort info:
> [    2.169663]   ESR = 0x0000000096000004
> [    2.169670]   EC = 0x25: DABT (current EL), IL = 32 bits
> [    2.176773] ehci-platform 92040100.usb: irq 64, io mem 0x92040100
> [    2.186626]   SET = 0, FnV = 0
> [    2.204264] renesas_sdhi_internal_dmac 92090000.mmc: mmc1 base at 
> 0x0000000092090000, max clock rate 200 MHz
> [    2.204308] ehci-platform 92040100.usb: USB 2.0 started, EHCI 1.10
> [    2.208714] hub 2-0:1.0: USB hub found
> [    2.214345]   EA = 0, S1PTW = 0
> [    2.222949] hub 2-0:1.0: 1 port detected
> [    2.231502]   FSC = 0x04: level 0 translation fault
> [    2.239779] hub 1-0:1.0: USB hub found
> [    2.248022] Data abort info:
> [    2.256579] hub 1-0:1.0: 1 port detected
> [    2.264724]   ISV = 0, ISS = 0x00000004, ISS2 = 0x00000000
> [    2.487291]   CM = 0, WnR = 0, TnD = 0, TagAccess = 0
> [    2.493148]   GCS = 0, Overlay = 0, DirtyBit = 0, Xs = 0
> [    2.499300] [0000000000000000] user address but active_mm is swapper
> [    2.506556] Internal error: Oops: 0000000096000004 [#1]  SMP
> [    2.513092] Modules linked in:
> [    2.527564] Hardware name: Renesas RZ/T2H EVK Board based on 
> r9a09g077m44 (DT)
> [    2.535903] Workqueue: async async_run_entry_fn
> [    2.541141] pstate: 80400005 (Nzcv daif +PAN -UAO -TCO -DIT -SSBS 
> BTYPE=--)
> [    2.549076] pc : gpiod_direction_input_nonotify 
> (drivers/gpio/gpiolib.h:231 (discriminator 1) 
> drivers/gpio/gpiolib.c:2863 (discriminator 1))
> [    2.555408] lr : gpiod_configure_flags (drivers/gpio/gpiolib.c:4956)
> [    2.560835] sp : ffff8000830e38c0
> [    2.564530] x29: ffff8000830e38c0 x28: ffff80008243894f x27: 
> ffff0001808f6410
> [    2.572773] x26: ffff800081e77160 x25: 0000000000000000 x24: 
> 0000000000000000
> [    2.580914] x23: 0000000000000000 x22: ffff800081e77160 x21: 
> ffff80008243892f
> [    2.589054] x20: 0000000000000000 x19: 0000000000000000 x18: 
> 000000000000000a
> [    2.597101] x17: ffff80007fffffff x16: ffff000180054480 x15: 
> 0000000000000000
> [    2.605345] x14: 0000000000000000 x13: ffff800080ed7600 x12: 
> 0000000000000000
> [    2.613485] x11: 0000000000000000 x10: 00007fff01adbb48 x9 : 
> 0000000000000005
> [    2.621626] x8 : 0101010101010101 x7 : fffffdffc6009b00 x6 : 
> 051573657480ffff
> [    2.629869] x5 : 0000000000000138 x4 : 0000000000000000 x3 : 
> 0000000000000001
> [    2.638113] x2 : 0000000000000000 x1 : 0000000000000000 x0 : 
> 0000000000000000
> [    2.646254] Call trace:
> [    2.648968]  gpiod_direction_input_nonotify 
> (drivers/gpio/gpiolib.h:231 (discriminator 1) 
> drivers/gpio/gpiolib.c:2863 (discriminator 1)) (P)
> [    2.655299]  gpiod_configure_flags (drivers/gpio/gpiolib.c:4956)
> [    2.660428]  gpiod_find_and_request (drivers/gpio/gpiolib.c:4774)
> [    2.665650]  gpiod_get_index (drivers/gpio/gpiolib.c:4986)
> [    2.669874]  devm_gpiod_get_index (drivers/gpio/gpiolib-devres.c:102)
> [    2.674695]  mmc_gpiod_request_ro (drivers/mmc/core/slot-gpio.c:255)
> [    2.679525]  mmc_of_parse (drivers/mmc/core/host.c:348)
> [    2.683749]  tmio_mmc_host_alloc 
> (drivers/mmc/host/tmio_mmc_core.c:1149)
> [    2.688571]  renesas_sdhi_probe 
> (drivers/mmc/host/renesas_sdhi_core.c:1119)
> [    2.693392]  renesas_sdhi_internal_dmac_probe 
> (drivers/mmc/host/renesas_sdhi_internal_dmac.c:599)
> [    2.699357]  platform_probe (drivers/base/platform.c:1449 
> (discriminator 1))
> [    2.703581]  really_probe (drivers/base/dd.c:581 
> drivers/base/dd.c:659)
> [    2.707703]  __driver_probe_device (drivers/base/dd.c:801)
> [    2.712728]  driver_probe_device (drivers/base/dd.c:831)
> [    2.717550]  __device_attach_driver (drivers/base/dd.c:960)
> [    2.722678]  bus_for_each_drv (drivers/base/bus.c:500)
> [    2.727099]  __device_attach_async_helper 
> (include/linux/pm_runtime.h:447 drivers/base/dd.c:991)
> [    2.732526]  async_run_entry_fn (kernel/async.c:136 (discriminator 3))
> [    2.737048]  process_one_work (kernel/workqueue.c:3290 
> (discriminator 1))
> [    2.741673]  worker_thread (kernel/workqueue.c:3356 (discriminator 
> 2) kernel/workqueue.c:3443 (discriminator 2))
> [    2.745915]  kthread (kernel/kthread.c:467)
> [    2.749635]  ret_from_fork (arch/arm64/kernel/entry.S:861)
> [    2.753762] Code: 910003fd a90153f3 aa0003f4 a9025bf5 (f9400013)
>
>> Signed-off-by: Bartosz Golaszewski 
>> <bartosz.golaszewski@oss.qualcomm.com>
>> ---
>> Changes in v2:
>> - Fix mutex deadlock
>> - Fix lockdep false-positives
>> - Add a patch reducing the number of memory allocations
>> - Link to v1: 
>> https://lore.kernel.org/r/20260105-gpio-shared-fixes-v1-0-76d6ff0afcd8@oss.qualcomm.com
>>
>> ---
>> Bartosz Golaszewski (3):
>>        gpio: shared: assign the correct firmware node for reset-gpio 
>> use-case
>>        gpio: shared: fix a race condition
>>        gpio: shared: don't allocate the lookup table until we really 
>> need it
>>
>>   drivers/gpio/gpiolib-shared.c | 34 ++++++++++++++++------------------
>>   1 file changed, 16 insertions(+), 18 deletions(-)
>> ---
>> base-commit: 19fb766a1e5ed5943a62fc671c09d45352a81b1d
>> change-id: 20260105-gpio-shared-fixes-40a8ec3b6b25
>>
>
Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland
Re: [PATCH v2 0/3] gpio: shared: another set of small fixes
Posted by Bartosz Golaszewski 4 weeks, 1 day ago 
On Thu, Jan 8, 2026 at 1:13 PM Cosmin Tanislav <demonsingur@gmail.com> wrote:
>
> On 1/6/26 11:34 AM, Bartosz Golaszewski wrote:
> > Here are three more patches fixing issues reported with shared GPIO
> > management.
> >
>
> I'm still getting some intermittent failures on today's next (which
> contains this series).
>
> This does not seem to happen on every boot, so it might be some race.
>

Should be fixed by:
https://lore.kernel.org/all/20260108102314.18816-1-bartosz.golaszewski@oss.qualcomm.com/

Please give it a try and I'll queue it shortly.

Bart
Re: [PATCH v2 0/3] gpio: shared: another set of small fixes
Posted by Cosmin Tanislav 4 weeks, 1 day ago 

On 1/8/26 2:18 PM, Bartosz Golaszewski wrote:
> On Thu, Jan 8, 2026 at 1:13 PM Cosmin Tanislav <demonsingur@gmail.com> wrote:
>>
>> On 1/6/26 11:34 AM, Bartosz Golaszewski wrote:
>>> Here are three more patches fixing issues reported with shared GPIO
>>> management.
>>>
>>
>> I'm still getting some intermittent failures on today's next (which
>> contains this series).
>>
>> This does not seem to happen on every boot, so it might be some race.
>>
> 
> Should be fixed by:
> https://lore.kernel.org/all/20260108102314.18816-1-bartosz.golaszewski@oss.qualcomm.com/
> 
> Please give it a try and I'll queue it shortly.
>

Thank you, I can't seem to hit the issue anymore.

> Bart
Re: [PATCH v2 0/3] gpio: shared: another set of small fixes
Posted by Bartosz Golaszewski 1 month ago 
On Tue, 06 Jan 2026 10:34:20 +0100, Bartosz Golaszewski wrote:
> Here are three more patches fixing issues reported with shared GPIO
> management.
> 
> 

Applied, thanks!

[1/3] gpio: shared: assign the correct firmware node for reset-gpio use-case
      commit: 0fe50631791bd3504dc7f32af6421bd4041f14aa
[2/3] gpio: shared: fix a race condition
      commit: 476e44d06fc107f8cd99695d8e4f1c792dfc3379
[3/3] gpio: shared: don't allocate the lookup table until we really need it
      commit: a80208072df8f4ceb53cd905c1f4362f84ce397f

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
Re: [PATCH v2 0/3] gpio: shared: another set of small fixes
Posted by Mark Brown 1 month ago 
On Tue, Jan 06, 2026 at 10:34:20AM +0100, Bartosz Golaszewski wrote:
> Here are three more patches fixing issues reported with shared GPIO
> management.

This fixes the problems I'm seeing on at least
k3-am625-verdin-wifi-mallow:

Tested-by: Mark Brown <broonie@kernel.org>

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.