Commit ddb20bcf authored by Luiz Sampaio's avatar Luiz Sampaio Committed by Greg Kroah-Hartman

w1: ds2438: fixed a coding style issue

There is an if statement and, if the function goes into it, it returns. So,
the next else is not required.
Signed-off-by: default avatarLuiz Sampaio <sampaio.ime@gmail.com>
Link: https://lore.kernel.org/r/20210519223046.13798-2-sampaio.ime@gmail.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent c6fa1a96
...@@ -154,11 +154,11 @@ static int w1_ds2438_change_config_bit(struct w1_slave *sl, u8 mask, u8 value) ...@@ -154,11 +154,11 @@ static int w1_ds2438_change_config_bit(struct w1_slave *sl, u8 mask, u8 value)
if ((status & mask) == value) if ((status & mask) == value)
return 0; /* already set as requested */ return 0; /* already set as requested */
else {
/* changing bit */ /* changing bit */
status ^= mask; status ^= mask;
perform_write = 1; perform_write = 1;
}
break; break;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment