Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1262752 > unrolled thread

[PATCH V2] Intel device IDs Audio

Started byAlexandra Yates <alexandra.yates@linux.intel.com>
First post2015-11-05 01:00 +0100
Last post2015-11-05 22:20 +0100
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH V2] Intel device IDs Audio Alexandra Yates <alexandra.yates@linux.intel.com> - 2015-11-05 01:00 +0100
    Re: [PATCH V2] Intel device IDs Audio Takashi Iwai <tiwai@suse.de> - 2015-11-05 11:30 +0100
      Re: [PATCH V2] Intel device IDs Audio Alexandra Yates <alexandra.yates@linux.intel.com> - 2015-11-05 20:50 +0100
        Re: [PATCH V2] Intel device IDs Audio Takashi Iwai <tiwai@suse.de> - 2015-11-05 22:20 +0100

#1262752 — [PATCH V2] Intel device IDs Audio

FromAlexandra Yates <alexandra.yates@linux.intel.com>
Date2015-11-05 01:00 +0100
Subject[PATCH V2] Intel device IDs Audio
Message-ID<qrg8a-7fJ-15@gated-at.bofh.it>
Adding Intel codename Lewisburg platform device IDs for audio.

Signed-off-by: Alexandra Yates <alexandra.yates@linux.intel.com>
---
 sound/pci/hda/hda_intel.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index c38c68f..e8d1151 100644
