01 Oct, 2017
3 commits
-
For xHCI it is not possible to read a device descriptor before it
has been assigned an address. That's why usb_setup_descriptor()
was called with 'do_read' being false. But we really need try to
read the device descriptor before starting any real communication
with the default control endpoint.Signed-off-by: Bin Meng
-
Full speed device endpoint 0 can have 8/16/32/64 bMaxPacketSize0.
Other speed devices report fixed value per USB spec. So it only
makes sense if we send a get device descriptor with 64 bytes to
full speed devices.While we are here, update the comment block to be within 80 cols.
Signed-off-by: Bin Meng
-
Normal endpoint descriptor size is 7, but for audio extension it is
9. Handle that correctly when parsing endpoint descriptor.Signed-off-by: Bin Meng
04 Jul, 2017
1 commit
-
There was for long time no activity in the 4xx area.
We need to go further and convert to Kconfig, but it
turned out, nobody is interested anymore in 4xx,
so remove it.Signed-off-by: Heiko Schocher
24 Sep, 2016
1 commit
-
This makes functions much simpler.
Signed-off-by: Masahiro Yamada
04 Jun, 2016
1 commit
-
Move CONFIG_USB_XHCI to defconfig files for all boards, renaming it
into CONFIG_USB_XHCI_HCD.As commented in the help of "config USB_XHCI" entry, this has been
a TODO for a long time; now CONFIG_USB_XHCI_HCD and CONFIG_USB_XHCI
have been unified in favor of the former.Note:
Some boards define CONFIG_USB_XHCI in their headers without
CONFIG_USB, which does not meet the "depends on" in Kconfig.
I added CONFIG_USB=y for those boards when converting.
Otherwise, they would fail to build.Signed-off-by: Masahiro Yamada
07 May, 2016
3 commits
-
The Kingston DT Ultimate USB 3.0 stick is sensitive to this first
Get Descriptor request and if the request is not in a separate
microframe, the stick refuses to operate. Add slight delay, which
is enough for one microframe to pass on any USB spec revision.Signed-off-by: Marek Vasut
Cc: Chin Liang See
Cc: Dinh Nguyen
Cc: Hans de Goede
Cc: Stefan Roese
Cc: Stephen Warren -
Some devices, like the SanDisk Cruzer Pop need some time to process
the Set Configuration request, so wait a little until they are ready.Signed-off-by: Marek Vasut
Cc: Chin Liang See
Cc: Dinh Nguyen
Cc: Hans de Goede
Cc: Stefan Roese
Cc: Stephen Warren -
The pointer should always be inited to NULL, not zero (0). These are
two different things and not necessarily equal.Signed-off-by: Marek Vasut
Cc: Chin Liang See
Cc: Dinh Nguyen
Cc: Hans de Goede
Cc: Stefan Roese
Cc: Stephen Warren
23 Mar, 2016
1 commit
-
Signed-off-by: Vagrant Cascadian
Acked-by: Marek Vasut
Reviewed-by: Simon Glass
Reviewed-by: Tom Rini
21 Mar, 2016
1 commit
-
Debugging has shown, that all USB hubs are being reset twice while
USB scanning. This introduces additional delays and makes USB scanning
even more slow. Testing has shown that this 2nd USB hub reset doesn't
seem to be necessary.This patch now removes this 2nd USB hub reset. Resulting in faster USB
scan time. Here the current numbers:Without this patch:
=> time usb start
starting USB...
USB0: USB EHCI 1.00
scanning bus 0 for devices... 9 USB Device(s) foundtime: 24.003 seconds
With this patch:
=> time usb start
starting USB...
USB0: USB EHCI 1.00
scanning bus 0 for devices... 9 USB Device(s) foundtime: 20.392 seconds
So ~3.6 seconds of USB scanning time reduction.
Signed-off-by: Stefan Roese
Cc: Simon Glass
Acked-by: Hans de Goede
Tested-by: Stephen Warren
Cc: Marek Vasut
16 Jan, 2016
1 commit
-
We support max USB_MAXENDPOINTS, so need to use
"epno >= USB_MAXENDPOINTS", but not "epno > USB_MAXENDPOINTS".
If use ">", we may exceeds the array of if_desc->ep_desc.Signed-off-by: Peng Fan
Cc: Simon Glass
Cc: Hans de Goede
Cc: Marek Vasut
Cc: Paul Kocialkowski
Cc: "Stefan Brüns"
Cc: Vincent Palatin
31 Dec, 2015
2 commits
-
Start split and complete split tokens need the hub address and the
downstream port of the first HS hub (device view).The core of the function was duplicated in both host/ehci_hcd and
musb-new/usb-compat.h.Signed-off-by: Stefan Brüns
Reviewed-by: Marek Vasut
Reviewed-by: Hans de Goede
Tested-by: Hans de Goede -
The configuration descriptor includes all interface, endpoint and
auxiliary descriptors (e.g. report, union) so 512 bytes may not be enough.Signed-off-by: Stefan Brüns
Reviewed-by: Marek Vasut
Reviewed-by: Simon Glass
12 Sep, 2015
1 commit
-
Now that we have a new header file for cache-aligned allocation, we should
move the stack-based allocation macro there also.Signed-off-by: Simon Glass
22 Jul, 2015
4 commits
-
Add an usb_device parameter to usb_reset_root_port so that it knows which
root-port it is resetting. This is necessary for proper device-model support
for usb_reset_root_port.Also remove a duplicate declaration of usb_reset_root_port() from usb.h .
Signed-off-by: Hans de Goede
Acked-by: Simon Glass -
Pass the usb_device instead of the portnr to usb_legacy_port_reset and
rename it to usb_hub_port_reset as there is nothing legacy about it.Signed-off-by: Hans de Goede
Acked-by: Simon Glass -
Drop the unneeded portnr function argument, the portnr is part of the
usb_device struct which is passed via the dev argument.Signed-off-by: Hans de Goede
Acked-by: Simon Glass -
The device-model usb_legacy_port_reset function calls the device-model
usb_port_reset function which is a 1 on 1 copy of the non dm
usb_legacy_port_reset and this is the only use of usb_port_reset in all
of u-boot.Drop both, and alway use the usb_legacy_port_reset() version in
common/usb.c .Also while at it make it static as it is only used in common/usb.c .
Signed-off-by: Hans de Goede
Acked-by: Simon Glass
29 May, 2015
1 commit
-
The compiler did not catch this as it was marked __maybe_unused.
Signed-off-by: Hans de Goede
15 May, 2015
2 commits
-
Provide a function to detect USB device insertion/removal in order to
avoid having to do USB enumeration in a tight loop when trying to detect
peripheral hotplugging.Signed-off-by: Vincent Palatin
Reviewed-by: Stefan Reinauer
Tested-by: Stefan Reinauer
Signed-off-by: Simon Glass -
Propagate the error returned by submit_control_msg() rather then always
returning -EIO when the hcd code indicates an error.Signed-off-by: Hans de Goede
06 May, 2015
1 commit
-
This fixes descriptor reading of lowspeed devices through ohci not working.
Signed-off-by: Hans de Goede
Acked-by: Marek Vasut
19 Apr, 2015
7 commits
-
Add the required #ifdefs and remove unwanted data structures so that the
USB uclass will be able to use this file.Signed-off-by: Simon Glass
Reviewed-by: Marek Vasut -
This function now calls usb_setup_device() to set up the device and
usb_hub_probe() to check if it is a hub. The XHCI special case is now a
parameter to usb_setup_device(). The latter will be used by the USB uclass
when it is added, since it does not rely on any CONFIGs or legacy data
structures.Signed-off-by: Simon Glass
Bug-fixes for descriptor reading and usb_new_device() return value
Signed-off-by: Hans de Goede
Reviewed-by: Marek Vasut
Tested-by: Tom Rini -
Move the code that sets up the device with a new address into its own
function, usb_prepare_device().Signed-off-by: Simon Glass
Reviewed-by: Marek Vasut -
usb_new_device() is far too long and does far too much. As a first step, move
the code that does initial setup and reads a descriptor into its own function
called usb_setup_descriptor().For XHCI the init order is different - we set up the device but don't
actually read the descriptor until after we set an address. Support this
option as a parameter to usb_setup_descriptor().Avoid changing this torturous code more than necessary to make it easy to
review.Signed-off-by: Simon Glass
Reviewed-by: Marek Vasut -
Move the port reset code into its own function. Rename usb_hub_reset() to
indicate that is is now a legacy function.Signed-off-by: Simon Glass
Reviewed-by: Marek Vasut -
This CONFIG is not used anywhere in U-Boot, so drop it.
Signed-off-by: Simon Glass
Acked-by: Marek Vasut -
This function returns NULL on error at present. Adjust it so that we can
return a real error, as is needed with driver model. Also improve the
error handling in its caller, usb_hub_port_connect_change(), and adjust
the code order to prepare for driver model.Signed-off-by: Simon Glass
Reviewed-by: Marek Vasut
14 Apr, 2015
5 commits
-
Invoked board_usb_cleanup for cleaning up initialized USB. It
will be invoked if the user enterts ctrl-C.Signed-off-by: Kishon Vijay Abraham I
Reviewed-by: Lukasz Majewski -
When fetching the first descriptor from a new device, only validate that
we received at least 8 bytes, not that we received the entire descriptor.
The reasoning is:
- The code only uses fields in the first 8 bytes, so that's all we need
to have fetched at this stage.
- The smallest maxpacket size is 8 bytes. Before we know the actual
maxpacket the device uses, the USB controller may only accept a single
packet (see the DWC2 note in the comment added in the commit).
Consequently we are only guaranteed to receive 1 packet (at least 8
bytes) even in a non-error case.Fixes: 1a7758044b04 ("usb: Early failure when the first descriptor read
fails or is invalid")
Cc: Paul Kocialkowski
Signed-off-by: Stephen Warren -
This may happen when using an USB1 device on a controller that only supports
USB2 (e.g. EHCI). Reading the first descriptor will fail (read 0 byte), so we
can abort the process at this point instead of failing later and wasting time.Signed-off-by: Paul Kocialkowski
-
This checks that a new USB device is correctly initialized and frees it if not.
In addition, this doesn't report that USB was started when no device was found.Signed-off-by: Paul Kocialkowski
-
This makes use of errno return codes for representing error codes in a unified
way.Signed-off-by: Paul Kocialkowski
18 Jan, 2015
2 commits
-
This commit fixes a number of issues with the reset sequence of musb-new
in host mode:1) Our usb device probe relies on a second device reset being done after the
first descriptors read. Factor the musb reset code into a usb_reset_root_port
function (and add this as an empty define for other controllers), and call
this when a device has no parent.2) Just like with normal usb controllers there needs to be a delay after
reset, for normal usb controllers, this is handled in hub_port_reset, add a
delay to usb_reset_root_port.3) Sync the musb reset sequence with the upstream kernel, clear all bits of
power except bits 4-7, and increase the time reset is asserted to 50 ms.With these fixes an usb keyboard I have now always enumerates properly, where
as earlier it would only enumerare properly once every 5 tries.Signed-off-by: Hans de Goede
-
Before this commit u-boot would print the following on boot with musb and
no usb device plugged in:starting USB...
USB0: Port not available.
USB error: all controllers failed lowlevel initThis commit changes this to:
starting USB...
USB0: Port not available.Which is the correct thing to do since the low-level init went fine.
Signed-off-by: Hans de Goede
08 Dec, 2014
1 commit
-
Including is enough for general use.
Signed-off-by: Masahiro Yamada
04 Nov, 2014
1 commit
-
If the device has a parent, it is instantiated from usb_hub_port_connect_change
and the portnr is right there in dev->portnr, so there is no need for this
whole dance to look it up.Signed-off-by: Hans de Goede
29 Aug, 2014
1 commit
-
As we support both Host and Device mode operation, an OTG controller
can return -ENODEV on a port which it found to be in Device mode during
Host mode scan for devices. In case -ENODEV is returned, print that the
port is not available and continue instead of screaming a bloody error
message.Signed-off-by: Marek Vasut