[PATCH AUTOSEL 6.18-6.1] ASoC: amd: yc: Add ASUS ExpertBook PM1503CDA to quirks list

[Sasha Levin <sashal@kernel.org>]

From: Anatolii Shirykalov <pipocavsobake@gmail.com>

[ Upstream commit 018b211b1d321a52ed8d8de74ce83ce52a2e1224 ]

Add ASUS ExpertBook PM1503CDA to the DMI quirks table to enable
internal DMIC support via the ACP6x machine driver.

Signed-off-by: Anatolii Shirykalov <pipocavsobake@gmail.com>
Link: https://patch.msgid.link/20260119145618.3171435-1-pipocavsobake@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
---

LLM Generated explanations, may be completely bogus:

## Analysis of ASoC: amd: yc: Add ASUS ExpertBook PM1503CDA to quirks
list

### 1. COMMIT MESSAGE ANALYSIS

The commit message is clear and straightforward:
- **Purpose**: Add a new device (ASUS ExpertBook PM1503CDA) to the DMI
  quirks table
- **Goal**: Enable internal DMIC (Digital Microphone) support via the
  ACP6x machine driver
- **Tags present**: Signed-off-by (author and maintainer), Link to patch
  submission
- **No Fixes: or Cc: stable tags**: As expected for commits requiring
  manual review

The commit is submitted by an external contributor and accepted by Mark
Brown (the ASoC maintainer), indicating it has been reviewed.

### 2. CODE CHANGE ANALYSIS

The diff shows:
- **Single file changed**: `sound/soc/amd/yc/acp6x-mach.c`
- **Lines added**: 7 lines (one new DMI table entry)
- **Lines removed**: 0

The change adds a new entry to the `yc_acp_quirk_table[]` array:
```c
{
    .driver_data = &acp6x_card,
    .matches = {
        DMI_MATCH(DMI_BOARD_VENDOR, "ASUSTeK COMPUTER INC."),
        DMI_MATCH(DMI_PRODUCT_NAME, "ASUS EXPERTBOOK PM1503CDA"),
    }
},
```

This is a **boilerplate DMI quirk entry** identical in structure to all
surrounding entries in the table. It simply associates this specific
ASUS laptop model with the `acp6x_card` driver data.

### 3. CLASSIFICATION

This commit falls squarely into the **"QUIRKS and WORKAROUNDS"**
exception category for stable backports:

- It adds a hardware-specific quirk for a device that needs special
  handling
- The driver already exists in stable trees (ACP6x AMD audio driver)
- Without this entry, the internal microphone on this laptop model does
  not work
- This is a **real hardware enablement fix** - users of this ASUS laptop
  cannot use their built-in microphone without it

### 4. SCOPE AND RISK ASSESSMENT

- **Size**: Minimal - 7 lines of trivial, templated code
- **Complexity**: None - exact copy of existing patterns in the same
  file
- **Files touched**: 1
- **Subsystem**: ASoC AMD audio driver (mature, stable)
- **Risk of regression**: **Extremely low**
  - The DMI matching only triggers on this specific laptop model
  - It cannot affect any other system
  - The `acp6x_card` driver data is already used by dozens of other
    entries

### 5. USER IMPACT

- **Who is affected**: Users of ASUS ExpertBook PM1503CDA laptops
  running Linux
- **Impact without the fix**: Internal microphone does not work
- **Impact with the fix**: Internal microphone works correctly
- **Severity**: Medium-high for affected users (audio input is a core
  feature)

This is a real-world hardware enablement issue. Users of this specific
laptop model cannot use their built-in microphone for video calls, voice
recording, etc. without this quirk entry.

### 6. STABILITY INDICATORS

- **Maintainer sign-off**: Mark Brown (ASoC subsystem maintainer)
  accepted the patch
- **Pattern**: This exact pattern has been used hundreds of times in
  this file and across the kernel
- **Testing**: Presumably tested by the submitter on their hardware
  (standard for device quirk submissions)

### 7. DEPENDENCY CHECK

- **Dependencies**: None - this is a self-contained table entry
- **Code existence in stable**: The ACP6x driver and this quirks table
  exist in all recent stable trees (6.1+, 6.6+, etc.)
- **Backport adjustments**: None needed - the code will apply cleanly as
  the table structure is identical

### SUMMARY

This commit is a textbook example of a **stable-appropriate DMI quirk
addition**:

1. ✅ **Obviously correct**: Follows exact pattern of 80+ other entries
   in the same table
2. ✅ **Fixes a real bug**: Enables non-functional hardware (internal
   microphone)
3. ✅ **Small and contained**: 7 lines, single file, no logic changes
4. ✅ **No new features**: Just enables existing driver for new hardware
5. ✅ **No new APIs**: Pure data addition
6. ✅ **Zero regression risk**: Only affects one specific laptop model
7. ✅ **Falls under documented exception**: Hardware quirks are
   explicitly allowed in stable

The benefit to users of this ASUS laptop model is significant (working
microphone), while the risk to all other systems is essentially zero
(the DMI match ensures no other system is affected).

**YES**

 sound/soc/amd/yc/acp6x-mach.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/sound/soc/amd/yc/acp6x-mach.c b/sound/soc/amd/yc/acp6x-mach.c
index bf4d9d3365617..fe6cc3c74991f 100644
--- a/sound/soc/amd/yc/acp6x-mach.c
+++ b/sound/soc/amd/yc/acp6x-mach.c
@@ -535,6 +535,13 @@ static const struct dmi_system_id yc_acp_quirk_table[] = {
 			DMI_MATCH(DMI_PRODUCT_NAME, "15NBC1011"),
 		}
 	},
+	{
+		.driver_data = &acp6x_card,
+		.matches = {
+			DMI_MATCH(DMI_BOARD_VENDOR, "ASUSTeK COMPUTER INC."),
+			DMI_MATCH(DMI_PRODUCT_NAME, "ASUS EXPERTBOOK PM1503CDA"),
+		}
+	},
 	{
 		.driver_data = &acp6x_card,
 		.matches = {
-- 
2.51.0