From: Colin Ian King Date: Thu, 17 Jan 2019 00:35:43 +0000 (+0000) Subject: sb1000: fix a couple of indentation issues and remove assignment in if statements X-Git-Tag: v5.1-rc1~178^2~498 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=6394d98df6b47a4fe28715536bd6c8e003847071;p=thirdparty%2Fkernel%2Flinux.git sb1000: fix a couple of indentation issues and remove assignment in if statements There is an if statement and a return statement that are incorrectly indented. Fix these. Also replace the assignment-in-if statements to assignment followed by an if to keep to the coding style. Signed-off-by: Colin Ian King Signed-off-by: David S. Miller --- diff --git a/drivers/net/sb1000.c b/drivers/net/sb1000.c index 7820fced33f62..941cfa8f1c2a5 100644 --- a/drivers/net/sb1000.c +++ b/drivers/net/sb1000.c @@ -535,17 +535,20 @@ sb1000_activate(const int ioaddr[], const char* name) int status; ssleep(1); - if ((status = card_send_command(ioaddr, name, Command0, st))) + status = card_send_command(ioaddr, name, Command0, st); + if (status) return status; - if ((status = card_send_command(ioaddr, name, Command1, st))) + status = card_send_command(ioaddr, name, Command1, st); + if (status) return status; if (st[3] != 0xf1) { - if ((status = sb1000_start_get_set_command(ioaddr, name))) + status = sb1000_start_get_set_command(ioaddr, name); + if (status) return status; return -EIO; } udelay(1000); - return sb1000_start_get_set_command(ioaddr, name); + return sb1000_start_get_set_command(ioaddr, name); } /* get SB1000 firmware version */