--- a/sound/pci/hda/hda_intel.c
+++ b/sound/pci/hda/hda_intel.c
@@ -2168,6 +2168,11 @@ static const struct pci_device_id azx_ids[] = {
 	/* ICH10 */
 	{ PCI_DEVICE(0x8086, 0x3a6e),
 	  .driver_data = AZX_DRIVER_ICH | AZX_DCAPS_INTEL_ICH },
+	/* Lewisburg */
+	{ PCI_DEVICE(0x8086, 0xa1f0),
+	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
+	{ PCI_DEVICE(0x8086, 0xa270),
+	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
 	/* Generic Intel */
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_ANY_ID),
 	  .class = PCI_CLASS_MULTIMEDIA_HD_AUDIO << 8,
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1263071

FromTakashi Iwai <tiwai@suse.de>
Date2015-11-05 11:30 +0100
Message-ID<qrpXP-5kx-5@gated-at.bofh.it>
In reply to#1262752
On Thu, 05 Nov 2015 00:56:09 +0100,
Alexandra Yates wrote:
> 
> Adding Intel codename Lewisburg platform device IDs for audio.
> 
> Signed-off-by: Alexandra Yates <alexandra.yates@linux.intel.com>

Thanks.  Though, I still believe the position after Wellsburg is best
(which is the similar chip that has the same driver type and driver
 caps bits), so I rearranged it.  Also the subject was aligned.
And last but not least, Cc to stable was added.


Takashi

> ---
>  sound/pci/hda/hda_intel.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
> index c38c68f..e8d1151 100644
> --- a/sound/pci/hda/hda_intel.c
> +++ b/sound/pci/hda/hda_intel.c
> @@ -2168,6 +2168,11 @@ static const struct pci_device_id azx_ids[] = {
>  	/* ICH10 */
>  	{ PCI_DEVICE(0x8086, 0x3a6e),
>  	  .driver_data = AZX_DRIVER_ICH | AZX_DCAPS_INTEL_ICH },
> +	/* Lewisburg */
> +	{ PCI_DEVICE(0x8086, 0xa1f0),
> +	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
> +	{ PCI_DEVICE(0x8086, 0xa270),
> +	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
>  	/* Generic Intel */
>  	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_ANY_ID),
>  	  .class = PCI_CLASS_MULTIMEDIA_HD_AUDIO << 8,
> -- 
> 1.9.1
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1263514

FromAlexandra Yates <alexandra.yates@linux.intel.com>
Date2015-11-05 20:50 +0100
Message-ID<qryHL-2vb-5@gated-at.bofh.it>
In reply to#1263071
Takashi,

On 11/05/2015 02:29 AM, Takashi Iwai wrote:
> On Thu, 05 Nov 2015 00:56:09 +0100,
> Alexandra Yates wrote:
>>
>> Adding Intel codename Lewisburg platform device IDs for audio.
>>
>> Signed-off-by: Alexandra Yates <alexandra.yates@linux.intel.com>
>
> Thanks.  Though, I still believe the position after Wellsburg is best
> (which is the similar chip that has the same driver type and driver
>   caps bits), so I rearranged it.  Also the subject was aligned.
> And last but not least, Cc to stable was added.
>

Thank you for your review and making the additional changes. I'll keep 
your methodology for future reference.

Is this mean that the patch was accepted upstream?

>
> Takashi
>
>> ---
>>   sound/pci/hda/hda_intel.c | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
>> index c38c68f..e8d1151 100644
>> --- a/sound/pci/hda/hda_intel.c
>> +++ b/sound/pci/hda/hda_intel.c
>> @@ -2168,6 +2168,11 @@ static const struct pci_device_id azx_ids[] = {
>>   	/* ICH10 */
>>   	{ PCI_DEVICE(0x8086, 0x3a6e),
>>   	  .driver_data = AZX_DRIVER_ICH | AZX_DCAPS_INTEL_ICH },
>> +	/* Lewisburg */
>> +	{ PCI_DEVICE(0x8086, 0xa1f0),
>> +	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
>> +	{ PCI_DEVICE(0x8086, 0xa270),
>> +	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
>>   	/* Generic Intel */
>>   	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_ANY_ID),
>>   	  .class = PCI_CLASS_MULTIMEDIA_HD_AUDIO << 8,
>> --
>> 1.9.1
>>
>>

-- 
Thank you,
<Alexandra>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1263558

FromTakashi Iwai <tiwai@suse.de>
Date2015-11-05 22:20 +0100
Message-ID<qrA6S-3va-19@gated-at.bofh.it>
In reply to#1263514
On Thu, 05 Nov 2015 20:51:52 +0100,
Alexandra Yates wrote:
> 
> Takashi,
> 
> On 11/05/2015 02:29 AM, Takashi Iwai wrote:
> > On Thu, 05 Nov 2015 00:56:09 +0100,
> > Alexandra Yates wrote:
> >>
> >> Adding Intel codename Lewisburg platform device IDs for audio.
> >>
> >> Signed-off-by: Alexandra Yates <alexandra.yates@linux.intel.com>
> >
> > Thanks.  Though, I still believe the position after Wellsburg is best
> > (which is the similar chip that has the same driver type and driver
> >   caps bits), so I rearranged it.  Also the subject was aligned.
> > And last but not least, Cc to stable was added.
> >
> 
> Thank you for your review and making the additional changes. I'll keep 
> your methodology for future reference.
> 
> Is this mean that the patch was accepted upstream?

Yes :)


Takashi

> >
> > Takashi
> >
> >> ---
> >>   sound/pci/hda/hda_intel.c | 5 +++++
> >>   1 file changed, 5 insertions(+)
> >>
> >> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
> >> index c38c68f..e8d1151 100644
> >> --- a/sound/pci/hda/hda_intel.c
> >> +++ b/sound/pci/hda/hda_intel.c
> >> @@ -2168,6 +2168,11 @@ static const struct pci_device_id azx_ids[] = {
> >>   	/* ICH10 */
> >>   	{ PCI_DEVICE(0x8086, 0x3a6e),
> >>   	  .driver_data = AZX_DRIVER_ICH | AZX_DCAPS_INTEL_ICH },
> >> +	/* Lewisburg */
> >> +	{ PCI_DEVICE(0x8086, 0xa1f0),
> >> +	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
> >> +	{ PCI_DEVICE(0x8086, 0xa270),
> >> +	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH },
> >>   	/* Generic Intel */
> >>   	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_ANY_ID),
> >>   	  .class = PCI_CLASS_MULTIMEDIA_HD_AUDIO << 8,
> >> --
> >> 1.9.1
> >>
> >>
> 
> -- 
> Thank you,
> <Alexandra>
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web