ASoC: cs-amp-lib: Simplify overcomplicated error return

In cs_amp_devm_get_dell_ssidex() remove an unnecessary special case check
on -ENOENT that just returned -ENOENT. The other branch of the if()
statement returned the error, which would of course return -ENOENT if the
error was -ENOENT and so do exactly the same as the first branch.

The whole if statement is identical to just returning the original pointer
if it is an error value.

Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Link: https://patch.msgid.link/20260528144632.130197-1-rf@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
Richard Fitzgerald 2026-05-28 15:46:32 +01:00 committed by Mark Brown
parent ec4489c43e
commit 3b28b14b79
No known key found for this signature in database
GPG Key ID: 24D68B725D5487D0

View File

@ -748,10 +748,7 @@ static const char *cs_amp_devm_get_dell_ssidex(struct device *dev,
char *ssidex_buf __free(kfree) = cs_amp_alloc_get_efi_variable(DELL_SSIDEXV2_EFI_NAME,
&DELL_SSIDEXV2_EFI_GUID,
NULL);
ret = PTR_ERR_OR_ZERO(ssidex_buf);
if (ret == -ENOENT)
return ERR_PTR(-ENOENT);
else if (ret < 0)
if (IS_ERR(ssidex_buf))
return ssidex_buf;
/*