[PATCH 0/3] dt-bindings: trivial-devices: misc (whitespace) fixes

All of lore.kernel.org
 help / color / mirror / Atom feed

* [PATCH 0/3] dt-bindings: trivial-devices: misc (whitespace) fixes
@ 2022-01-13 13:38 Stanislav Jakubek
  2022-01-13 13:39 ` [PATCH 1/3] dt-bindings: trivial-devices: fix swapped comments Stanislav Jakubek
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Stanislav Jakubek @ 2022-01-13 13:38 UTC (permalink / raw)
  To: Rob Herring; +Cc: devicetree, linux-kernel

This patchset fixes some minor issues in trivial-devices DT bindings.

Stanislav Jakubek (3):
  dt-bindings: trivial-devices: fix swapped comments
  dt-bindings: trivial-devices: fix double spaces in comments
  dt-bindings: trivial-devices: fix comment indentation

 .../devicetree/bindings/trivial-devices.yaml  | 314 +++++++++---------
 1 file changed, 157 insertions(+), 157 deletions(-)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 1/3] dt-bindings: trivial-devices: fix swapped comments
  2022-01-13 13:38 [PATCH 0/3] dt-bindings: trivial-devices: misc (whitespace) fixes Stanislav Jakubek
@ 2022-01-13 13:39 ` Stanislav Jakubek
  2022-01-14 15:11   ` Rob Herring
  2022-01-13 13:39 ` [PATCH 2/3] dt-bindings: trivial-devices: fix double spaces in comments Stanislav Jakubek
  2022-01-13 13:39 ` [PATCH 3/3] dt-bindings: trivial-devices: fix comment indentation Stanislav Jakubek
  2 siblings, 1 reply; 8+ messages in thread
From: Stanislav Jakubek @ 2022-01-13 13:39 UTC (permalink / raw)
  To: Rob Herring; +Cc: devicetree, linux-kernel

sparkfun,qwiic-joystick and st,24c256 had their comments incorrectly
swapped. Swap them to make them correct.

Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com>
---
 Documentation/devicetree/bindings/trivial-devices.yaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index 9af1b0f4ecea..098811b7aa97 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -303,9 +303,9 @@ properties:
           - skyworks,sky81452
             # Socionext SynQuacer TPM MMIO module
           - socionext,synquacer-tpm-mmio
-            # i2c serial eeprom  (24cxx)
-          - sparkfun,qwiic-joystick
             # SparkFun Qwiic Joystick (COM-15168) with i2c interface
+          - sparkfun,qwiic-joystick
+            # i2c serial eeprom  (24cxx)
           - st,24c256
             # Ambient Light Sensor with SMBUS/Two Wire Serial Interface
           - taos,tsl2550
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/3] dt-bindings: trivial-devices: fix swapped comments
  2022-01-13 13:39 ` [PATCH 1/3] dt-bindings: trivial-devices: fix swapped comments Stanislav Jakubek
@ 2022-01-14 15:11   ` Rob Herring
  0 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2022-01-14 15:11 UTC (permalink / raw)
  To: Stanislav Jakubek; +Cc: Rob Herring, devicetree, linux-kernel

On Thu, 13 Jan 2022 14:39:01 +0100, Stanislav Jakubek wrote:
> sparkfun,qwiic-joystick and st,24c256 had their comments incorrectly
> swapped. Swap them to make them correct.
> 
> Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com>
> ---
>  Documentation/devicetree/bindings/trivial-devices.yaml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 

Applied, thanks!

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 2/3] dt-bindings: trivial-devices: fix double spaces in comments
  2022-01-13 13:38 [PATCH 0/3] dt-bindings: trivial-devices: misc (whitespace) fixes Stanislav Jakubek
  2022-01-13 13:39 ` [PATCH 1/3] dt-bindings: trivial-devices: fix swapped comments Stanislav Jakubek
