| Instructions for Functional Requirements Peer Review |
|
|
|
|
|
| PRE-MEETING |
|
|
| Function Owner | Confirm the requirements are completed and frozen prior to the review (no changes after initiating) |
| Function Owner | Start with latest version of the template for any "first reviews" - Continue to use existing template for re-reviews |
| Function Owner | Provide the functional requirements (changed documents) to the invited attendees 1-2 working days in advance of review |
| Function Owner | Notify the assigned peer reviewer and make sure they are prepared to do their function in the meeting |
| Function Owner | Identify necessary attendance and invite to meeting |
| Function Owner | Complete the "Author" column information for sections 1 through 3 prior to the review |
| Function Owner | Complete the attendance invitation list in section 5 |
| Function Owner | For Re-reviews only: Complete the column "remarks by author" to identify actions taken to address items found in earlier reviews. |
|
|
|
| DURING MEETING |
|
|
| Function Owner | Present document changes to the review team |
| Peer Reviewer | Capture attendance of the review |
| Peer Reviewer | Capture Review Date, Review Action, Review Action Status during the review |
| Peer Reviewer | Capture actions and issues in section 4. Identify issue summary, Document type, Reference (Requirement ID, section number, etc), Defect Type and indicate status as "OPEN" |
|
|
|
| POST MEETING |
|
|
| Function Owner | Follow up on all "open" items. Update "Summary of Resolution" to indicate what was done or decided. |
| Function Owner | Schedule follow up review OR review open items with peer reviewer and obtain agreement to close |
| Peer Reviewer | Close change request in system and confirm all associated tasks are complete. Upload peer review checklist (this document) with any FR updates |
| Technical Review Checklist - Template Version 01.00.00 |
|
|
|
|
|
|
|
|
| Product Name | Electric Power Steering | Electrical Arch. | 4 | Review Scope | Defect Type | Numbers |
|
|
|
|
| Yes | Closed | FR |
| Function Name | ES005A Temporal Monitor | Target Version | 2.3 | Added requirment for SysFlt2A and 2B | Incorrect Requirement | 0 |
|
|
|
|
| No | Rejected | FDD |
| Author | Raghavendran Mohan |
|
| Ambiguous | 0 |
|
|
|
|
| NA | Open | Model |
|
| Effort | Conflicting | 0 |
|
|
|
|
|
|
| FMEA |
|
| Review Effort(Hrs.) | 1.00 | Incorrect Format | 0 |
|
|
|
|
|
|
| *.m File |
|
| Corr+Verf effort(Hrs.) | 0.00 |
| 0 |
|
|
|
|
|
|
| Cal Process |
|
| Total Effort (Hrs.) | 1.00 | Others | 0 |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Total | 0 |
|
|
|
|
|
|
|
|
| Checklist No. | Description of Check | Author:
This column is for Self review. Author shall fill Yes/No/NA against each point in checklist.
Author | Author:
This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist.
Reviewer | Author:
Detailed Description of the finding shall be provided by the reviewer.
Description of finding by reviewer | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed. Status |
|
|
|
|
|
|
|
|
| 1 | Section 1: TECHNICAL CHECK |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1.1 | All requirements indicated by setting "Is Requirement" attribute to TRUE | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1.2 | All requirements start with "The "function" shall…" | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1.3 | All IO to the function have been described | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1.4 | All reviewed requirements documented in the DOORs attributes for review (Review Date, Review Action, Review Action Status) | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1.5 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 2 | Section 2: Safety CHECK | Author:
This column is for Self review. Author shall fill Yes/No/NA against each point in checklist.
Author | Author:
This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist.
Reviewer | Author:
Detailed Description of the finding shall be provided by the reviewer.
Description of finding by reviewer | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed. Status |
|
|
|
|
|
|
|
|
| 2.1 | All "Is Requirements" attributes set to TRUE have a completed ASIL attribute identified | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| 2.2 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 3 | Section 3: Lessons Learned | Author:
This column is for Self review. Author shall fill Yes/No/NA against each point in checklist.
Author | Author:
This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist.
Reviewer | Author:
Detailed Description of the finding shall be provided by the reviewer.
Description of finding by reviewer | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed. Status |
|
|
|
|
|
|
|
|
| 3.1 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 3.2 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4 | Section 4: Issues / Actions Identified | Document | Reference | Summary of resolution | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed. Status |
|
|
|
|
|
|
|
|
| 4.1 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.2 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.3 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.4 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.5 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.6 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.7 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.8 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.9 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.10 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.11 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.12 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.13 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.14 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.15 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.16 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.17 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.18 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.19 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.20 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.21 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.22 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.23 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.24 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 4.25 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 5 | Section 5: APPROVALS |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Role | First Review | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
| Function Owner* |
Raghavendran Mohan | 11/28/2016 | Yes | Yes |
|
|
|
|
|
|
|
|
|
|
|
| Peer Reviewer* | Sai Desu | Yes |
|
|
|
|
|
|
|
|
|
|
|
| EPDT Engineer | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
| ES Engineer |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Software Lead | Jayakrishnan Thundathil | Yes |
|
|
|
|
|
|
|
|
|
|
|
| Hardware Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
| Test Lead | Pooja Kusnoor | Yes |
|
|
|
|
|
|
|
|
|
|
|
| Safety Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
| Role | Second Review (if required) | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
| Function Owner* |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Peer Reviewer* |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| EPDT Engineer | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| ES Engineer | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Software Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Hardware Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Test Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Safety Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Role | Third Review (if required) | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
| Function Owner* | <Owner Name> |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Peer Reviewer* | <Name> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| EPDT Engineer | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| ES Engineer | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Software Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Hardware Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Test Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Safety Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Role | Fourth Review (if required) | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
| Function Owner* | <Owner Name> |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Peer Reviewer* | <Name> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| EPDT Engineer | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| ES Engineer | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Software Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Hardware Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Test Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Safety Lead | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Role | Add more if necessary | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| P.S.: | Yes indicates adherence |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| No indicates non-adherence, reviewer shall provide suitable comments at the end of this document for each point. |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| NA indicates not applicable |
|
|
|
|
|
|
|
|
|
|
|
|
|
|