From: "Nuno Sá" <noname.nuno@gmail.com>
To: "David Lechner" <dlechner@baylibre.com>,
"Lars-Peter Clausen" <lars@metafoo.de>,
"Michael Hennerich" <Michael.Hennerich@analog.com>,
"Nuno Sá" <nuno.sa@analog.com>,
"Esteban Blanc" <eblanc@baylibre.com>,
"Jonathan Cameron" <jic23@kernel.org>,
"Andy Shevchenko" <andy@kernel.org>,
"Oleksij Rempel" <o.rempel@pengutronix.de>,
kernel@pengutronix.de, "Song Qiang" <songqiang1304521@gmail.com>
Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 00/14] iio: remove bits_per_word = 8 assignments
Date: Wed, 07 May 2025 07:28:12 +0100 [thread overview]
Message-ID: <4678d4f0c5634f8709f84745054ae16484617b8d.camel@gmail.com> (raw)
In-Reply-To: <20250505-iio-remove-bits_per_word-8-v1-0-341f85fcfe11@baylibre.com>
On Mon, 2025-05-05 at 14:20 -0500, David Lechner wrote:
> While grepping the IIO subsystem for drivers that set bits_per_word to
> unusual values to check for potential bugs, I found it to be a bit of a
> nuisance that 2/3s of the matches were setting it to the default value
> of 8. So here is a series removing the unnecessary assignments.
>
> ---
LGTM,
Reviewed-by: Nuno Sá <nuno.sa@analog.com>
> David Lechner (14):
> iio: adc: ad4030: remove bits_per_word = 8
> iio: adc: ti-tsc2046: remove bits_per_word = 8
> iio: chemical: bme680_spi: remove bits_per_word = 8
> iio: dac: ad5761: remove bits_per_word = 8
> iio: dac: ad5766: remove bits_per_word = 8
> iio: dac: ad5791: remove bits_per_word = 8
> iio: dac: ltc2688: remove bits_per_word = 8
> iio: gyro: adxrs450: remove bits_per_word = 8
> iio: imu: adis: remove bits_per_word = 8
> iio: magnetometer: hmc5843_spi: remove bits_per_word = 8
> iio: magnetometer: rm3100-spi: remove bits_per_word = 8
> iio: pressure: bmp280-spi: remove bits_per_word = 8
> iio: pressure: ms5611_spi: remove bits_per_word = 8
> iio: pressure: zpa2326_spi: remove bits_per_word = 8
>
> drivers/iio/adc/ad4030.c | 4 ----
> drivers/iio/adc/ti-tsc2046.c | 1 -
> drivers/iio/chemical/bme680_spi.c | 8 --------
> drivers/iio/dac/ad5761.c | 2 --
> drivers/iio/dac/ad5766.c | 2 --
> drivers/iio/dac/ad5791.c | 2 --
> drivers/iio/dac/ltc2688.c | 2 --
> drivers/iio/gyro/adxrs450.c | 5 -----
> drivers/iio/imu/adis.c | 9 ---------
> drivers/iio/imu/adis_buffer.c | 3 ---
> drivers/iio/magnetometer/hmc5843_spi.c | 1 -
> drivers/iio/magnetometer/rm3100-spi.c | 1 -
> drivers/iio/pressure/bmp280-spi.c | 8 --------
> drivers/iio/pressure/ms5611_spi.c | 1 -
> drivers/iio/pressure/zpa2326_spi.c | 1 -
> 15 files changed, 50 deletions(-)
> ---
> base-commit: 7e9a82ab5b861d3c33c99a22c1245a5b262ee502
> change-id: 20250505-iio-remove-bits_per_word-8-db80654b1c17
>
> Best regards,
next prev parent reply other threads:[~2025-05-07 7:27 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-05 19:20 [PATCH 00/14] iio: remove bits_per_word = 8 assignments David Lechner
2025-05-05 19:20 ` [PATCH 01/14] iio: adc: ad4030: remove bits_per_word = 8 David Lechner
2025-05-05 19:20 ` [PATCH 02/14] iio: adc: ti-tsc2046: " David Lechner
2025-05-06 4:48 ` Oleksij Rempel
2025-05-05 19:20 ` [PATCH 03/14] iio: chemical: bme680_spi: " David Lechner
2025-05-05 19:20 ` [PATCH 04/14] iio: dac: ad5761: " David Lechner
2025-05-05 19:20 ` [PATCH 05/14] iio: dac: ad5766: " David Lechner
2025-05-05 19:20 ` [PATCH 06/14] iio: dac: ad5791: " David Lechner
2025-05-05 19:20 ` [PATCH 07/14] iio: dac: ltc2688: " David Lechner
2025-05-05 19:20 ` [PATCH 08/14] iio: gyro: adxrs450: " David Lechner
2025-05-05 19:20 ` [PATCH 09/14] iio: imu: adis: " David Lechner
2025-05-05 19:20 ` [PATCH 10/14] iio: magnetometer: hmc5843_spi: " David Lechner
2025-05-05 19:20 ` [PATCH 11/14] iio: magnetometer: rm3100-spi: " David Lechner
2025-05-05 19:20 ` [PATCH 12/14] iio: pressure: bmp280-spi: " David Lechner
2025-05-05 19:20 ` [PATCH 13/14] iio: pressure: ms5611_spi: " David Lechner
2025-05-05 19:20 ` [PATCH 14/14] iio: pressure: zpa2326_spi: " David Lechner
2025-05-07 6:28 ` Nuno Sá [this message]
2025-05-07 19:57 ` [PATCH 00/14] iio: remove bits_per_word = 8 assignments Jonathan Cameron
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=4678d4f0c5634f8709f84745054ae16484617b8d.camel@gmail.com \
--to=noname.nuno@gmail.com \
--cc=Michael.Hennerich@analog.com \
--cc=andy@kernel.org \
--cc=dlechner@baylibre.com \
--cc=eblanc@baylibre.com \
--cc=jic23@kernel.org \
--cc=kernel@pengutronix.de \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nuno.sa@analog.com \
--cc=o.rempel@pengutronix.de \
--cc=songqiang1304521@gmail.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