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

USB transfer warnings #22502

Closed
StefJar opened this issue Feb 5, 2020 · 4 comments · Fixed by #24415
Closed

USB transfer warnings #22502

StefJar opened this issue Feb 5, 2020 · 4 comments · Fixed by #24415
Assignees
Labels
area: USB Universal Serial Bus

Comments

@StefJar
Copy link

StefJar commented Feb 5, 2020

after pulling the latest master branch. I see so much more logging on the usb and warnings. I am pretty sure that it is related to that commits:

commit 7a1ab248911837688ec9db75f615fe261b80e688
Author: Johann Fischer <j.fischer@phytec.de>
Date:   Thu Dec 12 17:09:44 2019 +0100    usb: rework USB transfer logging    Rework USB transfer logging.    Signed-off-by: Johann Fischer <j.fischer@phytec.de>

commit 7bd0de0c61ac279697cd9afefe7d05fa2979903c
Author: Johann Fischer <j.fischer@phytec.de>
Date:   Thu Dec 12 16:38:28 2019 +0100    usb: move USB transfer functions to appropriate file    Move USB transfer functions to appropriate file.    Signed-off-by: Johann Fischer <j.fischer@phytec.de>

Together with my stm32f412 using the usb as an acm I am seeing these warnings etc:

[00:04:37.206,000] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x82
[00:04:37.215,000] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x82
[00:04:37.333,000] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x01
[00:04:37.341,000] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x01
[00:04:37.350,000] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x01

the usb acm seems to work like before.

@jfischer-phytec-iot can you pls have a check on that?

@StefJar StefJar added the bug The issue is a bug, or the PR is fixing a bug label Feb 5, 2020
@carlescufi carlescufi added question area: USB Universal Serial Bus and removed bug The issue is a bug, or the PR is fixing a bug labels Feb 5, 2020
@jfischer-no
Copy link
Collaborator

jfischer-no commented Feb 5, 2020

@StefJar I will check it asap. We did not log it earlier and I did not change any logic in the transfers, so it just improper log level.

@carlescufi carlescufi changed the title after pulling latest master the usb transfer brings up warnings USB transfer warnings Feb 5, 2020
@sslupsky
Copy link
Contributor

sslupsky commented Apr 1, 2020

Has this issue been investigated?

I updated my zephyr from 2.1.0 to 2.2.0 and USB is broken. I am seeing the same thing reported here on sam0:

[00:00:19.518,615] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x01
[00:00:19.518,981] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x82
[00:00:19.519,165] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x82
[00:00:19.519,348] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x82
[00:00:19.519,714] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x82
[00:00:19.520,080] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x82
[00:00:19.520,446] <wrn> usb_transfer: Transfer cancelled or completed, ep 0x82

usb_transfer is spewing out hundreds of log messages per second. This issue was not present in 2.1.0.

@sslupsky
Copy link
Contributor

sslupsky commented Apr 1, 2020

Ok, I see what you mean about the log level. There is "new" log warning added that is causing this issue. So, what is causing this? Is this some kind of error?

jfischer-no added a commit to jfischer-no/zephyr that referenced this issue Apr 16, 2020
Lower log level in a debug message.

Fixes: zephyrproject-rtos#22502

Signed-off-by: Johann Fischer <j.fischer@phytec.de>
@jfischer-no
Copy link
Collaborator

So, what is causing this? Is this some kind of error?

Just a wrong log level, probably accidentally changed for debugging.

MaureenHelm pushed a commit that referenced this issue Apr 17, 2020
Lower log level in a debug message.

Fixes: #22502

Signed-off-by: Johann Fischer <j.fischer@phytec.de>
hakehuang pushed a commit to hakehuang/zephyr that referenced this issue Jun 20, 2020
Lower log level in a debug message.

Fixes: zephyrproject-rtos#22502

Signed-off-by: Johann Fischer <j.fischer@phytec.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: USB Universal Serial Bus
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants