[PATCH v4 1/2] dt-bindings: clock: xilinx: add description of user monitor interrupt

Krzysztof Kozlowski krzk at kernel.org
Sat Feb 14 12:59:30 PST 2026


On 14/02/2026 21:57, Harry Austen wrote:
>>>
>>> +  interrupts:
>>> +    items:
>>> +      - description: user clock monitor interrupt
>>> +
>>> +  interrupt-names:
>>> +    items:
>>> +      - const: monitor
>>> +
>>>    xlnx,static-config:
>>>      $ref: /schemas/types.yaml#/definitions/flag
>>>      description:
>>> @@ -66,12 +74,24 @@ required:
>>>    - xlnx,speed-grade
>>>    - xlnx,nr-outputs
>>>
>>> +if:
>>
>> Why isn't this in allOf? Code is fine, but why changing it?
> 
> There is only a single if condition, so the allOf is not required. Removed
> for simplicity and reduced indentation.
> 

Heh, you might need to add it soon re-indenting everything on adding
just one more if, so for me that was wrong choice. And you got a review :/

Best regards,
Krzysztof



More information about the linux-arm-kernel mailing list