* Re: [PATCH] dt-bindings: Cleanup standard unit properties
[not found] <20210128194515.743252-1-robh@kernel.org>
@ 2021-01-28 20:16 ` Guenter Roeck
2021-01-28 20:23 ` Wolfram Sang
2021-01-28 20:22 ` Mark Brown
` (2 subsequent siblings)
3 siblings, 1 reply; 5+ messages in thread
From: Guenter Roeck @ 2021-01-28 20:16 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Jean Delvare, Jonathan Cameron,
Lars-Peter Clausen, Alexandre Torgue, Dmitry Torokhov,
Ulf Hansson, David S. Miller, Jakub Kicinski, Sebastian Reichel,
Mark Brown, Alexandre Belloni, Greg Kroah-Hartman, Serge Semin,
Wolfram Sang, linux-hwmon, linux-i2c, linux-iio,
linux-arm-kernel, linux-input, linux-mmc, netdev, linux-pm,
linux-rtc, linux-serial, alsa-devel, linux-watchdog
On Thu, Jan 28, 2021 at 01:45:15PM -0600, Rob Herring wrote:
> Properties with standard unit suffixes already have a type and don't need
> type definitions. They also default to a single entry, so 'maxItems: 1'
> can be dropped.
>
> adi,ad5758 is an oddball which defined an enum of arrays. While a valid
> schema, it is simpler as a whole to only define scalar constraints.
>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Alexandre Torgue <alexandre.torgue@st.com>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Sebastian Reichel <sre@kernel.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Serge Semin <fancer.lancer@gmail.com>
> Cc: Wolfram Sang <wolfram@the-dreams.de>
> Cc: linux-hwmon@vger.kernel.org
Acked-by: Guenter Roeck <linux@roeck-us.net>
> Cc: linux-i2c@vger.kernel.org
> Cc: linux-iio@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-input@vger.kernel.org
> Cc: linux-mmc@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Cc: linux-pm@vger.kernel.org
> Cc: linux-rtc@vger.kernel.org
> Cc: linux-serial@vger.kernel.org
> Cc: alsa-devel@alsa-project.org
> Cc: linux-watchdog@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> .../devicetree/bindings/arm/cpus.yaml | 1 -
> .../bindings/extcon/wlf,arizona.yaml | 1 -
> .../bindings/hwmon/adi,ltc2947.yaml | 1 -
> .../bindings/hwmon/baikal,bt1-pvt.yaml | 8 ++--
> .../devicetree/bindings/hwmon/ti,tmp513.yaml | 1 -
> .../devicetree/bindings/i2c/i2c-gpio.yaml | 2 -
> .../bindings/i2c/snps,designware-i2c.yaml | 3 --
> .../bindings/iio/adc/maxim,max9611.yaml | 1 -
> .../bindings/iio/adc/st,stm32-adc.yaml | 1 -
> .../bindings/iio/adc/ti,palmas-gpadc.yaml | 2 -
> .../bindings/iio/dac/adi,ad5758.yaml | 41 ++++++++++++-------
> .../bindings/iio/health/maxim,max30100.yaml | 1 -
> .../input/touchscreen/touchscreen.yaml | 2 -
> .../bindings/mmc/mmc-controller.yaml | 1 -
> .../bindings/mmc/mmc-pwrseq-simple.yaml | 2 -
> .../bindings/net/ethernet-controller.yaml | 2 -
> .../devicetree/bindings/net/snps,dwmac.yaml | 1 -
> .../bindings/power/supply/battery.yaml | 3 --
> .../bindings/power/supply/bq2515x.yaml | 1 -
> .../bindings/regulator/dlg,da9121.yaml | 1 -
> .../bindings/regulator/fixed-regulator.yaml | 2 -
> .../devicetree/bindings/rtc/rtc.yaml | 2 -
> .../devicetree/bindings/serial/pl011.yaml | 2 -
> .../devicetree/bindings/sound/sgtl5000.yaml | 2 -
> .../bindings/watchdog/watchdog.yaml | 1 -
> 25 files changed, 29 insertions(+), 56 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
> index 14cd727d3c4b..f02fd10de604 100644
> --- a/Documentation/devicetree/bindings/arm/cpus.yaml
> +++ b/Documentation/devicetree/bindings/arm/cpus.yaml
> @@ -232,7 +232,6 @@ properties:
> by this cpu (see ./idle-states.yaml).
>
> capacity-dmips-mhz:
> - $ref: '/schemas/types.yaml#/definitions/uint32'
> description:
> u32 value representing CPU capacity (see ./cpu-capacity.txt) in
> DMIPS/MHz, relative to highest capacity-dmips-mhz
> diff --git a/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml b/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml
> index 5fe784f487c5..efdf59abb2e1 100644
> --- a/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml
> +++ b/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml
> @@ -85,7 +85,6 @@ properties:
> wlf,micd-timeout-ms:
> description:
> Timeout for microphone detection, specified in milliseconds.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
>
> wlf,micd-force-micbias:
> description:
> diff --git a/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml b/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> index eef614962b10..bf04151b63d2 100644
> --- a/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> @@ -49,7 +49,6 @@ properties:
> description:
> This property controls the Accumulation Dead band which allows to set the
> level of current below which no accumulation takes place.
> - $ref: /schemas/types.yaml#/definitions/uint32
> maximum: 255
> default: 0
>
> diff --git a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> index 00a6511354e6..5d3ce641fcde 100644
> --- a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> @@ -73,11 +73,9 @@ properties:
> description: |
> Temperature sensor trimming factor. It can be used to manually adjust the
> temperature measurements within 7.130 degrees Celsius.
> - maxItems: 1
> - items:
> - default: 0
> - minimum: 0
> - maximum: 7130
> + default: 0
> + minimum: 0
> + maximum: 7130
>
> additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml b/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml
> index 8020d739a078..1502b22c77cc 100644
> --- a/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml
> @@ -52,7 +52,6 @@ properties:
> ti,bus-range-microvolt:
> description: |
> This is the operating range of the bus voltage in microvolt
> - $ref: /schemas/types.yaml#/definitions/uint32
> enum: [16000000, 32000000]
> default: 32000000
>
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml b/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml
> index cc3aa2a5e70b..ff99344788ab 100644
> --- a/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml
> +++ b/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml
> @@ -39,11 +39,9 @@ properties:
>
> i2c-gpio,delay-us:
> description: delay between GPIO operations (may depend on each platform)
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> i2c-gpio,timeout-ms:
> description: timeout to get data
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> # Deprecated properties, do not use in new device tree sources:
> gpios:
> diff --git a/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml b/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml
> index c22b66b6219e..d9293c57f573 100644
> --- a/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml
> +++ b/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml
> @@ -66,21 +66,18 @@ properties:
> default: 400000
>
> i2c-sda-hold-time-ns:
> - maxItems: 1
> description: |
> The property should contain the SDA hold time in nanoseconds. This option
> is only supported in hardware blocks version 1.11a or newer or on
> Microsemi SoCs.
>
> i2c-scl-falling-time-ns:
> - maxItems: 1
> description: |
> The property should contain the SCL falling time in nanoseconds.
> This value is used to compute the tLOW period.
> default: 300
>
> i2c-sda-falling-time-ns:
> - maxItems: 1
> description: |
> The property should contain the SDA falling time in nanoseconds.
> This value is used to compute the tHIGH period.
> diff --git a/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml b/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml
> index 9475a9e6e920..95774a55629d 100644
> --- a/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml
> @@ -23,7 +23,6 @@ properties:
> maxItems: 1
>
> shunt-resistor-micro-ohms:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> Value in micro Ohms of the shunt resistor connected between the RS+ and
> RS- inputs, across which the current is measured. Value needed to compute
> diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> index 6364ede9bb5f..a58334c3bb76 100644
> --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> @@ -248,7 +248,6 @@ patternProperties:
> Resolution (bits) to use for conversions:
> - can be 6, 8, 10 or 12 on stm32f4
> - can be 8, 10, 12, 14 or 16 on stm32h7 and stm32mp1
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> st,adc-channels:
> description: |
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml
> index 692dacd0fee5..7b895784e008 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml
> @@ -42,7 +42,6 @@ properties:
> const: 1
>
> ti,channel0-current-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: Channel 0 current in uA.
> enum:
> - 0
> @@ -51,7 +50,6 @@ properties:
> - 20
>
> ti,channel3-current-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: Channel 3 current in uA.
> enum:
> - 0
> diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml
> index 626ccb6fe21e..fd4edca34a28 100644
> --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml
> @@ -46,31 +46,42 @@ properties:
> two properties must be present:
>
> adi,range-microvolt:
> - $ref: /schemas/types.yaml#/definitions/int32-array
> description: |
> Voltage output range specified as <minimum, maximum>
> - enum:
> - - [[0, 5000000]]
> - - [[0, 10000000]]
> - - [[-5000000, 5000000]]
> - - [[-10000000, 10000000]]
> + oneOf:
> + - items:
> + - const: 0
> + - enum: [5000000, 10000000]
> + - items:
> + - const: -5000000
> + - const: 5000000
> + - items:
> + - const: -10000000
> + - const: 10000000
>
> adi,range-microamp:
> - $ref: /schemas/types.yaml#/definitions/int32-array
> description: |
> Current output range specified as <minimum, maximum>
> - enum:
> - - [[0, 20000]]
> - - [[0, 24000]]
> - - [[4, 24000]]
> - - [[-20000, 20000]]
> - - [[-24000, 24000]]
> - - [[-1000, 22000]]
> + oneOf:
> + - items:
> + - const: 0
> + - enum: [20000, 24000]
> + - items:
> + - const: 4
> + - const: 24000
> + - items:
> + - const: -20000
> + - const: 20000
> + - items:
> + - const: -24000
> + - const: 24000
> + - items:
> + - const: -1000
> + - const: 22000
>
> reset-gpios: true
>
> adi,dc-dc-ilim-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> enum: [150000, 200000, 250000, 300000, 350000, 400000]
> description: |
> The dc-to-dc converter current limit.
> diff --git a/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml b/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml
> index 64b862637039..967778fb0ce8 100644
> --- a/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml
> +++ b/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml
> @@ -21,7 +21,6 @@ properties:
> description: Connected to ADC_RDY pin.
>
> maxim,led-current-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> minItems: 2
> maxItems: 2
> description: |
> diff --git a/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml b/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> index a771a15f053f..046ace461cc9 100644
> --- a/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> +++ b/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> @@ -70,11 +70,9 @@ properties:
>
> touchscreen-x-mm:
> description: horizontal length in mm of the touchscreen
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> touchscreen-y-mm:
> description: vertical length in mm of the touchscreen
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> dependencies:
> touchscreen-size-x: [ touchscreen-size-y ]
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-controller.yaml b/Documentation/devicetree/bindings/mmc/mmc-controller.yaml
> index df4ee4c778ae..e141330c1114 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-controller.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-controller.yaml
> @@ -261,7 +261,6 @@ properties:
> waiting for I/O signalling and card power supply to be stable,
> regardless of whether pwrseq-simple is used. Default to 10ms if
> no available.
> - $ref: /schemas/types.yaml#/definitions/uint32
> default: 10
>
> supports-cqe:
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> index 6cd57863c1db..226fb191913d 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> @@ -41,13 +41,11 @@ properties:
> description:
> Delay in ms after powering the card and de-asserting the
> reset-gpios (if any).
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> power-off-delay-us:
> description:
> Delay in us after asserting the reset-gpios (if any)
> during power off of the card.
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/net/ethernet-controller.yaml b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> index 0965f6515f9e..dac4aadb6e2e 100644
> --- a/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> +++ b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> @@ -122,7 +122,6 @@ properties:
> such as flow control thresholds.
>
> rx-internal-delay-ps:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> RGMII Receive Clock Delay defined in pico seconds.
> This is used for controllers that have configurable RX internal delays.
> @@ -140,7 +139,6 @@ properties:
> is used for components that can have configurable fifo sizes.
>
> tx-internal-delay-ps:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> RGMII Transmit Clock Delay defined in pico seconds.
> This is used for controllers that have configurable TX internal delays.
> diff --git a/Documentation/devicetree/bindings/net/snps,dwmac.yaml b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> index b2f6083f556a..9ac77b8cb767 100644
> --- a/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> @@ -208,7 +208,6 @@ properties:
> Triplet of delays. The 1st cell is reset pre-delay in micro
> seconds. The 2nd cell is reset pulse in micro seconds. The 3rd
> cell is reset post-delay in micro seconds.
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> minItems: 3
> maxItems: 3
>
> diff --git a/Documentation/devicetree/bindings/power/supply/battery.yaml b/Documentation/devicetree/bindings/power/supply/battery.yaml
> index 0c7e2e44793b..c3b4b7543591 100644
> --- a/Documentation/devicetree/bindings/power/supply/battery.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/battery.yaml
> @@ -83,21 +83,18 @@ properties:
> for each of the battery capacity lookup table.
>
> operating-range-celsius:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> description: operating temperature range of a battery
> items:
> - description: minimum temperature at which battery can operate
> - description: maximum temperature at which battery can operate
>
> ambient-celsius:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> description: safe range of ambient temperature
> items:
> - description: alert when ambient temperature is lower than this value
> - description: alert when ambient temperature is higher than this value
>
> alert-celsius:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> description: safe range of battery temperature
> items:
> - description: alert when battery temperature is lower than this value
> diff --git a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> index 75a56773be4a..813d6afde606 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> @@ -50,7 +50,6 @@ properties:
> maxItems: 1
>
> input-current-limit-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: Maximum input current in micro Amps.
> minimum: 50000
> maximum: 500000
> diff --git a/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml b/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml
> index 6f2164f7bc57..228018c87bea 100644
> --- a/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml
> +++ b/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml
> @@ -62,7 +62,6 @@ properties:
> description: IRQ line information.
>
> dlg,irq-polling-delay-passive-ms:
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> minimum: 1000
> maximum: 10000
> description: |
> diff --git a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
> index d3d0dc13dd8b..8850c01bd470 100644
> --- a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
> @@ -72,11 +72,9 @@ properties:
>
> startup-delay-us:
> description: startup time in microseconds
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> off-on-delay-us:
> description: off delay time in microseconds
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> enable-active-high:
> description:
> diff --git a/Documentation/devicetree/bindings/rtc/rtc.yaml b/Documentation/devicetree/bindings/rtc/rtc.yaml
> index d30dc045aac6..0ec3551f12dd 100644
> --- a/Documentation/devicetree/bindings/rtc/rtc.yaml
> +++ b/Documentation/devicetree/bindings/rtc/rtc.yaml
> @@ -27,7 +27,6 @@ properties:
> 1: chargeable
>
> quartz-load-femtofarads:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description:
> The capacitive load of the quartz(x-tal), expressed in femto
> Farad (fF). The default value shall be listed (if optional),
> @@ -47,7 +46,6 @@ properties:
> deprecated: true
>
> trickle-resistor-ohms:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description:
> Selected resistor for trickle charger. Should be given
> if trickle charger should be enabled.
> diff --git a/Documentation/devicetree/bindings/serial/pl011.yaml b/Documentation/devicetree/bindings/serial/pl011.yaml
> index c23c93b400f0..07fa6d26f2b4 100644
> --- a/Documentation/devicetree/bindings/serial/pl011.yaml
> +++ b/Documentation/devicetree/bindings/serial/pl011.yaml
> @@ -88,14 +88,12 @@ properties:
> description:
> Rate at which poll occurs when auto-poll is set.
> default 100ms.
> - $ref: /schemas/types.yaml#/definitions/uint32
> default: 100
>
> poll-timeout-ms:
> description:
> Poll timeout when auto-poll is set, default
> 3000ms.
> - $ref: /schemas/types.yaml#/definitions/uint32
> default: 3000
>
> required:
> diff --git a/Documentation/devicetree/bindings/sound/sgtl5000.yaml b/Documentation/devicetree/bindings/sound/sgtl5000.yaml
> index d116c174b545..70b4a8831073 100644
> --- a/Documentation/devicetree/bindings/sound/sgtl5000.yaml
> +++ b/Documentation/devicetree/bindings/sound/sgtl5000.yaml
> @@ -41,14 +41,12 @@ properties:
> values of 2k, 4k or 8k. If set to 0 it will be off. If this node is not
> mentioned or if the value is unknown, then micbias resistor is set to
> 4k.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> enum: [ 0, 2, 4, 8 ]
>
> micbias-voltage-m-volts:
> description: The bias voltage to be used in mVolts. The voltage can take
> values from 1.25V to 3V by 250mV steps. If this node is not mentioned
> or the value is unknown, then the value is set to 1.25V.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> enum: [ 1250, 1500, 1750, 2000, 2250, 2500, 2750, 3000 ]
>
> lrclk-strength:
> diff --git a/Documentation/devicetree/bindings/watchdog/watchdog.yaml b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> index 4e2c26cd981d..e3dfb02f0ca5 100644
> --- a/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> @@ -19,7 +19,6 @@ properties:
> pattern: "^watchdog(@.*|-[0-9a-f])?$"
>
> timeout-sec:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description:
> Contains the watchdog timeout in seconds.
>
> --
> 2.27.0
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] dt-bindings: Cleanup standard unit properties
2021-01-28 20:16 ` [PATCH] dt-bindings: Cleanup standard unit properties Guenter Roeck
@ 2021-01-28 20:23 ` Wolfram Sang
0 siblings, 0 replies; 5+ messages in thread
From: Wolfram Sang @ 2021-01-28 20:23 UTC (permalink / raw)
To: Guenter Roeck
Cc: Rob Herring, devicetree, linux-kernel, Jean Delvare,
Jonathan Cameron, Lars-Peter Clausen, Alexandre Torgue,
Dmitry Torokhov, Ulf Hansson, David S. Miller, Jakub Kicinski,
Sebastian Reichel, Mark Brown, Alexandre Belloni,
Greg Kroah-Hartman, Serge Semin, Wolfram Sang, linux-hwmon,
linux-i2c, linux-iio, linux-arm-kernel, linux-input, linux-mmc,
netdev, linux-pm, linux-rtc, linux-serial, alsa-devel,
linux-watchdog
[-- Attachment #1: Type: text/plain, Size: 516 bytes --]
On Thu, Jan 28, 2021 at 12:16:14PM -0800, Guenter Roeck wrote:
> On Thu, Jan 28, 2021 at 01:45:15PM -0600, Rob Herring wrote:
> > Properties with standard unit suffixes already have a type and don't need
> > type definitions. They also default to a single entry, so 'maxItems: 1'
> > can be dropped.
> >
> > adi,ad5758 is an oddball which defined an enum of arrays. While a valid
> > schema, it is simpler as a whole to only define scalar constraints.
Acked-by: Wolfram Sang <wsa@kernel.org> # for I2C
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: Cleanup standard unit properties
[not found] <20210128194515.743252-1-robh@kernel.org>
2021-01-28 20:16 ` [PATCH] dt-bindings: Cleanup standard unit properties Guenter Roeck
@ 2021-01-28 20:22 ` Mark Brown
2021-01-28 20:54 ` Alexandre Belloni
2021-01-29 16:23 ` Alexandre Torgue
3 siblings, 0 replies; 5+ messages in thread
From: Mark Brown @ 2021-01-28 20:22 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Jean Delvare, Guenter Roeck,
Jonathan Cameron, Lars-Peter Clausen, Alexandre Torgue,
Dmitry Torokhov, Ulf Hansson, David S. Miller, Jakub Kicinski,
Sebastian Reichel, Alexandre Belloni, Greg Kroah-Hartman,
Serge Semin, Wolfram Sang, linux-hwmon, linux-i2c, linux-iio,
linux-arm-kernel, linux-input, linux-mmc, netdev, linux-pm,
linux-rtc, linux-serial, alsa-devel, linux-watchdog
[-- Attachment #1: Type: text/plain, Size: 272 bytes --]
On Thu, Jan 28, 2021 at 01:45:15PM -0600, Rob Herring wrote:
> Properties with standard unit suffixes already have a type and don't need
> type definitions. They also default to a single entry, so 'maxItems: 1'
> can be dropped.
Acked-by: Mark Brown <broonie@kernel.org>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] dt-bindings: Cleanup standard unit properties
[not found] <20210128194515.743252-1-robh@kernel.org>
2021-01-28 20:16 ` [PATCH] dt-bindings: Cleanup standard unit properties Guenter Roeck
2021-01-28 20:22 ` Mark Brown
@ 2021-01-28 20:54 ` Alexandre Belloni
2021-01-29 16:23 ` Alexandre Torgue
3 siblings, 0 replies; 5+ messages in thread
From: Alexandre Belloni @ 2021-01-28 20:54 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Jean Delvare, Guenter Roeck,
Jonathan Cameron, Lars-Peter Clausen, Alexandre Torgue,
Dmitry Torokhov, Ulf Hansson, David S. Miller, Jakub Kicinski,
Sebastian Reichel, Mark Brown, Greg Kroah-Hartman, Serge Semin,
Wolfram Sang, linux-hwmon, linux-i2c, linux-iio,
linux-arm-kernel, linux-input, linux-mmc, netdev, linux-pm,
linux-rtc, linux-serial, alsa-devel, linux-watchdog
On 28/01/2021 13:45:15-0600, Rob Herring wrote:
> Properties with standard unit suffixes already have a type and don't need
> type definitions. They also default to a single entry, so 'maxItems: 1'
> can be dropped.
>
> adi,ad5758 is an oddball which defined an enum of arrays. While a valid
> schema, it is simpler as a whole to only define scalar constraints.
>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Alexandre Torgue <alexandre.torgue@st.com>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Sebastian Reichel <sre@kernel.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Serge Semin <fancer.lancer@gmail.com>
> Cc: Wolfram Sang <wolfram@the-dreams.de>
> Cc: linux-hwmon@vger.kernel.org
> Cc: linux-i2c@vger.kernel.org
> Cc: linux-iio@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-input@vger.kernel.org
> Cc: linux-mmc@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Cc: linux-pm@vger.kernel.org
> Cc: linux-rtc@vger.kernel.org
> Cc: linux-serial@vger.kernel.org
> Cc: alsa-devel@alsa-project.org
> Cc: linux-watchdog@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> ---
> .../devicetree/bindings/arm/cpus.yaml | 1 -
> .../bindings/extcon/wlf,arizona.yaml | 1 -
> .../bindings/hwmon/adi,ltc2947.yaml | 1 -
> .../bindings/hwmon/baikal,bt1-pvt.yaml | 8 ++--
> .../devicetree/bindings/hwmon/ti,tmp513.yaml | 1 -
> .../devicetree/bindings/i2c/i2c-gpio.yaml | 2 -
> .../bindings/i2c/snps,designware-i2c.yaml | 3 --
> .../bindings/iio/adc/maxim,max9611.yaml | 1 -
> .../bindings/iio/adc/st,stm32-adc.yaml | 1 -
> .../bindings/iio/adc/ti,palmas-gpadc.yaml | 2 -
> .../bindings/iio/dac/adi,ad5758.yaml | 41 ++++++++++++-------
> .../bindings/iio/health/maxim,max30100.yaml | 1 -
> .../input/touchscreen/touchscreen.yaml | 2 -
> .../bindings/mmc/mmc-controller.yaml | 1 -
> .../bindings/mmc/mmc-pwrseq-simple.yaml | 2 -
> .../bindings/net/ethernet-controller.yaml | 2 -
> .../devicetree/bindings/net/snps,dwmac.yaml | 1 -
> .../bindings/power/supply/battery.yaml | 3 --
> .../bindings/power/supply/bq2515x.yaml | 1 -
> .../bindings/regulator/dlg,da9121.yaml | 1 -
> .../bindings/regulator/fixed-regulator.yaml | 2 -
> .../devicetree/bindings/rtc/rtc.yaml | 2 -
> .../devicetree/bindings/serial/pl011.yaml | 2 -
> .../devicetree/bindings/sound/sgtl5000.yaml | 2 -
> .../bindings/watchdog/watchdog.yaml | 1 -
> 25 files changed, 29 insertions(+), 56 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
> index 14cd727d3c4b..f02fd10de604 100644
> --- a/Documentation/devicetree/bindings/arm/cpus.yaml
> +++ b/Documentation/devicetree/bindings/arm/cpus.yaml
> @@ -232,7 +232,6 @@ properties:
> by this cpu (see ./idle-states.yaml).
>
> capacity-dmips-mhz:
> - $ref: '/schemas/types.yaml#/definitions/uint32'
> description:
> u32 value representing CPU capacity (see ./cpu-capacity.txt) in
> DMIPS/MHz, relative to highest capacity-dmips-mhz
> diff --git a/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml b/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml
> index 5fe784f487c5..efdf59abb2e1 100644
> --- a/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml
> +++ b/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml
> @@ -85,7 +85,6 @@ properties:
> wlf,micd-timeout-ms:
> description:
> Timeout for microphone detection, specified in milliseconds.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
>
> wlf,micd-force-micbias:
> description:
> diff --git a/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml b/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> index eef614962b10..bf04151b63d2 100644
> --- a/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> @@ -49,7 +49,6 @@ properties:
> description:
> This property controls the Accumulation Dead band which allows to set the
> level of current below which no accumulation takes place.
> - $ref: /schemas/types.yaml#/definitions/uint32
> maximum: 255
> default: 0
>
> diff --git a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> index 00a6511354e6..5d3ce641fcde 100644
> --- a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> @@ -73,11 +73,9 @@ properties:
> description: |
> Temperature sensor trimming factor. It can be used to manually adjust the
> temperature measurements within 7.130 degrees Celsius.
> - maxItems: 1
> - items:
> - default: 0
> - minimum: 0
> - maximum: 7130
> + default: 0
> + minimum: 0
> + maximum: 7130
>
> additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml b/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml
> index 8020d739a078..1502b22c77cc 100644
> --- a/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml
> @@ -52,7 +52,6 @@ properties:
> ti,bus-range-microvolt:
> description: |
> This is the operating range of the bus voltage in microvolt
> - $ref: /schemas/types.yaml#/definitions/uint32
> enum: [16000000, 32000000]
> default: 32000000
>
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml b/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml
> index cc3aa2a5e70b..ff99344788ab 100644
> --- a/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml
> +++ b/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml
> @@ -39,11 +39,9 @@ properties:
>
> i2c-gpio,delay-us:
> description: delay between GPIO operations (may depend on each platform)
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> i2c-gpio,timeout-ms:
> description: timeout to get data
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> # Deprecated properties, do not use in new device tree sources:
> gpios:
> diff --git a/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml b/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml
> index c22b66b6219e..d9293c57f573 100644
> --- a/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml
> +++ b/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml
> @@ -66,21 +66,18 @@ properties:
> default: 400000
>
> i2c-sda-hold-time-ns:
> - maxItems: 1
> description: |
> The property should contain the SDA hold time in nanoseconds. This option
> is only supported in hardware blocks version 1.11a or newer or on
> Microsemi SoCs.
>
> i2c-scl-falling-time-ns:
> - maxItems: 1
> description: |
> The property should contain the SCL falling time in nanoseconds.
> This value is used to compute the tLOW period.
> default: 300
>
> i2c-sda-falling-time-ns:
> - maxItems: 1
> description: |
> The property should contain the SDA falling time in nanoseconds.
> This value is used to compute the tHIGH period.
> diff --git a/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml b/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml
> index 9475a9e6e920..95774a55629d 100644
> --- a/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml
> @@ -23,7 +23,6 @@ properties:
> maxItems: 1
>
> shunt-resistor-micro-ohms:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> Value in micro Ohms of the shunt resistor connected between the RS+ and
> RS- inputs, across which the current is measured. Value needed to compute
> diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> index 6364ede9bb5f..a58334c3bb76 100644
> --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> @@ -248,7 +248,6 @@ patternProperties:
> Resolution (bits) to use for conversions:
> - can be 6, 8, 10 or 12 on stm32f4
> - can be 8, 10, 12, 14 or 16 on stm32h7 and stm32mp1
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> st,adc-channels:
> description: |
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml
> index 692dacd0fee5..7b895784e008 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml
> @@ -42,7 +42,6 @@ properties:
> const: 1
>
> ti,channel0-current-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: Channel 0 current in uA.
> enum:
> - 0
> @@ -51,7 +50,6 @@ properties:
> - 20
>
> ti,channel3-current-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: Channel 3 current in uA.
> enum:
> - 0
> diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml
> index 626ccb6fe21e..fd4edca34a28 100644
> --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml
> @@ -46,31 +46,42 @@ properties:
> two properties must be present:
>
> adi,range-microvolt:
> - $ref: /schemas/types.yaml#/definitions/int32-array
> description: |
> Voltage output range specified as <minimum, maximum>
> - enum:
> - - [[0, 5000000]]
> - - [[0, 10000000]]
> - - [[-5000000, 5000000]]
> - - [[-10000000, 10000000]]
> + oneOf:
> + - items:
> + - const: 0
> + - enum: [5000000, 10000000]
> + - items:
> + - const: -5000000
> + - const: 5000000
> + - items:
> + - const: -10000000
> + - const: 10000000
>
> adi,range-microamp:
> - $ref: /schemas/types.yaml#/definitions/int32-array
> description: |
> Current output range specified as <minimum, maximum>
> - enum:
> - - [[0, 20000]]
> - - [[0, 24000]]
> - - [[4, 24000]]
> - - [[-20000, 20000]]
> - - [[-24000, 24000]]
> - - [[-1000, 22000]]
> + oneOf:
> + - items:
> + - const: 0
> + - enum: [20000, 24000]
> + - items:
> + - const: 4
> + - const: 24000
> + - items:
> + - const: -20000
> + - const: 20000
> + - items:
> + - const: -24000
> + - const: 24000
> + - items:
> + - const: -1000
> + - const: 22000
>
> reset-gpios: true
>
> adi,dc-dc-ilim-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> enum: [150000, 200000, 250000, 300000, 350000, 400000]
> description: |
> The dc-to-dc converter current limit.
> diff --git a/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml b/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml
> index 64b862637039..967778fb0ce8 100644
> --- a/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml
> +++ b/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml
> @@ -21,7 +21,6 @@ properties:
> description: Connected to ADC_RDY pin.
>
> maxim,led-current-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> minItems: 2
> maxItems: 2
> description: |
> diff --git a/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml b/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> index a771a15f053f..046ace461cc9 100644
> --- a/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> +++ b/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> @@ -70,11 +70,9 @@ properties:
>
> touchscreen-x-mm:
> description: horizontal length in mm of the touchscreen
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> touchscreen-y-mm:
> description: vertical length in mm of the touchscreen
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> dependencies:
> touchscreen-size-x: [ touchscreen-size-y ]
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-controller.yaml b/Documentation/devicetree/bindings/mmc/mmc-controller.yaml
> index df4ee4c778ae..e141330c1114 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-controller.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-controller.yaml
> @@ -261,7 +261,6 @@ properties:
> waiting for I/O signalling and card power supply to be stable,
> regardless of whether pwrseq-simple is used. Default to 10ms if
> no available.
> - $ref: /schemas/types.yaml#/definitions/uint32
> default: 10
>
> supports-cqe:
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> index 6cd57863c1db..226fb191913d 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> @@ -41,13 +41,11 @@ properties:
> description:
> Delay in ms after powering the card and de-asserting the
> reset-gpios (if any).
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> power-off-delay-us:
> description:
> Delay in us after asserting the reset-gpios (if any)
> during power off of the card.
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/net/ethernet-controller.yaml b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> index 0965f6515f9e..dac4aadb6e2e 100644
> --- a/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> +++ b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> @@ -122,7 +122,6 @@ properties:
> such as flow control thresholds.
>
> rx-internal-delay-ps:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> RGMII Receive Clock Delay defined in pico seconds.
> This is used for controllers that have configurable RX internal delays.
> @@ -140,7 +139,6 @@ properties:
> is used for components that can have configurable fifo sizes.
>
> tx-internal-delay-ps:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> RGMII Transmit Clock Delay defined in pico seconds.
> This is used for controllers that have configurable TX internal delays.
> diff --git a/Documentation/devicetree/bindings/net/snps,dwmac.yaml b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> index b2f6083f556a..9ac77b8cb767 100644
> --- a/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> @@ -208,7 +208,6 @@ properties:
> Triplet of delays. The 1st cell is reset pre-delay in micro
> seconds. The 2nd cell is reset pulse in micro seconds. The 3rd
> cell is reset post-delay in micro seconds.
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> minItems: 3
> maxItems: 3
>
> diff --git a/Documentation/devicetree/bindings/power/supply/battery.yaml b/Documentation/devicetree/bindings/power/supply/battery.yaml
> index 0c7e2e44793b..c3b4b7543591 100644
> --- a/Documentation/devicetree/bindings/power/supply/battery.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/battery.yaml
> @@ -83,21 +83,18 @@ properties:
> for each of the battery capacity lookup table.
>
> operating-range-celsius:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> description: operating temperature range of a battery
> items:
> - description: minimum temperature at which battery can operate
> - description: maximum temperature at which battery can operate
>
> ambient-celsius:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> description: safe range of ambient temperature
> items:
> - description: alert when ambient temperature is lower than this value
> - description: alert when ambient temperature is higher than this value
>
> alert-celsius:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> description: safe range of battery temperature
> items:
> - description: alert when battery temperature is lower than this value
> diff --git a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> index 75a56773be4a..813d6afde606 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> @@ -50,7 +50,6 @@ properties:
> maxItems: 1
>
> input-current-limit-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: Maximum input current in micro Amps.
> minimum: 50000
> maximum: 500000
> diff --git a/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml b/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml
> index 6f2164f7bc57..228018c87bea 100644
> --- a/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml
> +++ b/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml
> @@ -62,7 +62,6 @@ properties:
> description: IRQ line information.
>
> dlg,irq-polling-delay-passive-ms:
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> minimum: 1000
> maximum: 10000
> description: |
> diff --git a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
> index d3d0dc13dd8b..8850c01bd470 100644
> --- a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
> @@ -72,11 +72,9 @@ properties:
>
> startup-delay-us:
> description: startup time in microseconds
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> off-on-delay-us:
> description: off delay time in microseconds
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> enable-active-high:
> description:
> diff --git a/Documentation/devicetree/bindings/rtc/rtc.yaml b/Documentation/devicetree/bindings/rtc/rtc.yaml
> index d30dc045aac6..0ec3551f12dd 100644
> --- a/Documentation/devicetree/bindings/rtc/rtc.yaml
> +++ b/Documentation/devicetree/bindings/rtc/rtc.yaml
> @@ -27,7 +27,6 @@ properties:
> 1: chargeable
>
> quartz-load-femtofarads:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description:
> The capacitive load of the quartz(x-tal), expressed in femto
> Farad (fF). The default value shall be listed (if optional),
> @@ -47,7 +46,6 @@ properties:
> deprecated: true
>
> trickle-resistor-ohms:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description:
> Selected resistor for trickle charger. Should be given
> if trickle charger should be enabled.
> diff --git a/Documentation/devicetree/bindings/serial/pl011.yaml b/Documentation/devicetree/bindings/serial/pl011.yaml
> index c23c93b400f0..07fa6d26f2b4 100644
> --- a/Documentation/devicetree/bindings/serial/pl011.yaml
> +++ b/Documentation/devicetree/bindings/serial/pl011.yaml
> @@ -88,14 +88,12 @@ properties:
> description:
> Rate at which poll occurs when auto-poll is set.
> default 100ms.
> - $ref: /schemas/types.yaml#/definitions/uint32
> default: 100
>
> poll-timeout-ms:
> description:
> Poll timeout when auto-poll is set, default
> 3000ms.
> - $ref: /schemas/types.yaml#/definitions/uint32
> default: 3000
>
> required:
> diff --git a/Documentation/devicetree/bindings/sound/sgtl5000.yaml b/Documentation/devicetree/bindings/sound/sgtl5000.yaml
> index d116c174b545..70b4a8831073 100644
> --- a/Documentation/devicetree/bindings/sound/sgtl5000.yaml
> +++ b/Documentation/devicetree/bindings/sound/sgtl5000.yaml
> @@ -41,14 +41,12 @@ properties:
> values of 2k, 4k or 8k. If set to 0 it will be off. If this node is not
> mentioned or if the value is unknown, then micbias resistor is set to
> 4k.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> enum: [ 0, 2, 4, 8 ]
>
> micbias-voltage-m-volts:
> description: The bias voltage to be used in mVolts. The voltage can take
> values from 1.25V to 3V by 250mV steps. If this node is not mentioned
> or the value is unknown, then the value is set to 1.25V.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> enum: [ 1250, 1500, 1750, 2000, 2250, 2500, 2750, 3000 ]
>
> lrclk-strength:
> diff --git a/Documentation/devicetree/bindings/watchdog/watchdog.yaml b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> index 4e2c26cd981d..e3dfb02f0ca5 100644
> --- a/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> @@ -19,7 +19,6 @@ properties:
> pattern: "^watchdog(@.*|-[0-9a-f])?$"
>
> timeout-sec:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description:
> Contains the watchdog timeout in seconds.
>
> --
> 2.27.0
>
--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] dt-bindings: Cleanup standard unit properties
[not found] <20210128194515.743252-1-robh@kernel.org>
` (2 preceding siblings ...)
2021-01-28 20:54 ` Alexandre Belloni
@ 2021-01-29 16:23 ` Alexandre Torgue
3 siblings, 0 replies; 5+ messages in thread
From: Alexandre Torgue @ 2021-01-29 16:23 UTC (permalink / raw)
To: Rob Herring, devicetree
Cc: linux-kernel, Jean Delvare, Guenter Roeck, Jonathan Cameron,
Lars-Peter Clausen, Dmitry Torokhov, Ulf Hansson,
David S. Miller, Jakub Kicinski, Sebastian Reichel, Mark Brown,
Alexandre Belloni, Greg Kroah-Hartman, Serge Semin, Wolfram Sang,
linux-hwmon, linux-i2c, linux-iio, linux-arm-kernel, linux-input,
linux-mmc, netdev, linux-pm, linux-rtc, linux-serial, alsa-devel,
linux-watchdog
On 1/28/21 8:45 PM, Rob Herring wrote:
> Properties with standard unit suffixes already have a type and don't need
> type definitions. They also default to a single entry, so 'maxItems: 1'
> can be dropped.
>
> adi,ad5758 is an oddball which defined an enum of arrays. While a valid
> schema, it is simpler as a whole to only define scalar constraints.
>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Alexandre Torgue <alexandre.torgue@st.com>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Sebastian Reichel <sre@kernel.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Serge Semin <fancer.lancer@gmail.com>
> Cc: Wolfram Sang <wolfram@the-dreams.de>
> Cc: linux-hwmon@vger.kernel.org
> Cc: linux-i2c@vger.kernel.org
> Cc: linux-iio@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-input@vger.kernel.org
> Cc: linux-mmc@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Cc: linux-pm@vger.kernel.org
> Cc: linux-rtc@vger.kernel.org
> Cc: linux-serial@vger.kernel.org
> Cc: alsa-devel@alsa-project.org
> Cc: linux-watchdog@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> .../devicetree/bindings/arm/cpus.yaml | 1 -
> .../bindings/extcon/wlf,arizona.yaml | 1 -
> .../bindings/hwmon/adi,ltc2947.yaml | 1 -
> .../bindings/hwmon/baikal,bt1-pvt.yaml | 8 ++--
> .../devicetree/bindings/hwmon/ti,tmp513.yaml | 1 -
> .../devicetree/bindings/i2c/i2c-gpio.yaml | 2 -
> .../bindings/i2c/snps,designware-i2c.yaml | 3 --
> .../bindings/iio/adc/maxim,max9611.yaml | 1 -
> .../bindings/iio/adc/st,stm32-adc.yaml | 1 -
> .../bindings/iio/adc/ti,palmas-gpadc.yaml | 2 -
> .../bindings/iio/dac/adi,ad5758.yaml | 41 ++++++++++++-------
> .../bindings/iio/health/maxim,max30100.yaml | 1 -
> .../input/touchscreen/touchscreen.yaml | 2 -
> .../bindings/mmc/mmc-controller.yaml | 1 -
> .../bindings/mmc/mmc-pwrseq-simple.yaml | 2 -
> .../bindings/net/ethernet-controller.yaml | 2 -
> .../devicetree/bindings/net/snps,dwmac.yaml | 1 -
> .../bindings/power/supply/battery.yaml | 3 --
> .../bindings/power/supply/bq2515x.yaml | 1 -
> .../bindings/regulator/dlg,da9121.yaml | 1 -
> .../bindings/regulator/fixed-regulator.yaml | 2 -
> .../devicetree/bindings/rtc/rtc.yaml | 2 -
> .../devicetree/bindings/serial/pl011.yaml | 2 -
> .../devicetree/bindings/sound/sgtl5000.yaml | 2 -
> .../bindings/watchdog/watchdog.yaml | 1 -
> 25 files changed, 29 insertions(+), 56 deletions(-)
For stm32:
Acked-by: Alexandre TORGUE <alexandre.torgue@foss.st.com>
>
> diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
> index 14cd727d3c4b..f02fd10de604 100644
> --- a/Documentation/devicetree/bindings/arm/cpus.yaml
> +++ b/Documentation/devicetree/bindings/arm/cpus.yaml
> @@ -232,7 +232,6 @@ properties:
> by this cpu (see ./idle-states.yaml).
>
> capacity-dmips-mhz:
> - $ref: '/schemas/types.yaml#/definitions/uint32'
> description:
> u32 value representing CPU capacity (see ./cpu-capacity.txt) in
> DMIPS/MHz, relative to highest capacity-dmips-mhz
> diff --git a/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml b/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml
> index 5fe784f487c5..efdf59abb2e1 100644
> --- a/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml
> +++ b/Documentation/devicetree/bindings/extcon/wlf,arizona.yaml
> @@ -85,7 +85,6 @@ properties:
> wlf,micd-timeout-ms:
> description:
> Timeout for microphone detection, specified in milliseconds.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
>
> wlf,micd-force-micbias:
> description:
> diff --git a/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml b/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> index eef614962b10..bf04151b63d2 100644
> --- a/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> @@ -49,7 +49,6 @@ properties:
> description:
> This property controls the Accumulation Dead band which allows to set the
> level of current below which no accumulation takes place.
> - $ref: /schemas/types.yaml#/definitions/uint32
> maximum: 255
> default: 0
>
> diff --git a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> index 00a6511354e6..5d3ce641fcde 100644
> --- a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> @@ -73,11 +73,9 @@ properties:
> description: |
> Temperature sensor trimming factor. It can be used to manually adjust the
> temperature measurements within 7.130 degrees Celsius.
> - maxItems: 1
> - items:
> - default: 0
> - minimum: 0
> - maximum: 7130
> + default: 0
> + minimum: 0
> + maximum: 7130
>
> additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml b/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml
> index 8020d739a078..1502b22c77cc 100644
> --- a/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml
> @@ -52,7 +52,6 @@ properties:
> ti,bus-range-microvolt:
> description: |
> This is the operating range of the bus voltage in microvolt
> - $ref: /schemas/types.yaml#/definitions/uint32
> enum: [16000000, 32000000]
> default: 32000000
>
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml b/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml
> index cc3aa2a5e70b..ff99344788ab 100644
> --- a/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml
> +++ b/Documentation/devicetree/bindings/i2c/i2c-gpio.yaml
> @@ -39,11 +39,9 @@ properties:
>
> i2c-gpio,delay-us:
> description: delay between GPIO operations (may depend on each platform)
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> i2c-gpio,timeout-ms:
> description: timeout to get data
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> # Deprecated properties, do not use in new device tree sources:
> gpios:
> diff --git a/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml b/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml
> index c22b66b6219e..d9293c57f573 100644
> --- a/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml
> +++ b/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml
> @@ -66,21 +66,18 @@ properties:
> default: 400000
>
> i2c-sda-hold-time-ns:
> - maxItems: 1
> description: |
> The property should contain the SDA hold time in nanoseconds. This option
> is only supported in hardware blocks version 1.11a or newer or on
> Microsemi SoCs.
>
> i2c-scl-falling-time-ns:
> - maxItems: 1
> description: |
> The property should contain the SCL falling time in nanoseconds.
> This value is used to compute the tLOW period.
> default: 300
>
> i2c-sda-falling-time-ns:
> - maxItems: 1
> description: |
> The property should contain the SDA falling time in nanoseconds.
> This value is used to compute the tHIGH period.
> diff --git a/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml b/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml
> index 9475a9e6e920..95774a55629d 100644
> --- a/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/maxim,max9611.yaml
> @@ -23,7 +23,6 @@ properties:
> maxItems: 1
>
> shunt-resistor-micro-ohms:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> Value in micro Ohms of the shunt resistor connected between the RS+ and
> RS- inputs, across which the current is measured. Value needed to compute
> diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> index 6364ede9bb5f..a58334c3bb76 100644
> --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> @@ -248,7 +248,6 @@ patternProperties:
> Resolution (bits) to use for conversions:
> - can be 6, 8, 10 or 12 on stm32f4
> - can be 8, 10, 12, 14 or 16 on stm32h7 and stm32mp1
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> st,adc-channels:
> description: |
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml
> index 692dacd0fee5..7b895784e008 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml
> @@ -42,7 +42,6 @@ properties:
> const: 1
>
> ti,channel0-current-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: Channel 0 current in uA.
> enum:
> - 0
> @@ -51,7 +50,6 @@ properties:
> - 20
>
> ti,channel3-current-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: Channel 3 current in uA.
> enum:
> - 0
> diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml
> index 626ccb6fe21e..fd4edca34a28 100644
> --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5758.yaml
> @@ -46,31 +46,42 @@ properties:
> two properties must be present:
>
> adi,range-microvolt:
> - $ref: /schemas/types.yaml#/definitions/int32-array
> description: |
> Voltage output range specified as <minimum, maximum>
> - enum:
> - - [[0, 5000000]]
> - - [[0, 10000000]]
> - - [[-5000000, 5000000]]
> - - [[-10000000, 10000000]]
> + oneOf:
> + - items:
> + - const: 0
> + - enum: [5000000, 10000000]
> + - items:
> + - const: -5000000
> + - const: 5000000
> + - items:
> + - const: -10000000
> + - const: 10000000
>
> adi,range-microamp:
> - $ref: /schemas/types.yaml#/definitions/int32-array
> description: |
> Current output range specified as <minimum, maximum>
> - enum:
> - - [[0, 20000]]
> - - [[0, 24000]]
> - - [[4, 24000]]
> - - [[-20000, 20000]]
> - - [[-24000, 24000]]
> - - [[-1000, 22000]]
> + oneOf:
> + - items:
> + - const: 0
> + - enum: [20000, 24000]
> + - items:
> + - const: 4
> + - const: 24000
> + - items:
> + - const: -20000
> + - const: 20000
> + - items:
> + - const: -24000
> + - const: 24000
> + - items:
> + - const: -1000
> + - const: 22000
>
> reset-gpios: true
>
> adi,dc-dc-ilim-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> enum: [150000, 200000, 250000, 300000, 350000, 400000]
> description: |
> The dc-to-dc converter current limit.
> diff --git a/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml b/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml
> index 64b862637039..967778fb0ce8 100644
> --- a/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml
> +++ b/Documentation/devicetree/bindings/iio/health/maxim,max30100.yaml
> @@ -21,7 +21,6 @@ properties:
> description: Connected to ADC_RDY pin.
>
> maxim,led-current-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> minItems: 2
> maxItems: 2
> description: |
> diff --git a/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml b/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> index a771a15f053f..046ace461cc9 100644
> --- a/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> +++ b/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> @@ -70,11 +70,9 @@ properties:
>
> touchscreen-x-mm:
> description: horizontal length in mm of the touchscreen
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> touchscreen-y-mm:
> description: vertical length in mm of the touchscreen
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> dependencies:
> touchscreen-size-x: [ touchscreen-size-y ]
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-controller.yaml b/Documentation/devicetree/bindings/mmc/mmc-controller.yaml
> index df4ee4c778ae..e141330c1114 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-controller.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-controller.yaml
> @@ -261,7 +261,6 @@ properties:
> waiting for I/O signalling and card power supply to be stable,
> regardless of whether pwrseq-simple is used. Default to 10ms if
> no available.
> - $ref: /schemas/types.yaml#/definitions/uint32
> default: 10
>
> supports-cqe:
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> index 6cd57863c1db..226fb191913d 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> @@ -41,13 +41,11 @@ properties:
> description:
> Delay in ms after powering the card and de-asserting the
> reset-gpios (if any).
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> power-off-delay-us:
> description:
> Delay in us after asserting the reset-gpios (if any)
> during power off of the card.
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/net/ethernet-controller.yaml b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> index 0965f6515f9e..dac4aadb6e2e 100644
> --- a/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> +++ b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> @@ -122,7 +122,6 @@ properties:
> such as flow control thresholds.
>
> rx-internal-delay-ps:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> RGMII Receive Clock Delay defined in pico seconds.
> This is used for controllers that have configurable RX internal delays.
> @@ -140,7 +139,6 @@ properties:
> is used for components that can have configurable fifo sizes.
>
> tx-internal-delay-ps:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> RGMII Transmit Clock Delay defined in pico seconds.
> This is used for controllers that have configurable TX internal delays.
> diff --git a/Documentation/devicetree/bindings/net/snps,dwmac.yaml b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> index b2f6083f556a..9ac77b8cb767 100644
> --- a/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> @@ -208,7 +208,6 @@ properties:
> Triplet of delays. The 1st cell is reset pre-delay in micro
> seconds. The 2nd cell is reset pulse in micro seconds. The 3rd
> cell is reset post-delay in micro seconds.
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> minItems: 3
> maxItems: 3
>
> diff --git a/Documentation/devicetree/bindings/power/supply/battery.yaml b/Documentation/devicetree/bindings/power/supply/battery.yaml
> index 0c7e2e44793b..c3b4b7543591 100644
> --- a/Documentation/devicetree/bindings/power/supply/battery.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/battery.yaml
> @@ -83,21 +83,18 @@ properties:
> for each of the battery capacity lookup table.
>
> operating-range-celsius:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> description: operating temperature range of a battery
> items:
> - description: minimum temperature at which battery can operate
> - description: maximum temperature at which battery can operate
>
> ambient-celsius:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> description: safe range of ambient temperature
> items:
> - description: alert when ambient temperature is lower than this value
> - description: alert when ambient temperature is higher than this value
>
> alert-celsius:
> - $ref: /schemas/types.yaml#/definitions/uint32-array
> description: safe range of battery temperature
> items:
> - description: alert when battery temperature is lower than this value
> diff --git a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> index 75a56773be4a..813d6afde606 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> @@ -50,7 +50,6 @@ properties:
> maxItems: 1
>
> input-current-limit-microamp:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description: Maximum input current in micro Amps.
> minimum: 50000
> maximum: 500000
> diff --git a/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml b/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml
> index 6f2164f7bc57..228018c87bea 100644
> --- a/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml
> +++ b/Documentation/devicetree/bindings/regulator/dlg,da9121.yaml
> @@ -62,7 +62,6 @@ properties:
> description: IRQ line information.
>
> dlg,irq-polling-delay-passive-ms:
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> minimum: 1000
> maximum: 10000
> description: |
> diff --git a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
> index d3d0dc13dd8b..8850c01bd470 100644
> --- a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
> @@ -72,11 +72,9 @@ properties:
>
> startup-delay-us:
> description: startup time in microseconds
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> off-on-delay-us:
> description: off delay time in microseconds
> - $ref: /schemas/types.yaml#/definitions/uint32
>
> enable-active-high:
> description:
> diff --git a/Documentation/devicetree/bindings/rtc/rtc.yaml b/Documentation/devicetree/bindings/rtc/rtc.yaml
> index d30dc045aac6..0ec3551f12dd 100644
> --- a/Documentation/devicetree/bindings/rtc/rtc.yaml
> +++ b/Documentation/devicetree/bindings/rtc/rtc.yaml
> @@ -27,7 +27,6 @@ properties:
> 1: chargeable
>
> quartz-load-femtofarads:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description:
> The capacitive load of the quartz(x-tal), expressed in femto
> Farad (fF). The default value shall be listed (if optional),
> @@ -47,7 +46,6 @@ properties:
> deprecated: true
>
> trickle-resistor-ohms:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description:
> Selected resistor for trickle charger. Should be given
> if trickle charger should be enabled.
> diff --git a/Documentation/devicetree/bindings/serial/pl011.yaml b/Documentation/devicetree/bindings/serial/pl011.yaml
> index c23c93b400f0..07fa6d26f2b4 100644
> --- a/Documentation/devicetree/bindings/serial/pl011.yaml
> +++ b/Documentation/devicetree/bindings/serial/pl011.yaml
> @@ -88,14 +88,12 @@ properties:
> description:
> Rate at which poll occurs when auto-poll is set.
> default 100ms.
> - $ref: /schemas/types.yaml#/definitions/uint32
> default: 100
>
> poll-timeout-ms:
> description:
> Poll timeout when auto-poll is set, default
> 3000ms.
> - $ref: /schemas/types.yaml#/definitions/uint32
> default: 3000
>
> required:
> diff --git a/Documentation/devicetree/bindings/sound/sgtl5000.yaml b/Documentation/devicetree/bindings/sound/sgtl5000.yaml
> index d116c174b545..70b4a8831073 100644
> --- a/Documentation/devicetree/bindings/sound/sgtl5000.yaml
> +++ b/Documentation/devicetree/bindings/sound/sgtl5000.yaml
> @@ -41,14 +41,12 @@ properties:
> values of 2k, 4k or 8k. If set to 0 it will be off. If this node is not
> mentioned or if the value is unknown, then micbias resistor is set to
> 4k.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> enum: [ 0, 2, 4, 8 ]
>
> micbias-voltage-m-volts:
> description: The bias voltage to be used in mVolts. The voltage can take
> values from 1.25V to 3V by 250mV steps. If this node is not mentioned
> or the value is unknown, then the value is set to 1.25V.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> enum: [ 1250, 1500, 1750, 2000, 2250, 2500, 2750, 3000 ]
>
> lrclk-strength:
> diff --git a/Documentation/devicetree/bindings/watchdog/watchdog.yaml b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> index 4e2c26cd981d..e3dfb02f0ca5 100644
> --- a/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> @@ -19,7 +19,6 @@ properties:
> pattern: "^watchdog(@.*|-[0-9a-f])?$"
>
> timeout-sec:
> - $ref: /schemas/types.yaml#/definitions/uint32
> description:
> Contains the watchdog timeout in seconds.
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread