-
Notifications
You must be signed in to change notification settings - Fork 27
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
rm: boot sw update: include information about custom hardware require… #775
Conversation
…ments Our reset logic for bootloader updates depends on having the WDOG_B pin connected to the PMIC, so add this to the docs. Signed-off-by: Vanessa Maegima <[email protected]>
@igoropaniuk based on our call today, I think this is a very important information to have in our docs. I don't know the best place to add it though. |
Docs for ab80adf are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2801/docs/artifacts/html/index.html |
@igoropaniuk does it apply to all imx or only imx8m? |
I think where it is placed in this commit works well. With a quick glance, I didn't find anywhere else equally or more appropriate where it could go. |
@kprosise I will need to extend this PR to cover zynqMP as well. I will do that later today or tomorrow. |
@@ -5,6 +5,9 @@ | |||
Boot Software Updates on iMX | |||
============================ | |||
|
|||
.. note:: | |||
The boot software update feature on iMX depends on having the WDOG_B signal of the SoC connected to the PMIC. |
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 boot software update feature on iMX depends on having the WDOG_B signal of the SoC connected to the PMIC. | |
The boot software update feature on iMX depends on having the ``WDOG_B`` signal of the SoC connected to the PMIC. |
I think we should rewrite this section completely, and include more details about the usage of watchdog and in which case we expect watchdog to assert WDOG_B signal, and in what cases we need "soft" reset. @vanmaegima |
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.
Added a comment above
@igoropaniuk I think it's a great idea, I'll close this one |
…ments
Our reset logic for bootloader updates depends on having the WDOG_B pin connected to the PMIC, so add this to the docs.
PR Template and Checklist
Please complete as much as possible to speed up the reviewing process.
Readiness and adding reviewers as appropriate is required.
All PRs should be reviewed by a technical writer/documentation team and a peer.
If effecting customers—which is a majority of content changes—a member of Customer Success must also review.
Readiness
Overview
Why merge this PR? What does it solve?
Checklist
make linkcheck
.-s, --signoff
).-S, --gpg-sign
).Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.