Skip to content

Add MB_PSU_SETTING command for MWOCP67 - #36

Merged
evan-oxide merged 1 commit into
masterfrom
evan/mwocp67-MB_PSU_SETTING
Jun 25, 2026
Merged

Add MB_PSU_SETTING command for MWOCP67#36
evan-oxide merged 1 commit into
masterfrom
evan/mwocp67-MB_PSU_SETTING

Conversation

@evan-oxide

Copy link
Copy Markdown
Contributor

We'll need to use the ClearFaults field of MB_PSU_SETTING to clear latched faults. I tested that field and the PowerCycle field (though we'll use the already-defined OPERATION command instead for disabling the output.)

This is part of oxidecomputer/hubris#2564.

@evan-oxide
evan-oxide requested a review from mkeeter June 24, 2026 21:32

@mkeeter mkeeter left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I checked against the datasheet and this looks correct!

@hawkw

hawkw commented Jun 24, 2026

Copy link
Copy Markdown
Member

Out of curiosity, does the MWOCP67 also honor the PMBus CLEAR_FAULTS command, or does that not work?

@evan-oxide

Copy link
Copy Markdown
Contributor Author

Out of curiosity, does the MWOCP67 also honor the PMBus CLEAR_FAULTS command, or does that not work?

CLEAR_FAULTS might work too, but I'm preferring MB_PSU_SETTING because the datasheet makes it sound simpler to use and more certain to get the PSU working again. Quotes from the datasheet:

Sending a CLEAR_FAULTS command to the PSU shall reset all STATUS_ bits to ‘0’. CLEAR_FAULTS shall clear all STATUS commands at a given PAGE if PAGE command is supported. If the PAGE is set to FFh; all STATUS bits in all PAGEs shall be cleared..

MB_PSU_SETTING bit 6:

Clear PSU faults and latch. write 1 for clear faults and start PSU to operate if not working
due to a fault.

I want to test with an interesting fault that shuts off the output, like overtemperature, but haven't been able to yet.

@evan-oxide
evan-oxide merged commit 3833d72 into master Jun 25, 2026
8 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.

3 participants