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

iobus: display power supply overload notifications #29

Merged
merged 4 commits into from
Sep 26, 2023

Conversation

hnez
Copy link
Member

@hnez hnez commented Jul 7, 2023

This should make troubleshooting easier in case of a short-circuit or overload on the IOBus bus.

IOBus fault

Related Pull Requests

@Emantor
Copy link
Member

Emantor commented Jul 10, 2023

There is a failing cargo check --tests run.

@hnez
Copy link
Member Author

hnez commented Aug 14, 2023

There is a failing cargo check --tests run.

You are right, and as usual they were actually quite valid. Thanks cargo check!
I've fixed those.

There were also prettier warnings for the web interface. Those were not quite as valid, as they were due to a new prettier release that caused the formatting rules to change.
I've fixed those in a separate PR (#30) and rebased this PR on top of it.
That PR should thus be merged first.

The demo_mode code for the IOBus supply was not migrated when
the control of the power supply moved from a GPIO to a
userspace-controllable regulator.

Signed-off-by: Leonard Göhrs <l.goehrs@pengutronix.de>
The fault pin from the smart fuse on board of the LXA TAC will rarely
(if ever) be active, as it only indicates a shutdown due to the smart
fuse overheating.

If the fuse is sufficiently cooled the IOBus may be in current limiting
condition indefinitly without the fault signal activating.

Signed-off-by: Leonard Göhrs <l.goehrs@pengutronix.de>
…obus

Signed-off-by: Leonard Göhrs <l.goehrs@pengutronix.de>
Signed-off-by: Leonard Göhrs <l.goehrs@pengutronix.de>
@hnez hnez merged commit 24bd697 into linux-automation:main Sep 26, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants