* [PATCH] ASoC: intel: rt5640: add support for sound card found on HP Pavilion x2 10-p000
@ 2017-01-03 20:40 Alexandrov Stanislav
2017-01-04 4:02 ` Vinod Koul
0 siblings, 1 reply; 3+ messages in thread
From: Alexandrov Stanislav @ 2017-01-03 20:40 UTC (permalink / raw)
To: Alsa Devel; +Cc: Vinod Koul, Pierre-Louis Bossart
Add support for rt5640 sound card found on HP Pavilion x2 10-p000
tablet.
Inside DSDT table there is record for this soundcard:
Device (RTKC)
{
Name (_ADR, Zero) // _ADR: Address
Name (_HID, "10EC3276") // _HID: Hardware ID
Name (_CID, "10EC3276") // _CID: Compatible ID
Name (_DDN, "ALC3276") // _DDN: DOS Device Name
Name (_SUB, "103C827C") // _SUB: Subsystem ID
Name (_PR0, Package (0x01) // _PR0: Power Resources for
D0
original bugreport: https://bugzilla.kernel.org/show_bug.cgi?id=187621
---
diff --git a/sound/soc/codecs/rt5640.c b/sound/soc/codecs/rt5640.c
index e29a6de..c5234b9 100644
--- a/sound/soc/codecs/rt5640.c
+++ b/sound/soc/codecs/rt5640.c
@@ -2315,6 +2315,7 @@ static const struct acpi_device_id
rt5640_acpi_match[] = {
{ "INT33CA", 0 },
{ "10EC5640", 0 },
{ "10EC5642", 0 },
+ { "10EC3276", 0 },
{ "INTCCFFD", 0 },
{ },
};
diff --git a/sound/soc/intel/atom/sst/sst_acpi.c
b/sound/soc/intel/atom/sst/sst_acpi.c
index f4d92bb..896ced2 100644
--- a/sound/soc/intel/atom/sst/sst_acpi.c
+++ b/sound/soc/intel/atom/sst/sst_acpi.c
@@ -463,6 +463,8 @@ static struct sst_acpi_mach sst_acpi_chv[] = {
/* some CHT-T platforms rely on RT5640, use Baytrail machine
driver */
{"10EC5640", "bytcr_rt5640", "intel/fw_sst_22a8.bin",
"bytcr_rt5640", cht_quirk,
&chv_platform_data },
+ {"10EC3276", "bytcr_rt5640", "intel/fw_sst_22a8.bin",
"bytcr_rt5640", NULL,
+ &chv_platform_data },
{},
};
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] ASoC: intel: rt5640: add support for sound card found on HP Pavilion x2 10-p000
2017-01-03 20:40 [PATCH] ASoC: intel: rt5640: add support for sound card found on HP Pavilion x2 10-p000 Alexandrov Stanislav
@ 2017-01-04 4:02 ` Vinod Koul
0 siblings, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2017-01-04 4:02 UTC (permalink / raw)
To: Alexandrov Stanislav; +Cc: Alsa Devel, Pierre-Louis Bossart
On Tue, Jan 03, 2017 at 11:40:04PM +0300, Alexandrov Stanislav wrote:
> Add support for rt5640 sound card found on HP Pavilion x2 10-p000
> tablet.
>
> Inside DSDT table there is record for this soundcard:
>
> Device (RTKC)
> {
> Name (_ADR, Zero) // _ADR: Address
> Name (_HID, "10EC3276") // _HID: Hardware ID
> Name (_CID, "10EC3276") // _CID: Compatible ID
> Name (_DDN, "ALC3276") // _DDN: DOS Device Name
> Name (_SUB, "103C827C") // _SUB: Subsystem ID
> Name (_PR0, Package (0x01) // _PR0: Power Resources
> for D0
>
> original bugreport: https://bugzilla.kernel.org/show_bug.cgi?id=187621
Hi,
Thanks for the patch, changes look okay, but they need to be split per
driver and you need to sign-off these changes.
See Documentation/process/submitting-patches.rst esp Section 11.
>
> ---
>
> diff --git a/sound/soc/codecs/rt5640.c b/sound/soc/codecs/rt5640.c
> index e29a6de..c5234b9 100644
> --- a/sound/soc/codecs/rt5640.c
> +++ b/sound/soc/codecs/rt5640.c
> @@ -2315,6 +2315,7 @@ static const struct acpi_device_id
> rt5640_acpi_match[] = {
> { "INT33CA", 0 },
> { "10EC5640", 0 },
> { "10EC5642", 0 },
> + { "10EC3276", 0 },
> { "INTCCFFD", 0 },
> { },
> };
> diff --git a/sound/soc/intel/atom/sst/sst_acpi.c
> b/sound/soc/intel/atom/sst/sst_acpi.c
> index f4d92bb..896ced2 100644
> --- a/sound/soc/intel/atom/sst/sst_acpi.c
> +++ b/sound/soc/intel/atom/sst/sst_acpi.c
> @@ -463,6 +463,8 @@ static struct sst_acpi_mach sst_acpi_chv[] = {
> /* some CHT-T platforms rely on RT5640, use Baytrail machine
> driver */
> {"10EC5640", "bytcr_rt5640", "intel/fw_sst_22a8.bin",
> "bytcr_rt5640", cht_quirk,
> &chv_platform_data },
> + {"10EC3276", "bytcr_rt5640", "intel/fw_sst_22a8.bin",
> "bytcr_rt5640", NULL,
> + &chv_platform_data },
>
> {},
> };
--
~Vinod
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] ASoC: intel: rt5640: add support for sound card found on HP Pavilion x2 10-p000
@ 2017-01-03 21:56 Alexandrov Stanislav
0 siblings, 0 replies; 3+ messages in thread
From: Alexandrov Stanislav @ 2017-01-03 21:56 UTC (permalink / raw)
To: Alsa Devel; +Cc: Vinod Koul, Pierre-Louis Bossart
Add support for rt5640 sound card found on HP Pavilion x2 10-p000
tablet.
original bugreport: https://bugzilla.kernel.org/show_bug.cgi?id=187621
Signed-off-by: Alexandrov Stanislav <neko@nya.ai>
---
sound/soc/codecs/rt5640.c | 1 +
sound/soc/intel/atom/sst/sst_acpi.c | 2 ++
2 files changed, 3 insertions(+)
diff --git a/sound/soc/codecs/rt5640.c b/sound/soc/codecs/rt5640.c
index e29a6de..c5234b9 100644
--- a/sound/soc/codecs/rt5640.c
+++ b/sound/soc/codecs/rt5640.c
@@ -2315,6 +2315,7 @@ static const struct acpi_device_id
rt5640_acpi_match[] = {
{ "INT33CA", 0 },
{ "10EC5640", 0 },
{ "10EC5642", 0 },
+ { "10EC3276", 0 },
{ "INTCCFFD", 0 },
{ },
};
diff --git a/sound/soc/intel/atom/sst/sst_acpi.c
b/sound/soc/intel/atom/sst/sst_acpi.c
index f4d92bb..896ced2 100644
--- a/sound/soc/intel/atom/sst/sst_acpi.c
+++ b/sound/soc/intel/atom/sst/sst_acpi.c
@@ -463,6 +463,8 @@ static struct sst_acpi_mach sst_acpi_chv[] = {
/* some CHT-T platforms rely on RT5640, use Baytrail machine driver */
{"10EC5640", "bytcr_rt5640", "intel/fw_sst_22a8.bin", "bytcr_rt5640",
cht_quirk,
&chv_platform_data },
+ {"10EC3276", "bytcr_rt5640", "intel/fw_sst_22a8.bin", "bytcr_rt5640",
NULL,
+ &chv_platform_data },
{},
};
--
2.10.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-01-04 4:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-03 20:40 [PATCH] ASoC: intel: rt5640: add support for sound card found on HP Pavilion x2 10-p000 Alexandrov Stanislav
2017-01-04 4:02 ` Vinod Koul
-- strict thread matches above, loose matches on Subject: below --
2017-01-03 21:56 Alexandrov Stanislav
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox