10 Apr, 2020
1 commit
-
Rather than keeping the asynchronous schedule running always, keep it
running only across USB mass storage transfers for now, as it seems
that keeping it running all the time interferes with certain control
transfers during device enumeration.Note that running the async schedule all the time should not be an
issue, especially on EHCI HCD, as that one implements most of the
transfers using async schedule.Note that we have usb_disable_asynch(), which however is utterly broken.
The usb_disable_asynch() blocks the USB core from doing async transfers
by setting a global flag. The async schedule should however be disabled
per USB controller. Moreover, setting a global flag does not prevent the
controller from using the async schedule, which e.g. the EHCI HCD does.This patch implements additional callback to the controller, which
permits it to lock the async schedule and keep it running across
multiple transfers. Once the schedule is unlocked, it must also be
disabled. This thus prevents the async schedule from running outside
of the USB mass storage transfers.Signed-off-by: Marek Vasut
Cc: Lukasz Majewski
Cc: Tom Rini
Tested-by: Tom Rini [omap3_beagle, previously failing]
06 Feb, 2020
1 commit
-
At present dm/device.h includes the linux-compatible features. This
requires including linux/compat.h which in turn includes a lot of headers.
One of these is malloc.h which we thus end up including in every file in
U-Boot. Apart from the inefficiency of this, it is problematic for sandbox
which needs to use the system malloc() in some files.Move the compatibility features into a separate header file.
Signed-off-by: Simon Glass
11 Sep, 2019
2 commits
-
This will be used to implement non-blocking keyboard polling in case of
errors.Signed-off-by: Michal Suchanek
-
This aligns naming with usb_bulk_msg and usb_control_msg.
Signed-off-by: Michal Suchanek
27 Nov, 2018
1 commit
-
This allows to disable the USB driver model in SPL because it checks
the CONFIG_SPL_DM_USB variable for SPL builds. Nothing changes for
regular non-SPL builds.Signed-off-by: Sven Schwermer
07 May, 2018
1 commit
-
When U-Boot started using SPDX tags we were among the early adopters and
there weren't a lot of other examples to borrow from. So we picked the
area of the file that usually had a full license text and replaced it
with an appropriate SPDX-License-Identifier: entry. Since then, the
Linux Kernel has adopted SPDX tags and they place it as the very first
line in a file (except where shebangs are used, then it's second line)
and with slightly different comment styles than us.In part due to community overlap, in part due to better tag visibility
and in part for other minor reasons, switch over to that style.This commit changes all instances where we have a single declared
license in the tag as both the before and after are identical in tag
contents. There's also a few places where I found we did not have a tag
and have introduced one.Signed-off-by: Tom Rini
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
4 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
