diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2025-11-25 16:55:28 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2025-11-25 15:17:46 +0000 |
| commit | 80339b3823bb76d383c82186b55ac836fed3f586 (patch) | |
| tree | b719d7241ad33961800f7327d6c54baf09e185ce | |
| parent | a13744f628c61a450f650442a118405242ac0db0 (diff) | |
ASoC: SDCA: Fix NULL vs IS_ERR() bug in sdca_dev_register_functions()
The sdca_dev_register() function never returns NULL, it returns error
pointers on error. Fix the error checking to match.
Fixes: 4496d1c65bad ("ASoC: SDCA: add function devices")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://patch.msgid.link/aSW1UOgMCiQIaZG8@stanley.mountain
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | sound/soc/sdca/sdca_function_device.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sound/soc/sdca/sdca_function_device.c b/sound/soc/sdca/sdca_function_device.c index 91c49d7389db..c6cc880a150e 100644 --- a/sound/soc/sdca/sdca_function_device.c +++ b/sound/soc/sdca/sdca_function_device.c @@ -96,8 +96,8 @@ int sdca_dev_register_functions(struct sdw_slave *slave) func_dev = sdca_dev_register(&slave->dev, &sdca_data->function[i]); - if (!func_dev) - return -ENODEV; + if (IS_ERR(func_dev)) + return PTR_ERR(func_dev); sdca_data->function[i].func_dev = func_dev; } |