Commit 387d0002 authored by Alex Elder's avatar Alex Elder Committed by Greg Kroah-Hartman
Browse files

dt-bindings: serial: 8250: move a constraint 



A block that required a "spacemit,k1-uart" compatible node to
specify two clocks was placed in the wrong spot in the binding.
Conor Dooley pointed out it belongs earlier in the file, as part
of the initial "allOf".

Fixes: 2c0594f9 ("dt-bindings: serial: 8250: support an optional second clock")
Cc: stable <stable@kernel.org>
Reported-by: default avatarConor Dooley <conor@kernel.org>
Closes: https://lore.kernel.org/lkml/20250729-reshuffle-contented-e6def76b540b@spud/


Signed-off-by: default avatarAlex Elder <elder@riscstar.com>
Acked-by: default avatarConor Dooley <conor.dooley@microchip.com>
Link: https://lore.kernel.org/r/20250813032151.2330616-1-elder@riscstar.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent ee047e1d

Documentation/devicetree/bindings/serial/8250.yaml

+22 −24
Original line number Diff line number Diff line
@@ -48,7 +48,6 @@ allOf: 
      oneOf:

        - required: [ clock-frequency ]

        - required: [ clocks ]



  - if:

      properties:

        compatible:
@@ -66,6 +65,28 @@ allOf: 
          items:

            - const: core

            - const: bus

  - if:

      properties:

        compatible:

          contains:

            enum:

              - spacemit,k1-uart

              - nxp,lpc1850-uart

    then:

      required:

        - clocks

        - clock-names

      properties:

        clocks:

          minItems: 2

        clock-names:

          minItems: 2

    else:

      properties:

        clocks:

          maxItems: 1

        clock-names:

          maxItems: 1



properties:

  compatible:
@@ -264,29 +285,6 @@ required: 
  - reg

  - interrupts



if:

  properties:

    compatible:

      contains:

        enum:

          - spacemit,k1-uart

          - nxp,lpc1850-uart

then:

  required:

    - clocks

    - clock-names

  properties:

    clocks:

      minItems: 2

    clock-names:

      minItems: 2

else:

  properties:

    clocks:

      maxItems: 1

    clock-names:

      maxItems: 1



unevaluatedProperties: false



examples: