Hello.
On 1/1/2016 11:31 PM, SF Markus Elfring wrote:
From: Markus Elfring <[email protected]>
Date: Fri, 1 Jan 2016 21:12:29 +0100
This issue was detected by using the Coccinelle software.
Adjust a jump target to avoid a check repetition before the function
call "il_eeprom_free".
One question: why?
Signed-off-by: Markus Elfring <[email protected]>
---
drivers/net/wireless/intel/iwlegacy/common.c | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/net/wireless/intel/iwlegacy/common.c
b/drivers/net/wireless/intel/iwlegacy/common.c
index c3afaf7..ae45fd3 100644
--- a/drivers/net/wireless/intel/iwlegacy/common.c
+++ b/drivers/net/wireless/intel/iwlegacy/common.c
[...]
@@ -772,9 +772,11 @@ il_eeprom_init(struct il_priv *il)
done:
il->ops->eeprom_release_semaphore(il);
-err:
- if (ret)
+ if (ret) {
+free_eeprom:
This is ugly, I'd say.
[...]
MBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
