| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20250724123046.4eced56f@jic23-huawei>
Date: Thu, 24 Jul 2025 12:30:46 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: David Heidelberg <david@...t.cz>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>, Lars-Peter Clausen
<lars@...afoo.de>, Michael Hennerich <Michael.Hennerich@...log.com>, David
Lechner <dlechner@...libre.com>, Nuno Sá
<nuno.sa@...log.com>, Andy Shevchenko <andy@...nel.org>, Rob Herring
<robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, Lucas Stankus <lucas.p.stankus@...il.com>, Puranjay
Mohan <puranjay@...nel.org>, Dan Robertson <dan@...obertson.com>, Marcelo
Schmitt <marcelo.schmitt@...log.com>, Alim Akhtar
<alim.akhtar@...sung.com>, Dragos Bogdan <dragos.bogdan@...log.com>,
Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@....com>, Ondřej Jirman <megi@....cz>, Alexandru Tachici
<alexandru.tachici@...log.com>, Stefan Popa <stefan.popa@...log.com>, Linus
Walleij <linus.walleij@...aro.org>, Stephan Gerhold <stephan@...hold.net>,
Ceclan Dumitru <dumitru.ceclan@...log.com>, Alexandru Lazar
<alazar@...rtmail.com>, Andy Gross <agross@...nel.org>, Bjorn Andersson
<andersson@...nel.org>, Matti Vaittinen <mazziesaccount@...il.com>, Angelo
Compagnucci <angelo.compagnucci@...il.com>, Mike Looijmans
<mike.looijmans@...ic.nl>, Manivannan Sadhasivam <mani@...nel.org>, Peter
Meerwald-Stadler <pmeerw@...erw.net>, Andreas Klinger <ak@...klinger.de>,
linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-samsung-soc@...r.kernel.org
Subject: Re: [PATCH 1/4] dt-bindings: iio: Drop unused header includes in
examples
On Thu, 24 Jul 2025 13:19:58 +0200
David Heidelberg <david@...t.cz> wrote:
> Reviewed-by: David Heidelberg <david@...t.cz>
I'm going to take the view these are utterly uncontroversial and pick them
up now (I'm trying to clear my review backlog).
Applied to the testing branch of iio.git which will be rebased on rc1
(so I can easily add tags etc)
Jonathan
>
> On 24/07/2025 13:13, Krzysztof Kozlowski wrote:
> > Drop includes of headers which example code does not use. No functional
> > impact.
> >
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> > ---
> > Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml | 1 -
> > Documentation/devicetree/bindings/iio/accel/adi,adxl313.yaml | 2 --
> > Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml | 2 --
> > Documentation/devicetree/bindings/iio/accel/adi,adxl355.yaml | 2 --
> > Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml | 2 --
> > Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml | 1 -
> > Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml | 1 -
> > Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml | 1 -
> > Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml | 1 -
> > Documentation/devicetree/bindings/iio/adc/adi,ad7173.yaml | 1 -
> > Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.yaml | 1 -
> > Documentation/devicetree/bindings/iio/adc/rohm,bd79104.yaml | 1 -
> > Documentation/devicetree/bindings/iio/adc/ti,adc128s052.yaml | 1 -
> > Documentation/devicetree/bindings/iio/adc/ti,ads1298.yaml | 1 -
> > Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml | 1 -
> > .../devicetree/bindings/iio/imu/invensense,icm42600.yaml | 2 --
> > Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 2 --
> > Documentation/devicetree/bindings/iio/light/st,vl6180.yaml | 1 -
> > .../bindings/iio/magnetometer/voltafield,af8133j.yaml | 1 -
> > Documentation/devicetree/bindings/iio/pressure/bmp085.yaml | 1 -
> > 20 files changed, 26 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> > index 5887021cc90f..3dc973b98f81 100644
> > --- a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> > +++ b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> > @@ -37,7 +37,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl313.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl313.yaml
> > index 0c5b64cae965..3a8c69eecfde 100644
> > --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl313.yaml
> > +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl313.yaml
> > @@ -57,7 +57,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > i2c {
> > #address-cells = <1>;
> > @@ -73,7 +72,6 @@ examples:
> > };
> > };
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> > index 84d949392012..a23a626bfab6 100644
> > --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> > +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> > @@ -56,7 +56,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > i2c {
> > #address-cells = <1>;
> > @@ -72,7 +71,6 @@ examples:
> > };
> > };
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl355.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl355.yaml
> > index c07261c71013..f39e2912731f 100644
> > --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl355.yaml
> > +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl355.yaml
> > @@ -58,7 +58,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > i2c {
> > #address-cells = <1>;
> > @@ -74,7 +73,6 @@ examples:
> > };
> > };
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> > index 62465e36a590..88aa67bf2280 100644
> > --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> > +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> > @@ -37,7 +37,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > i2c {
> > #address-cells = <1>;
> > @@ -52,7 +51,6 @@ examples:
> > };
> > };
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml b/Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml
> > index 457a709b583c..85c9537f1f02 100644
> > --- a/Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml
> > +++ b/Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml
> > @@ -107,7 +107,6 @@ examples:
> > };
> > };
> > - |
> > - # include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > diff --git a/Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml b/Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml
> > index 8723a336229e..c5fedcf998f2 100644
> > --- a/Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml
> > +++ b/Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml
> > @@ -40,7 +40,6 @@ additionalProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > i2c {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml b/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml
> > index f64d99b35492..53de921768ac 100644
> > --- a/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml
> > +++ b/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml
> > @@ -57,7 +57,6 @@ examples:
> > };
> > };
> > - |
> > - # include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml
> > index ddec9747436c..705adbe88def 100644
> > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml
> > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml
> > @@ -93,7 +93,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > i2c {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7173.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7173.yaml
> > index 21ee319d4675..62d906e24997 100644
> > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7173.yaml
> > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7173.yaml
> > @@ -379,7 +379,6 @@ unevaluatedProperties: false
> > examples:
> > # Example AD7173-8 with external reference connected to REF+/REF-:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> >
> > spi {
> > diff --git a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.yaml b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.yaml
> > index c28db0d635a0..b9dc04b0d307 100644
> > --- a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.yaml
> > +++ b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.yaml
> > @@ -278,7 +278,6 @@ examples:
> > - |
> > #include <dt-bindings/iio/qcom,spmi-adc7-pmk8350.h>
> > #include <dt-bindings/iio/qcom,spmi-adc7-pm8350.h>
> > - #include <dt-bindings/interrupt-controller/irq.h>
> >
> > pmic {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/adc/rohm,bd79104.yaml b/Documentation/devicetree/bindings/iio/adc/rohm,bd79104.yaml
> > index 2a8ad4fdfc6b..f0a1347ba4db 100644
> > --- a/Documentation/devicetree/bindings/iio/adc/rohm,bd79104.yaml
> > +++ b/Documentation/devicetree/bindings/iio/adc/rohm,bd79104.yaml
> > @@ -50,7 +50,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > diff --git a/Documentation/devicetree/bindings/iio/adc/ti,adc128s052.yaml b/Documentation/devicetree/bindings/iio/adc/ti,adc128s052.yaml
> > index 775eee972b12..044b66a3b00c 100644
> > --- a/Documentation/devicetree/bindings/iio/adc/ti,adc128s052.yaml
> > +++ b/Documentation/devicetree/bindings/iio/adc/ti,adc128s052.yaml
> > @@ -44,7 +44,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > diff --git a/Documentation/devicetree/bindings/iio/adc/ti,ads1298.yaml b/Documentation/devicetree/bindings/iio/adc/ti,ads1298.yaml
> > index bf5a43a81d59..71f9f9b745cb 100644
> > --- a/Documentation/devicetree/bindings/iio/adc/ti,ads1298.yaml
> > +++ b/Documentation/devicetree/bindings/iio/adc/ti,ads1298.yaml
> > @@ -59,7 +59,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> > index 4cacc9948726..3a725ece7ec4 100644
> > --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> > +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> > @@ -44,7 +44,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
> > index d4d4e5c3d856..119e28a833fd 100644
> > --- a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
> > +++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
> > @@ -74,7 +74,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > i2c {
> > #address-cells = <1>;
> > @@ -91,7 +90,6 @@ examples:
> > };
> > };
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> > index 688100b240bc..2930b3386703 100644
> > --- a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> > +++ b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> > @@ -47,7 +47,6 @@ unevaluatedProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > i2c {
> > #address-cells = <1>;
> > @@ -63,7 +62,6 @@ examples:
> > };
> > };
> > - |
> > - #include <dt-bindings/gpio/gpio.h>
> > #include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/light/st,vl6180.yaml b/Documentation/devicetree/bindings/iio/light/st,vl6180.yaml
> > index 27c36ab7990d..8598fb631aac 100644
> > --- a/Documentation/devicetree/bindings/iio/light/st,vl6180.yaml
> > +++ b/Documentation/devicetree/bindings/iio/light/st,vl6180.yaml
> > @@ -32,7 +32,6 @@ required:
> >
> > examples:
> > - |
> > - #include <dt-bindings/interrupt-controller/irq.h>
> > i2c {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > diff --git a/Documentation/devicetree/bindings/iio/magnetometer/voltafield,af8133j.yaml b/Documentation/devicetree/bindings/iio/magnetometer/voltafield,af8133j.yaml
> > index b6ab01a6914a..ed42dc5afb99 100644
> > --- a/Documentation/devicetree/bindings/iio/magnetometer/voltafield,af8133j.yaml
> > +++ b/Documentation/devicetree/bindings/iio/magnetometer/voltafield,af8133j.yaml
> > @@ -44,7 +44,6 @@ additionalProperties: false
> >
> > examples:
> > - |
> > - #include <dt-bindings/interrupt-controller/irq.h>
> > #include <dt-bindings/gpio/gpio.h>
> > i2c {
> > #address-cells = <1>;
> > diff --git a/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml b/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml
> > index 706b7e24f182..b9ea37317b53 100644
> > --- a/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml
> > +++ b/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml
> > @@ -109,7 +109,6 @@ examples:
> > };
> > - |
> > # include <dt-bindings/gpio/gpio.h>
> > - # include <dt-bindings/interrupt-controller/irq.h>
> > spi {
> > #address-cells = <1>;
> > #size-cells = <0>;
>
Powered by blists - more mailing lists