Re: [PATCH] dt-bindings: timer: Remove sifive,fine-ctr-bits property

[Conor Dooley]

On Sun, Apr 19, 2026 at 11:18:55PM -0700, Nick Hu wrote:
> The counter width can be inferred from the compatible string, making the
> explicit "sifive,fine-ctr-bits" property redundant. Remove the property
> to simplify the bindings.

And there are no users of this, because the only permitted compatible
string combination contains {}.
Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
pw-bot: not-applicable

> 
> Fixes: 0f920690a82c ("dt-bindings: timer: Add SiFive CLINT2")
> Suggested-by: Conor Dooley <conor+dt@xxxxxxxxxx>
> Link: https://lore.kernel.org/linux-riscv/20260330-relative-hardened-5ce35fe1ef57@spud/
> Signed-off-by: Nick Hu <nick.hu@xxxxxxxxxx>
> ---
>  .../devicetree/bindings/timer/sifive,clint.yaml          | 16 ----------------
>  1 file changed, 16 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/timer/sifive,clint.yaml b/Documentation/devicetree/bindings/timer/sifive,clint.yaml
> index 3c16b260db04..051edb1da0d7 100644
> --- a/Documentation/devicetree/bindings/timer/sifive,clint.yaml
> +++ b/Documentation/devicetree/bindings/timer/sifive,clint.yaml
> @@ -72,22 +72,6 @@ properties:
>      minItems: 1
>      maxItems: 4095
>  
> -  sifive,fine-ctr-bits:
> -    maximum: 15
> -    description: The width in bits of the fine counter.
> -
> -if:
> -  properties:
> -    compatible:
> -      contains:
> -        const: sifive,clint2
> -then:
> -  required:
> -    - sifive,fine-ctr-bits
> -else:
> -  properties:
> -    sifive,fine-ctr-bits: false
> -
>  additionalProperties: false
>  
>  required:
> 
> ---
> base-commit: c1f49dea2b8f335813d3b348fd39117fb8efb428
> change-id: 20260409-clintv2-remove-fine-ctr-d5caeda27863
> 
> Best regards,
> -- 
> Nick Hu <nick.hu@xxxxxxxxxx>
>

Attachment: signature.asc
Description: PGP signature