@ 2022-01-13 13:39 ` Stanislav Jakubek
  2022-01-14 15:11   ` Rob Herring
  2022-01-13 13:39 ` [PATCH 3/3] dt-bindings: trivial-devices: fix comment indentation Stanislav Jakubek
  2 siblings, 1 reply; 8+ messages in thread
From: Stanislav Jakubek @ 2022-01-13 13:39 UTC (permalink / raw)
  To: Rob Herring; +Cc: devicetree, linux-kernel

Cleanup double spaces in some of the comments.

Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com>
---
 Documentation/devicetree/bindings/trivial-devices.yaml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index 098811b7aa97..091792ba993e 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -31,7 +31,7 @@ properties:
       - enum:
             # SMBus/I2C Digital Temperature Sensor in 6-Pin SOT with SMBus Alert and Over Temperature Pin
           - ad,ad7414
-            # ADM9240:  Complete System Hardware Monitor for uProcessor-Based Systems
+            # ADM9240: Complete System Hardware Monitor for uProcessor-Based Systems
           - ad,adm9240
             # AD5110 - Nonvolatile Digital Potentiometer
           - adi,ad5110
@@ -43,7 +43,7 @@ properties:
           - adi,adp5589
             # AMS iAQ-Core VOC Sensor
           - ams,iaq-core
-            # i2c serial eeprom  (24cxx)
+            # i2c serial eeprom (24cxx)
           - at,24c08
             # i2c trusted platform module (TPM)
           - atmel,at97sc3204t
@@ -305,7 +305,7 @@ properties:
           - socionext,synquacer-tpm-mmio
             # SparkFun Qwiic Joystick (COM-15168) with i2c interface
           - sparkfun,qwiic-joystick
-            # i2c serial eeprom  (24cxx)
+            # i2c serial eeprom (24cxx)
           - st,24c256
             # Ambient Light Sensor with SMBUS/Two Wire Serial Interface
           - taos,tsl2550
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/3] dt-bindings: trivial-devices: fix double spaces in comments
  2022-01-13 13:39 ` [PATCH 2/3] dt-bindings: trivial-devices: fix double spaces in comments Stanislav Jakubek
