| From 5c61e223004b3b5c3f1dd25718e979bc17a3b12d Mon Sep 17 00:00:00 2001 |
| From: Nathan Chancellor <natechancellor@gmail.com> |
| Date: Mon, 9 Dec 2019 14:50:27 -0700 |
| Subject: net: smc911x: Adjust indentation in smc911x_phy_configure |
| |
| From: Nathan Chancellor <natechancellor@gmail.com> |
| |
| commit 5c61e223004b3b5c3f1dd25718e979bc17a3b12d upstream. |
| |
| Clang warns: |
| |
| ../drivers/net/ethernet/smsc/smc911x.c:939:3: warning: misleading |
| indentation; statement is not part of the previous 'if' |
| [-Wmisleading-indentation] |
| if (!lp->ctl_rfduplx) |
| ^ |
| ../drivers/net/ethernet/smsc/smc911x.c:936:2: note: previous statement |
| is here |
| if (lp->ctl_rspeed != 100) |
| ^ |
| 1 warning generated. |
| |
| This warning occurs because there is a space after the tab on this line. |
| Remove it so that the indentation is consistent with the Linux kernel |
| coding style and clang no longer warns. |
| |
| Fixes: 0a0c72c9118c ("[PATCH] RE: [PATCH 1/1] net driver: Add support for SMSC LAN911x line of ethernet chips") |
| Link: https://github.com/ClangBuiltLinux/linux/issues/796 |
| Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/net/ethernet/smsc/smc911x.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/net/ethernet/smsc/smc911x.c |
| +++ b/drivers/net/ethernet/smsc/smc911x.c |
| @@ -947,7 +947,7 @@ static void smc911x_phy_configure(struct |
| if (lp->ctl_rspeed != 100) |
| my_ad_caps &= ~(ADVERTISE_100BASE4|ADVERTISE_100FULL|ADVERTISE_100HALF); |
| |
| - if (!lp->ctl_rfduplx) |
| + if (!lp->ctl_rfduplx) |
| my_ad_caps &= ~(ADVERTISE_100FULL|ADVERTISE_10FULL); |
| |
| /* Update our Auto-Neg Advertisement Register */ |