diff options
| author |   Andrey Utkin <andrey.krieger.utkin@gmail.com> | 2014-07-17 16:53:54 +0300 |
|---|---|---|
| committer |   Sebastian Reichel <sre@kernel.org> | 2014-07-17 22:45:28 +0200 |
| commit | b74d4954ae6a6799f7ee832bc377795ab506f4b1 (patch) | |
| tree | e2246eadf907336db00ef020c6df0c1509bbd872 /drivers/hsi/controllers/omap_ssi.c | |
| parent | hsi: omap_ssi_port: use normal module refcounting (diff) | |
| download | linux-dev-b74d4954ae6a6799f7ee832bc377795ab506f4b1.tar.xz linux-dev-b74d4954ae6a6799f7ee832bc377795ab506f4b1.zip | |
drivers/hsi/controllers/omap_ssi{,_port}.c: fix failure checks
1.
[linux-3.16-rc5/drivers/hsi/controllers/omap_ssi.c:357]: (style) Checking if
unsigned variable 'gdd_irq' is less than zero.
Source code is
omap_ssi->gdd_irq = platform_get_irq_byname(pd, "gdd_mpu");
if (omap_ssi->gdd_irq < 0) {
2.
[linux-3.16-rc5/drivers/hsi/controllers/omap_ssi_port.c:1017]: (style) Checking
if unsigned variable 'irq' is less than zero.
Source code is
omap_port->irq = platform_get_irq(pd, 0);
if (omap_port->irq < 0) {
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=80441
Reported-by: David Binderman <dcb314@hotmail.com>
Signed-off-by: Andrey Utkin <andrey.krieger.utkin@gmail.com>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
Diffstat (limited to 'drivers/hsi/controllers/omap_ssi.c')
| -rw-r--r-- | drivers/hsi/controllers/omap_ssi.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/hsi/controllers/omap_ssi.c b/drivers/hsi/controllers/omap_ssi.c index 0fc7a7fd0140..232ab7340b76 100644 --- a/drivers/hsi/controllers/omap_ssi.c +++ b/drivers/hsi/controllers/omap_ssi.c @@ -353,12 +353,12 @@ static int __init ssi_add_controller(struct hsi_controller *ssi, err = ssi_get_iomem(pd, "gdd", &omap_ssi->gdd, NULL); if (err < 0) goto out_err; - omap_ssi->gdd_irq = platform_get_irq_byname(pd, "gdd_mpu"); - if (omap_ssi->gdd_irq < 0) { + err = platform_get_irq_byname(pd, "gdd_mpu"); + if (err < 0) { dev_err(&pd->dev, "GDD IRQ resource missing\n"); - err = omap_ssi->gdd_irq; goto out_err; } + omap_ssi->gdd_irq = err; tasklet_init(&omap_ssi->gdd_tasklet, ssi_gdd_tasklet, (unsigned long)ssi); err = devm_request_irq(&ssi->device, omap_ssi->gdd_irq, ssi_gdd_isr, |