Re: [PATCH v3 3/8] wifi: ath10k: snoc: support powering on the device via pwrseq
Dmitry Baryshkov <[email protected]> Sat, 18 Apr 2026 22:38:21 +0300
| Newsgroups | org.infradead.lists.ath10k,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-bluetooth,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.linux-pm,org.kernel.vger.linux-wireless |
|---|---|
| Message-ID | <hdypom3nioc6tk26gh647imy5ykhcjqvknideilnbc2b5p7eo7@hm7fsscleutf> |
On Thu, Apr 16, 2026 at 12:06:09PM +0200, Luca Weiss wrote: > Hi Dmitry, > > On Mon Jan 19, 2026 at 6:07 PM CET, Dmitry Baryshkov wrote: > > The WCN39xx family of WiFi/BT chips incorporates a simple PMU, spreading > > voltages over internal rails. Implement support for using powersequencer > > for this family of ATH10k devices in addition to using regulators. > > > > Reviewed-by: Bartosz Golaszewski <[email protected]> > > Signed-off-by: Dmitry Baryshkov <[email protected]> > > --- > > drivers/net/wireless/ath/ath10k/snoc.c | 53 ++++++++++++++++++++++++++++++++-- > > drivers/net/wireless/ath/ath10k/snoc.h | 3 ++ > > 2 files changed, 53 insertions(+), 3 deletions(-) > > > > + ar_snoc->pwrseq = devm_pwrseq_get(&pdev->dev, "wlan"); > > + if (IS_ERR(ar_snoc->pwrseq)) { > > + ret = PTR_ERR(ar_snoc->pwrseq); > > + ar_snoc->pwrseq = NULL; > > + if (ret != -EPROBE_DEFER) > > + goto err_free_irq; > > I'm fairly sure this is now broken with CONFIG_POWER_SEQUENCING=n since > then pwrseq_get() is returning ERR_PTR(-ENOSYS) which is not handled > here. > > I'm observing my ath10k_snoc is now failing to probe "with error -38" > which definitely seems to be related, but I haven't debugged it further > yet. Posted https://patch.msgid.link/[email protected] -- With best wishes Dmitry