sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

USB hub: Minor fix to reduce the amount of debug garbage on console in certain configurations

Browse Source
This commit is contained in:
Gregory Nutt 2015-04-30 13:03:28 -06:00
parent 9f34a34d8c
commit e4f538446b
2 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/usbhost/usbhost_findclass.c
View File

@ -160,7 +160,7 @@ const struct usbhost_registry_s *usbhost_findclass(const struct usbhost_id_s *id
uvdbg("Looking for class:%d subclass:%d protocol:%d vid:%04x pid:%04x\n",
id->base, id->subclass, id->proto, id->vid, id->pid);
/* g_classregistry is a singly-linkedlist of class ID information added by
/* g_classregistry is a singly-linked list of class ID information added by
* calls to usbhost_registerclass(). Since this list is accessed from USB
* host controller interrupt handling logic, accesses to this list must be
* protected by disabling interrupts.

14
drivers/usbhost/usbhost_hub.c
View File

@ -1161,13 +1161,23 @@ static void usbhost_callback(FAR void *arg, int result)
* transfer will pend until data is available (OHCI and EHCI). On lower
* end host controllers (like STM32 and EFM32), the transfer will fail
* immediately when the device NAKs the first attempted interrupt IN
* transfer (with result == EGAIN). In that case (or in the case of
* transfer (with result == EAGAIN). In that case (or in the case of
* other errors), we must fall back to polling.
*/
if (result != OK)
{
ulldbg("ERROR: Transfer failed: %d\n", result);
/* This debug output is good to know, but really a nuisance for
* those configurations where we have to fall back to polling.
* FIX: Don't output the message is the result is EAGAIN.
*/
#if defined(CONFIG_DEBUG_USB) && !defined(CONFIG_DEBUG_VERBOSE)
if (result != EAGAIN)
#endif
{
ulldbg("ERROR: Transfer failed: %d\n", result);
}
/* Indicate there there is nothing to do. So when the work is
* performed, nothing will happen other than we will set to receive