[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20220424092842.101307-1-wanjiabing@vivo.com>
Date: Sun, 24 Apr 2022 17:28:42 +0800
From: Wan Jiabing <wanjiabing@...o.com>
To: "Maciej W. Rozycki" <macro@...am.me.uk>,
"David S. Miller" <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org
Cc: kael_w@...h.net, Wan Jiabing <wanjiabing@...o.com>
Subject: [PATCH] FDDI: defxx: simplify if-if to if-else
Use if and else instead of if(A) and if (!A).
Signed-off-by: Wan Jiabing <wanjiabing@...o.com>
---
drivers/net/fddi/defxx.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/fddi/defxx.c b/drivers/net/fddi/defxx.c
index b584ffe38ad6..3edb2e96f763 100644
--- a/drivers/net/fddi/defxx.c
+++ b/drivers/net/fddi/defxx.c
@@ -585,10 +585,10 @@ static int dfx_register(struct device *bdev)
bp->mmio = false;
dfx_get_bars(bp, bar_start, bar_len);
}
- }
- if (!dfx_use_mmio)
+ } else {
region = request_region(bar_start[0], bar_len[0],
bdev->driver->name);
+ }
if (!region) {
dfx_register_res_err(print_name, dfx_use_mmio,
bar_start[0], bar_len[0]);
--
2.35.1
Powered by blists - more mailing lists