diff mbox series

[v1,10/13] dt-bindings: riscv: add Zihintntl ISA extension description

Message ID 20231011111438.909552-11-cleger@rivosinc.com (mailing list archive)
State Superseded
Headers show
Series riscv: report more ISA extensions through hwprobe | expand

Checks

Context Check Description
conchuod/vmtest-for-next-PR fail PR summary
conchuod/patch-10-test-1 success .github/scripts/patches/build_rv32_defconfig.sh
conchuod/patch-10-test-2 success .github/scripts/patches/build_rv64_clang_allmodconfig.sh
conchuod/patch-10-test-3 success .github/scripts/patches/build_rv64_gcc_allmodconfig.sh
conchuod/patch-10-test-4 success .github/scripts/patches/build_rv64_nommu_k210_defconfig.sh
conchuod/patch-10-test-5 success .github/scripts/patches/build_rv64_nommu_virt_defconfig.sh
conchuod/patch-10-test-6 success .github/scripts/patches/checkpatch.sh
conchuod/patch-10-test-7 success .github/scripts/patches/dtb_warn_rv64.sh
conchuod/patch-10-test-8 success .github/scripts/patches/header_inline.sh
conchuod/patch-10-test-9 success .github/scripts/patches/kdoc.sh
conchuod/patch-10-test-10 success .github/scripts/patches/module_param.sh
conchuod/patch-10-test-11 success .github/scripts/patches/verify_fixes.sh
conchuod/patch-10-test-12 success .github/scripts/patches/verify_signedoff.sh

Commit Message

Clément Léger Oct. 11, 2023, 11:14 a.m. UTC
Add description for Zihintntl ISA extension[1] which can now be reported
through hwprobe for userspace usage.

[1] https://drive.google.com/file/d/13_wsN8YmRfH8YWysFyTX-DjTkCnBd9hj/view

Signed-off-by: Clément Léger <cleger@rivosinc.com>
---
 Documentation/devicetree/bindings/riscv/extensions.yaml | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Conor Dooley Oct. 12, 2023, 1:50 p.m. UTC | #1
On Wed, Oct 11, 2023 at 01:14:35PM +0200, Clément Léger wrote:
> Add description for Zihintntl ISA extension[1] which can now be reported
> through hwprobe for userspace usage.
> 
> [1] https://drive.google.com/file/d/13_wsN8YmRfH8YWysFyTX-DjTkCnBd9hj/view

Ditto. Who is even hosting this google drive anyway? I'd rather see
GitHub links to docs from the releases, but since youre pointing at the
ISA manual I suppose those do not really exist.
Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
Conor.
> 
> Signed-off-by: Clément Léger <cleger@rivosinc.com>
> ---
>  Documentation/devicetree/bindings/riscv/extensions.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml
> index 4c923800d751..70c2b0351357 100644
> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml
> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml
> @@ -247,6 +247,12 @@ properties:
>              The standard Zihintpause extension for pause hints, as ratified in
>              commit d8ab5c7 ("Zihintpause is ratified") of the riscv-isa-manual.
>  
> +        - const: zihintntl
> +          description:
> +            The standard Zihintntl extension for non-temporal locality hints, as
> +            ratified in commit 0dc91f5 ("Zihintntl is ratified") of the
> +            riscv-isa-manual.
> +
>          - const: zihpm
>            description:
>              The standard Zihpm extension for hardware performance counters, as
> -- 
> 2.42.0
>
Clément Léger Oct. 12, 2023, 1:58 p.m. UTC | #2
On 12/10/2023 15:50, Conor Dooley wrote:
> On Wed, Oct 11, 2023 at 01:14:35PM +0200, Clément Léger wrote:
>> Add description for Zihintntl ISA extension[1] which can now be
>> reported through hwprobe for userspace usage.
>> 
>> [1]
>> https://drive.google.com/file/d/13_wsN8YmRfH8YWysFyTX-DjTkCnBd9hj/view
>
>> 
> Ditto. Who is even hosting this google drive anyway? I'd rather see 
> GitHub links to docs from the releases, but since youre pointing at
> the ISA manual I suppose those do not really exist.

Acked, I'll fix all these as "Links:"

Regarding the pdf sources themselves, I actually used the links as
referred on the RISC-V wiki [1] which is the wiki mentioned from the
riscv.org website [2]. I do not like it but there does not seems to have
a proper specification release website...

Maybe pointing only to a specific commit on the github repo that
mentioned that the extension was ratified is better.

Thanks,

Clément

[1] https://wiki.riscv.org/display/HOME/Recently+Ratified+Extensions
[2] https://riscv.org/technical/specifications/

> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> 
> Thanks, Conor.
>> 
>> Signed-off-by: Clément Léger <cleger@rivosinc.com> --- 
>> Documentation/devicetree/bindings/riscv/extensions.yaml | 6 ++++++ 
>> 1 file changed, 6 insertions(+)
>> 
>> diff --git
>> a/Documentation/devicetree/bindings/riscv/extensions.yaml
>> b/Documentation/devicetree/bindings/riscv/extensions.yaml index
>> 4c923800d751..70c2b0351357 100644 ---
>> a/Documentation/devicetree/bindings/riscv/extensions.yaml +++
>> b/Documentation/devicetree/bindings/riscv/extensions.yaml @@ -247,6
>> +247,12 @@ properties: The standard Zihintpause extension for pause
>> hints, as ratified in commit d8ab5c7 ("Zihintpause is ratified") of
>> the riscv-isa-manual.
>> 
>> +        - const: zihintntl +          description: +
>> The standard Zihintntl extension for non-temporal locality hints,
>> as +            ratified in commit 0dc91f5 ("Zihintntl is
>> ratified") of the +            riscv-isa-manual. + - const: zihpm 
>> description: The standard Zihpm extension for hardware performance
>> counters, as -- 2.42.0
>>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml
index 4c923800d751..70c2b0351357 100644
--- a/Documentation/devicetree/bindings/riscv/extensions.yaml
+++ b/Documentation/devicetree/bindings/riscv/extensions.yaml
@@ -247,6 +247,12 @@  properties:
             The standard Zihintpause extension for pause hints, as ratified in
             commit d8ab5c7 ("Zihintpause is ratified") of the riscv-isa-manual.
 
+        - const: zihintntl
+          description:
+            The standard Zihintntl extension for non-temporal locality hints, as
+            ratified in commit 0dc91f5 ("Zihintntl is ratified") of the
+            riscv-isa-manual.
+
         - const: zihpm
           description:
             The standard Zihpm extension for hardware performance counters, as