Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Mon, 20 Jul 2020 08:51:34 +0100
From:   Lee Jones <lee.jones@linaro.org>
To:     Jonathan Cameron <jic23@kernel.org>
Cc:     knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-iio@vger.kernel.org
Subject: Re: [PATCH 00/30] First batch of W=1 fixes for IIO
Message-ID: <20200720075134.GM3165313@dell>
References: <20200716135928.1456727-1-lee.jones@linaro.org>
 <20200718154204.280fcb83@archlinux>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20200718154204.280fcb83@archlinux>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Sat, 18 Jul 2020, Jonathan Cameron wrote:

> On Thu, 16 Jul 2020 14:58:58 +0100
> Lee Jones <lee.jones@linaro.org> wrote:
> 
> > This set is part of a larger effort attempting to clean-up W=1
> > kernel builds, which are currently overwhelmingly riddled with
> > niggly little warnings.
> Thanks for these.
> 
> One general comment is I'd have appreciated a spot of grouping to bring
> a particular group of identical problems together in the series.
> Example being the @lock(:) ones in this set.

There are a few reasons for me not doing this.  Firstly, some of the
files experiencing 'lock:' issues had other problems which needed to
be dealt with anyway.  I also do not know what other issues exist
before I start drafting patches - until now I have been going through
the list of issues, file-by-file, from top (first complained about) to
bottom.  I have no mechanism (or desire) to pre-read 100's of lines of
warnings to see which ones could be easily grouped.  I also wanted to
keep changes encapsulated into 1 patch per file, for bisectability and
revertability proposes.

I'm certainly not disagreeing or arguing with you, just explaining
that it is something that I've thought about.

> It doesn't matter that much though. I'm just being fussy :)
> 
> Jonathan
> 
> > 
> > Lee Jones (30):
> >   iio: adc: ad_sigma_delta: Remove unused variable 'ret'
> >   iio: accel: bma220_spi: Do not define 'struct acpi_device_id' when
> >     !CONFIG_ACPI
> >   iio: common: ms_sensors: ms_sensors_i2c: Fix misspelling of parameter
> >     'client'
> >   iio: adc: ad7298: Demote obvious misuse of kerneldoc to standard
> >     comment blocks
> >   iio: chemical: sgp30: Add description for sgp_read_cmd()'s
> >     'duration_us'
> >   iio: adc: ad7923: Demote obvious misuse of kerneldoc to standard
> >     comment blocks
> >   iio: dac: ad5360: Fix kerneldoc attribute formatting for 'lock'
> >   iio: dac: ad5380: Fix kerneldoc attribute formatting for 'lock'
> >   iio: dummy: iio_simple_dummy: Demote unworthy kerneldocs and correct
> >     misspelling
> >   iio: dummy: iio_simple_dummy: Add newline after function-end
> >   iio: dac: ad5421: Fix kerneldoc attribute formatting for 'lock'
> >   iio: gyro: adis16080: Fix formatting issue and compiler attribute
> >     ordering
> >   iio: dummy: iio_simple_dummy_events: Demote file header and supply
> >     descriptions for val2 params
> >   iio: dac: ad5064: Value returned by ad5064_vref_name may not be 'const
> >     * const'
> >   iio: dummy: iio_dummy_evgen: Demote file header and supply description
> >     for 'irq_sim_domain'
> >   iio: adc: ad7887: Demote seemingly unintentional kerneldoc header
> >   iio: adc: ad7949: Fix misspelling issue and compiler attribute
> >     ordering
> >   iio: dummy: iio_simple_dummy_buffer: Demote file header and correct
> >     misspelling
> >   iio: dac: ad5064: Fix a few kerneldoc misdemeanours
> >   iio: dac: ad5446: Complete 'struct ad5446_state' doc and demote
> >     unworthy kerneldocs
> >   iio: dac: ad5449: Fix kerneldoc attribute formatting for 'lock'
> >   iio: dac: ad5755: Fix kerneldoc attribute formatting for 'lock'
> >   iio: dac: ad5758: Move and fix-up kerneldoc header and demote unworthy
> >     kerneldoc
> >   iio: dac: ad5761: Fix kerneldoc attribute formatting for 'lock'
> >   iio: dac: ad5764: Fix misdocumenting and formatting error
> >   iio: dac: ad5791: Complete 'struct ad5791_chip_info' documentation
> >   iio: light: cm32181: Fix formatting and docrot issues in
> >     cm32181_acpi_get_cpm()
> >   iio: magnetometer: mmc35240: Fix function header formatting
> >   iio: imu: kmx61: Fix formatting in kerneldoc function headers
> >   iio: dac: ad7303: Complete 'struct ad7303_state' doc and reorder
> >     compiler attribute
> > 
> >  drivers/iio/accel/bma220_spi.c                 |  2 ++
> >  drivers/iio/adc/ad7298.c                       |  8 ++++----
> >  drivers/iio/adc/ad7887.c                       |  2 +-
> >  drivers/iio/adc/ad7923.c                       |  8 ++++----
> >  drivers/iio/adc/ad7949.c                       |  4 ++--
> >  drivers/iio/adc/ad_sigma_delta.c               |  7 ++-----
> >  drivers/iio/chemical/sgp30.c                   |  1 +
> >  drivers/iio/common/ms_sensors/ms_sensors_i2c.c |  2 +-
> >  drivers/iio/dac/ad5064.c                       | 10 +++++-----
> >  drivers/iio/dac/ad5360.c                       |  2 +-
> >  drivers/iio/dac/ad5380.c                       |  2 +-
> >  drivers/iio/dac/ad5421.c                       |  2 +-
> >  drivers/iio/dac/ad5446.c                       | 11 +++++++----
> >  drivers/iio/dac/ad5449.c                       |  2 +-
> >  drivers/iio/dac/ad5755.c                       |  2 +-
> >  drivers/iio/dac/ad5758.c                       | 18 +++++++++---------
> >  drivers/iio/dac/ad5761.c                       |  2 +-
> >  drivers/iio/dac/ad5764.c                       |  5 ++---
> >  drivers/iio/dac/ad5791.c                       | 10 ++++------
> >  drivers/iio/dac/ad7303.c                       |  5 ++++-
> >  drivers/iio/dummy/iio_dummy_evgen.c            |  4 +++-
> >  drivers/iio/dummy/iio_simple_dummy.c           |  7 ++++---
> >  drivers/iio/dummy/iio_simple_dummy_buffer.c    |  4 ++--
> >  drivers/iio/dummy/iio_simple_dummy_events.c    |  4 +++-
> >  drivers/iio/gyro/adis16080.c                   |  4 ++--
> >  drivers/iio/imu/kmx61.c                        | 14 +++++++-------
> >  drivers/iio/light/cm32181.c                    |  8 ++++----
> >  drivers/iio/magnetometer/mmc35240.c            |  4 ++--
> >  28 files changed, 81 insertions(+), 73 deletions(-)
> > 
> 

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog