Re: [PATCH v3 3/8] irqchip/qcom-pdc: Remove pdc_enable_intr() wrapper
From: Maulik Shah (mkshah)
Date: Mon Jun 29 2026 - 23:52:59 EST
On 6/28/2026 11:24 PM, Val Packett wrote:
>
> On 6/16/26 6:25 AM, Maulik Shah wrote:
[...]
> whoops..
>
> [ 0.197090] BUG: spinlock bad magic on CPU#7, swapper/0/1
> [ 0.197104] lock: 0xffff0001022e37b0, .magic: 00000000, .owner: <none>/-1, .owner_cpu: 0
> [ 0.197122] CPU: 7 UID: 0 PID: 1 Comm: swapper/0 Not tainted 7.1.0-next-20260626-uwu+ #128 PREEMPT(full)
> [ 0.197129] Hardware name: motorola Motorola Edge 30 (Tianma)/Motorola Edge 30 (Tianma), BIOS 2026.07-rc2-g432bcf301c03-dirty 07/01/2026
> [ 0.197133] Call trace:
> [ 0.197135] show_stack+0x24/0x38 (C)
> [ 0.197148] __dump_stack+0x28/0x38
> [ 0.197156] dump_stack_lvl+0x7c/0xa8
> [ 0.197165] dump_stack+0x18/0x30
> [ 0.197172] spin_dump+0x7c/0x98
> [ 0.197179] do_raw_spin_lock+0xa4/0x140
> [ 0.197189] _raw_spin_lock+0x2c/0x40
> [ 0.197195] pdc_enable_intr_bank+0x40/0x128
> [ 0.197201] qcom_pdc_probe+0x3bc/0x520
>
> Gotta also move the spinlock init to before the call to pdc_setup_pin_mapping..
Yes, Moving spinlock will be added in v4 of series.
Thanks,
Maulik