diff mbox series

ASoC: rt1015p: add acpi device id for rt1015p

Message ID 20210226020517.17124-1-jack.yu@realtek.com (mailing list archive)
State Superseded
Headers show
Series ASoC: rt1015p: add acpi device id for rt1015p | expand

Commit Message

Jack Yu Feb. 26, 2021, 2:05 a.m. UTC
From: Jack Yu <jack.yu@realtek.com>

Add acpi device id for rt1015p.

Signed-off-by: Jack Yu <jack.yu@realtek.com>
---
 sound/soc/codecs/rt1015p.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

Comments

Pierre-Louis Bossart Feb. 26, 2021, 9:48 p.m. UTC | #1
On 2/25/21 8:05 PM, jack.yu@realtek.com wrote:
> From: Jack Yu <jack.yu@realtek.com>
> 
> Add acpi device id for rt1015p.
> 
> Signed-off-by: Jack Yu <jack.yu@realtek.com>
> ---
>   sound/soc/codecs/rt1015p.c | 10 ++++++++++
>   1 file changed, 10 insertions(+)
> 
> diff --git a/sound/soc/codecs/rt1015p.c b/sound/soc/codecs/rt1015p.c
> index 671f2a2130fe..61694de080e6 100644
> --- a/sound/soc/codecs/rt1015p.c
> +++ b/sound/soc/codecs/rt1015p.c
> @@ -4,6 +4,7 @@
>   //
>   // Copyright 2020 The Linux Foundation. All rights reserved.
>   
> +#include <linux/acpi.h>
>   #include <linux/delay.h>
>   #include <linux/device.h>
>   #include <linux/err.h>
> @@ -130,10 +131,19 @@ static const struct of_device_id rt1015p_device_id[] = {
>   MODULE_DEVICE_TABLE(of, rt1015p_device_id);
>   #endif
>   
> +#ifdef CONFIG_ACPI
> +static const struct acpi_device_id rt1015p_acpi_match[] = {
> +	{ "ALCP1015", 0},

This isn't much better, ALC is not a vendor ID as defined by 
https://uefi.org/PNP_ACPI_Registry

I've ask for guidance from Intel ACPI folks on this, please stay tuned.

> +	{ },
> +};
> +MODULE_DEVICE_TABLE(acpi, rt1015p_acpi_match);
> +#endif
> +
>   static struct platform_driver rt1015p_platform_driver = {
>   	.driver = {
>   		.name = "rt1015p",
>   		.of_match_table = of_match_ptr(rt1015p_device_id),
> +		.acpi_match_table = ACPI_PTR(rt1015p_acpi_match),
>   	},
>   	.probe = rt1015p_platform_probe,
>   };
>
Jack Yu March 8, 2021, 6:10 a.m. UTC | #2
> -----Original Message-----
> From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> Sent: Saturday, February 27, 2021 5:48 AM
> To: Jack Yu <jack.yu@realtek.com>; broonie@kernel.org;
> lgirdwood@gmail.com
> Cc: Oder Chiou <oder_chiou@realtek.com>; alsa-devel@alsa-project.org;
> lars@metafoo.de; kent_chen@realtek.com [陳建宏]
> <kent_chen@realtek.com>; 陳昶志 <kenny_chen@realtek.com>; Derek [方德
> 義] <derek.fang@realtek.com>; Shuming [范書銘] <shumingf@realtek.com>;
> Flove(HsinFu) <flove@realtek.com>
> Subject: Re: [PATCH] ASoC: rt1015p: add acpi device id for rt1015p
> 
> On 2/25/21 8:05 PM, jack.yu@realtek.com wrote:
> > From: Jack Yu <jack.yu@realtek.com>
> >
> > Add acpi device id for rt1015p.
> >
> > Signed-off-by: Jack Yu <jack.yu@realtek.com>
> > ---
> >   sound/soc/codecs/rt1015p.c | 10 ++++++++++
> >   1 file changed, 10 insertions(+)
> >
> > diff --git a/sound/soc/codecs/rt1015p.c b/sound/soc/codecs/rt1015p.c
> > index 671f2a2130fe..61694de080e6 100644
> > --- a/sound/soc/codecs/rt1015p.c
> > +++ b/sound/soc/codecs/rt1015p.c
> > @@ -4,6 +4,7 @@
> >   //
> >   // Copyright 2020 The Linux Foundation. All rights reserved.
> >
> > +#include <linux/acpi.h>
> >   #include <linux/delay.h>
> >   #include <linux/device.h>
> >   #include <linux/err.h>
> > @@ -130,10 +131,19 @@ static const struct of_device_id
> rt1015p_device_id[] = {
> >   MODULE_DEVICE_TABLE(of, rt1015p_device_id);
> >   #endif
> >
> > +#ifdef CONFIG_ACPI
> > +static const struct acpi_device_id rt1015p_acpi_match[] = {
> > +	{ "ALCP1015", 0},
> 
> This isn't much better, ALC is not a vendor ID as defined by
> https://uefi.org/PNP_ACPI_Registry
> 
> I've ask for guidance from Intel ACPI folks on this, please stay tuned.
> 

Hi Pierre,

Is there any update from Intel ACPI team regarding to this ID format issue?


BR,
Jack

> > +	{ },
> > +};
> > +MODULE_DEVICE_TABLE(acpi, rt1015p_acpi_match); #endif
> > +
> >   static struct platform_driver rt1015p_platform_driver = {
> >   	.driver = {
> >   		.name = "rt1015p",
> >   		.of_match_table = of_match_ptr(rt1015p_device_id),
> > +		.acpi_match_table = ACPI_PTR(rt1015p_acpi_match),
> >   	},
> >   	.probe = rt1015p_platform_probe,
> >   };
> >
> 
> ------Please consider the environment before printing this e-mail.
Pierre-Louis Bossart March 9, 2021, 8:26 p.m. UTC | #3
>>> +#ifdef CONFIG_ACPI
>>> +static const struct acpi_device_id rt1015p_acpi_match[] = {
>>> +	{ "ALCP1015", 0},
>>
>> This isn't much better, ALC is not a vendor ID as defined by
>> https://uefi.org/PNP_ACPI_Registry
>>
>> I've ask for guidance from Intel ACPI folks on this, please stay tuned.
>>
> 
> Hi Pierre,
> 
> Is there any update from Intel ACPI team regarding to this ID format issue?

Sorry about the delay.

It seems that the options are limited. The HID needs to be made of two 
parts, vendor ID and part ID.

For the vendor ID, you may use the PNP ID (RTL) or the PCI ID (10EC).

For the part ID, you are limited to 4 hex-digits. I didn't get any 
positive answers on how to support '1015P', the only solution seems to 
be to create a new part number, or detect at run-time which of the 
'1015' skews is actually used.
Jack Yu March 10, 2021, 2:09 a.m. UTC | #4
> >>> +#ifdef CONFIG_ACPI
> >>> +static const struct acpi_device_id rt1015p_acpi_match[] = {
> >>> +	{ "ALCP1015", 0},
> >>
> >> This isn't much better, ALC is not a vendor ID as defined by
> >> https://uefi.org/PNP_ACPI_Registry
> >>
> >> I've ask for guidance from Intel ACPI folks on this, please stay tuned.
> >>
> >
> > Hi Pierre,
> >
> > Is there any update from Intel ACPI team regarding to this ID format issue?
> 
> Sorry about the delay.
> 
> It seems that the options are limited. The HID needs to be made of two parts,
> vendor ID and part ID.
> 
> For the vendor ID, you may use the PNP ID (RTL) or the PCI ID (10EC).
> 
> For the part ID, you are limited to 4 hex-digits. I didn't get any positive answers
> on how to support '1015P', the only solution seems to be to create a new part
> number, or detect at run-time which of the '1015' skews is actually used.
> 

Actually this driver is also used for rt1015 IC, the only difference is that rt1015.c is for i2c mode
and rt1015p is for non-i2c mode of rt1015 IC, so I think it's ok to use the same part ID (different vendor ID) for two drivers.
How do you think if we use "RTL1015" instead?
Pierre-Louis Bossart March 10, 2021, 3:39 p.m. UTC | #5
>> It seems that the options are limited. The HID needs to be made of two parts,
>> vendor ID and part ID.
>>
>> For the vendor ID, you may use the PNP ID (RTL) or the PCI ID (10EC).
>>
>> For the part ID, you are limited to 4 hex-digits. I didn't get any positive answers
>> on how to support '1015P', the only solution seems to be to create a new part
>> number, or detect at run-time which of the '1015' skews is actually used.
>>
> 
> Actually this driver is also used for rt1015 IC, the only difference is that rt1015.c is for i2c mode
> and rt1015p is for non-i2c mode of rt1015 IC, so I think it's ok to use the same part ID (different vendor ID) for two drivers.
> How do you think if we use "RTL1015" instead?

that would work indeed, e.g. 10EC1015 refers to the I2C version and 
RTL1015 to the non-I2C version. Thanks!
Mark Brown March 12, 2021, 8:23 p.m. UTC | #6
On Fri, 26 Feb 2021 10:05:17 +0800, jack.yu@realtek.com wrote:
> Add acpi device id for rt1015p.

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/1] ASoC: rt1015p: add acpi device id for rt1015p
      commit: 464b489a7885d6a89b0dfa7b7f1f56e2db80700d

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
diff mbox series

Patch

diff --git a/sound/soc/codecs/rt1015p.c b/sound/soc/codecs/rt1015p.c
index 671f2a2130fe..61694de080e6 100644
--- a/sound/soc/codecs/rt1015p.c
+++ b/sound/soc/codecs/rt1015p.c
@@ -4,6 +4,7 @@ 
 //
 // Copyright 2020 The Linux Foundation. All rights reserved.
 
+#include <linux/acpi.h>
 #include <linux/delay.h>
 #include <linux/device.h>
 #include <linux/err.h>
@@ -130,10 +131,19 @@  static const struct of_device_id rt1015p_device_id[] = {
 MODULE_DEVICE_TABLE(of, rt1015p_device_id);
 #endif
 
+#ifdef CONFIG_ACPI
+static const struct acpi_device_id rt1015p_acpi_match[] = {
+	{ "ALCP1015", 0},
+	{ },
+};
+MODULE_DEVICE_TABLE(acpi, rt1015p_acpi_match);
+#endif
+
 static struct platform_driver rt1015p_platform_driver = {
 	.driver = {
 		.name = "rt1015p",
 		.of_match_table = of_match_ptr(rt1015p_device_id),
+		.acpi_match_table = ACPI_PTR(rt1015p_acpi_match),
 	},
 	.probe = rt1015p_platform_probe,
 };