From: "Nuno Sá" <noname.nuno@gmail.com>
To: "Uwe Kleine-König (The Capable Hub)" <u.kleine-koenig@baylibre.com>
Cc: "Jonathan Cameron" <jic23@kernel.org>,
"David Lechner" <dlechner@baylibre.com>,
"Nuno Sá" <nuno.sa@analog.com>,
"Andy Shevchenko" <andy@kernel.org>,
"Vladislav Kulikov" <vlad.kulikov.c@gmail.com>,
"Kevin Tung" <kevin.tung.openbmc@gmail.com>,
"ChiYuan Huang" <cy_huang@richtek.com>,
linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
"Oleksij Rempel" <o.rempel@pengutronix.de>,
kernel@pengutronix.de,
"Antoniu Miclaus" <antoniu.miclaus@analog.com>,
"Michael Hennerich" <Michael.Hennerich@analog.com>,
linux@analog.com, "Marcelo Schmitt" <marcelo.schmitt@analog.com>,
"Petre Rodan" <petre.rodan@subdimension.ro>,
"Dan Robertson" <dan@dlrobertson.com>,
"Rui Miguel Silva" <rmfrfs@gmail.com>,
"Dragos Bogdan" <dragos.bogdan@analog.com>,
"Dixit Parmar" <dixitparmar19@gmail.com>,
chuguangqing <chuguangqing@inspur.com>,
"Sebastian Andrzej Siewior" <bigeasy@linutronix.de>,
"Marcus Folkesson" <marcus.folkesson@gmail.com>,
"Andrew Davis" <afd@ti.com>,
"Puranjay Mohan" <puranjay@kernel.org>,
"Ramona Gradinariu" <ramona.gradinariu@analog.com>,
"Esteban Blanc" <eblanc@baylibre.com>,
"Sergiu Cuciurean" <sergiu.cuciurean@analog.com>,
"Alisa-Dariana Roman" <alisa.roman@analog.com>,
"Matti Vaittinen" <mazziesaccount@gmail.com>,
"Renato Lui Geh" <renatogeh@gmail.com>,
"Ramona Bolboaca" <ramona.bolboaca@analog.com>,
"Kim Seer Paller" <kimseer.paller@analog.com>,
"Marilene Andrade Garcia" <marilene.agarcia@gmail.com>,
"Marius Cristea" <marius.cristea@microchip.com>,
"Kent Gustavsson" <kent@minoris.se>,
"Kurt Borja" <kuurtb@gmail.com>,
"Leonard Göhrs" <l.goehrs@pengutronix.de>,
"Rodrigo Alencar" <rodrigo.alencar@analog.com>,
"Gustavo Silva" <gustavograzs@gmail.com>,
"Alexis Czezar Torreno" <alexisczezar.torreno@analog.com>,
"Janani Sunil" <janani.sunil@analog.com>,
"Anshul Dalal" <anshulusr@gmail.com>,
"Ricardo Ribalda" <ribalda@kernel.org>,
"Lorenzo Bianconi" <lorenzo@kernel.org>,
"Alex Lanzano" <lanzano.alex@gmail.com>,
"Jagath Jog J" <jagathjog1996@gmail.com>,
"Jean-Baptiste Maneyrol" <jean-baptiste.maneyrol@tdk.com>,
"Remi Buisson" <remi.buisson@tdk.com>,
"Herve Codina" <herve.codina@bootlin.com>,
"Andreas Klinger" <ak@it-klinger.de>,
"Harshit Mogalapalli" <harshit.m.mogalapalli@oracle.com>,
"Andrew Ijano" <andrew.ijano@gmail.com>,
"Giorgi Tchankvetadze" <giorgitchankvetadze1997@gmail.com>,
"Krzysztof Kozlowski" <krzk@kernel.org>,
"Md Shofiqul Islam" <shofiqtest@gmail.com>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Stepan Ionichev" <sozdayvek@gmail.com>,
"Sam Daly" <sam@samdaly.ie>,
"Colin Ian King" <colin.i.king@gmail.com>,
"David Jander" <david@protonic.nl>,
"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
"Bartosz Golaszewski" <brgl@kernel.org>,
"Linus Walleij" <linusw@kernel.org>,
"Gabriel Rondon" <grondon@gmail.com>,
"David Marinovic" <david.marinovic@pupin.rs>,
"Lukas Schmid" <lukas.schmid@netcube.li>,
"Shi Hao" <i.shihao.999@gmail.com>,
"Rahman Mahmutović" <mahmutovicrahman5@gmail.com>,
linux-staging@lists.linux.dev
Subject: Re: [PATCH v1 00/13] iio: Use named initializers for device_id structures
Date: Mon, 22 Jun 2026 12:43:21 +0100 [thread overview]
Message-ID: <ajkflUnWXSWPRegL@nsa> (raw)
In-Reply-To: <cover.1781883685.git.u.kleine-koenig@baylibre.com>
On Fri, Jun 19, 2026 at 05:54:28PM +0200, Uwe Kleine-König (The Capable Hub) wrote:
> Hello,
>
> the first patch is a follow up for making iio's i2c_device_id arrays use
> named initializers. The two drivers adapted in it were missed before
> because they were not yet in the tree I used to prepare the patch. The
> remainder is a similar adaption for spi_device_id. Instead of adapting
> arrays with a single entry and .driver_data, rework these to not use
> .driver_data.
>
> The ad9523 driver is a bit special here. It can only probe devices that
> have platform data. A device instantiated using dt never has platform
> data, so this driver only works for boards registering the device using
> a spi_board_info struct. There is no such board, and there never was
> one. So an alternative is to just drop this driver?!
>
> I was unsure if I should post the staging patches in a separate series,
> I hope it's fine to include them here.
>
> Best regards
> Uwe
>
Tend to agree with Andy on the squash. Not a big deal to me though.
Hence:
Reviewed-by: Nuno Sá <nuno.sa@analog.com>
> Uwe Kleine-König (The Capable Hub) (13):
> iio: Initialize i2c_device_id arrays using member names (part 2)
> iio: adc: ti-tsc2046: Simplify device abstraction
> iio: adc: ti-tsc2046: Drop unused member from private data
> iio: adc: max1241: Simplify device abstraction
> iio: dac: max5522: Simplify device abstraction
> iio: frequency: ad9523: Simplify driver a bit
> iio: imu: adis16550: Simplify device abstraction
> iio: magnetometer: hmc5843: Simplify device abstraction
> iio: accel: bmc150: Explicitly set spi .driver_data
> iio: Drop unused assignment of spi_device_id driver data
> iio: Initialize spi_device_id arrays using member names
> staging: iio: Drop unused assignment of spi_device_id driver data
> staging: iio: Initialize spi_device_id arrays using member names
>
> drivers/iio/accel/adxl345_spi.c | 4 +-
> drivers/iio/accel/adxl355_spi.c | 4 +-
> drivers/iio/accel/adxl367_spi.c | 2 +-
> drivers/iio/accel/adxl372_spi.c | 4 +-
> drivers/iio/accel/adxl380_spi.c | 8 +-
> drivers/iio/accel/bma220_spi.c | 2 +-
> drivers/iio/accel/bma400_spi.c | 2 +-
> drivers/iio/accel/bmc150-accel-spi.c | 18 +--
> drivers/iio/accel/bmi088-accel-spi.c | 6 +-
> drivers/iio/accel/fxls8962af-spi.c | 4 +-
> drivers/iio/accel/kxsd9-spi.c | 2 +-
> drivers/iio/accel/mma7455_spi.c | 4 +-
> drivers/iio/accel/sca3000.c | 8 +-
> drivers/iio/accel/sca3300.c | 4 +-
> drivers/iio/accel/st_accel_spi.c | 40 +++---
> drivers/iio/adc/ad4000.c | 62 ++++-----
> drivers/iio/adc/ad4030.c | 14 +-
> drivers/iio/adc/ad4080.c | 22 +--
> drivers/iio/adc/ad4130.c | 12 +-
> drivers/iio/adc/ad4134.c | 2 +-
> drivers/iio/adc/ad4170-4.c | 6 +-
> drivers/iio/adc/ad4851.c | 18 +--
> drivers/iio/adc/ad7091r8.c | 6 +-
> drivers/iio/adc/ad7124.c | 4 +-
> drivers/iio/adc/ad7173.c | 26 ++--
> drivers/iio/adc/ad7191.c | 2 +-
> drivers/iio/adc/ad7192.c | 10 +-
> drivers/iio/adc/ad7266.c | 4 +-
> drivers/iio/adc/ad7280a.c | 2 +-
> drivers/iio/adc/ad7292.c | 2 +-
> drivers/iio/adc/ad7298.c | 2 +-
> drivers/iio/adc/ad7380.c | 36 ++---
> drivers/iio/adc/ad7476.c | 60 ++++----
> drivers/iio/adc/ad7606_spi.c | 22 +--
> drivers/iio/adc/ad7766.c | 12 +-
> drivers/iio/adc/ad7768-1.c | 8 +-
> drivers/iio/adc/ad7780.c | 8 +-
> drivers/iio/adc/ad7791.c | 10 +-
> drivers/iio/adc/ad7793.c | 18 +--
> drivers/iio/adc/ad7887.c | 2 +-
> drivers/iio/adc/ad7923.c | 14 +-
> drivers/iio/adc/ad7944.c | 6 +-
> drivers/iio/adc/ad7949.c | 6 +-
> drivers/iio/adc/ad9467.c | 14 +-
> drivers/iio/adc/ade9000.c | 2 +-
> drivers/iio/adc/hi8435.c | 2 +-
> drivers/iio/adc/max1027.c | 12 +-
> drivers/iio/adc/max1118.c | 6 +-
> drivers/iio/adc/max11205.c | 4 +-
> drivers/iio/adc/max11410.c | 2 +-
> drivers/iio/adc/max1241.c | 8 +-
> drivers/iio/adc/max14001.c | 4 +-
> drivers/iio/adc/mcp320x.c | 26 ++--
> drivers/iio/adc/mcp3564.c | 24 ++--
> drivers/iio/adc/mcp3911.c | 14 +-
> drivers/iio/adc/rohm-bd79112.c | 2 +-
> drivers/iio/adc/rtq6056.c | 4 +-
> drivers/iio/adc/ti-adc0832.c | 8 +-
> drivers/iio/adc/ti-adc084s021.c | 2 +-
> drivers/iio/adc/ti-adc108s102.c | 2 +-
> drivers/iio/adc/ti-adc12138.c | 6 +-
> drivers/iio/adc/ti-adc128s052.c | 24 ++--
> drivers/iio/adc/ti-adc161s626.c | 4 +-
> drivers/iio/adc/ti-ads1018.c | 4 +-
> drivers/iio/adc/ti-ads124s08.c | 4 +-
> drivers/iio/adc/ti-ads1298.c | 2 +-
> drivers/iio/adc/ti-ads131e08.c | 6 +-
> drivers/iio/adc/ti-ads131m02.c | 10 +-
> drivers/iio/adc/ti-ads7950.c | 24 ++--
> drivers/iio/adc/ti-ads8688.c | 4 +-
> drivers/iio/adc/ti-lmp92064.c | 2 +-
> drivers/iio/adc/ti-tlc4541.c | 4 +-
> drivers/iio/adc/ti-tsc2046.c | 12 +-
> drivers/iio/addac/ad74115.c | 2 +-
> drivers/iio/amplifiers/ad8366.c | 26 ++--
> drivers/iio/amplifiers/ada4250.c | 2 +-
> drivers/iio/chemical/bme680_spi.c | 2 +-
> drivers/iio/chemical/ens160_spi.c | 2 +-
> drivers/iio/dac/ad3530r.c | 8 +-
> drivers/iio/dac/ad5064.c | 32 ++---
> drivers/iio/dac/ad5360.c | 16 +--
> drivers/iio/dac/ad5380.c | 32 ++---
> drivers/iio/dac/ad5446-spi.c | 62 ++++-----
> drivers/iio/dac/ad5449.c | 14 +-
> drivers/iio/dac/ad5504.c | 4 +-
> drivers/iio/dac/ad5624r_spi.c | 12 +-
> drivers/iio/dac/ad5706r.c | 2 +-
> drivers/iio/dac/ad5755.c | 10 +-
> drivers/iio/dac/ad5758.c | 2 +-
> drivers/iio/dac/ad5761.c | 8 +-
> drivers/iio/dac/ad5764.c | 8 +-
> drivers/iio/dac/ad5766.c | 4 +-
> drivers/iio/dac/ad5770r.c | 2 +-
> drivers/iio/dac/ad5791.c | 10 +-
> drivers/iio/dac/ad7293.c | 2 +-
> drivers/iio/dac/ad7303.c | 2 +-
> drivers/iio/dac/ad8801.c | 4 +-
> drivers/iio/dac/ad9739a.c | 2 +-
> drivers/iio/dac/ltc1660.c | 4 +-
> drivers/iio/dac/ltc2632.c | 44 +++---
> drivers/iio/dac/ltc2664.c | 4 +-
> drivers/iio/dac/ltc2688.c | 2 +-
> drivers/iio/dac/max22007.c | 2 +-
> drivers/iio/dac/max5522.c | 31 +----
> drivers/iio/dac/mcp4821.c | 12 +-
> drivers/iio/dac/mcp4922.c | 8 +-
> drivers/iio/dac/rohm-bd79703.c | 8 +-
> drivers/iio/dac/ti-dac082s085.c | 12 +-
> drivers/iio/dac/ti-dac7311.c | 6 +-
> drivers/iio/dac/ti-dac7612.c | 2 +-
> drivers/iio/filter/admv8818.c | 2 +-
> drivers/iio/frequency/ad9523.c | 3 +-
> drivers/iio/frequency/adf4350.c | 4 +-
> drivers/iio/frequency/adf4371.c | 4 +-
> drivers/iio/frequency/adf4377.c | 4 +-
> drivers/iio/frequency/admv1013.c | 2 +-
> drivers/iio/frequency/admv1014.c | 2 +-
> drivers/iio/frequency/adrf6780.c | 2 +-
> drivers/iio/gyro/adis16080.c | 4 +-
> drivers/iio/gyro/adis16136.c | 8 +-
> drivers/iio/gyro/adis16260.c | 12 +-
> drivers/iio/gyro/adxrs450.c | 4 +-
> drivers/iio/gyro/bmg160_spi.c | 7 +-
> drivers/iio/gyro/fxas21002c_spi.c | 2 +-
> drivers/iio/gyro/st_gyro_spi.c | 18 +--
> drivers/iio/health/afe4403.c | 2 +-
> drivers/iio/humidity/hts221_spi.c | 2 +-
> drivers/iio/imu/adis16400.c | 30 ++--
> drivers/iio/imu/adis16460.c | 2 +-
> drivers/iio/imu/adis16475.c | 54 ++++----
> drivers/iio/imu/adis16480.c | 40 +++---
> drivers/iio/imu/adis16550.c | 9 +-
> drivers/iio/imu/bmi160/bmi160_spi.c | 4 +-
> drivers/iio/imu/bmi270/bmi270_spi.c | 4 +-
> drivers/iio/imu/bmi323/bmi323_spi.c | 2 +-
> drivers/iio/imu/fxos8700_spi.c | 2 +-
> .../iio/imu/inv_icm42600/inv_icm42600_spi.c | 14 +-
> .../iio/imu/inv_icm45600/inv_icm45600_spi.c | 16 +--
> drivers/iio/imu/inv_mpu6050/inv_mpu_spi.c | 34 ++---
> drivers/iio/imu/smi240.c | 2 +-
> drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c | 48 +++----
> drivers/iio/imu/st_lsm9ds0/st_lsm9ds0_spi.c | 4 +-
> drivers/iio/light/st_uvis25_spi.c | 2 +-
> drivers/iio/magnetometer/bmc150_magn_spi.c | 6 +-
> drivers/iio/magnetometer/hmc5843_spi.c | 6 +-
> drivers/iio/magnetometer/mmc5983.c | 2 +-
> drivers/iio/magnetometer/st_magn_spi.c | 12 +-
> drivers/iio/potentiometer/max5481.c | 8 +-
> drivers/iio/potentiometer/max5487.c | 6 +-
> drivers/iio/potentiometer/mcp41010.c | 12 +-
> drivers/iio/potentiometer/mcp4131.c | 128 +++++++++---------
> drivers/iio/potentiometer/x9250.c | 4 +-
> drivers/iio/pressure/abp2030pa_spi.c | 2 +-
> drivers/iio/pressure/bmp280-spi.c | 14 +-
> drivers/iio/pressure/hsc030pa_spi.c | 2 +-
> drivers/iio/pressure/mpl115_spi.c | 2 +-
> drivers/iio/pressure/mprls0025pa_spi.c | 2 +-
> drivers/iio/pressure/ms5611_spi.c | 4 +-
> drivers/iio/pressure/st_pressure_spi.c | 24 ++--
> drivers/iio/pressure/zpa2326_spi.c | 2 +-
> drivers/iio/proximity/as3935.c | 2 +-
> drivers/iio/resolver/ad2s1200.c | 4 +-
> drivers/iio/resolver/ad2s1210.c | 2 +-
> drivers/iio/resolver/ad2s90.c | 2 +-
> drivers/iio/temperature/ltc2983.c | 10 +-
> drivers/iio/temperature/max31856.c | 2 +-
> drivers/iio/temperature/max31865.c | 2 +-
> drivers/iio/temperature/maxim_thermocouple.c | 18 +--
> drivers/staging/iio/adc/ad7816.c | 6 +-
> drivers/staging/iio/addac/adt7316-spi.c | 13 +-
> drivers/staging/iio/frequency/ad9832.c | 4 +-
> drivers/staging/iio/frequency/ad9834.c | 8 +-
> 172 files changed, 886 insertions(+), 927 deletions(-)
>
>
> base-commit: 3ce97bd3c4f18608335e709c24d6a40e7036cab8
> --
> 2.47.3
>
prev parent reply other threads:[~2026-06-22 11:42 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-19 15:54 [PATCH v1 00/13] iio: Use named initializers for device_id structures Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 01/13] iio: Initialize i2c_device_id arrays using member names (part 2) Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 02/13] iio: adc: ti-tsc2046: Simplify device abstraction Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 03/13] iio: adc: ti-tsc2046: Drop unused member from private data Uwe Kleine-König (The Capable Hub)
2026-06-22 10:30 ` Andy Shevchenko
2026-06-22 13:32 ` Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 04/13] iio: adc: max1241: Simplify device abstraction Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 05/13] iio: dac: max5522: " Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 06/13] iio: frequency: ad9523: Simplify driver a bit Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 07/13] iio: imu: adis16550: Simplify device abstraction Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 08/13] iio: magnetometer: hmc5843: " Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 09/13] iio: accel: bmc150: Explicitly set spi .driver_data Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 10/13] iio: Drop unused assignment of spi_device_id driver data Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 11/13] iio: Initialize spi_device_id arrays using member names Uwe Kleine-König (The Capable Hub)
2026-06-19 20:19 ` Linus Walleij
2026-06-19 15:54 ` [PATCH v1 12/13] staging: iio: Drop unused assignment of spi_device_id driver data Uwe Kleine-König (The Capable Hub)
2026-06-19 15:54 ` [PATCH v1 13/13] staging: iio: Initialize spi_device_id arrays using member names Uwe Kleine-König (The Capable Hub)
2026-06-22 11:43 ` Nuno Sá [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=ajkflUnWXSWPRegL@nsa \
--to=noname.nuno@gmail.com \
--cc=Michael.Hennerich@analog.com \
--cc=afd@ti.com \
--cc=ak@it-klinger.de \
--cc=alexisczezar.torreno@analog.com \
--cc=alisa.roman@analog.com \
--cc=andrew.ijano@gmail.com \
--cc=andy@kernel.org \
--cc=anshulusr@gmail.com \
--cc=antoniu.miclaus@analog.com \
--cc=bigeasy@linutronix.de \
--cc=brgl@kernel.org \
--cc=chuguangqing@inspur.com \
--cc=colin.i.king@gmail.com \
--cc=cy_huang@richtek.com \
--cc=dan@dlrobertson.com \
--cc=david.marinovic@pupin.rs \
--cc=david@protonic.nl \
--cc=dixitparmar19@gmail.com \
--cc=dlechner@baylibre.com \
--cc=dmitry.torokhov@gmail.com \
--cc=dragos.bogdan@analog.com \
--cc=eblanc@baylibre.com \
--cc=giorgitchankvetadze1997@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=grondon@gmail.com \
--cc=gustavograzs@gmail.com \
--cc=harshit.m.mogalapalli@oracle.com \
--cc=herve.codina@bootlin.com \
--cc=i.shihao.999@gmail.com \
--cc=jagathjog1996@gmail.com \
--cc=janani.sunil@analog.com \
--cc=jean-baptiste.maneyrol@tdk.com \
--cc=jic23@kernel.org \
--cc=kent@minoris.se \
--cc=kernel@pengutronix.de \
--cc=kevin.tung.openbmc@gmail.com \
--cc=kimseer.paller@analog.com \
--cc=krzk@kernel.org \
--cc=kuurtb@gmail.com \
--cc=l.goehrs@pengutronix.de \
--cc=lanzano.alex@gmail.com \
--cc=linusw@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=linux@analog.com \
--cc=lorenzo@kernel.org \
--cc=lukas.schmid@netcube.li \
--cc=mahmutovicrahman5@gmail.com \
--cc=marcelo.schmitt@analog.com \
--cc=marcus.folkesson@gmail.com \
--cc=marilene.agarcia@gmail.com \
--cc=marius.cristea@microchip.com \
--cc=mazziesaccount@gmail.com \
--cc=nuno.sa@analog.com \
--cc=o.rempel@pengutronix.de \
--cc=petre.rodan@subdimension.ro \
--cc=puranjay@kernel.org \
--cc=ramona.bolboaca@analog.com \
--cc=ramona.gradinariu@analog.com \
--cc=remi.buisson@tdk.com \
--cc=renatogeh@gmail.com \
--cc=ribalda@kernel.org \
--cc=rmfrfs@gmail.com \
--cc=rodrigo.alencar@analog.com \
--cc=sam@samdaly.ie \
--cc=sergiu.cuciurean@analog.com \
--cc=shofiqtest@gmail.com \
--cc=sozdayvek@gmail.com \
--cc=u.kleine-koenig@baylibre.com \
--cc=vlad.kulikov.c@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 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.