Linux IIO development
 help / color / mirror / Atom feed
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(-)
> 


      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