Commit b1809ee9 authored by Rob Herring's avatar Rob Herring
Browse files

dt-bindings: i2c: stm32-i2c: Move st,syscfg-fmp definition to top level



It is preferred to define all properties in the main schema and leave
if/then/else schemas to just be further constraints on properties.

Rework the schema to use be more specific for each cell. Previously,
multiple entries of 3 cells each was allowed.

Signed-off-by: default avatarRob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/20220119174407.3810088-1-robh@kernel.org
parent b588772a
Loading
Loading
Loading
Loading
+13 −11
Original line number Diff line number Diff line
@@ -25,16 +25,9 @@ allOf:

        i2c-scl-falling-time-ns:
          default: 10

        st,syscfg-fmp:
          description: Use to set Fast Mode Plus bit within SYSCFG when
                       Fast Mode Plus speed is selected by slave.
                       Format is phandle to syscfg / register offset within
                       syscfg / register bitmask for FMP bit.
          $ref: "/schemas/types.yaml#/definitions/phandle-array"
          items:
            minItems: 3
            maxItems: 3
    else:
      properties:
        st,syscfg-fmp: false

  - if:
      properties:
@@ -87,6 +80,16 @@ properties:
    minimum: 1
    maximum: 1000000

  st,syscfg-fmp:
    description: Use to set Fast Mode Plus bit within SYSCFG when Fast Mode
      Plus speed is selected by slave.
    $ref: "/schemas/types.yaml#/definitions/phandle-array"
    items:
      - items:
          - description: phandle to syscfg
          - description: register offset within syscfg
          - description: register bitmask for FMP bit

required:
  - compatible
  - reg
@@ -147,4 +150,3 @@ examples:
          i2c-scl-falling-time-ns = <20>;
          st,syscfg-fmp = <&syscfg 0x4 0x2>;
      };
...