All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: Aleksa Paunovic <aleksa.paunovic@htecgroup.com>
Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Paul Walmsley <pjw@kernel.org>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Albert Ou <aou@eecs.berkeley.edu>,
	Alexandre Ghiti <alex@ghiti.fr>,
	Djordje Todorovic <djordje.todorovic@htecgroup.com>,
	Anup Patel <anup@brainfault.org>,
	Thomas Gleixner <tglx@kernel.org>, Radu Rendec <radu@rendec.net>,
	Inochi Amaoto <inochiama@outlook.com>,
	Daniel Lezcano <daniel.lezcano@kernel.org>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	Conor Dooley <conor@kernel.org>,
	linux-riscv@lists.infradead.org,
	Vladimir Kondratiev <vladimir.kondratiev@mobileye.com>
Subject: Re: [PATCH 6/9] dt-bindings: timer: Add MIPS P8700 ACLINT MTIMER
Date: Mon, 21 Sep 2026 13:21:25 -0500	[thread overview]
Message-ID: <20260921182125.GA1219949-robh@kernel.org> (raw)
In-Reply-To: <20260914-p8700-dts-v1-6-3267f14ebce1@htecgroup.com>

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'.

Rob

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Aleksa Paunovic <aleksa.paunovic@htecgroup.com>
Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Paul Walmsley <pjw@kernel.org>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Albert Ou <aou@eecs.berkeley.edu>,
	Alexandre Ghiti <alex@ghiti.fr>,
	Djordje Todorovic <djordje.todorovic@htecgroup.com>,
	Anup Patel <anup@brainfault.org>,
	Thomas Gleixner <tglx@kernel.org>, Radu Rendec <radu@rendec.net>,
	Inochi Amaoto <inochiama@outlook.com>,
	Daniel Lezcano <daniel.lezcano@kernel.org>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	Conor Dooley <conor@kernel.org>,
	linux-riscv@lists.infradead.org,
	Vladimir Kondratiev <vladimir.kondratiev@mobileye.com>
Subject: Re: [PATCH 6/9] dt-bindings: timer: Add MIPS P8700 ACLINT MTIMER
Date: Mon, 21 Sep 2026 13:21:25 -0500	[thread overview]
Message-ID: <20260921182125.GA1219949-robh@kernel.org> (raw)
In-Reply-To: <20260914-p8700-dts-v1-6-3267f14ebce1@htecgroup.com>

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'.

Rob

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

  reply	other threads:[~2026-09-21 18:21 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-14 13:55 [PATCH 0/9] Add support for MIPS P8700 Aleksa Paunovic
2026-09-14 13:55 ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55 ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55 ` [PATCH 1/9] dt-bindings: vendor-prefixes: Add GlobalFoundries, Inc Aleksa Paunovic
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-15 17:23   ` Conor Dooley
2026-09-15 17:23     ` Conor Dooley
2026-09-17 11:50     ` Maciej W. Rozycki
2026-09-17 11:50       ` Maciej W. Rozycki
2026-09-14 13:55 ` [PATCH 2/9] dt-bindings: riscv: cpus: add gf,mips-p8700 Aleksa Paunovic
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55 ` [PATCH 3/9] dt-bindings: riscv: Add MIPS Boston board compatible Aleksa Paunovic
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-15 17:58   ` Conor Dooley
2026-09-15 17:58     ` Conor Dooley
2026-09-17 13:23     ` Maciej W. Rozycki
2026-09-17 13:23       ` Maciej W. Rozycki
2026-09-14 13:55 ` [PATCH 4/9] dt-bindings: interrupt-controller: Add MIPS P8700 APLIC Aleksa Paunovic
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-15 18:16   ` Conor Dooley
2026-09-15 18:16     ` Conor Dooley
2026-09-14 13:55 ` [PATCH 5/9] dt-bindings: interrupt-controller: Add MIPS P8700 ACLINT MSWI Aleksa Paunovic
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-15 18:15   ` Conor Dooley
2026-09-15 18:15     ` Conor Dooley
2026-09-14 13:55 ` [PATCH 6/9] dt-bindings: timer: Add MIPS P8700 ACLINT MTIMER Aleksa Paunovic
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-21 18:21   ` Rob Herring [this message]
2026-09-21 18:21     ` Rob Herring
2026-09-14 13:55 ` [PATCH 7/9] dt-bindings: interrupt-controller: thead,c900-aclint-sswi: Add MIPS P8700 gf compatible Aleksa Paunovic
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-18 22:03   ` Rob Herring
2026-09-18 22:03     ` Rob Herring
2026-09-14 13:55 ` [PATCH 8/9] riscv: dts: gf: Initial support for the P8700 Boston board Aleksa Paunovic
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 15:08   ` sashiko-bot
2026-09-14 13:55 ` [PATCH 9/9] riscv: defconfig: Enable GF SoCs Aleksa Paunovic
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay
2026-09-14 13:55   ` Aleksa Paunovic via B4 Relay

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260921182125.GA1219949-robh@kernel.org \
    --to=robh@kernel.org \
    --cc=aleksa.paunovic@htecgroup.com \
    --cc=alex@ghiti.fr \
    --cc=anup@brainfault.org \
    --cc=aou@eecs.berkeley.edu \
    --cc=conor+dt@kernel.org \
    --cc=conor@kernel.org \
    --cc=daniel.lezcano@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=djordje.todorovic@htecgroup.com \
    --cc=inochiama@outlook.com \
    --cc=krzk+dt@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=palmer@dabbelt.com \
    --cc=pjw@kernel.org \
    --cc=radu@rendec.net \
    --cc=tglx@kernel.org \
    --cc=vladimir.kondratiev@mobileye.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.