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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.