[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