Re: [PATCH 6/9] dt-bindings: timer: Add MIPS P8700 ACLINT MTIMER

[Aleksa Paunovic <aleksa.paunovic@htecgroup.com>]

Hi Rob,

On 9/21/26 20:21, Rob Herring wrote:
> On Mon, Sep 14, 2026 at 03:55:54PM +0200, Aleksa Paunovic wrote:
>> Document the MIPS P8700 ACLINT MTIMER.
>> Extend support for both mtimecmp and mtime registers
>> within the ACLINT MTIMER, identified via reg-names.
>>
>> Signed-off-by: Aleksa Paunovic <aleksa.paunovic@htecgroup.com>
>> ---
>>  .../bindings/timer/thead,c900-aclint-mtimer.yaml   | 43 ++++++++++++++++++++--
>>  1 file changed, 39 insertions(+), 4 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/timer/thead,c900-aclint-mtimer.yaml b/Documentation/devicetree/bindings/timer/thead,c900-aclint-mtimer.yaml
>> index cf7c82e980f674afcf59fc726742dc4f49c1a820..35e02da675becb3c8bb187b8e1e623994d52df8c 100644
>> --- a/Documentation/devicetree/bindings/timer/thead,c900-aclint-mtimer.yaml
>> +++ b/Documentation/devicetree/bindings/timer/thead,c900-aclint-mtimer.yaml
>> @@ -17,18 +17,20 @@ properties:
>>                - sophgo,sg2042-aclint-mtimer
>>                - sophgo,sg2044-aclint-mtimer
>>            - const: thead,c900-aclint-mtimer
>> +      - items:
>> +          - const: gf,mips-p8700-aclint-mtimer
>>        - items:
>>            - enum:
>>                - anlogic,dr1v90-aclint-mtimer
>>            - const: nuclei,ux900-aclint-mtimer
>>
>>    reg:
>> -    items:
>> -      - description: MTIMECMP Registers
>> +    minItems: 1
>> +    maxItems: 2
>>
>>    reg-names:
>> -    items:
>> -      - const: mtimecmp
>> +    minItems: 1
>> +    maxItems: 2
>>
>>    interrupts-extended:
>>      minItems: 1
>> @@ -42,6 +44,28 @@ required:
>>    - reg-names
>>    - interrupts-extended
>>
>> +allOf:
>> +  - if:
>> +      properties:
>> +        compatible:
>> +          contains:
>> +            const: gf,mips-p8700-aclint-mtimer
>> +    then:
>> +      properties:
>> +        reg:
>> +          minItems: 2
>> +        reg-names:
>> +          items:
>> +            - const: mtime
>> +            - const: mtimecmp
> New entries go at the end. And then you can define 'mtime' at the top
> level with this just being 'minItems: 2'.
Thanks, will fix in v2!

Best regards,
Aleksa