[PATCH v5 08/17] dt-bindings: riscv: Require block-size for Zicbom, Zicbop, and Zicboz
Conor Dooley
conor at kernel.org
Wed Jul 1 13:39:19 PDT 2026
On Wed, Jul 01, 2026 at 08:52:21AM -0400, Guodong Xu wrote:
> Zicbom, Zicbop, and Zicboz have no default cache block size, so a
> devicetree that declares one must also provide the matching
> riscv,cbom/cbop/cboz-block-size property. Make it required so a
> missing block-size property can be caught by dtbs_check.
>
> Suggested-by: Conor Dooley <conor.dooley at microchip.com>
> Signed-off-by: Guodong Xu <docular.xu at gmail.com>
> ---
> v5: New patch.
> ---
> .../devicetree/bindings/riscv/extensions.yaml | 26 ++++++++++++++++++++++
> 1 file changed, 26 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml
> index 5ffc40d599c02..f4dd34d446c02 100644
> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml
> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml
> @@ -1142,6 +1142,32 @@ allOf:
> not:
> contains:
> const: zilsd
> + # All three Zicbo* extensions require their block size property as there's no
> + # default.
> + - if:
> + properties:
> + riscv,isa-extensions:
> + contains:
> + const: zicbom
I think the way to fix this is to add
required:
- riscv,isa-extensions
to each of these if conditions
(not got time to test it right now, sorry)
> + then:
> + required:
> + - riscv,cbom-block-size
> + - if:
> + properties:
> + riscv,isa-extensions:
> + contains:
> + const: zicbop
> + then:
> + required:
> + - riscv,cbop-block-size
> + - if:
> + properties:
> + riscv,isa-extensions:
> + contains:
> + const: zicboz
> + then:
> + required:
> + - riscv,cboz-block-size
>
> additionalProperties: true
> ...
>
> --
> 2.43.0
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-riscv/attachments/20260701/2408abd4/attachment.sig>
More information about the linux-riscv
mailing list