|
Takashi Iwai |
a3a0d1 |
From cd0d465bb697a9c7bf66a9fe940f7981232f1676 Mon Sep 17 00:00:00 2001
|
|
Takashi Iwai |
a3a0d1 |
From: YueHaibing <yuehaibing@huawei.com>
|
|
Takashi Iwai |
a3a0d1 |
Date: Mon, 19 Nov 2018 20:48:19 +0800
|
|
Takashi Iwai |
a3a0d1 |
Subject: [PATCH] e100: Fix passing zero to 'PTR_ERR' warning in e100_load_ucode_wait
|
|
Takashi Iwai |
a3a0d1 |
Git-commit: cd0d465bb697a9c7bf66a9fe940f7981232f1676
|
|
Takashi Iwai |
a3a0d1 |
Patch-mainline: v5.0-rc1
|
|
Takashi Iwai |
a3a0d1 |
References: bsc#1051510
|
|
Takashi Iwai |
a3a0d1 |
|
|
Takashi Iwai |
a3a0d1 |
Fix a static code checker warning:
|
|
Takashi Iwai |
a3a0d1 |
drivers/net/ethernet/intel/e100.c:1349
|
|
Takashi Iwai |
a3a0d1 |
e100_load_ucode_wait() warn: passing zero to 'PTR_ERR'
|
|
Takashi Iwai |
a3a0d1 |
|
|
Takashi Iwai |
a3a0d1 |
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
|
|
Takashi Iwai |
a3a0d1 |
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
|
|
Takashi Iwai |
a3a0d1 |
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
|
|
Takashi Iwai |
a3a0d1 |
Acked-by: Takashi Iwai <tiwai@suse.de>
|
|
Takashi Iwai |
a3a0d1 |
|
|
Takashi Iwai |
a3a0d1 |
---
|
|
Takashi Iwai |
a3a0d1 |
drivers/net/ethernet/intel/e100.c | 4 ++--
|
|
Takashi Iwai |
a3a0d1 |
1 file changed, 2 insertions(+), 2 deletions(-)
|
|
Takashi Iwai |
a3a0d1 |
|
|
Takashi Iwai |
a3a0d1 |
diff --git a/drivers/net/ethernet/intel/e100.c b/drivers/net/ethernet/intel/e100.c
|
|
Takashi Iwai |
a3a0d1 |
index 5e5c57db0d3f..0fd268070fb4 100644
|
|
Takashi Iwai |
a3a0d1 |
--- a/drivers/net/ethernet/intel/e100.c
|
|
Takashi Iwai |
a3a0d1 |
+++ b/drivers/net/ethernet/intel/e100.c
|
|
Takashi Iwai |
a3a0d1 |
@@ -1345,8 +1345,8 @@ static inline int e100_load_ucode_wait(struct nic *nic)
|
|
Takashi Iwai |
a3a0d1 |
|
|
Takashi Iwai |
a3a0d1 |
fw = e100_request_firmware(nic);
|
|
Takashi Iwai |
a3a0d1 |
/* If it's NULL, then no ucode is required */
|
|
Takashi Iwai |
a3a0d1 |
- if (!fw || IS_ERR(fw))
|
|
Takashi Iwai |
a3a0d1 |
- return PTR_ERR(fw);
|
|
Takashi Iwai |
a3a0d1 |
+ if (IS_ERR_OR_NULL(fw))
|
|
Takashi Iwai |
a3a0d1 |
+ return PTR_ERR_OR_ZERO(fw);
|
|
Takashi Iwai |
a3a0d1 |
|
|
Takashi Iwai |
a3a0d1 |
if ((err = e100_exec_cb(nic, (void *)fw, e100_setup_ucode)))
|
|
Takashi Iwai |
a3a0d1 |
netif_err(nic, probe, nic->netdev,
|
|
Takashi Iwai |
a3a0d1 |
--
|
|
Takashi Iwai |
a3a0d1 |
2.16.4
|
|
Takashi Iwai |
a3a0d1 |
|