On 17/01/2019 14:54, Kalle Valo wrote: > Colin King <colin.king@xxxxxxxxxxxxx> writes: > >> From: Colin Ian King <colin.king@xxxxxxxxxxxxx> >> >> There are two lines that have indentation issues, fix these. >> >> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> >> --- >> drivers/net/wireless/st/cw1200/fwio.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/wireless/st/cw1200/fwio.c b/drivers/net/wireless/st/cw1200/fwio.c >> index 30e7646d04af..b7881232499c 100644 >> --- a/drivers/net/wireless/st/cw1200/fwio.c >> +++ b/drivers/net/wireless/st/cw1200/fwio.c >> @@ -465,8 +465,8 @@ int cw1200_load_firmware(struct cw1200_common *priv) >> >> if (!(val32 & ST90TDS_CONFIG_ACCESS_MODE_BIT)) { >> pr_err("Device is already in QUEUE mode!\n"); >> - ret = -EINVAL; >> - goto out; >> + ret = -EINVAL; >> + goto out; > > Just out of curiosity, how do you find these? > smatch, grep for "warn: inconsistent indenting" Colin