Re: [PATCH v3 1/3] dt-bindings: timer: atmel,at91sam9260-pit: convert to yaml

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

 



On 09.06.2023 13:48, Krzysztof Kozlowski wrote:

[ ... ]

> 
>>>> +      properties:
>>>> +        clocks:
>>>> +          minItems: 2
>>>> +        clock-names:
>>>> +          items:
>>>> +            - const: pclk
>>>> +            - const: gclk
>>>
>>> interrupts? They are still required, so why no description here?
>>
>> It was here in the previous versions but Conor suggested to remove it as it
>> was nothing specific about this description. For the if-then branch I kept
>> it to specify that the interrupt is share with other devices. In this
>> branch the interrupt is only for the timer itself. With this, would you
>> still prefer to add it back?
> 
> I just don't understand why interrupts are in one arm of the if: and not
> in the other.
> 

As previously mentioned, Conor suggested to have it like this.





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux