| Age | Commit message (Collapse) | Author |
|---|
|
This code is written very carefully to always use an uncached view
of memory to read/write EDs. An uncached view must *always* be used,
or else cache behavior can corrupt the ED.
As part of this change, combine access into as few word-sized accesses
as possible. This makes the code perform better. Doing this involves
giving type names to the bitfields that make up the ED's data words.
|
|
|
|
The initial motivation for doing this is to remove the `used` flag
in the TD. If we use this flag, we end up being required to read from TDs
that the OHCI controller might be modifying (i.e. the OHCI controller
logically owns the TD). This happens when we try to allocate a
new, empty TD while the OHCI host controller is working on a transfer.
Move the `used` flag to `gtd_extra_data_t`. This data is only used
by the CPU, and the OHCI controller never accesses it.
The existing allocation method for TDs does *not* put an empty TD
onto each ED (i.e it does *not* do what is shown in Figure 5-6
of the OHCI specification). Instead, the NextTD field of the last
TD is set to 0. The TailP field of the ED is also set to 0.
This works in many cases. However, this implementation means that
the CPU may end up trying to write to the NextTD field of an
in-progress transfer while the OHCI host controller logically owns it.
Change the implementation to use an empty TD, as suggested by the
specification, for endpoints other than EP0. This avoids the
above issue. It is not necessary to make the change for EP0
because only at most one TD can ever be pending at a time.
The above change should also remove the need for the stall workaround.
In the future, we want to modify the code to access EDs through
an uncached mapping. Because uncached mappings are slow, we want to
access EDs as little as possible. Currently, when a TD completes,
we access an ED in order to figure out the device address and
endpoint number of the TD which was completed.
Because moving `used` to `gtd_extra_data_t` necessitates expanding it,
we have enough room to also store the device address and endpoint number
of the TD. This patch does so.
With the above two changes, we no longer need to access an ED when
a TD completes. Also remove the `index` field from TDs as it is
no longer necessary.
|
|
|
|
|
|
driver can execute xfer_is()
|
|
support at32 mcu (2)
|
|
|
|
mcu/sdk instead
add cmake support for f403a_407 and f423
|
|
|
|
enable host example build for samd21/d51
|
|
|
|
Add DWC2 cache maintenance routines for STM32
|
|
use stock iar linker
|
|
|
|
Signed-off-by: HiFiPhile <[email protected]>
|
|
support host mode
reformat hcd usbfs
add uart rx for ch32v20x bsp
|
|
|
|
Signed-off-by: HiFiPhile <[email protected]>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
add spinlock implementation for most rtos
|
|
|
|
|
|
|
|
TinyUSB does not provide any locking means to protect the DCD variables.
This can lead to race conditions when the user is trying to submit
a transfer while the device is being disconnected. This can cause
the device to be in an inconsistent state, leading to a crash or
undefined behavior.
This commit adds a spin-lock to protect the DCD variables during
device disconnect.
Closes https://github.com/espressif/esp-idf/issues/9691
Also reported in https://github.com/espressif/esp-usb/pull/131
|
|
|
|
Signed-off-by: HiFiPhile <[email protected]>
|
|
|
|
roothub that is currently doing debouncing delay
|
|
|
|
hcd_bus_info_get
streamline bus info to usbh_devies, also replace dev0 (renamed to dev0_bus)
|
|
dwc2/host: attach debouncing fixes
|
|
Fix 1st nak retry one frame shorter.
|
|
Signed-off-by: HiFiPhile <[email protected]>
|
|
command
- correctly carry out OUT transfer when PING is ack
|
|
Signed-off-by: HiFiPhile <[email protected]>
|
|
|
|
# Conflicts:
# src/portable/synopsys/dwc2/hcd_dwc2.c
|
|
Signed-off-by: HiFiPhile <[email protected]>
|
|
dwc2/host: immediately retry IN token for bInterval=1
|
