Re: [PATCH 2/2] media: dt-bindings: Use additionalProperties: false for endpoint: properties:

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 14/10/2024 09:47, Krzysztof Kozlowski wrote:
If a common binding for a group of devices encourages you to list its
subset, then it is not that common.

Solution is to fix that, e.g. split it per classes of devices.

It might be possible to have

         $ref: /schemas/media/video-interfaces-endpoint-defaults.yaml#

which declares the typical list ->

$ref: /schemas/media/video-interfaces.yaml#
additonalProperties:false

properties:
    data-lanes: true
    link-frequencies: true
    remote-endpoints: true

required:
    data-lanes
    link-frequencies
    remote-endpoints

and then if you need say clock-noncontinuous you'd just include

$ref: /schemas/media/video-interfaces.yaml#
unevaluatedProperties: false

and then list whatever you need

Or don't care and use unevaluatedProps because it makes people's life
easier and is still correct. If it is not correct, then this should be
used as an argument.

I'll wait to see what people think before progressing this patch further.

---
bod




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux