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

"[!] send_recv" error message is a little cryptic #425

Closed
galexander1 opened this issue Jun 4, 2016 · 5 comments · Fixed by #426
Closed

"[!] send_recv" error message is a little cryptic #425

galexander1 opened this issue Jun 4, 2016 · 5 comments · Fixed by #426

Comments

@galexander1
Copy link
Contributor

To diagnose a recent problem (#423), I found screens full of "[!] send_recv", which was not meaningful to me - I could not even be certain it was an error message. Pull request forthcoming.

@galexander1
Copy link
Contributor Author

I made pull request #426 to address this issue. When the send_recv() fails, now it outputs the libusb error name and what it was trying to do (send request/read reply) when the failure happened. And when the caller notices send_recv() failed, it also outputs the STLINK_... command name that it was attempting. Makes it particularly easy to unambiguously diagnose when one stage in stlink access causes future requests to fail.

@xor-gate
Copy link
Member

xor-gate commented Jun 4, 2016

Hi @galexander1 looks very good to me. @texane could you also review the PR #426? You may merge this after review.

@texane
Copy link
Collaborator

texane commented Jun 5, 2016

sorry for delay, but looks good to me too

2016-06-05 20:37 GMT+02:00 Jerry Jacobs notifications@github.com:

Closed #425 #425.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#425 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AAEeH_5QzgO5BGqzkJkARzplfzeiGvKsks5qIxd2gaJpZM4IuNSP
.

@xor-gate
Copy link
Member

xor-gate commented Jun 5, 2016

Good, so my "silent" merge effort is not lost :=)

@texane
Copy link
Collaborator

texane commented Jun 5, 2016

Of course it is not :)

2016-06-05 20:45 GMT+02:00 Jerry Jacobs notifications@github.com:

Good, so my "silent" merge effort is not lost :=)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#425 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AAEeH59OSL9yTD_IN5epxBe-9HNf3Qusks5qIxlagaJpZM4IuNSP
.

@Nightwalker-87 Nightwalker-87 added this to the v1.3.0 milestone Mar 16, 2020
@Nightwalker-87 Nightwalker-87 linked a pull request Mar 17, 2020 that will close this issue
@stlink-org stlink-org locked as resolved and limited conversation to collaborators Apr 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants