diff options
author | Nico Huber <nico.huber@secunet.com> | 2012-11-20 17:27:46 +0100 |
---|---|---|
committer | Patrick Georgi <patrick@georgi-clan.de> | 2012-11-24 08:43:21 +0100 |
commit | ce407e470d36bd446e7e1f02884c97a98423d5c4 (patch) | |
tree | 989ff1e649743b60b08224c7c88704bf9097dada /payloads/libpayload/configs/defconfig | |
parent | cd587f110bec55ff24a73a775a8c757332715f3f (diff) | |
download | coreboot-ce407e470d36bd446e7e1f02884c97a98423d5c4.tar.xz |
libpayload: Implement correct interrupt-queue linking for UHCI
The linking of interrupt queues into UHCI controller's framelist (in
uhci_create_intr_queue()) was incomplete. The implementation of
uhci_destroy_intr_queue() was even worse, looking like it wanted to
clean up more than uhci_create_intr_queue() did.
This patch follows the simple approach that we used for OHCI and EHCI:
Each slot in the framelist holds only one interrupt queue. Therefore, we
have to look for free slots each time we want to link an interrupt queue
into the framelist. In return, we have a much simpler structured
framelist.
With this, USB devices using interrupt transfers (e.g. keyboards) can be
detached cleanly from UHCI controllers. Also, more than one of such
devices can be attached without further risk.
Change-Id: I07b81a3b6f2cb3ff69515c973b3ae6321ad969aa
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: http://review.coreboot.org/1897
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
Diffstat (limited to 'payloads/libpayload/configs/defconfig')
0 files changed, 0 insertions, 0 deletions