[PATCH] ASoC: Intel: atom: Remove redundant check to simplify the code
Tang Bin
tangbin at cmss.chinamobile.com
Thu Nov 25 08:50:28 CET 2021
In the function sst_platform_get_resources(), if platform_get_irq()
failed, the return should not be zero, as the example in
platform.c is
* int irq = platform_get_irq(pdev, 0)
* if (irq < 0)
* return irq;
So remove the redundant check to simplify the code.
Signed-off-by: Tang Bin <tangbin at cmss.chinamobile.com>
---
sound/soc/intel/atom/sst/sst_acpi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/intel/atom/sst/sst_acpi.c b/sound/soc/intel/atom/sst/sst_acpi.c
index 3be64430c..696d547c5 100644
--- a/sound/soc/intel/atom/sst/sst_acpi.c
+++ b/sound/soc/intel/atom/sst/sst_acpi.c
@@ -226,8 +226,8 @@ static int sst_platform_get_resources(struct intel_sst_drv *ctx)
/* Find the IRQ */
ctx->irq_num = platform_get_irq(pdev,
ctx->pdata->res_info->acpi_ipc_irq_index);
- if (ctx->irq_num <= 0)
- return ctx->irq_num < 0 ? ctx->irq_num : -EIO;
+ if (ctx->irq_num < 0)
+ return ctx->irq_num;
return 0;
}
--
2.20.1.windows.1
More information about the Alsa-devel
mailing list