When a component has a prefix, and uses a SND_SOC_DAPM_REGULATOR_SUPPLY,
the name of the regulator should not use the prefix, otherwise it won't
be properly matched in the DT/ACPI.
Fixes: 3caac759681e ("ASoC: soc-dapm.c: fixup snd_soc_dapm_new_control_unlocked() error handling")
Signed-off-by: Paul Cercueil <[email protected]>
---
sound/soc/soc-dapm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c
index d515e7a78ea8..879cf1be67a9 100644
--- a/sound/soc/soc-dapm.c
+++ b/sound/soc/soc-dapm.c
@@ -3645,7 +3645,7 @@ snd_soc_dapm_new_control_unlocked(struct snd_soc_dapm_context *dapm,
switch (w->id) {
case snd_soc_dapm_regulator_supply:
- w->regulator = devm_regulator_get(dapm->dev, w->name);
+ w->regulator = devm_regulator_get(dapm->dev, widget->name);
if (IS_ERR(w->regulator)) {
ret = PTR_ERR(w->regulator);
goto request_failed;
--
2.35.1
On Tue, 25 Oct 2022 16:01:49 +0100, Paul Cercueil wrote:
> When a component has a prefix, and uses a SND_SOC_DAPM_REGULATOR_SUPPLY,
> the name of the regulator should not use the prefix, otherwise it won't
> be properly matched in the DT/ACPI.
>
>
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/1] ASoC: dapm: Don't use prefix for regulator name
commit: 1683d3282f240336a2b4b6b541d435facfe8bbb6
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
Hi Mark,
Le mer. 26 oct. 2022 ? 19:07:28 +0100, Mark Brown <[email protected]>
a ?crit :
> On Tue, 25 Oct 2022 16:01:49 +0100, Paul Cercueil wrote:
>> When a component has a prefix, and uses a
>> SND_SOC_DAPM_REGULATOR_SUPPLY,
>> the name of the regulator should not use the prefix, otherwise it
>> won't
>> be properly matched in the DT/ACPI.
>>
>>
>
> Applied to
>
> https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
> for-next
Small reminder that this is a fix for a bug introduced in -rc1, so it
should be queued for the next RC.
Cheers,
-Paul
> Thanks!
>
> [1/1] ASoC: dapm: Don't use prefix for regulator name
> commit: 1683d3282f240336a2b4b6b541d435facfe8bbb6
>
> 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
Le lun. 7 nov. 2022 ? 12:57:05 +0000, Mark Brown <[email protected]>
a ?crit :
> On Mon, Nov 07, 2022 at 09:52:24AM +0000, Paul Cercueil wrote:
>> Le mer. 26 oct. 2022 ? 19:07:28 +0100, Mark Brown
>> <[email protected]> a
>
>> > Applied to
>> >
>> >
>> https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
>> > for-next
>
>> Small reminder that this is a fix for a bug introduced in -rc1, so
>> it should
>> be queued for the next RC.
>
> This is already queued as a fix.
Ok, thanks. I didn't see it in -rc4 and thought that it was queued for
6.2 instead.
Cheers,
-Paul
On Mon, Nov 07, 2022 at 09:52:24AM +0000, Paul Cercueil wrote:
> Le mer. 26 oct. 2022 ? 19:07:28 +0100, Mark Brown <[email protected]> a
> > Applied to
> >
> > https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
> > for-next
> Small reminder that this is a fix for a bug introduced in -rc1, so it should
> be queued for the next RC.
This is already queued as a fix.