Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Lucas De Marchi <lucas.demarchi@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: Jani Nikula <jani.nikula@linux.intel.com>,
	Rodrigo Vivi <rodrigo.vivi@intel.com>,
	Lucas De Marchi <lucas.demarchi@intel.com>
Subject: [PATCH topic/core-for-CI 2/2] Revert "HAX sound: Disable probing snd_hda with DG1"
Date: Wed, 23 Oct 2024 14:18:34 -0700	[thread overview]
Message-ID: <20241023211834.3370346-2-lucas.demarchi@intel.com> (raw)
In-Reply-To: <20241023211834.3370346-1-lucas.demarchi@intel.com>

This reverts commit 852510ccd334063bec2c2c3e5a61a071599ac015.

This should be solved already since commit e6d0c13e9f46 ("ALSA: hda:
i915: Remove extra argument from snd_hdac_i915_init").

We can try to drop it and see if anything explodes on dg1 land.

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
 sound/hda/hdac_i915.c     | 23 -----------------------
 sound/pci/hda/hda_intel.c |  3 +++
 2 files changed, 3 insertions(+), 23 deletions(-)

diff --git a/sound/hda/hdac_i915.c b/sound/hda/hdac_i915.c
index a214636ed9757..e9425213320ea 100644
--- a/sound/hda/hdac_i915.c
+++ b/sound/hda/hdac_i915.c
@@ -170,26 +170,6 @@ static int i915_gfx_present(struct pci_dev *hdac_pci)
 	return false;
 }
 
-static bool dg1_gfx_present(void)
-{
-	static const struct pci_device_id ids[] = {
-		{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x4905),
-		  .class = PCI_BASE_CLASS_DISPLAY << 16,
-		  .class_mask = 0xff << 16 },
-		{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x4906),
-		  .class = PCI_BASE_CLASS_DISPLAY << 16,
-		  .class_mask = 0xff << 16 },
-		{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x4907),
-		  .class = PCI_BASE_CLASS_DISPLAY << 16,
-		  .class_mask = 0xff << 16 },
-		{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x4908),
-		  .class = PCI_BASE_CLASS_DISPLAY << 16,
-		  .class_mask = 0xff << 16 },
-		{}
-	};
-	return pci_dev_present(ids);
-}
-
 /**
  * snd_hdac_i915_init - Initialize i915 audio component
  * @bus: HDA core bus
@@ -210,9 +190,6 @@ int snd_hdac_i915_init(struct hdac_bus *bus)
 	if (!i915_gfx_present(to_pci_dev(bus->dev)))
 		return -ENODEV;
 
-	if (dg1_gfx_present())
-		return -ENODEV;
-
 	err = snd_hdac_acomp_init(bus, NULL,
 				  i915_component_master_match,
 				  sizeof(struct i915_audio_component) - sizeof(*acomp));
diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index 282e87cc2166b..b4540c5cd2a6f 100644
--- a/sound/pci/hda/hda_intel.c
+++ b/sound/pci/hda/hda_intel.c
@@ -2143,6 +2143,9 @@ static int azx_probe(struct pci_dev *pci,
 			 * codecs can be on the same link.
 			 */
 			if (HDA_CONTROLLER_IN_GPU(pci)) {
+				dev_err_probe(card->dev, err,
+					     "HSW/BDW HD-audio HDMI/DP requires binding with gfx driver\n");
+
 				goto out_free;
 			} else {
 				/* don't bother any longer */
-- 
2.47.0


  reply	other threads:[~2024-10-23 21:18 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-23 21:18 [PATCH topic/core-for-CI 1/2] Revert "HAX suspend: Disable S3/S4 for fi-bdw-samus" Lucas De Marchi
2024-10-23 21:18 ` Lucas De Marchi [this message]
2024-10-24 10:23   ` [PATCH topic/core-for-CI 2/2] Revert "HAX sound: Disable probing snd_hda with DG1" Jani Nikula
2024-10-24  8:30 ` ✗ Fi.CI.CHECKPATCH: warning for series starting with [topic/core-for-CI,1/2] Revert "HAX suspend: Disable S3/S4 for fi-bdw-samus" Patchwork
2024-10-24  8:30 ` ✓ Fi.CI.BAT: success " Patchwork
2024-10-24 10:00 ` ✗ Fi.CI.IGT: failure " Patchwork
2024-10-24 10:22 ` [PATCH topic/core-for-CI 1/2] " Jani Nikula
2024-10-28 20:59 ` Lucas De Marchi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20241023211834.3370346-2-lucas.demarchi@intel.com \
    --to=lucas.demarchi@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jani.nikula@linux.intel.com \
    --cc=rodrigo.vivi@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox