[PATCH 2/3] staging: pi433: - style fix, space before tabs
Dan Carpenter
dan.carpenter at oracle.com
Mon Jul 24 07:48:58 UTC 2017
On Sat, Jul 22, 2017 at 10:48:24AM +0200, Marcus Wolf wrote:
> Reviewed-by: Marcus Wolf <linux at wolf-entwicklungen.de>
>
> >From my point of view, the rearrangement of the block of SET_CHECKED reduces the
> readability a lot. I like same stuff to be aligned (all brakets below each other
> as a column, all spi->dev below each other and so on) But if it is necessary to
> fullfill the rules, we have to do it the new way.
It's not necessary. The rule is just complaining about this:
SET_CHECKED(rf69_set_bandwidth (dev->spi, rx_cfg->bw_mantisse, rx_cfg->bw_exponent));
^
There is a space after the 'h' character. The old code has
"[space][tab][tab][space](dev->spi,..." it should be:
"[tab][tab][space](dev->spi, ..."
Also the rules are there to improve readability. Sometimes they're
debatable but if they make readability worse, then we can ignore the
rules.
regards,
dan carpenter
More information about the devel
mailing list