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

Selftest failed on serial remote session when temp sensor is not detected #161

Closed
prasimix opened this issue Aug 28, 2017 · 3 comments
Closed
Assignees
Milestone

Comments

@prasimix
Copy link
Member

prasimix commented Aug 28, 2017

If error selftest is initiated via serial remote session (e.g. SYST:POW 1 or *TST?) and any of channel's temp sensor is not detected (e.g. the Power board is detached or sensor is damaged) a firmware will stuck and only hard reset can help.
To reproduce it unplug one Power board from the Arduino shield, power up PSU, initiate serial session and execute *TST?

mvladic added a commit that referenced this issue Aug 29, 2017
@prasimix prasimix changed the title Selftest failed on serial remote session (via Native USB) if error condition exists on channels Selftest failed on serial remote session when temp sensor is not detected Aug 29, 2017
@mvladic
Copy link
Contributor

mvladic commented Aug 29, 2017

Apparently, there is an limit of max. 64 characters when printing strings using Serial.println(const char *).

Following string was printed when CH2 temperature sensor test failed:

**ERROR [2017-08-29 13:03:14]: 723,"CH2 temperature sensor test failed"

which has 71 characters. That was cause of blocking.

In this fix, I replaced single println with multiple print's followed by the println at the end.

@prasimix prasimix added this to the v1.02 milestone Sep 24, 2017
@prasimix
Copy link
Member Author

prasimix commented Oct 2, 2017

Now it works but for some reason temp error is reported twice:

*TST?
**ERROR [2017-10-02 10:41:20]: 723,"CH2 temperature sensor test failed"
**ERROR [2017-10-02 10:41:20]: 723,"CH2 temperature sensor test failed"
**TRACE [2017-10-02 10:41:20]: Ch2 IO expander reg check failure: reg=0, expected=101, got=0
**ERROR [2017-10-02 10:41:20]: 211,"CH2 IOEXP test failed"
**TRACE [2017-10-02 10:41:20]: Ch2 ADC test failed reg1: expected=160, got=0
**TRACE [2017-10-02 10:41:20]: Ch2 ADC test failed reg2: expected=96, got=0
**ERROR [2017-10-02 10:41:20]: 221,"CH2 ADC test failed"
**TRACE [2017-10-02 10:41:20]: Ch2 DAC test skipped because of IO expander
**TRACE [2017-10-02 10:41:20]: Fan RPM: 3654
1

mvladic added a commit that referenced this issue Oct 2, 2017
mvladic added a commit that referenced this issue Oct 2, 2017
@prasimix
Copy link
Member Author

prasimix commented Oct 2, 2017

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants