sdw_master_read_intel_prop() declares quirk_mask without an initialiser
and ignores the return value of fwnode_property_read_u32(). When a BIOS
does not describe "intel-quirk-mask" for a link, the property read leaves
the variable untouched and stack garbage decides whether
SDW_INTEL_QUIRK_MASK_BUS_DISABLE is honoured, so the link may be marked
hw_disabled at random.

Initialise it to 0, matching is_link_enabled() in
sound/hda/core/intel-sdw-acpi.c.
Found by code inspection, not observed at run time.

Fixes: 395713d8ca43 ("soundwire: intel: handle disabled links")
Cc: stable@vger.kernel.org
Signed-off-by: Boris Faure <boris@fau.re>
---
 drivers/soundwire/intel_auxdevice.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/soundwire/intel_auxdevice.c b/drivers/soundwire/intel_auxdevice.c
index 0b8107bec9ab..26bdca95c7a0 100644
--- a/drivers/soundwire/intel_auxdevice.c
+++ b/drivers/soundwire/intel_auxdevice.c
@@ -158,7 +158,7 @@ static int sdw_master_read_intel_prop(struct sdw_bus *bus)
 	struct sdw_intel_prop *intel_prop;
 	struct fwnode_handle *link;
 	char name[32];
-	u32 quirk_mask;
+	u32 quirk_mask = 0;
 
 	/* Find master handle */
 	snprintf(name, sizeof(name),
-- 
2.55.0

On Thu, 20 Aug 2026 21:22:57 +0000, Boris Faure wrote:
> sdw_master_read_intel_prop() declares quirk_mask without an initialiser
> and ignores the return value of fwnode_property_read_u32(). When a BIOS
> does not describe "intel-quirk-mask" for a link, the property read leaves
> the variable untouched and stack garbage decides whether
> SDW_INTEL_QUIRK_MASK_BUS_DISABLE is honoured, so the link may be marked
> hw_disabled at random.
> 
> [...]

Applied, thanks!

[1/1] soundwire: intel: initialise quirk_mask before reading it
      commit: ee8a8cfe4e8d3c67456e554c988017c252710a25

Best regards,
-- 
~Vinod