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

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1786 bytes --]

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@microchip.com>
pw-bot: not-applicable

> 
> Fixes: 0f920690a82c ("dt-bindings: timer: Add SiFive CLINT2")
> Suggested-by: Conor Dooley <conor+dt@kernel.org>
> Link: https://lore.kernel.org/linux-riscv/20260330-relative-hardened-5ce35fe1ef57@spud/
> Signed-off-by: Nick Hu <nick.hu@sifive.com>
> ---
>  .../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@sifive.com>
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]