Re: [PATCH v5 3/7] dt-bindings: auxdisplay: add Titan Micro Electronics TM16xx

["Jean-François Lessard" <jefflessard3@gmail.com>]

Hi Rob,

Thank you for the review and Reviewed-by tag. I've addressed all your feedback 
except one item that's causing validation issues.

Le 1 octobre 2025 22 h 58 min 38 s HAE, "Jean-François Lessard" <jefflessard3@gmail.com> a écrit :
>Le 1 octobre 2025 22 h 44 min 31 s HAE, Rob Herring <robh@kernel.org> a écrit :
>>On Fri, Sep 26, 2025 at 10:19:04AM -0400, Jean-François Lessard wrote:
>>> Add documentation for TM16xx-compatible 7-segment LED display controllers
>>> with keyscan.

...

>>> +
>>> +allOf:
>>> +  - $ref: /schemas/leds/common.yaml#
>>> +    properties:
>>> +      color: false
>>> +      function: false
>>> +      function-enumerator: false
>>> +  - $ref: /schemas/input/input.yaml#
>>> +  - $ref: /schemas/input/matrix-keymap.yaml#
>>> +  # SPI controllers require 3-wire (combined MISO/MOSI line)
>>> +  - if:
>>> +      properties:
>>> +        compatible:
>>> +          contains:
>>> +            enum:
>>> +              - fdhisi,fd620
>>> +              - fdhisi,fd628
>>> +              - princeton,pt6964
>>> +              - titanmec,tm1618
>>> +              - titanmec,tm1620
>>> +              - titanmec,tm1628
>>> +              - titanmec,tm1638
>>> +              - wxicore,aip1618
>>> +              - wxicore,aip1628
>>> +    then:
>>> +      $ref: /schemas/spi/spi-peripheral-props.yaml#
>>> +      properties:
>>> +        spi-3wire: true
>>
>>You can drop properties.
>>
>
>The issue is spi-3wire is defined in the child node of spi/spi-controller.yaml,
>not in spi-peripheral-props.yaml.
>
>Removing properties did not pass dt validation. Am I missing something?
>

You suggested dropping "properties:" in the SPI 3-wire section:

then:
  $ref: /schemas/spi/spi-peripheral-props.yaml#
  spi-3wire: true
  required:
  - spi-3wire

However, this causes dt_binding_check to fail with:

'spi-3wire' is not one of ['$ref', 'additionalItems', ... 'properties', 
'required', 'then', ...]

Unevaluated properties are not allowed ('spi-3wire' was unexpected)

It appears the schema requires "properties:" to recognize spi-3wire as a 
property constraint rather than a schema keyword. Should I keep the properties 
wrapper, or is there a different way to structure this that I'm missing?

>>> +      required:
>>> +        - spi-3wire
>>
>>With those nits fixed,
>>
>>Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
>>

Thanks for clarifying.

Best regards,
Jean-François