lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <20210128205450.GP1196852@piout.net>
Date:   Thu, 28 Jan 2021 21:54:50 +0100
From:   Alexandre Belloni <alexandre.belloni@...tlin.com>
To:     Rob Herring <robh@...nel.org>
Cc:     devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        Jean Delvare <jdelvare@...e.com>,
        Guenter Roeck <linux@...ck-us.net>,
        Jonathan Cameron <jic23@...nel.org>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Alexandre Torgue <alexandre.torgue@...com>,
        Dmitry Torokhov <dmitry.torokhov@...il.com>,
        Ulf Hansson <ulf.hansson@...aro.org>,
        "David S. Miller" <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>,
        Sebastian Reichel <sre@...nel.org>,
        Mark Brown <broonie@...nel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Serge Semin <fancer.lancer@...il.com>,
        Wolfram Sang <wolfram@...-dreams.de>,
        linux-hwmon@...r.kernel.org, linux-i2c@...r.kernel.org,
        linux-iio@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-input@...r.kernel.org, linux-mmc@...r.kernel.org,
        netdev@...r.kernel.org, linux-pm@...r.kernel.org,
        linux-rtc@...r.kernel.org, linux-serial@...r.kernel.org,
        alsa-devel@...a-project.org, linux-watchdog@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: Cleanup standard unit properties

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@...e.com>
> Cc: Guenter Roeck <linux@...ck-us.net>
> Cc: Jonathan Cameron <jic23@...nel.org>
> Cc: Lars-Peter Clausen <lars@...afoo.de>
> Cc: Alexandre Torgue <alexandre.torgue@...com>
> Cc: Dmitry Torokhov <dmitry.torokhov@...il.com>
> Cc: Ulf Hansson <ulf.hansson@...aro.org>
> Cc: "David S. Miller" <davem@...emloft.net>
> Cc: Jakub Kicinski <kuba@...nel.org>
> Cc: Sebastian Reichel <sre@...nel.org>
> Cc: Mark Brown <broonie@...nel.org>
> Cc: Alexandre Belloni <alexandre.belloni@...tlin.com>
> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> Cc: Serge Semin <fancer.lancer@...il.com>
> Cc: Wolfram Sang <wolfram@...-dreams.de>
> Cc: linux-hwmon@...r.kernel.org
> Cc: linux-i2c@...r.kernel.org
> Cc: linux-iio@...r.kernel.org
> Cc: linux-arm-kernel@...ts.infradead.org
> Cc: linux-input@...r.kernel.org
> Cc: linux-mmc@...r.kernel.org
> Cc: netdev@...r.kernel.org
> Cc: linux-pm@...r.kernel.org
> Cc: linux-rtc@...r.kernel.org
> Cc: linux-serial@...r.kernel.org
> Cc: alsa-devel@...a-project.org
> Cc: linux-watchdog@...r.kernel.org
> Signed-off-by: Rob Herring <robh@...nel.org>
Acked-by: Alexandre Belloni <alexandre.belloni@...tlin.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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