Re: [PATCH v4 2/6] Update terminology and specification tables


On Sat, 2021-08-07 at 11:42 +0530, Anup Patel wrote:
This patch updates terminology and specification table in
following ways:
1) Provide complete expansion of terms for APLIC and ACLINT
2) Add links to unprivileged and privileged specifications
3) Add link to legacy PLIC specification
4) Add links to AIA and ACLINT specifications
5) Remove platform policy from specification table
6) Adjust width of columns in both tables
7) Use "RISC-V" prefix for specifications owned by RISC-V international

Reviewed-by: Abner Chang <abner.chang@...>
Signed-off-by: Anup Patel <anup.patel@...>
 riscv-platform-spec.adoc | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/riscv-platform-spec.adoc b/riscv-platform-spec.adoc
index 0b30724..59b53fa 100644
--- a/riscv-platform-spec.adoc
+++ b/riscv-platform-spec.adoc
@@ -50,7 +50,7 @@ The M platform has the following extensions:
 == OS-A Platform
 === Terminology
-[cols="1,2", width=80%, align="left", options="header"]
+[cols="1,4", width=80%, align="left", options="header"]
 |SBI       | Supervisor Binary Interface   
@@ -67,9 +67,9 @@ The M platform has the following extensions:
 |RV64GC    | RISC-V 64-bit general purpose ISA described as
 |RAS       | Reliability, Availability, and Serviceability
 |CLINT     | Legacy Core-Local Interrupt Controller
-|ACLINT    | Advanced CLINT
+|ACLINT    | Advanced Core-Local Interrupt Controller
 |PLIC      | Legacy Platform-Level Interrupt Controller
-|APLIC     | Advanced PLIC
+|APLIC     | Advanced Platform-Level Interrupt Controller
 |AIA       | Advanced Interrupt Architecture
 |IMSIC     | Incomning MSI Controller
 |L1D       | L1 Data cache
@@ -92,18 +92,22 @@ The M platform has the following extensions:
 === Specifications
-[cols="1,2", width=80%, align="left", options="header"]
+[cols="3,1", width=80%, align="left", options="header"]
 Specification]         | v2.9   
 Specification]  | v0.3
-|link:[SBI S
pecification]                    | v0.3
                          | TBD
 Unprivileged Architecture Specification] | 20191214-draft
 Privileged Architecture Specification] | v1.12-draft
 SBI Specification] | v0.3
 PLIC Specification] | v1.0
Currently, there is no released version of the PLIC spec. We should add
that to a TODO list before the platform spec goes out for a public

 ACLINT Specification] | v1.0-draft2
 AIA Specification] | v0.2-draft.24
 Profiles Specification]| Draft
 |link:[EBBR Specification]        
                                        | v2.0.0   
It should be "v2.0.1"

Not sure if we should refer to the github repo or the released section.
In case you want to change the latter, here is the link.

 Specification]              | v6.4
 Specification]              | v6.4
 Specification]    | v3.4.0
                       | TBD
 // Base feature set for OS-A Platform

Other than that, LGTM.

Reviewed-by: Atish Patra <atish.patra@...>


Join to automatically receive all group messages.