Commit 33cd9343 authored by Dan Carpenter's avatar Dan Carpenter Committed by Herbert Xu
Browse files

crypto: qat/qat_6xxx - Fix NULL vs IS_ERR() check in adf_probe() 



The pcim_iomap_region() returns error pointers.  It doesn't return NULL
pointers.  Update the check to match.

Fixes: 17fd7514 ("crypto: qat - add qat_6xxx driver")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Acked-by: default avatarGiovanni Cabiddu <giovanni.cabiddu@intel.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 57999ed1

drivers/crypto/intel/qat/qat_6xxx/adf_drv.c

+2 −2
Original line number Diff line number Diff line
@@ -156,8 +156,8 @@ static int adf_probe(struct pci_dev *pdev, const struct pci_device_id *ent) 


		/* Map 64-bit PCIe BAR */

		bar->virt_addr = pcim_iomap_region(pdev, bar_map[i], pci_name(pdev));

		if (!bar->virt_addr) {

			ret = -ENOMEM;

		if (IS_ERR(bar->virt_addr)) {

			ret = PTR_ERR(bar->virt_addr);

			return dev_err_probe(dev, ret, "Failed to ioremap PCI region.\n");

		}

	}