Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

This PR updates the PC and PL template instructions for SW and ARVM p… #557

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
46 changes: 29 additions & 17 deletions process/PC_PL_Template_Markdown.md
@@ -1,18 +1,30 @@
# OpenHW Project Concept And Project Launch Markdown Template: Instructions
*This template is divided into two parts:*
- *The **Project Concept (PC)** required fields are shown in the first part.*
- *The additional **Project Launch (PL)** required and optional fields are shown in the second part.*

*Delete any sections not needed for your proposal*

*The normal proposal flow is:*
- ***The PC proposal*** *is prepared and presented to TWG. The PC proposal introduces the project and explains the market drivers and the "why"*
- *TWG grants PC gate with feedback, or rejects PC gate with feedback*
- *If PC granted, additional work is carried out to prepare the **PL proposal***.
- *The **PL proposal** contains updates to the PC fields and adds additional fields. The PL proposal explains the "what" of the project.*
- *For software porting projects, the PL should contain the feature list*
- *For IP core or other complex projects, the PL should contain a high level feature list (the user manual with feature specification is developed for the Plan Approved gate).*

# OpenHW Project Concept and Project Launch markdown template: Instructions
*This template presents both PC and PL templates as well as instructions for use*


*In the **normal OpenHW project flow**:*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about the word normal in this context. Seems to imply that OpenHW also has an abnormal project flow.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps "standard"?

- *The **PC***
- *Introduces the intitial project concept and team, associated market drivers or user requirements (the "why") along with basic scope of the project*
- *Typically lacks detail such as a feature list, architecture or complete project team.*
- *If PC approved by the TWG, the project team completes the project description in the **PL proposal***.
- *The PC as approved is retained in the programs repository*

- *The **PL***:
- *Is a separate document to the PC, which it updates and completes.*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the it in this sentence?

- *The architecture, major features and project phases (the "what" - but not necessarily final level of detail)*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can break this out to influence the authors of the PL to focus primarily on the features and project phases.

  • The major features of the IP (the "what")
  • Top-level project phases of the IP development
  • High level architecture of the IP (not necessarily final level of detail)

- *The project team is identified*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doubt this will be identified for a software project. You probably just have a lead, with the team identified at PA

- *If PL approved by the TWG, the project team completes the feature specification and project plan in the **Plan Approved (PA) proposal***. *

*The flow for **OpenHW software projects** whose scope/target is a family of related targets (such as CORE-V 32 bit 4 stage cores):*
- *PC and PL are written as top-level projects addressing the target family.*
- *Any PC and PL sections intended for a particular target, e.g., preliminary feature list, are identified as target-specific*
- *The PA describes a "subproject" for a particular target. Each PA should contain the full feature list and project plan for a particular target*
- *There would normally be multiple PA sub-projects for top-level SW projects*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't help feeling an example would make this clear. E.g. PC/PL for the GNU Compiler Tool Chain and separate PAs for the GNU Compiler specifically for the CV32E40Pv1 and CV32E40Pv2 targets.


*The flow for **OpenHW Advanced RISC-V Verification Methodology (ARVM)** subprojects:*
- *The ARVM PC is written as a top-level project addressing multiple ARVM subprojects*
- *The PL and PA describe an ARVM "subproject". The PA contains the full feature list and project plan for the subproject*
- *There will be multiple PL/PA subprojects for the ARVM top-level project*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commentary is really helpful. Any chance of adding this to all the sections below?


<hr/>

Expand Down Expand Up @@ -113,7 +125,7 @@
### Feature Requirements
*Features are more granular than Components.*
*For SW porting projects, this list serves as the detailed project reference for features*
*For IP Cores or more complext projects, a user manual with requirements specification is produced at the PA gate, which may supercede this list of features*
*For IP Cores or more complext projects, a user manual with requirements specification is produced at the PA gate, which may supersede this list of features*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: complext

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to check, I presume we are using US spelling throughout. "Supercede" would be British English spelling


#### Feature 1
#### Feature 2
Expand All @@ -134,7 +146,7 @@
### Funding for project aspects - requirement and availability

## Architecture and/or context diagrams
*Architecture (internal blocks and interconnections), and context (depiction of the the project content within its operational context), are both encouraged where appropriate to depict functionality to both subject matter experts and to non-experts*
*Architecture (internal blocks and interconnections), and context (depiction of the project content within its operational context), are both encouraged where appropriate to depict functionality to both subject matter experts and to non-experts*



Expand Down