]> git.ipfire.org Git - thirdparty/u-boot.git/commit
usb: xhci: fix DMA address corruption in abort_td
authorANANDHAKRISHNAN S <anandhakrishnansasidharan@gmail.com>
Thu, 22 Jan 2026 05:27:54 +0000 (10:57 +0530)
committerMarek Vasut <marek.vasut+usb@mailbox.org>
Sun, 25 Jan 2026 09:40:42 +0000 (10:40 +0100)
commitba94fb3003bd4155284c00c1b37613dd8e284816
tree1a464aab3215a15aef6af23be4a786cc3f8c6771
parent8de6e8f8a076d2c9b6d38d8563db135c167077ec
usb: xhci: fix DMA address corruption in abort_td

When aborting a Transfer Descriptor (TD), the xHCI driver updates the
device dequeue pointer by converting the virtual enqueue TRB pointer
into a DMA address.

Previously, the code OR-ed the ring's Dequeue Cycle State (DCS) bit into
the virtual TRB pointer before passing it to xhci_trb_virt_to_dma().
This produced an unaligned virtual address (e.g. ending in 0x...1).

Inside xhci_trb_virt_to_dma(), the offset calculation:

segment_offset = trb - seg->trbs;

operated on this unaligned pointer, resulting in an incorrect TRB index.
In wraparound cases, this caused the bounds check to fail and the
function to return 0.

As a result, a SET_DEQ_PTR command was issued with a DMA address of 0x0,
leading to controller hangs and transfer timeouts, most commonly when
aborting TDs near the end of a ring segment (e.g. index 63).

Fix this by translating the aligned virtual TRB pointer to a DMA address
first, and only then applying the DCS bit to the resulting physical
address.

Reviewed-by: Marek Vasut <marek.vasut@mailbox.org>
Signed-off-by: ANANDHAKRISHNAN S <anandhakrishnan.s@dicortech.com>
drivers/usb/host/xhci-ring.c