diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2025-10-29 16:13:56 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-11-26 15:09:34 +0100 |
| commit | 05d36a5931d92310819e64fdee0cc1a35e14944c (patch) | |
| tree | 24b441ba1cd179a4c985281d6ee03b6f72bf1bb1 /drivers/misc | |
| parent | 4863cb2b0f505c77f7b4632246d4de4859b86ed2 (diff) | |
| download | tip-05d36a5931d92310819e64fdee0cc1a35e14944c.tar.gz | |
misc: cb710: Fix a NULL vs IS_ERR() check in probe()
The pcim_iomap_region() function never returns NULL, it returns error
pointers. Update the checking to match.
Fixes: b91c13534a63 ("misc: cb710: Replace deprecated PCI functions")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Link: https://patch.msgid.link/aQITFDPyuzjNN4GN@stanley.mountain
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/misc')
| -rw-r--r-- | drivers/misc/cb710/core.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/misc/cb710/core.c b/drivers/misc/cb710/core.c index a1e6ba62c29806..2dd212f04fed2a 100644 --- a/drivers/misc/cb710/core.c +++ b/drivers/misc/cb710/core.c @@ -226,8 +226,8 @@ static int cb710_probe(struct pci_dev *pdev, spin_lock_init(&chip->irq_lock); chip->pdev = pdev; chip->iobase = pcim_iomap_region(pdev, 0, KBUILD_MODNAME); - if (!chip->iobase) - return -ENOMEM; + if (IS_ERR(chip->iobase)) + return PTR_ERR(chip->iobase); pci_set_drvdata(pdev, chip); |