Re: [PATCH -fixes v2 2/4] dt-bindings: riscv: Add ratified privileged ISA versions

From: Andrew Jones
Date: Tue Feb 13 2024 - 09:26:00 EST


On Mon, Feb 12, 2024 at 07:37:33PM -0800, Samuel Holland wrote:
> The baseline for the RISC-V privileged ISA is version 1.10. Using
> features from newer versions of the privileged ISA requires the
> supported version to be reported by platform firmware, either in the ISA
> string (where the binding already accepts version numbers) or in the
> riscv,isa-extensions property. So far two newer versions are ratified.
>
> Signed-off-by: Samuel Holland <samuel.holland@xxxxxxxxxx>
> ---
>
> Changes in v2:
> - New patch for v2
>
> .../devicetree/bindings/riscv/extensions.yaml | 20 +++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml
> index 63d81dc895e5..7faf22df01af 100644
> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml
> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml
> @@ -121,6 +121,16 @@ properties:
> version of the privileged ISA specification.
>
> # multi-letter extensions, sorted alphanumerically
> + - const: sm1p11
> + description:
> + The standard Machine ISA v1.11, as ratified in the 20190608
> + version of the privileged ISA specification.
> +
> + - const: sm1p12
> + description:
> + The standard Machine ISA v1.12, as ratified in the 20211203
> + version of the privileged ISA specification.
> +
> - const: smaia
> description: |
> The standard Smaia supervisor-level extension for the advanced
> @@ -134,6 +144,16 @@ properties:
> added by other RISC-V extensions in H/S/VS/U/VU modes and as
> ratified at commit a28bfae (Ratified (#7)) of riscv-state-enable.
>
> + - const: ss1p11
> + description:
> + The standard Supervisor ISA v1.11, as ratified in the 20190608
> + version of the privileged ISA specification.
> +
> + - const: ss1p12
> + description:
> + The standard Supervisor ISA v1.12, as ratified in the 20211203
> + version of the privileged ISA specification.
> +
> - const: ssaia
> description: |
> The standard Ssaia supervisor-level extension for the advanced
> --
> 2.43.0
>

Reviewed-by: Andrew Jones <ajones@xxxxxxxxxxxxxxxx>

Note, QEMU doesn't add these extensions to the ISA string yet, but I think
it should start, particularly the profile CPU types which should ensure
all the profile's mandatory extensions are added to the ISA string in
order to avoid any confusion.

Thanks,
drew