@ 2022-01-14 15:11   ` Rob Herring
  0 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2022-01-14 15:11 UTC (permalink / raw)
  To: Stanislav Jakubek; +Cc: Rob Herring, devicetree, linux-kernel

On Thu, 13 Jan 2022 14:39:20 +0100, Stanislav Jakubek wrote:
> Cleanup double spaces in some of the comments.
> 
> Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com>
> ---
>  Documentation/devicetree/bindings/trivial-devices.yaml | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 

Applied, thanks!

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 3/3] dt-bindings: trivial-devices: fix comment indentation
  2022-01-13 13:38 [PATCH 0/3] dt-bindings: trivial-devices: misc (whitespace) fixes Stanislav Jakubek
  2022-01-13 13:39 ` [PATCH 1/3] dt-bindings: trivial-devices: fix swapped comments Stanislav Jakubek
  2022-01-13 13:39 ` [PATCH 2/3] dt-bindings: trivial-devices: fix double spaces in comments Stanislav Jakubek
@ 2022-01-13 13:39 ` Stanislav Jakubek
  2022-01-14 15:21   ` Rob Herring
  2 siblings, 1 reply; 8+ messages in thread
From: Stanislav Jakubek @ 2022-01-13 13:39 UTC (permalink / raw)
  To: Rob Herring; +Cc: devicetree, linux-kernel

Indent comments the same as the content they describe.
Fixes yamllint warnings like:
32:13     warning  comment not indented like content  (comments-indentation)

Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com>
---
 .../devicetree/bindings/trivial-devices.yaml  | 314 +++++++++---------
 1 file changed, 157 insertions(+), 157 deletions(-)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index 091792ba993e..1fb565c8f967 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -29,332 +29,332 @@ properties:
   compatible:
     items:
       - enum:
-            # SMBus/I2C Digital Temperature Sensor in 6-Pin SOT with SMBus Alert and Over Temperature Pin
+          # SMBus/I2C Digital Temperature Sensor in 6-Pin SOT with SMBus Alert and Over Temperature Pin
           - ad,ad7414
-            # ADM9240: Complete System Hardware Monitor for uProcessor-Based Systems
+          # ADM9240: Complete System Hardware Monitor for uProcessor-Based Systems
           - ad,adm9240
-            # AD5110 - Nonvolatile Digital Potentiometer
+          # AD5110 - Nonvolatile Digital Potentiometer
           - adi,ad5110
-            # Analog Devices ADP5585 Keypad Decoder and I/O Expansion
+          # Analog Devices ADP5585 Keypad Decoder and I/O Expansion
           - adi,adp5585
-            # Analog Devices ADP5585 Keypad Decoder and I/O Expansion with support for Row5
+          # Analog Devices ADP5585 Keypad Decoder and I/O Expansion with support for Row5
           - adi,adp5585-02
-            # Analog Devices ADP5589 Keypad Decoder and I/O Expansion
+          # Analog Devices ADP5589 Keypad Decoder and I/O Expansion
           - adi,adp5589
-            # AMS iAQ-Core VOC Sensor
+          # AMS iAQ-Core VOC Sensor
           - ams,iaq-core
-            # i2c serial eeprom (24cxx)
+          # i2c serial eeprom (24cxx)
           - at,24c08
-            # i2c trusted platform module (TPM)
+          # i2c trusted platform module (TPM)
           - atmel,at97sc3204t
-            # i2c h/w symmetric crypto module
+          # i2c h/w symmetric crypto module
           - atmel,atsha204a
-            # i2c h/w elliptic curve crypto module
+          # i2c h/w elliptic curve crypto module
           - atmel,atecc508a
-            # BPA-RS600: Power Supply
+          # BPA-RS600: Power Supply
           - blutek,bpa-rs600
-            # Bosch Sensortec pressure, temperature, humididty and VOC sensor
+          # Bosch Sensortec pressure, temperature, humididty and VOC sensor
           - bosch,bme680
-            # CM32181: Ambient Light Sensor
+          # CM32181: Ambient Light Sensor
           - capella,cm32181
-            # CM3232: Ambient Light Sensor
+          # CM3232: Ambient Light Sensor
           - capella,cm3232
-            # CM3323: Ambient Light Sensor
+          # CM3323: Ambient Light Sensor
           - capella,cm3323
-            # High-Precision Digital Thermometer
+          # High-Precision Digital Thermometer
           - dallas,ds1631
-            # Total-Elapsed-Time Recorder with Alarm
+          # Total-Elapsed-Time Recorder with Alarm
           - dallas,ds1682
-            # Tiny Digital Thermometer and Thermostat
+          # Tiny Digital Thermometer and Thermostat
           - dallas,ds1775
-            # CPU Peripheral Monitor
+          # CPU Peripheral Monitor
           - dallas,ds1780
-            # CPU Supervisor with Nonvolatile Memory and Programmable I/O
+          # CPU Supervisor with Nonvolatile Memory and Programmable I/O
           - dallas,ds4510
-            # Digital Thermometer and Thermostat
+          # Digital Thermometer and Thermostat
           - dallas,ds75
-            # Delta AHE-50DC Open19 power shelf fan control module
+          # Delta AHE-50DC Open19 power shelf fan control module
           - delta,ahe50dc-fan
-            # Delta Electronics DPS-650-AB power supply
+          # Delta Electronics DPS-650-AB power supply
           - delta,dps650ab
           # Delta Electronics DPS920AB 920W 54V Power Supply
           - delta,dps920ab
-            # 1/4 Brick DC/DC Regulated Power Module
+          # 1/4 Brick DC/DC Regulated Power Module
           - delta,q54sj108a2
-            # Devantech SRF02 ultrasonic ranger in I2C mode
+          # Devantech SRF02 ultrasonic ranger in I2C mode
           - devantech,srf02
-            # Devantech SRF08 ultrasonic ranger
+          # Devantech SRF08 ultrasonic ranger
           - devantech,srf08
-            # Devantech SRF10 ultrasonic ranger
+          # Devantech SRF10 ultrasonic ranger
           - devantech,srf10
-            # DA9053: flexible system level PMIC with multicore support
+          # DA9053: flexible system level PMIC with multicore support
           - dlg,da9053
-            # DA9063: system PMIC for quad-core application processors
+          # DA9063: system PMIC for quad-core application processors
           - dlg,da9063
-            # DMARD05: 3-axis I2C Accelerometer
+          # DMARD05: 3-axis I2C Accelerometer
           - domintech,dmard05
-            # DMARD06: 3-axis I2C Accelerometer
+          # DMARD06: 3-axis I2C Accelerometer
           - domintech,dmard06
-            # DMARD05: 3-axis I2C Accelerometer
+          # DMARD05: 3-axis I2C Accelerometer
           - domintech,dmard07
-            # DMARD09: 3-axis Accelerometer
+          # DMARD09: 3-axis Accelerometer
           - domintech,dmard09
-            # DMARD10: 3-axis Accelerometer
+          # DMARD10: 3-axis Accelerometer
           - domintech,dmard10
-            # MMA7660FC: 3-Axis Orientation/Motion Detection Sensor
+          # MMA7660FC: 3-Axis Orientation/Motion Detection Sensor
           - fsl,mma7660
-            # MMA8450Q: Xtrinsic Low-power, 3-axis Xtrinsic Accelerometer
+          # MMA8450Q: Xtrinsic Low-power, 3-axis Xtrinsic Accelerometer
           - fsl,mma8450
-            # MPL3115: Absolute Digital Pressure Sensor
+          # MPL3115: Absolute Digital Pressure Sensor
           - fsl,mpl3115
-            # MPR121: Proximity Capacitive Touch Sensor Controller
+          # MPR121: Proximity Capacitive Touch Sensor Controller
           - fsl,mpr121
-            # Monolithic Power Systems Inc. multi-phase controller mp2888
+          # Monolithic Power Systems Inc. multi-phase controller mp2888
           - mps,mp2888
-            # Monolithic Power Systems Inc. multi-phase controller mp2975
+          # Monolithic Power Systems Inc. multi-phase controller mp2975
           - mps,mp2975
-            # Honeywell Humidicon HIH-6130 humidity/temperature sensor
+          # Honeywell Humidicon HIH-6130 humidity/temperature sensor
           - honeywell,hi6130
-            # IBM Common Form Factor Power Supply Versions (all versions)
+          # IBM Common Form Factor Power Supply Versions (all versions)
           - ibm,cffps
-            # IBM Common Form Factor Power Supply Versions 1
+          # IBM Common Form Factor Power Supply Versions 1
           - ibm,cffps1
-            # IBM Common Form Factor Power Supply Versions 2
+          # IBM Common Form Factor Power Supply Versions 2
           - ibm,cffps2
-            # Infineon IR36021 digital POL buck controller
+          # Infineon IR36021 digital POL buck controller
           - infineon,ir36021
           # Infineon IR38060 Voltage Regulator
           - infineon,ir38060
-            # Infineon IR38064 Voltage Regulator
+          # Infineon IR38064 Voltage Regulator
           - infineon,ir38064
           # Infineon IR38164 Voltage Regulator
           - infineon,ir38164
           # Infineon IR38263 Voltage Regulator
           - infineon,ir38263
-            # Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
+          # Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
           - infineon,slb9635tt
-            # Infineon SLB9645 I2C TPM (new protocol, max 400khz)
+          # Infineon SLB9645 I2C TPM (new protocol, max 400khz)
           - infineon,slb9645tt
-            # Infineon TLV493D-A1B6 I2C 3D Magnetic Sensor
+          # Infineon TLV493D-A1B6 I2C 3D Magnetic Sensor
           - infineon,tlv493d-a1b6
-            # Infineon Multi-phase Digital VR Controller xdpe12254
+          # Infineon Multi-phase Digital VR Controller xdpe12254
           - infineon,xdpe12254
-            # Infineon Multi-phase Digital VR Controller xdpe12284
+          # Infineon Multi-phase Digital VR Controller xdpe12284
           - infineon,xdpe12284
-            # Inspur Power System power supply unit version 1
+          # Inspur Power System power supply unit version 1
           - inspur,ipsps1
-            # Intersil ISL29028 Ambient Light and Proximity Sensor
+          # Intersil ISL29028 Ambient Light and Proximity Sensor
           - isil,isl29028
-            # Intersil ISL29030 Ambient Light and Proximity Sensor
+          # Intersil ISL29030 Ambient Light and Proximity Sensor
           - isil,isl29030
-            # Intersil ISL68137 Digital Output Configurable PWM Controller
+          # Intersil ISL68137 Digital Output Configurable PWM Controller
           - isil,isl68137
-            # 5 Bit Programmable, Pulse-Width Modulator
+          # 5 Bit Programmable, Pulse-Width Modulator
           - maxim,ds1050
-            # 10 kOhm digital potentiometer with I2C interface
+          # 10 kOhm digital potentiometer with I2C interface
           - maxim,ds1803-010
-            # 50 kOhm digital potentiometer with I2C interface
+          # 50 kOhm digital potentiometer with I2C interface
           - maxim,ds1803-050
-            # 100 kOhm digital potentiometer with I2C interface
+          # 100 kOhm digital potentiometer with I2C interface
           - maxim,ds1803-100
-            # Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
+          # Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
           - maxim,max1237
-            # Temperature Sensor, I2C interface
+          # Temperature Sensor, I2C interface
           - maxim,max1619
-            # 10-bit 10 kOhm linear programable voltage divider
+          # 10-bit 10 kOhm linear programable voltage divider
           - maxim,max5481
-            # 10-bit 50 kOhm linear programable voltage divider
+          # 10-bit 50 kOhm linear programable voltage divider
           - maxim,max5482
-            # 10-bit 10 kOhm linear programable variable resistor
+          # 10-bit 10 kOhm linear programable variable resistor
           - maxim,max5483
-            # 10-bit 50 kOhm linear programable variable resistor
+          # 10-bit 50 kOhm linear programable variable resistor
           - maxim,max5484
-            # PECI-to-I2C translator for PECI-to-SMBus/I2C protocol conversion
+          # PECI-to-I2C translator for PECI-to-SMBus/I2C protocol conversion
           - maxim,max6621
-            # 9-Bit/12-Bit Temperature Sensors with I²C-Compatible Serial Interface
+          # 9-Bit/12-Bit Temperature Sensors with I²C-Compatible Serial Interface
           - maxim,max6625
-            # 3-Channel Remote Temperature Sensor
+          # 3-Channel Remote Temperature Sensor
           - maxim,max31730
-            # mCube 3-axis 8-bit digital accelerometer
+          # mCube 3-axis 8-bit digital accelerometer
           - mcube,mc3230
-            # Measurement Specialities I2C temperature and humidity sensor
+          # Measurement Specialities I2C temperature and humidity sensor
           - meas,htu21
-            # Measurement Specialities I2C pressure and temperature sensor
+          # Measurement Specialities I2C pressure and temperature sensor
           - meas,ms5637
-            # Measurement Specialities I2C pressure and temperature sensor
+          # Measurement Specialities I2C pressure and temperature sensor
           - meas,ms5803
-            # Measurement Specialities I2C pressure and temperature sensor
+          # Measurement Specialities I2C pressure and temperature sensor
           - meas,ms5805
-            # Measurement Specialities I2C pressure and temperature sensor
+          # Measurement Specialities I2C pressure and temperature sensor
           - meas,ms5837
-            # Measurement Specialities temp and humidity part of ms8607 device
+          # Measurement Specialities temp and humidity part of ms8607 device
           - meas,ms8607-humidity
-            # Measurement Specialities temp and pressure part of ms8607 device
+          # Measurement Specialities temp and pressure part of ms8607 device
           - meas,ms8607-temppressure
-            # Measurement Specialties temperature sensor
+          # Measurement Specialties temperature sensor
           - meas,tsys01
-            # MEMSIC magnetometer
+          # MEMSIC magnetometer
           - memsic,mmc35240
-            # MEMSIC 3-axis accelerometer
+          # MEMSIC 3-axis accelerometer
           - memsic,mx4005
-            # MEMSIC 2-axis 8-bit digital accelerometer
+          # MEMSIC 2-axis 8-bit digital accelerometer
           - memsic,mxc6225
-            # MEMSIC 2-axis 8-bit digital accelerometer
+          # MEMSIC 2-axis 8-bit digital accelerometer
           - memsic,mxc6255
-            # MEMSIC 3-axis accelerometer
+          # MEMSIC 3-axis accelerometer
           - memsic,mxc6655
-            # Microchip differential I2C ADC, 1 Channel, 18 bit
+          # Microchip differential I2C ADC, 1 Channel, 18 bit
           - microchip,mcp3421
-            # Microchip differential I2C ADC, 2 Channel, 18 bit
+          # Microchip differential I2C ADC, 2 Channel, 18 bit
           - microchip,mcp3422
-            # Microchip differential I2C ADC, 2 Channel, 18 bit
+          # Microchip differential I2C ADC, 2 Channel, 18 bit
           - microchip,mcp3423
-            # Microchip differential I2C ADC, 4 Channel, 18 bit
+          # Microchip differential I2C ADC, 4 Channel, 18 bit
           - microchip,mcp3424
-            # Microchip differential I2C ADC, 1 Channel, 16 bit
+          # Microchip differential I2C ADC, 1 Channel, 16 bit
           - microchip,mcp3425
-            # Microchip differential I2C ADC, 2 Channel, 16 bit
+          # Microchip differential I2C ADC, 2 Channel, 16 bit
           - microchip,mcp3426
-            # Microchip differential I2C ADC, 2 Channel, 16 bit
+          # Microchip differential I2C ADC, 2 Channel, 16 bit
           - microchip,mcp3427
-            # Microchip differential I2C ADC, 4 Channel, 16 bit
+          # Microchip differential I2C ADC, 4 Channel, 16 bit
           - microchip,mcp3428
-            # Microchip 7-bit Single I2C Digital POT (5k)
+          # Microchip 7-bit Single I2C Digital POT (5k)
           - microchip,mcp4017-502
-            # Microchip 7-bit Single I2C Digital POT (10k)
+          # Microchip 7-bit Single I2C Digital POT (10k)
           - microchip,mcp4017-103
-            # Microchip 7-bit Single I2C Digital POT (50k)
+          # Microchip 7-bit Single I2C Digital POT (50k)
           - microchip,mcp4017-503
-            # Microchip 7-bit Single I2C Digital POT (100k)
+          # Microchip 7-bit Single I2C Digital POT (100k)
           - microchip,mcp4017-104
-            # Microchip 7-bit Single I2C Digital POT (5k)
+          # Microchip 7-bit Single I2C Digital POT (5k)
           - microchip,mcp4018-502
-            # Microchip 7-bit Single I2C Digital POT (10k)
+          # Microchip 7-bit Single I2C Digital POT (10k)
           - microchip,mcp4018-103
-            # Microchip 7-bit Single I2C Digital POT (50k)
+          # Microchip 7-bit Single I2C Digital POT (50k)
           - microchip,mcp4018-503
-            # Microchip 7-bit Single I2C Digital POT (100k)
+          # Microchip 7-bit Single I2C Digital POT (100k)
           - microchip,mcp4018-104
-            # Microchip 7-bit Single I2C Digital POT (5k)
+          # Microchip 7-bit Single I2C Digital POT (5k)
           - microchip,mcp4019-502
-            # Microchip 7-bit Single I2C Digital POT (10k)
+          # Microchip 7-bit Single I2C Digital POT (10k)
           - microchip,mcp4019-103
-            # Microchip 7-bit Single I2C Digital POT (50k)
+          # Microchip 7-bit Single I2C Digital POT (50k)
           - microchip,mcp4019-503
-            # Microchip 7-bit Single I2C Digital POT (100k)
+          # Microchip 7-bit Single I2C Digital POT (100k)
           - microchip,mcp4019-104
-            # PWM Fan Speed Controller With Fan Fault Detection
+          # PWM Fan Speed Controller With Fan Fault Detection
           - microchip,tc654
-            # PWM Fan Speed Controller With Fan Fault Detection
+          # PWM Fan Speed Controller With Fan Fault Detection
           - microchip,tc655
-            # MiraMEMS DA226 2-axis 14-bit digital accelerometer
+          # MiraMEMS DA226 2-axis 14-bit digital accelerometer
           - miramems,da226
-            # MiraMEMS DA280 3-axis 14-bit digital accelerometer
+          # MiraMEMS DA280 3-axis 14-bit digital accelerometer
           - miramems,da280
-            # MiraMEMS DA311 3-axis 12-bit digital accelerometer
+          # MiraMEMS DA311 3-axis 12-bit digital accelerometer
           - miramems,da311
-            # Temperature sensor with integrated fan control
+          # Temperature sensor with integrated fan control
           - national,lm63
-            # I2C TEMP SENSOR
+          # I2C TEMP SENSOR
           - national,lm75
-            # Serial Interface ACPI-Compatible Microprocessor System Hardware Monitor
+          # Serial Interface ACPI-Compatible Microprocessor System Hardware Monitor
           - national,lm80
-            # Serial Interface ACPI-Compatible Microprocessor System Hardware Monitor
+          # Serial Interface ACPI-Compatible Microprocessor System Hardware Monitor
           - national,lm81
-            # Temperature sensor with integrated fan control
+          # Temperature sensor with integrated fan control
           - national,lm85
-            # I2C ±0.33°C Accurate, 12-Bit + Sign Temperature Sensor and Thermal Window Comparator
+          # I2C ±0.33°C Accurate, 12-Bit + Sign Temperature Sensor and Thermal Window Comparator
           - national,lm92
-            # i2c trusted platform module (TPM)
+          # i2c trusted platform module (TPM)
           - nuvoton,npct501
-            # i2c trusted platform module (TPM2)
+          # i2c trusted platform module (TPM2)
           - nuvoton,npct601
-            # Nuvoton Temperature Sensor
+          # Nuvoton Temperature Sensor
           - nuvoton,w83773g
-            # OKI ML86V7667 video decoder
+          # OKI ML86V7667 video decoder
           - oki,ml86v7667
-            # OV5642: Color CMOS QSXGA (5-megapixel) Image Sensor with OmniBSI and Embedded TrueFocus
+          # OV5642: Color CMOS QSXGA (5-megapixel) Image Sensor with OmniBSI and Embedded TrueFocus
           - ovti,ov5642
-            # 48-Lane, 12-Port PCI Express Gen 2 (5.0 GT/s) Switch
+          # 48-Lane, 12-Port PCI Express Gen 2 (5.0 GT/s) Switch
           - plx,pex8648
-            # Pulsedlight LIDAR range-finding sensor
+          # Pulsedlight LIDAR range-finding sensor
           - pulsedlight,lidar-lite-v2
-            # Renesas ISL29501 time-of-flight sensor
+          # Renesas ISL29501 time-of-flight sensor
           - renesas,isl29501
-            # S524AD0XF1 (128K/256K-bit Serial EEPROM for Low Power)
+          # S524AD0XF1 (128K/256K-bit Serial EEPROM for Low Power)
           - samsung,24ad0xd1
-            # Sensirion low power multi-pixel gas sensor with I2C interface
+          # Sensirion low power multi-pixel gas sensor with I2C interface
           - sensirion,sgpc3
-            # Sensirion multi-pixel gas sensor with I2C interface
+          # Sensirion multi-pixel gas sensor with I2C interface
           - sensirion,sgp30
-            # Sensirion gas sensor with I2C interface
+          # Sensirion gas sensor with I2C interface
           - sensirion,sgp40
-            # Sensirion temperature & humidity sensor with I2C interface
+          # Sensirion temperature & humidity sensor with I2C interface
           - sensirion,sht4x
-            # Sensortek 3 axis accelerometer
+          # Sensortek 3 axis accelerometer
           - sensortek,stk8312
-            # Sensortek 3 axis accelerometer
+          # Sensortek 3 axis accelerometer
           - sensortek,stk8ba50
-            # SGX Sensortech VZ89X Sensors
+          # SGX Sensortech VZ89X Sensors
           - sgx,vz89x
-            # Relative Humidity and Temperature Sensors
+          # Relative Humidity and Temperature Sensors
           - silabs,si7020
-            # Skyworks SKY81452: Six-Channel White LED Driver with Touch Panel Bias Supply
+          # Skyworks SKY81452: Six-Channel White LED Driver with Touch Panel Bias Supply
           - skyworks,sky81452
-            # Socionext SynQuacer TPM MMIO module
+          # Socionext SynQuacer TPM MMIO module
           - socionext,synquacer-tpm-mmio
-            # SparkFun Qwiic Joystick (COM-15168) with i2c interface
+          # SparkFun Qwiic Joystick (COM-15168) with i2c interface
           - sparkfun,qwiic-joystick
-            # i2c serial eeprom (24cxx)
+          # i2c serial eeprom (24cxx)
           - st,24c256
-            # Ambient Light Sensor with SMBUS/Two Wire Serial Interface
+          # Ambient Light Sensor with SMBUS/Two Wire Serial Interface
           - taos,tsl2550
-            # Temperature Monitoring and Fan Control
+          # Temperature Monitoring and Fan Control
           - ti,amc6821
-            # Temperature and humidity sensor with i2c interface
+          # Temperature and humidity sensor with i2c interface
           - ti,hdc1000
-            # Temperature and humidity sensor with i2c interface
+          # Temperature and humidity sensor with i2c interface
           - ti,hdc1008
-            # Temperature and humidity sensor with i2c interface
+          # Temperature and humidity sensor with i2c interface
           - ti,hdc1010
-            # Temperature and humidity sensor with i2c interface
+          # Temperature and humidity sensor with i2c interface
           - ti,hdc1050
-            # Temperature and humidity sensor with i2c interface
+          # Temperature and humidity sensor with i2c interface
           - ti,hdc1080
-            # Thermometer with SPI interface
+          # Thermometer with SPI interface
           - ti,lm70
           - ti,lm71
-            # Temperature sensor with 2-wire interface
+          # Temperature sensor with 2-wire interface
           - ti,lm73
-            # Thermometer with SPI interface
+          # Thermometer with SPI interface
           - ti,lm74
-            # Temperature sensor with integrated fan control
+          # Temperature sensor with integrated fan control
           - ti,lm96000
-            # I2C Touch-Screen Controller
+          # I2C Touch-Screen Controller
           - ti,tsc2003
-            # Low Power Digital Temperature Sensor with SMBUS/Two Wire Serial Interface
+          # Low Power Digital Temperature Sensor with SMBUS/Two Wire Serial Interface
           - ti,tmp103
-            # Thermometer with SPI interface
+          # Thermometer with SPI interface
           - ti,tmp121
           - ti,tmp122
-            # Digital Temperature Sensor
+          # Digital Temperature Sensor
           - ti,tmp275
-            # TI DC-DC converter on PMBus
+          # TI DC-DC converter on PMBus
           - ti,tps40400
-            # TI Dual channel DCAP+ multiphase controller TPS53676 with AVSBus
+          # TI Dual channel DCAP+ multiphase controller TPS53676 with AVSBus
           - ti,tps53676
-            # TI Dual channel DCAP+ multiphase controller TPS53679
+          # TI Dual channel DCAP+ multiphase controller TPS53679
           - ti,tps53679
-            # TI Dual channel DCAP+ multiphase controller TPS53688
+          # TI Dual channel DCAP+ multiphase controller TPS53688
           - ti,tps53688
-            # TI DC-DC converters on PMBus
+          # TI DC-DC converters on PMBus
           - ti,tps544b20
           - ti,tps544b25
           - ti,tps544c20
           - ti,tps544c25
-            # Winbond/Nuvoton H/W Monitor
+          # Winbond/Nuvoton H/W Monitor
           - winbond,w83793
-            # i2c trusted platform module (TPM)
+          # i2c trusted platform module (TPM)
           - winbond,wpct301
 
 required:
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 3/3] dt-bindings: trivial-devices: fix comment indentation
  2022-01-13 13:39 ` [PATCH 3/3] dt-bindings: trivial-devices: fix comment indentation Stanislav Jakubek
