Skip to contentSkip to conversation

Add ID and patches for Intel BXT

PATCH Applied [email protected] 2 participants 4 messages Nov 19, 2015 Sound

lore
Share & tools
lore

[email protected]

author
#0 lore raw

cover Add ID and patches for Intel BXT

From: "Lu, Han" <[email protected]>

Add PCI ID for Intel BXT(Broxton) platform, and add necessary patches for the new platform.

Lu, Han (2):
ALSA: hda - add PCI IDs for Intel Broxton
ALSA: hda - apply SKL display power request/release patch to BXT

 sound/pci/hda/hda_intel.c  | 7 +++++++
 sound/pci/hda/patch_hdmi.c | 3 ++-
 2 files changed, 9 insertions(+), 1 deletion(-)
Signature · 1 line
2.5.0
author
#1 lore raw

PATCH 1/2 ALSA: hda - add PCI IDs for Intel Broxton

From: "Lu, Han" <[email protected]>

Add HD Audio Device PCI ID for the Intel Broxton platform. It is an HDA Intel PCH controller.

Signed-off-by Lu, Han <[email protected]>
sound/pci/hda/hda_intel.c +7 -0
index 8a7fbdc..963f824 100644
@@ -312,6 +312,10 @@ enum {
 	(AZX_DCAPS_INTEL_PCH | AZX_DCAPS_SEPARATE_STREAM_TAG |\
 	 AZX_DCAPS_I915_POWERWELL)
 
+#define AZX_DCAPS_INTEL_BROXTON \
+	(AZX_DCAPS_INTEL_PCH | AZX_DCAPS_SEPARATE_STREAM_TAG |\
+	 AZX_DCAPS_I915_POWERWELL)
+
 /* quirks for ATI SB / AMD Hudson */
 #define AZX_DCAPS_PRESET_ATI_SB \
 	(AZX_DCAPS_NO_TCSEL | AZX_DCAPS_SYNC_WRITE | AZX_DCAPS_POSFIX_LPIB |\

@@ -2124,6 +2128,9 @@ static const struct pci_device_id azx_ids[] = {
 	/* Sunrise Point-LP */
 	{ PCI_DEVICE(0x8086, 0x9d70),
 	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_SKYLAKE },
+	/* Broxton-P(Apollolake) */
+	{ PCI_DEVICE(0x8086, 0x5a98),
+	  .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_BROXTON },
 	/* Haswell */
 	{ PCI_DEVICE(0x8086, 0x0a0c),
 	  .driver_data = AZX_DRIVER_HDMI | AZX_DCAPS_INTEL_HASWELL },
Signature · 1 line
2.5.0
author
#2 lore raw

PATCH 2/2 ALSA: hda - apply SKL display power request/release patch to BXT

In reply to [email protected]

From: "Lu, Han" <[email protected]>

For SKL, only the HDMI codec is in the display power well while the HD-A controller isn't. So the codec flag 'link_power_control' is set to request/release the display power via bus link_power ops. For BXT, the power well design is the same as SKL, so the patch should be applied to BXT too.

Signed-off-by Lu, Han <[email protected]>
sound/pci/hda/patch_hdmi.c +2 -1
index 309274b..aa68db6 100644
@@ -2375,7 +2375,8 @@ static int patch_generic_hdmi(struct hda_codec *codec)
 	 * can cover the codec power request, and so need not set this flag.
 	 * For previous platforms, there is no such power well feature.
 	 */
-	if (is_valleyview_plus(codec) || is_skylake(codec))
+	if (is_valleyview_plus(codec) || is_skylake(codec) ||
+			is_broxton(codec))
 		codec->core.link_power_control = 1;
 
 	if (is_haswell_plus(codec) || is_valleyview_plus(codec)) {
Signature · 1 line
2.5.0
Applied
#3 lore raw
In reply to [email protected]
Quoted 9 lines · On Thu, 19 Nov 2015 16:25:11 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Add PCI ID for Intel BXT(Broxton) platform, and add necessary patches for the new platform.

Lu, Han (2):
ALSA: hda - add PCI IDs for Intel Broxton
ALSA: hda - apply SKL display power request/release patch to BXT

Applied both patches. Thanks.

Takashi

Quoted 13 lines
 sound/pci/hda/hda_intel.c  | 7 +++++++
 sound/pci/hda/patch_hdmi.c | 3 ++-
 2 files changed, 9 insertions(+), 1 deletion(-)
Signature · 6 lines
2.5.0

_______________________________________________
Alsa-devel mailing list
[email protected]
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel