[alsa-devel] [PATCH] ALSA: hda controller - handle -1 value as error in snd_hdac_bus_parse_capabilities()

Jaroslav Kysela perex at perex.cz
Tue Oct 24 18:56:47 CEST 2017


We have seen this value on a development platform.

Signed-off-by: Jaroslav Kysela <perex at perex.cz>
---
 sound/hda/hdac_controller.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/sound/hda/hdac_controller.c b/sound/hda/hdac_controller.c
index 978dc1801b3a..4a1823290b2d 100644
--- a/sound/hda/hdac_controller.c
+++ b/sound/hda/hdac_controller.c
@@ -278,6 +278,11 @@ int snd_hdac_bus_parse_capabilities(struct hdac_bus *bus)
 	do {
 		cur_cap = _snd_hdac_chip_readl(bus, offset);
 
+		if (cur_cap == 0xffffffff) {
+			dev_dbg(bus->dev, "Wrong capability -1\n");
+			break;
+		}
+
 		dev_dbg(bus->dev, "Capability version: 0x%x\n",
 			(cur_cap & AZX_CAP_HDR_VER_MASK) >> AZX_CAP_HDR_VER_OFF);
 
-- 
2.13.6


More information about the Alsa-devel mailing list