diff options
| author | Zhu Wang <[email protected]> | 2023-08-03 17:27:01 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2023-08-04 16:19:55 +0200 |
| commit | ac19020be0e2e6ff62d18439d7f81251bdd1bb5a (patch) | |
| tree | e110588c35a9d29954b31da153db3eeefd12ec50 | |
| parent | 2f59ee3f8172d89693bae7c007870c4172387345 (diff) | |
staging: fieldbus: arcx-anybus: Do not check 0 for platform_get_irq()
Since platform_get_irq() never returned zero, so it need not to check
whether it returned zero, and we use the return error code of
platform_get_irq() to replace the current return error code, for that
platform_get_irq() may return -EINVAL or -ENXIO.
Signed-off-by: Zhu Wang <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Greg Kroah-Hartman <[email protected]>
| -rw-r--r-- | drivers/staging/fieldbus/anybuss/arcx-anybus.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/fieldbus/anybuss/arcx-anybus.c b/drivers/staging/fieldbus/anybuss/arcx-anybus.c index f135b9f52c8d..be28165b4f05 100644 --- a/drivers/staging/fieldbus/anybuss/arcx-anybus.c +++ b/drivers/staging/fieldbus/anybuss/arcx-anybus.c @@ -156,8 +156,8 @@ create_anybus_host(struct platform_device *pdev, int idx) if (IS_ERR(ops.regmap)) return ERR_CAST(ops.regmap); ops.irq = platform_get_irq(pdev, idx); - if (ops.irq <= 0) - return ERR_PTR(-EINVAL); + if (ops.irq < 0) + return ERR_PTR(ops.irq); return devm_anybuss_host_common_probe(&pdev->dev, &ops); } |