@ 2022-01-14 15:21   ` Rob Herring
  2022-01-14 16:50     ` Stanislav Jakubek
  0 siblings, 1 reply; 8+ messages in thread
From: Rob Herring @ 2022-01-14 15:21 UTC (permalink / raw)
  To: Stanislav Jakubek; +Cc: devicetree, linux-kernel

On Thu, Jan 13, 2022 at 02:39:40PM +0100, Stanislav Jakubek wrote:
> Indent comments the same as the content they describe.
> Fixes yamllint warnings like:
> 32:13     warning  comment not indented like content  (comments-indentation)

But that's not a warning we enable or is this something new in newer 
yamllint?

In any case, a change like this can only be done at the end of the merge 
window just before rc1. Otherwise, it is merge conflict hell. I would do 
that, but I really prefer it the way it is. The reason is it makes the 
'-' stand out more and those are easy to miss in yaml. For example, 
these look the same but are a bit different:

- items:
  - const: foo

- items:
    const: foo

> 
> Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com>
> ---
>  .../devicetree/bindings/trivial-devices.yaml  | 314 +++++++++---------
>  1 file changed, 157 insertions(+), 157 deletions(-)

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 3/3] dt-bindings: trivial-devices: fix comment indentation
  2022-01-14 15:21   ` Rob Herring
@ 2022-01-14 16:50     ` Stanislav Jakubek
  0 siblings, 0 replies; 8+ messages in thread
