diff options
author | Peng Li <lipeng321@huawei.com> | 2021-06-15 15:54:20 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2021-06-15 20:03:17 +0200 |
commit | b9282333efff96c19bc58a36f4929471de02b4f3 (patch) | |
tree | 990aed0a8b3c18bf0eba0b6aef3efbff80fd7c14 /drivers/net/wan/pci200syn.c | |
parent | net: pci200syn: add blank line after declarations (diff) | |
download | linux-b9282333efff96c19bc58a36f4929471de02b4f3.tar.xz linux-b9282333efff96c19bc58a36f4929471de02b4f3.zip |
net: pci200syn: replace comparison to NULL with "!card"
According to the chackpatch.pl, comparison to NULL could
be written "!card".
Signed-off-by: Peng Li <lipeng321@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wan/pci200syn.c')
-rw-r--r-- | drivers/net/wan/pci200syn.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/wan/pci200syn.c b/drivers/net/wan/pci200syn.c index a7eac9099de9..cee3d658a3e8 100644 --- a/drivers/net/wan/pci200syn.c +++ b/drivers/net/wan/pci200syn.c @@ -279,7 +279,7 @@ static int pci200_pci_init_one(struct pci_dev *pdev, } card = kzalloc(sizeof(card_t), GFP_KERNEL); - if (card == NULL) { + if (!card) { pci_release_regions(pdev); pci_disable_device(pdev); return -ENOBUFS; @@ -310,9 +310,7 @@ static int pci200_pci_init_one(struct pci_dev *pdev, ramphys = pci_resource_start(pdev,3) & PCI_BASE_ADDRESS_MEM_MASK; card->rambase = pci_ioremap_bar(pdev, 3); - if (card->plxbase == NULL || - card->scabase == NULL || - card->rambase == NULL) { + if (!card->plxbase || !card->scabase || !card->rambase) { pr_err("ioremap() failed\n"); pci200_pci_remove_one(pdev); return -EFAULT; |