| Message ID | 20211230174911.78291-1-jic23@kernel.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | IIO: Tidy up inconsistent spacing in ID tables. | expand |
On Thu, 30 Dec 2021 17:48:55 +0000 Jonathan Cameron <jic23@kernel.org> wrote: > From: Jonathan Cameron <Jonathan.Cameron@huawei.com> > > Andy's review of Gwendal's sx9324 patch included a comment on having > a space before the } in id tables. It is a more general comment, but > this series only addresses the id table cases where things are > inconsistent. I said I would send a clean up set when I got bored. That > happened sooner than expected (I blame tax returns). > > Found using: > git grep -re "[0-9A-Za-z]}" drivers/iio/ | grep "{\ " > > Note that the fact that we have a clump of these in proximity drivers > suggests that copy and paste is leading to this odd bit of formatting > getting copied into new drivers. > > As ever, some of these don't have active driver maintainers so please > take a quick look at the whole series if you have time. Series applied to the togreg branch of iio.git, pushed out as testing for 0-day to poke at it until after the merge window. Thanks, Jonathan > > Thanks, > > Jonathan > > Cc: Matt Ranostay <matt.ranostay@konsulko.com> > Cc: Parthiban Nallathambi <pn@denx.de> > Cc: Nikita Travkin <nikita@trvn.ru> > Cc: Maslov Dmitry <maslovdmitry@seeed.cc> > Cc: Andreas Klinger <ak@it-klinger.de> > Cc: Antoniu Miclaus <antoniu.miclaus@analog.com> > Cc: Zhiyong Tao <zhiyong.tao@mediatek.com> > Cc: Hui Liu <hui.liu@mediatek.com> > Cc: Gwendal Grignou <gwendal@chromium.org> > Cc: Ivan Drobyshevskyi <drobyshevskyi@gmail.com> > > Jonathan Cameron (16): > iio:chemical:atlas: Trivial white space cleanup to add space before } > iio:light:pa12203001: Tidy up white space change to add spaces after { > and before } > iio:light:vcnl4035: Trivial whitespace cleanup to add space before } > iio:light:us5182: White space cleanup of spacing around {} in id > tables > iio:light:ltr501: White space cleanup of spacing around {} in id > tables > iio:proximity:ping: White space cleanup of spacing around {} in id > tables > iio:proximity:rfd77402: White space cleanup of spacing around {} in id > tables > iio:proximity:srf04: White space cleanup of spacing around {} in id > tables > iio:proximity:srf08: White space cleanup of spacing around {} in id > tables > iio:frequency:admv1013: White space cleanup of spacing around {} in id > tables > iio:adc:mt6577_auxadc: Tidy up white space around {} in id tables > iio:adc:hi8435: Tidy up white space around {} in id tables > iio:adc:ti-adc084s021: Tidy up white space around {} > iio:light:tsl2722: Fix inconsistent spacing before } in id table > iio:proximity:vl53l0x: Tidy up white space around {} in id tables > iio:accel:dmard09: Tidy up white space around {} in id table > > drivers/iio/accel/dmard09.c | 2 +- > drivers/iio/adc/hi8435.c | 2 +- > drivers/iio/adc/mt6577_auxadc.c | 10 +++++----- > drivers/iio/adc/ti-adc084s021.c | 2 +- > drivers/iio/chemical/atlas-sensor.c | 10 +++++----- > drivers/iio/frequency/admv1013.c | 2 +- > drivers/iio/light/ltr501.c | 14 +++++++------- > drivers/iio/light/pa12203001.c | 4 ++-- > drivers/iio/light/tsl2772.c | 2 +- > drivers/iio/light/us5182d.c | 6 +++--- > drivers/iio/light/vcnl4035.c | 2 +- > drivers/iio/proximity/ping.c | 4 ++-- > drivers/iio/proximity/rfd77402.c | 2 +- > drivers/iio/proximity/srf04.c | 12 ++++++------ > drivers/iio/proximity/srf08.c | 6 +++--- > drivers/iio/proximity/vl53l0x-i2c.c | 2 +- > 16 files changed, 41 insertions(+), 41 deletions(-) >
From: Jonathan Cameron <Jonathan.Cameron@huawei.com> Andy's review of Gwendal's sx9324 patch included a comment on having a space before the } in id tables. It is a more general comment, but this series only addresses the id table cases where things are inconsistent. I said I would send a clean up set when I got bored. That happened sooner than expected (I blame tax returns). Found using: git grep -re "[0-9A-Za-z]}" drivers/iio/ | grep "{\ " Note that the fact that we have a clump of these in proximity drivers suggests that copy and paste is leading to this odd bit of formatting getting copied into new drivers. As ever, some of these don't have active driver maintainers so please take a quick look at the whole series if you have time. Thanks, Jonathan Cc: Matt Ranostay <matt.ranostay@konsulko.com> Cc: Parthiban Nallathambi <pn@denx.de> Cc: Nikita Travkin <nikita@trvn.ru> Cc: Maslov Dmitry <maslovdmitry@seeed.cc> Cc: Andreas Klinger <ak@it-klinger.de> Cc: Antoniu Miclaus <antoniu.miclaus@analog.com> Cc: Zhiyong Tao <zhiyong.tao@mediatek.com> Cc: Hui Liu <hui.liu@mediatek.com> Cc: Gwendal Grignou <gwendal@chromium.org> Cc: Ivan Drobyshevskyi <drobyshevskyi@gmail.com> Jonathan Cameron (16): iio:chemical:atlas: Trivial white space cleanup to add space before } iio:light:pa12203001: Tidy up white space change to add spaces after { and before } iio:light:vcnl4035: Trivial whitespace cleanup to add space before } iio:light:us5182: White space cleanup of spacing around {} in id tables iio:light:ltr501: White space cleanup of spacing around {} in id tables iio:proximity:ping: White space cleanup of spacing around {} in id tables iio:proximity:rfd77402: White space cleanup of spacing around {} in id tables iio:proximity:srf04: White space cleanup of spacing around {} in id tables iio:proximity:srf08: White space cleanup of spacing around {} in id tables iio:frequency:admv1013: White space cleanup of spacing around {} in id tables iio:adc:mt6577_auxadc: Tidy up white space around {} in id tables iio:adc:hi8435: Tidy up white space around {} in id tables iio:adc:ti-adc084s021: Tidy up white space around {} iio:light:tsl2722: Fix inconsistent spacing before } in id table iio:proximity:vl53l0x: Tidy up white space around {} in id tables iio:accel:dmard09: Tidy up white space around {} in id table drivers/iio/accel/dmard09.c | 2 +- drivers/iio/adc/hi8435.c | 2 +- drivers/iio/adc/mt6577_auxadc.c | 10 +++++----- drivers/iio/adc/ti-adc084s021.c | 2 +- drivers/iio/chemical/atlas-sensor.c | 10 +++++----- drivers/iio/frequency/admv1013.c | 2 +- drivers/iio/light/ltr501.c | 14 +++++++------- drivers/iio/light/pa12203001.c | 4 ++-- drivers/iio/light/tsl2772.c | 2 +- drivers/iio/light/us5182d.c | 6 +++--- drivers/iio/light/vcnl4035.c | 2 +- drivers/iio/proximity/ping.c | 4 ++-- drivers/iio/proximity/rfd77402.c | 2 +- drivers/iio/proximity/srf04.c | 12 ++++++------ drivers/iio/proximity/srf08.c | 6 +++--- drivers/iio/proximity/vl53l0x-i2c.c | 2 +- 16 files changed, 41 insertions(+), 41 deletions(-)