From: Stanislav Jakubek @ 2022-01-14 16:50 UTC (permalink / raw)
  To: Rob Herring; +Cc: devicetree, linux-kernel

On Fri, Jan 14, 2022 at 09:21:52AM -0600, Rob Herring wrote:
> On Thu, Jan 13, 2022 at 02:39:40PM +0100, Stanislav Jakubek wrote:
> > Indent comments the same as the content they describe.
> > Fixes yamllint warnings like:
> > 32:13     warning  comment not indented like content  (comments-indentation)
> 
> But that's not a warning we enable or is this something new in newer 
> yamllint?

Don't know, I just noticed that the comment indentation is inconsistent and
decided to fix that, but checkpatch wasn't complaining about it, so I ran it
through yamllint to check which one is "correct". I guess the default
yamllint config complains about it.

It also gave me errors about some line lengths being > 80, but I decided to
leave those alone. Not sure if it's worth it to break up the comments into
multiple lines for that.

Thanks,

Stanislav

> 
> In any case, a change like this can only be done at the end of the merge 
> window just before rc1. Otherwise, it is merge conflict hell. I would do 
> that, but I really prefer it the way it is. The reason is it makes the 
> '-' stand out more and those are easy to miss in yaml. For example, 
> these look the same but are a bit different:
> 
> - items:
>   - const: foo
> 
> - items:
>     const: foo
> 
> > 
> > Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com>
> > ---
> >  .../devicetree/bindings/trivial-devices.yaml  | 314 +++++++++---------
> >  1 file changed, 157 insertions(+), 157 deletions(-)

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2022-01-14 16:50 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-13 13:38 [PATCH 0/3] dt-bindings: trivial-devices: misc (whitespace) fixes Stanislav Jakubek
2022-01-13 13:39 ` [PATCH 1/3] dt-bindings: trivial-devices: fix swapped comments Stanislav Jakubek
2022-01-14 15:11   ` Rob Herring
2022-01-13 13:39 ` [PATCH 2/3] dt-bindings: trivial-devices: fix double spaces in comments Stanislav Jakubek
2022-01-14 15:11   ` Rob Herring
2022-01-13 13:39 ` [PATCH 3/3] dt-bindings: trivial-devices: fix comment indentation Stanislav Jakubek
2022-01-14 15:21   ` Rob Herring
2022-01-14 16:50     ` Stanislav Jakubek

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.