| Message ID | 20220721153155.245336-4-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | Handled Elsewhere |
| Headers | show |
| Series | dt-bindings: iio/panel/eeprom/misc/net/spi: drop SPI CPHA and CPOL | expand |
On Thu, Jul 21, 2022 at 05:31:52PM +0200, Krzysztof Kozlowski wrote: > The spi-cpha and spi-cpol properties are device specific and should be > accepted only if device really needs them. Explicitly list them in > device bindings in preparation of their removal from generic > spi-peripheral-props.yaml schema. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/iio/accel/adi,adxl345.yaml | 10 ++++++++-- > .../devicetree/bindings/iio/adc/adi,ad7192.yaml | 10 ++++++++-- > .../devicetree/bindings/iio/adc/adi,ad7292.yaml | 5 ++++- > .../devicetree/bindings/iio/adc/adi,ad7606.yaml | 10 ++++++++-- > .../devicetree/bindings/iio/adc/adi,ad7768-1.yaml | 10 ++++++++-- > .../bindings/iio/adc/microchip,mcp3201.yaml | 12 ++++++++++-- > .../devicetree/bindings/iio/adc/ti,adc084s021.yaml | 11 +++++++++-- > .../devicetree/bindings/iio/adc/ti,ads124s08.yaml | 5 ++++- > .../devicetree/bindings/iio/adc/ti,ads131e08.yaml | 5 ++++- > .../devicetree/bindings/iio/addac/adi,ad74413r.yaml | 5 ++++- > .../devicetree/bindings/iio/dac/adi,ad5592r.yaml | 5 ++++- > .../devicetree/bindings/iio/dac/adi,ad5755.yaml | 10 ++++++++-- > .../devicetree/bindings/iio/dac/adi,ad5758.yaml | 6 +++++- > .../devicetree/bindings/iio/dac/adi,ad5766.yaml | 5 ++++- > .../devicetree/bindings/iio/dac/ti,dac082s085.yaml | 9 +++++++-- > .../bindings/iio/gyroscope/adi,adxrs290.yaml | 10 ++++++++-- > .../devicetree/bindings/iio/imu/adi,adis16460.yaml | 12 +++++++++--- > .../devicetree/bindings/iio/imu/adi,adis16475.yaml | 10 ++++++++-- > .../devicetree/bindings/iio/imu/adi,adis16480.yaml | 11 +++++++++-- > .../bindings/iio/imu/invensense,icm42600.yaml | 12 ++++++++++-- > .../bindings/iio/proximity/ams,as3935.yaml | 5 ++++- > .../devicetree/bindings/iio/resolver/adi,ad2s90.yaml | 10 ++++++++-- > .../bindings/iio/temperature/maxim,max31855k.yaml | 6 +++++- > .../bindings/iio/temperature/maxim,max31856.yaml | 6 +++++- > .../bindings/iio/temperature/maxim,max31865.yaml | 6 +++++- > 25 files changed, 166 insertions(+), 40 deletions(-) This whole patch can be dropped which will make merging easier. Rob
diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml index 9bb039e2f533..0b498b9c9823 100644 --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml @@ -28,9 +28,15 @@ properties: reg: maxItems: 1 - spi-cpha: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. - spi-cpol: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. spi-max-frequency: true diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml index 22b7ed3723f6..d533eb6e9233 100644 --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml @@ -26,9 +26,15 @@ properties: reg: maxItems: 1 - spi-cpol: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. - spi-cpha: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. spi-max-frequency: true diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml index a3e39a40c9b3..c0be5c87bd5c 100644 --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml @@ -28,7 +28,10 @@ properties: description: | The regulator supply for ADC and DAC reference voltage. - spi-cpha: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. spi-max-frequency: true diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml index 73775174cf57..181358c90f8e 100644 --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml @@ -29,9 +29,15 @@ properties: reg: maxItems: 1 - spi-cpha: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. - spi-cpol: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. spi-max-frequency: true diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml index a85a28145ef6..6f9457f41ac3 100644 --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml @@ -52,9 +52,15 @@ properties: spi-max-frequency: true - spi-cpol: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. - spi-cpha: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. "#io-channel-cells": const: 1 diff --git a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3201.yaml b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3201.yaml index fcc1ba53b20d..b880354567e3 100644 --- a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3201.yaml +++ b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3201.yaml @@ -33,8 +33,16 @@ properties: maxItems: 1 spi-max-frequency: true - spi-cpha: true - spi-cpol: true + + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. + + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. vref-supply: description: External reference. diff --git a/Documentation/devicetree/bindings/iio/adc/ti,adc084s021.yaml b/Documentation/devicetree/bindings/iio/adc/ti,adc084s021.yaml index 1a113b30a414..07e1d54e93fe 100644 --- a/Documentation/devicetree/bindings/iio/adc/ti,adc084s021.yaml +++ b/Documentation/devicetree/bindings/iio/adc/ti,adc084s021.yaml @@ -24,8 +24,15 @@ properties: vref-supply: description: External reference, needed to establish input scaling - spi-cpol: true - spi-cpha: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. + + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. "#io-channel-cells": const: 1 diff --git a/Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml b/Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml index 9f5e96439c01..74be1b4a4c27 100644 --- a/Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml +++ b/Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml @@ -20,7 +20,10 @@ properties: spi-max-frequency: true - spi-cpha: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. reset-gpios: maxItems: 1 diff --git a/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml b/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml index e0670e3fbb72..b05426a4cae5 100644 --- a/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml +++ b/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml @@ -30,7 +30,10 @@ properties: spi-max-frequency: true - spi-cpha: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. clocks: description: | diff --git a/Documentation/devicetree/bindings/iio/addac/adi,ad74413r.yaml b/Documentation/devicetree/bindings/iio/addac/adi,ad74413r.yaml index baa65a521bad..5f5354601a0f 100644 --- a/Documentation/devicetree/bindings/iio/addac/adi,ad74413r.yaml +++ b/Documentation/devicetree/bindings/iio/addac/adi,ad74413r.yaml @@ -39,7 +39,10 @@ properties: spi-max-frequency: maximum: 1000000 - spi-cpol: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. interrupts: maxItems: 1 diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5592r.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5592r.yaml index 30194880f457..7f094e31bddc 100644 --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5592r.yaml +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5592r.yaml @@ -21,7 +21,10 @@ properties: spi-max-frequency: maximum: 30000000 - spi-cpol: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. "#address-cells": const: 1 diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5755.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5755.yaml index f866b88e1440..11cec9c991c0 100644 --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5755.yaml +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5755.yaml @@ -22,8 +22,14 @@ properties: maxItems: 1 spi-cpha: - description: Either this or spi-cpol but not both. - spi-cpol: true + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. + + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. spi-max-frequency: true diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml index fd4edca34a28..4efcc2c7eaf8 100644 --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml @@ -17,7 +17,11 @@ properties: maxItems: 1 spi-max-frequency: true - spi-cpha: true + + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. adi,dc-dc-mode: $ref: /schemas/types.yaml#/definitions/uint32 diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml index a8f7720d1e3e..ceea30f3af55 100644 --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml @@ -30,7 +30,10 @@ properties: spi-max-frequency: maximum: 1000000 - spi-cpol: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. reset-gpios: description: GPIO spec for the RESET pin. As the line is active low, it diff --git a/Documentation/devicetree/bindings/iio/dac/ti,dac082s085.yaml b/Documentation/devicetree/bindings/iio/dac/ti,dac082s085.yaml index b0157050f1ee..31e909eca988 100644 --- a/Documentation/devicetree/bindings/iio/dac/ti,dac082s085.yaml +++ b/Documentation/devicetree/bindings/iio/dac/ti,dac082s085.yaml @@ -25,10 +25,15 @@ properties: reg: maxItems: 1 - spi-cpha: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. + spi-cpol: + type: boolean description: - Must be either spi-cpha, or spi-cpol but not both. + The device requires inverse clock polarity (CPOL) mode. vref-supply: description: Needed to provide output scaling. diff --git a/Documentation/devicetree/bindings/iio/gyroscope/adi,adxrs290.yaml b/Documentation/devicetree/bindings/iio/gyroscope/adi,adxrs290.yaml index 662ec59ca0af..5462efc1b87f 100644 --- a/Documentation/devicetree/bindings/iio/gyroscope/adi,adxrs290.yaml +++ b/Documentation/devicetree/bindings/iio/gyroscope/adi,adxrs290.yaml @@ -24,9 +24,15 @@ properties: spi-max-frequency: maximum: 5000000 - spi-cpol: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. - spi-cpha: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. interrupts: maxItems: 1 diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml index 340be256f283..f28833915f2b 100644 --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml @@ -21,9 +21,15 @@ properties: reg: maxItems: 1 - spi-cpha: true - - spi-cpol: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. + + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. spi-max-frequency: true diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml index a7574210175a..5ae163819a24 100644 --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml @@ -40,9 +40,15 @@ properties: reg: maxItems: 1 - spi-cpha: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. - spi-cpol: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. spi-max-frequency: maximum: 2000000 diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16480.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16480.yaml index dd29dc6c4c19..dab503b54ad0 100644 --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16480.yaml +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16480.yaml @@ -49,8 +49,15 @@ properties: spi-max-frequency: true - spi-cpha: true - spi-cpol: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. + + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. reset-gpios: maxItems: 1 diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml index 4c1c083d0e92..9fe3c5993601 100644 --- a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml +++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml @@ -48,8 +48,16 @@ properties: description: Regulator that provides power to the bus spi-max-frequency: true - spi-cpha: true - spi-cpol: true + + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. + + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. required: - compatible diff --git a/Documentation/devicetree/bindings/iio/proximity/ams,as3935.yaml b/Documentation/devicetree/bindings/iio/proximity/ams,as3935.yaml index 7fcba5d6d508..1245a4134256 100644 --- a/Documentation/devicetree/bindings/iio/proximity/ams,as3935.yaml +++ b/Documentation/devicetree/bindings/iio/proximity/ams,as3935.yaml @@ -23,7 +23,10 @@ properties: spi-max-frequency: maximum: 2000000 - spi-cpha: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. interrupts: maxItems: 1 diff --git a/Documentation/devicetree/bindings/iio/resolver/adi,ad2s90.yaml b/Documentation/devicetree/bindings/iio/resolver/adi,ad2s90.yaml index 81e4bdfc17c4..38c0acb96dd6 100644 --- a/Documentation/devicetree/bindings/iio/resolver/adi,ad2s90.yaml +++ b/Documentation/devicetree/bindings/iio/resolver/adi,ad2s90.yaml @@ -29,9 +29,15 @@ properties: most 2 * 600ns, so the max frequency should be 1 / (2 * 6e-7), which gives roughly 830000Hz. - spi-cpol: true + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. - spi-cpha: true + spi-cpol: + type: boolean + description: + The device requires inverse clock polarity (CPOL) mode. additionalProperties: false diff --git a/Documentation/devicetree/bindings/iio/temperature/maxim,max31855k.yaml b/Documentation/devicetree/bindings/iio/temperature/maxim,max31855k.yaml index 9969bac66aa1..9bbff1a5c0a7 100644 --- a/Documentation/devicetree/bindings/iio/temperature/maxim,max31855k.yaml +++ b/Documentation/devicetree/bindings/iio/temperature/maxim,max31855k.yaml @@ -33,7 +33,11 @@ properties: maxItems: 1 spi-max-frequency: true - spi-cpha: true + + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. required: - compatible diff --git a/Documentation/devicetree/bindings/iio/temperature/maxim,max31856.yaml b/Documentation/devicetree/bindings/iio/temperature/maxim,max31856.yaml index 873b34766676..44e53f0c84ba 100644 --- a/Documentation/devicetree/bindings/iio/temperature/maxim,max31856.yaml +++ b/Documentation/devicetree/bindings/iio/temperature/maxim,max31856.yaml @@ -20,7 +20,11 @@ properties: maxItems: 1 spi-max-frequency: true - spi-cpha: true + + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. thermocouple-type: $ref: /schemas/types.yaml#/definitions/uint32 diff --git a/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml b/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml index aafb33b16549..f1b6b151ebc9 100644 --- a/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml +++ b/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml @@ -26,7 +26,11 @@ properties: type: boolean spi-max-frequency: true - spi-cpha: true + + spi-cpha: + type: boolean + description: + The device requires shifted clock phase (CPHA) mode. required: - compatible
The spi-cpha and spi-cpol properties are device specific and should be accepted only if device really needs them. Explicitly list them in device bindings in preparation of their removal from generic spi-peripheral-props.yaml schema. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../devicetree/bindings/iio/accel/adi,adxl345.yaml | 10 ++++++++-- .../devicetree/bindings/iio/adc/adi,ad7192.yaml | 10 ++++++++-- .../devicetree/bindings/iio/adc/adi,ad7292.yaml | 5 ++++- .../devicetree/bindings/iio/adc/adi,ad7606.yaml | 10 ++++++++-- .../devicetree/bindings/iio/adc/adi,ad7768-1.yaml | 10 ++++++++-- .../bindings/iio/adc/microchip,mcp3201.yaml | 12 ++++++++++-- .../devicetree/bindings/iio/adc/ti,adc084s021.yaml | 11 +++++++++-- .../devicetree/bindings/iio/adc/ti,ads124s08.yaml | 5 ++++- .../devicetree/bindings/iio/adc/ti,ads131e08.yaml | 5 ++++- .../devicetree/bindings/iio/addac/adi,ad74413r.yaml | 5 ++++- .../devicetree/bindings/iio/dac/adi,ad5592r.yaml | 5 ++++- .../devicetree/bindings/iio/dac/adi,ad5755.yaml | 10 ++++++++-- .../devicetree/bindings/iio/dac/adi,ad5758.yaml | 6 +++++- .../devicetree/bindings/iio/dac/adi,ad5766.yaml | 5 ++++- .../devicetree/bindings/iio/dac/ti,dac082s085.yaml | 9 +++++++-- .../bindings/iio/gyroscope/adi,adxrs290.yaml | 10 ++++++++-- .../devicetree/bindings/iio/imu/adi,adis16460.yaml | 12 +++++++++--- .../devicetree/bindings/iio/imu/adi,adis16475.yaml | 10 ++++++++-- .../devicetree/bindings/iio/imu/adi,adis16480.yaml | 11 +++++++++-- .../bindings/iio/imu/invensense,icm42600.yaml | 12 ++++++++++-- .../bindings/iio/proximity/ams,as3935.yaml | 5 ++++- .../devicetree/bindings/iio/resolver/adi,ad2s90.yaml | 10 ++++++++-- .../bindings/iio/temperature/maxim,max31855k.yaml | 6 +++++- .../bindings/iio/temperature/maxim,max31856.yaml | 6 +++++- .../bindings/iio/temperature/maxim,max31865.yaml | 6 +++++- 25 files changed, 166 insertions(+), 40 deletions(-)