2023-12-13 11:34:09

by Clément Léger

[permalink] [raw]
Subject: [PATCH 3/9] dt-bindings: riscv: add Zam ISA extension description

Add description for the Zam ISA extension.

Signed-off-by: Clément Léger <[email protected]>
---
Documentation/devicetree/bindings/riscv/extensions.yaml | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml
index 3574a0b70be4..912cc6a42eb4 100644
--- a/Documentation/devicetree/bindings/riscv/extensions.yaml
+++ b/Documentation/devicetree/bindings/riscv/extensions.yaml
@@ -171,6 +171,11 @@ properties:
memory types as ratified in the 20191213 version of the privileged
ISA specification.

+ - const: zam
+ description: |
+ The standard Zam extension for misaligned atomics is supported as
+ ratified in version 20191213 of the riscv-isa-manual.
+
- const: zba
description: |
The standard Zba bit-manipulation extension for address generation
--
2.43.0


2023-12-14 14:12:02

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH 3/9] dt-bindings: riscv: add Zam ISA extension description

On Wed, Dec 13, 2023 at 12:32:59PM +0100, Cl?ment L?ger wrote:
> Add description for the Zam ISA extension.
>
> Signed-off-by: Cl?ment L?ger <[email protected]>
> ---
> Documentation/devicetree/bindings/riscv/extensions.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml
> index 3574a0b70be4..912cc6a42eb4 100644
> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml
> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml
> @@ -171,6 +171,11 @@ properties:
> memory types as ratified in the 20191213 version of the privileged
> ISA specification.
>
> + - const: zam
> + description: |
> + The standard Zam extension for misaligned atomics is supported as
> + ratified in version 20191213 of the riscv-isa-manual.

Is "20191213" an actual tag in that repo? Looking at that version of the
spec (because it is a spec version, but I don't think it is a valid
reference to that repo) Zam is listed as Draft. In fact, in the most
recent thing I could find, Zam was still listed as draft.

Are you sure this is ratified?


Attachments:
(No filename) (1.23 kB)
signature.asc (235.00 B)
Download all attachments

2023-12-15 08:27:04

by Clément Léger

[permalink] [raw]
Subject: Re: [PATCH 3/9] dt-bindings: riscv: add Zam ISA extension description



On 14/12/2023 15:11, Conor Dooley wrote:
> On Wed, Dec 13, 2023 at 12:32:59PM +0100, Clément Léger wrote:
>> Add description for the Zam ISA extension.
>>
>> Signed-off-by: Clément Léger <[email protected]>
>> ---
>> Documentation/devicetree/bindings/riscv/extensions.yaml | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml
>> index 3574a0b70be4..912cc6a42eb4 100644
>> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml
>> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml
>> @@ -171,6 +171,11 @@ properties:
>> memory types as ratified in the 20191213 version of the privileged
>> ISA specification.
>>
>> + - const: zam
>> + description: |
>> + The standard Zam extension for misaligned atomics is supported as
>> + ratified in version 20191213 of the riscv-isa-manual.
>
> Is "20191213" an actual tag in that repo? Looking at that version of the
> spec (because it is a spec version, but I don't think it is a valid
> reference to that repo) Zam is listed as Draft. In fact, in the most
> recent thing I could find, Zam was still listed as draft.

Whoops, my bad, I assumed that the chapter being present in the spec
meant it was ratified and I did not checked the listing stating it is
still in draft.

Will remove that from the next version.

Thanks,

Clément

>
> Are you sure this is ratified?