| From 321198daa8cf91acafebe0ab6ae1071deccf4351 Mon Sep 17 00:00:00 2001 |
| From: Nathan Chancellor <natechancellor@gmail.com> |
| Date: Mon, 9 Dec 2019 14:50:27 -0700 |
| Subject: [PATCH] net: smc911x: Adjust indentation in smc911x_phy_configure |
| |
| 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: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/drivers/net/ethernet/smsc/smc911x.c b/drivers/net/ethernet/smsc/smc911x.c |
| index bd14803545de..6a67485828d6 100644 |
| --- a/drivers/net/ethernet/smsc/smc911x.c |
| +++ b/drivers/net/ethernet/smsc/smc911x.c |
| @@ -935,7 +935,7 @@ static void smc911x_phy_configure(struct work_struct *work) |
| 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 */ |
| -- |
| 2.7.4 |
| |