* [PATCH] ASoC: tlv320aic3x: Use dev_err_probe
@ 2025-05-12 18:57 Nishanth Menon
2025-05-13 16:34 ` Mark Brown
0 siblings, 1 reply; 2+ messages in thread
From: Nishanth Menon @ 2025-05-12 18:57 UTC (permalink / raw)
To: Takashi Iwai, Jaroslav Kysela, Mark Brown, Liam Girdwood
Cc: linux-kernel, alsa-devel, Nishanth Menon
During probe the regulator supply drivers may not yet be available.
Use dev_err_probe to provide just the pertinent log.
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Testing based on next-20250512 (includes additional patch) on AM62a7-sk
Before: https://gist.github.com/nmenon/be94f21e83b4260ad3f89e1ae8f0d188#file-before-L851
After: https://gist.github.com/nmenon/be94f21e83b4260ad3f89e1ae8f0d188#file-after-L806
sound/soc/codecs/tlv320aic3x.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c
index 56e795a00e22..f1649df19738 100644
--- a/sound/soc/codecs/tlv320aic3x.c
+++ b/sound/soc/codecs/tlv320aic3x.c
@@ -1818,10 +1818,8 @@ int aic3x_probe(struct device *dev, struct regmap *regmap, kernel_ulong_t driver
ret = devm_regulator_bulk_get(dev, ARRAY_SIZE(aic3x->supplies),
aic3x->supplies);
- if (ret) {
- dev_err(dev, "Failed to request supplies: %d\n", ret);
- return ret;
- }
+ if (ret)
+ return dev_err_probe(dev, ret, "Failed to request supplies\n");
aic3x_configure_ocmv(dev, aic3x);
--
2.47.0
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH] ASoC: tlv320aic3x: Use dev_err_probe
2025-05-12 18:57 [PATCH] ASoC: tlv320aic3x: Use dev_err_probe Nishanth Menon
@ 2025-05-13 16:34 ` Mark Brown
0 siblings, 0 replies; 2+ messages in thread
From: Mark Brown @ 2025-05-13 16:34 UTC (permalink / raw)
To: Takashi Iwai, Jaroslav Kysela, Liam Girdwood, Nishanth Menon
Cc: linux-kernel, alsa-devel
On Mon, 12 May 2025 13:57:39 -0500, Nishanth Menon wrote:
> During probe the regulator supply drivers may not yet be available.
> Use dev_err_probe to provide just the pertinent log.
>
>
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/1] ASoC: tlv320aic3x: Use dev_err_probe
commit: 85f8c2d56caf56aa3379bbc5f1a19fef9aabd23e
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-05-13 16:34 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-05-12 18:57 [PATCH] ASoC: tlv320aic3x: Use dev_err_probe Nishanth Menon
2025-05-13 16:34 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®