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

accton-as4610: do not enable v_out/i_out/p_out caps #1012

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KanjiMonster
Copy link
Contributor

Some PSUs on AS4610-30 and AS4610-54 have a defect where reading out v_out/i_out/p_out values while only one PSU is connected causes system lockups and subsequent resets from the watchdog.

Since these values are non-essential, just disable the readout of the values. As we do not know exactly which PSU models are affected, disable it for everyone for now.

This is a partial revert of 24efd4f.

Fixes: 24efd4f ("[as4610] Support Delta PSU DPS-920AB")

Some PSUs on AS4610-30 and AS4610-54 have a defect where reading out
v_out/i_out/p_out values while only one PSU is connected causes system
lockups and subsequent resets from the watchdog.

Since these values are non-essential, just disable the readout of the
values. As we do not know exactly which PSU models are affected, disable
it for everyone for now.

This is a partial revert of 24efd4f.

Fixes: 24efd4f ("[as4610] Support Delta PSU DPS-920AB")
Signed-off-by: Jonas Gorski <[email protected]>
@paulmenzel
Copy link

Thank you for your patch. Could you list (a non-complete) list of PSUs you saw this with? That’d be grand.

@KanjiMonster
Copy link
Contributor Author

Personally we experienced it with YM-1921 on AS4610-54P, and we got a report for YM-1601A on AS4610-30P affected by the same issue.

since the reset triggers shortly after boot, it is not easy to work around on an affected device.

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