diff options
author | Charles Keepax <ckeepax@opensource.cirrus.com> | 2025-03-21 13:53:24 +0000 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2025-03-21 15:45:25 +0000 |
commit | 9ef52d529bb75071e03cf85078f724d69c4abe89 (patch) | |
tree | 27361b3a444b13341b2cf47ac189fc0ee91f3ca2 /sound | |
parent | 5a4dd520ef8a94ecf81ac77b90d6a03e91c100a9 (diff) |
ASoC: SDCA: Correct handling of selected mode DisCo property
mipi-sdca-ge-selectedmode-controls-affected is actually required by the
specification so the code should return an error if it is missing.
Reported-by: Maciej Strozek <mstrozek@opensource.cirrus.com>
Fixes: 13fe7497af19 ("ASoC: SDCA: Add support for GE Entity properties")
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://patch.msgid.link/20250321135324.380237-1-ckeepax@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/soc/sdca/sdca_functions.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/sdca/sdca_functions.c b/sound/soc/sdca/sdca_functions.c index c8efdc5301b5..493f390f087a 100644 --- a/sound/soc/sdca/sdca_functions.c +++ b/sound/soc/sdca/sdca_functions.c @@ -1159,7 +1159,7 @@ static int find_sdca_entity_ge(struct device *dev, num_affected = fwnode_property_count_u8(entity_node, "mipi-sdca-ge-selectedmode-controls-affected"); - if (!num_affected || num_affected == -EINVAL) { + if (!num_affected) { return 0; } else if (num_affected < 0) { dev_err(dev, "%s: failed to read affected controls: %d\n", |