dm: usb: Fix finding of first upstream usb-2 hub in the ehci dm code
authorHans de Goede <hdegoede@redhat.com>
Tue, 5 May 2015 09:54:34 +0000 (11:54 +0200)
committerSimon Glass <sjg@chromium.org>
Wed, 6 May 2015 02:58:42 +0000 (20:58 -0600)
commitfcdd8aaa48dfcac492cbac5436d9edc3f5928325
treee3f2589dae5d6b02310f61edd3cf1c0bbb0f3227
parent25c8ebdfcd9b2fca08ccf6bcc37de9ba388ad458
dm: usb: Fix finding of first upstream usb-2 hub in the ehci dm code

The ehci driver model code for finding the first upstream usb-2 hub before
this commit has a number of issues:

1) "if (!ttdev->speed != USB_SPEED_HIGH)" does not work because the '!'
   takes presedence over the '!=' this should simply be
   "if (ttdev->speed == USB_SPEED_HIGH)"
2) It makes ttdev point to the first upstream usb-2 hub, but ttdev should
   point to the last usb-1 device before the first usb-2 hub (when going
   upstream from the device), as ttdev is used to find the port of the
   first usb-2 hub to which the the last usb-1 device is connected.
3) parent_devnum however should be set to the devnum of the first usb-2
   hub, so we need to keep pointers around to both usb_device structs.

To complicate things further during enumeration usb_device.dev will point
to the parent udevice, where as during normal use it will point to
the actual udevice, we must handle both cases correctly.

This commit fixes all this making usb-1 devices attached to usb-2 hubs,
including usb-1 devices attached to usb-1 hubs attached to usb-2 hubs, work.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Simon Glass <sjg@chromium.org>
drivers/usb/host/ehci-hcd.c