usb; xhci: Fix TRB offset calculations.
authorSarah Sharp <sarah.a.sharp@linux.intel.com>
Thu, 14 May 2009 18:44:14 +0000 (11:44 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 16 Jun 2009 04:44:51 +0000 (21:44 -0700)
commit6071d8363b7b284038069f1795a98372fbc1a48e
treee5b56ace01d240c49b1acbdc02f17cf6a2102fdc
parent7dd19e69d131ea34f74397559b422511e54d2911
usb; xhci: Fix TRB offset calculations.

Greg KH introduced a bug into xhci_trb_virt_to_dma() when he changed the
type of offset to dma_addr_t from unsigned int and dropped the casts to
unsigned int around the virtual address pointer subtraction.

trb and seg->trbs are both valid pointers to virtual addresses, so the
compiler will mod the subtraction by the size of union trb (16 bytes).
segment_offset is an unsigned long, which is guaranteed to be at least as
big as a void *.

Drop the void * casts in the first if statement because trb and seg->trbs
are both pointers of the same type (pointers to union trb).

Signed-off-by: Sarah Sharp <sarah.a.sharp@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/host/xhci-ring.c