-
Notifications
You must be signed in to change notification settings - Fork 61
Correct Baselibs reqs ASIL #1409
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
Conversation
4dad958
to
90210a4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The baselibs part looks good. Thank you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed the changes in commit "lola: adapt ASIL Level for checks"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine for me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please refer to the latest released version
3b5c7bf
to
364cf8f
Compare
364cf8f
to
2fc0dc4
Compare
The created documentation from the pull request is available at: docu-html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only minor changes
:status: valid | ||
|
||
The base libraries shall implement functionality necessary to support safety-relevant platform components up to ASIL-B. | ||
The base libraries shall implement functionality necessary to support safety-relevant platform components up to ASIL-B for selected functionalities. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The base libraries shall implement functionality necessary to support safety-relevant platform components up to ASIL-B for selected functionalities. | |
The base libraries shall implement functionality necessary to support safety-relevant platform components up to ASIL-B. |
The additional restriction does not really add value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
without the restriction all libraries functions would need ASIL B implementation and I think this is not planned. Merged nevertheless as this blocks some other PR work.
@aschemmel-tech , as some PR depends now on that, can we merge it? |
No description provided.