Thread (7 messages) 7 messages, 3 authors, 2d ago

Re: [PATCH v2] remoteproc: qcom_wcnss: Fix handling the lack of PD regulators in v3

From: Konrad Dybcio <hidden>
Date: 2026-02-02 10:57:23
Also in: linux-arm-msm, linux-remoteproc, lkml, llvm, phone-devel

On 2/1/26 9:55 PM, Val Packett wrote:
The changes introduced to handle single power domain platforms have
swapped the info pointer increment from num_pd_vregs to num_pds, which
would shift the info pointer past the end of the array for pronto-v3,
which does not list power domain regulators in vregs.

This showed up as a difference between GCC- and LLVM-compiled kernels
on SDM632 devices, where only with LLVM one would get the
"regulator request with no identifier" error, because the out-of-bounds
memory ended up being zeroed. Fix by skipping the increment when there
are more power domains than regulators.

Signed-off-by: Val Packett <redacted>
---
Reviewed-by: Konrad Dybcio <redacted>

Konrad
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help