Re: [PATCH v3 1/2] dt-bindings: sifive: add cache-set value of 2048

[Krzysztof Kozlowski]

On 26/07/2022 19:07, Conor Dooley wrote:
> From: Atul Khare <atulkhare@xxxxxxxxxxxx>
> 
> Fixes Running device tree schema validation error messages like
> '... cache-sets:0:0: 1024 was expected'.
> 
> The existing bindings had a single enumerated value of 1024, which
> trips up the dt-schema checks. The ISA permits any arbitrary power
> of two for the cache-sets value, but we decided to add the single
> additional value of 2048 because we couldn't spot an obvious way
> to express the constraint in the schema.

There is no way to express "power of two" but enum for multiple values
would work. Is there a reason to limit only to 2048?

Best regards,
Krzysztof