Devicetree
 help / color / mirror / Atom feed
From: Conor Dooley <conor@kernel.org>
To: Guenter Roeck <linux@roeck-us.net>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	Charles Hsu <ythsu0511@gmail.com>,
	linux-hwmon@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: trivial-devices: sort entries alphanumerically
Date: Thu, 1 Feb 2024 18:32:09 +0000	[thread overview]
Message-ID: <20240201-silliness-unfair-265a0d896377@spud> (raw)
In-Reply-To: <5461a237-1df4-4077-86ef-e9ff6ff17e27@roeck-us.net>

[-- Attachment #1: Type: text/plain, Size: 9824 bytes --]

On Thu, Feb 01, 2024 at 05:25:13AM -0800, Guenter Roeck wrote:
> On 1/31/24 23:58, Krzysztof Kozlowski wrote:
> > Sort entries alphanumerically.  This was a semi manual job with help of:
> > 
> >    cat Documentation/devicetree/bindings/trivial-devices.yaml | grep '    - ' > old
> >    cat old | sort -n > new
> >    diff -ubB old new
> > 
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > 
> 
> Acked-by: Guenter Roeck <linux@roeck-us.net>

z sorts before a, please fix in the whole file.

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
Conor.

> 
> > ---
> > 
> > Cc: Charles Hsu <ythsu0511@gmail.com>
> > Cc: linux-hwmon@vger.kernel.org
> > Cc: Guenter Roeck <linux@roeck-us.net>
> > ---
> >   .../devicetree/bindings/trivial-devices.yaml  | 73 ++++++++++---------
> >   1 file changed, 37 insertions(+), 36 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
> > index 5b52950e6bfc..41982a41398a 100644
> > --- a/Documentation/devicetree/bindings/trivial-devices.yaml
> > +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
> > @@ -28,6 +28,7 @@ properties:
> >     compatible:
> >       items:
> > +      # Entries are sorted alphanumerically by the compatible
> >         - enum:
> >               # Acbel fsg032 power supply
> >             - acbel,fsg032
> > @@ -49,12 +50,12 @@ properties:
> >             - ams,iaq-core
> >               # i2c serial eeprom (24cxx)
> >             - at,24c08
> > +            # i2c h/w elliptic curve crypto module
> > +          - atmel,atecc508a
> >               # ATSHA204 - i2c h/w symmetric crypto module
> >             - atmel,atsha204
> >               # ATSHA204A - i2c h/w symmetric crypto module
> >             - atmel,atsha204a
> > -            # i2c h/w elliptic curve crypto module
> > -          - atmel,atecc508a
> >               # BPA-RS600: Power Supply
> >             - blutek,bpa-rs600
> >               # Bosch Sensortec pressure, temperature, humididty and VOC sensor
> > @@ -115,20 +116,6 @@ properties:
> >             - fsl,mpl3115
> >               # MPR121: Proximity Capacitive Touch Sensor Controller
> >             - fsl,mpr121
> > -            # Monolithic Power Systems Inc. multi-phase controller mp2856
> > -          - mps,mp2856
> > -            # Monolithic Power Systems Inc. multi-phase controller mp2857
> > -          - mps,mp2857
> > -            # Monolithic Power Systems Inc. multi-phase controller mp2888
> > -          - mps,mp2888
> > -            # Monolithic Power Systems Inc. multi-phase controller mp2971
> > -          - mps,mp2971
> > -            # Monolithic Power Systems Inc. multi-phase controller mp2973
> > -          - mps,mp2973
> > -            # Monolithic Power Systems Inc. multi-phase controller mp2975
> > -          - mps,mp2975
> > -            # Monolithic Power Systems Inc. multi-phase hot-swap controller mp5990
> > -          - mps,mp5990
> >               # Honeywell Humidicon HIH-6130 humidity/temperature sensor
> >             - honeywell,hi6130
> >               # IBM Common Form Factor Power Supply Versions (all versions)
> > @@ -197,6 +184,8 @@ properties:
> >             - maxim,max1237
> >               # Temperature Sensor, I2C interface
> >             - maxim,max1619
> > +            # 3-Channel Remote Temperature Sensor
> > +          - maxim,max31730
> >               # 10-bit 10 kOhm linear programmable voltage divider
> >             - maxim,max5481
> >               # 10-bit 50 kOhm linear programmable voltage divider
> > @@ -209,8 +198,6 @@ properties:
> >             - maxim,max6621
> >               # 9-Bit/12-Bit Temperature Sensors with I²C-Compatible Serial Interface
> >             - maxim,max6625
> > -            # 3-Channel Remote Temperature Sensor
> > -          - maxim,max31730
> >               # mCube 3-axis 8-bit digital accelerometer
> >             - mcube,mc3230
> >               # Measurement Specialities I2C temperature and humidity sensor
> > @@ -241,8 +228,6 @@ properties:
> >             - memsic,mxc6655
> >               # Menlo on-board CPLD trivial SPI device
> >             - menlo,m53cpld
> > -            # Micron SPI NOR Authenta
> > -          - micron,spi-authenta
> >               # Microchip differential I2C ADC, 1 Channel, 18 bit
> >             - microchip,mcp3421
> >               # Microchip differential I2C ADC, 2 Channel, 18 bit
> > @@ -259,40 +244,56 @@ properties:
> >             - microchip,mcp3427
> >               # Microchip differential I2C ADC, 4 Channel, 16 bit
> >             - microchip,mcp3428
> > -            # Microchip 7-bit Single I2C Digital POT (5k)
> > -          - microchip,mcp4017-502
> >               # Microchip 7-bit Single I2C Digital POT (10k)
> >             - microchip,mcp4017-103
> > -            # Microchip 7-bit Single I2C Digital POT (50k)
> > -          - microchip,mcp4017-503
> >               # Microchip 7-bit Single I2C Digital POT (100k)
> >             - microchip,mcp4017-104
> >               # Microchip 7-bit Single I2C Digital POT (5k)
> > -          - microchip,mcp4018-502
> > +          - microchip,mcp4017-502
> > +            # Microchip 7-bit Single I2C Digital POT (50k)
> > +          - microchip,mcp4017-503
> >               # Microchip 7-bit Single I2C Digital POT (10k)
> >             - microchip,mcp4018-103
> > -            # Microchip 7-bit Single I2C Digital POT (50k)
> > -          - microchip,mcp4018-503
> >               # Microchip 7-bit Single I2C Digital POT (100k)
> >             - microchip,mcp4018-104
> >               # Microchip 7-bit Single I2C Digital POT (5k)
> > -          - microchip,mcp4019-502
> > +          - microchip,mcp4018-502
> > +            # Microchip 7-bit Single I2C Digital POT (50k)
> > +          - microchip,mcp4018-503
> >               # Microchip 7-bit Single I2C Digital POT (10k)
> >             - microchip,mcp4019-103
> > -            # Microchip 7-bit Single I2C Digital POT (50k)
> > -          - microchip,mcp4019-503
> >               # Microchip 7-bit Single I2C Digital POT (100k)
> >             - microchip,mcp4019-104
> > +            # Microchip 7-bit Single I2C Digital POT (5k)
> > +          - microchip,mcp4019-502
> > +            # Microchip 7-bit Single I2C Digital POT (50k)
> > +          - microchip,mcp4019-503
> >               # PWM Fan Speed Controller With Fan Fault Detection
> >             - microchip,tc654
> >               # PWM Fan Speed Controller With Fan Fault Detection
> >             - microchip,tc655
> > +            # Micron SPI NOR Authenta
> > +          - micron,spi-authenta
> >               # MiraMEMS DA226 2-axis 14-bit digital accelerometer
> >             - miramems,da226
> >               # MiraMEMS DA280 3-axis 14-bit digital accelerometer
> >             - miramems,da280
> >               # MiraMEMS DA311 3-axis 12-bit digital accelerometer
> >             - miramems,da311
> > +            # Monolithic Power Systems Inc. multi-phase controller mp2856
> > +          - mps,mp2856
> > +            # Monolithic Power Systems Inc. multi-phase controller mp2857
> > +          - mps,mp2857
> > +            # Monolithic Power Systems Inc. multi-phase controller mp2888
> > +          - mps,mp2888
> > +            # Monolithic Power Systems Inc. multi-phase controller mp2971
> > +          - mps,mp2971
> > +            # Monolithic Power Systems Inc. multi-phase controller mp2973
> > +          - mps,mp2973
> > +            # Monolithic Power Systems Inc. multi-phase controller mp2975
> > +          - mps,mp2975
> > +            # Monolithic Power Systems Inc. multi-phase hot-swap controller mp5990
> > +          - mps,mp5990
> >               # Temperature sensor with integrated fan control
> >             - national,lm63
> >               # Serial Interface ACPI-Compatible Microprocessor System Hardware Monitor
> > @@ -323,12 +324,12 @@ properties:
> >             - samsung,exynos-sataphy-i2c
> >               # Semtech sx1301 baseband processor
> >             - semtech,sx1301
> > -            # Sensirion low power multi-pixel gas sensor with I2C interface
> > -          - sensirion,sgpc3
> >               # Sensirion multi-pixel gas sensor with I2C interface
> >             - sensirion,sgp30
> >               # Sensirion gas sensor with I2C interface
> >             - sensirion,sgp40
> > +            # Sensirion low power multi-pixel gas sensor with I2C interface
> > +          - sensirion,sgpc3
> >               # Sensirion temperature & humidity sensor with I2C interface
> >             - sensirion,sht4x
> >               # Sensortek 3 axis accelerometer
> > @@ -374,8 +375,6 @@ properties:
> >             - ti,lm74
> >               # Temperature sensor with integrated fan control
> >             - ti,lm96000
> > -            # I2C Touch-Screen Controller
> > -          - ti,tsc2003
> >               # Low Power Digital Temperature Sensor with SMBUS/Two Wire Serial Interface
> >             - ti,tmp103
> >               # Thermometer with SPI interface
> > @@ -397,10 +396,12 @@ properties:
> >             - ti,tps544b25
> >             - ti,tps544c20
> >             - ti,tps544c25
> > -            # Winbond/Nuvoton H/W Monitor
> > -          - winbond,w83793
> > +            # I2C Touch-Screen Controller
> > +          - ti,tsc2003
> >               # Vicor Corporation Digital Supervisor
> >             - vicor,pli1209bc
> > +            # Winbond/Nuvoton H/W Monitor
> > +          - winbond,w83793
> >   required:
> >     - compatible
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

  reply	other threads:[~2024-02-01 18:32 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-01  7:58 [PATCH] dt-bindings: trivial-devices: sort entries alphanumerically Krzysztof Kozlowski
2024-02-01 13:25 ` Guenter Roeck
2024-02-01 18:32   ` Conor Dooley [this message]
2024-02-02 20:21     ` Rob Herring
2024-02-02 20:37       ` Conor Dooley
2024-02-05  9:29       ` Krzysztof Kozlowski
2024-02-05 17:20 ` Rob Herring
2024-02-05 18:33   ` Guenter Roeck

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=20240201-silliness-unfair-265a0d896377@spud \
    --to=conor@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linux-hwmon@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=robh@kernel.org \
    --cc=ythsu0511@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