From: Jonathan Cameron <jic23@kernel.org>
To: linux-iio@vger.kernel.org
Cc: Gwendal Grignou <gwendal@chromium.org>,
Andy Shevchenko <andy.shevchenko@gmail.com>,
Jonathan Cameron <Jonathan.Cameron@huawei.com>,
Matt Ranostay <matt.ranostay@konsulko.com>,
Parthiban Nallathambi <pn@denx.de>,
Nikita Travkin <nikita@trvn.ru>,
Maslov Dmitry <maslovdmitry@seeed.cc>,
Andreas Klinger <ak@it-klinger.de>,
Antoniu Miclaus <antoniu.miclaus@analog.com>,
Zhiyong Tao <zhiyong.tao@mediatek.com>,
Hui Liu <hui.liu@mediatek.com>,
Ivan Drobyshevskyi <drobyshevskyi@gmail.com>
Subject: Re: [PATCH 00/16] IIO: Tidy up inconsistent spacing in ID tables.
Date: Sun, 9 Jan 2022 15:00:16 +0000 [thread overview]
Message-ID: <20220109150016.0a593e46@jic23-huawei> (raw)
In-Reply-To: <20211230174911.78291-1-jic23@kernel.org>
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(-)
>
prev parent reply other threads:[~2022-01-09 14:54 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-30 17:48 [PATCH 00/16] IIO: Tidy up inconsistent spacing in ID tables Jonathan Cameron
2021-12-30 17:48 ` [PATCH 01/16] iio:chemical:atlas: Trivial white space cleanup to add space before } Jonathan Cameron
2021-12-31 5:46 ` Gwendal Grignou
2021-12-30 17:48 ` [PATCH 02/16] iio:light:pa12203001: Tidy up white space change to add spaces after { and " Jonathan Cameron
2021-12-31 5:45 ` Gwendal Grignou
2021-12-30 17:48 ` [PATCH 03/16] iio:light:vcnl4035: Trivial whitespace cleanup to add space " Jonathan Cameron
2021-12-31 5:46 ` Gwendal Grignou
2021-12-30 17:48 ` [PATCH 04/16] iio:light:us5182: White space cleanup of spacing around {} in id tables Jonathan Cameron
2021-12-31 5:47 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 05/16] iio:light:ltr501: " Jonathan Cameron
2021-12-31 5:47 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 06/16] iio:proximity:ping: " Jonathan Cameron
2021-12-31 5:48 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 07/16] iio:proximity:rfd77402: " Jonathan Cameron
2021-12-31 5:48 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 08/16] iio:proximity:srf04: " Jonathan Cameron
2021-12-31 5:48 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 09/16] iio:proximity:srf08: " Jonathan Cameron
2021-12-31 5:49 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 10/16] iio:frequency:admv1013: " Jonathan Cameron
2021-12-31 5:49 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 11/16] iio:adc:mt6577_auxadc: Tidy up white space " Jonathan Cameron
2021-12-31 5:49 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 12/16] iio:adc:hi8435: " Jonathan Cameron
2021-12-31 5:49 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 13/16] iio:adc:ti-adc084s021: Tidy up white space around {} Jonathan Cameron
2021-12-31 5:50 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 14/16] iio:light:tsl2722: Fix inconsistent spacing before } in id table Jonathan Cameron
2021-12-31 5:50 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 15/16] iio:proximity:vl53l0x: Tidy up white space around {} in id tables Jonathan Cameron
2021-12-31 5:50 ` Gwendal Grignou
2021-12-30 17:49 ` [PATCH 16/16] iio:accel:dmard09: Tidy up white space around {} in id table Jonathan Cameron
2021-12-31 5:50 ` Gwendal Grignou
2022-01-09 15:00 ` Jonathan Cameron [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220109150016.0a593e46@jic23-huawei \
--to=jic23@kernel.org \
--cc=Jonathan.Cameron@huawei.com \
--cc=ak@it-klinger.de \
--cc=andy.shevchenko@gmail.com \
--cc=antoniu.miclaus@analog.com \
--cc=drobyshevskyi@gmail.com \
--cc=gwendal@chromium.org \
--cc=hui.liu@mediatek.com \
--cc=linux-iio@vger.kernel.org \
--cc=maslovdmitry@seeed.cc \
--cc=matt.ranostay@konsulko.com \
--cc=nikita@trvn.ru \
--cc=pn@denx.de \
--cc=zhiyong.tao@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox