On Wed, 19 Dec 2012, Jan Beulich wrote:

> Debug port in-use determination must be done before the controller gets
> reset the first time, i.e. before the call to ehci_setup() as of commit
> 1a49e2ac9651df7349867a5cf44e2c83de1046af. That commit effectively
> rendered commit 9fa5780beea1274d498a224822397100022da7d4 useless.

Quite right, thank you for finding and fixing this.

> While moving that code around, also fix the BAR determination - the
> respective capability field is a 3- rather than a 2-bit one -, and use
> PCI_CAP_ID_DBG instead of the literal 0x0a.
> 
> It's unclear to me whether the debug port functionality is important
> enough to warrant fixing this in stable kernels too.

I think we should.  Greg, please queue this for all stable kernels that 
include the 1a49e2ac commit.

> Signed-off-by: Jan Beulich <jbeul...@suse.com>
> Cc: Alan Stern <st...@rowland.harvard.edu>
> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>

Acked-by: Alan Stern <st...@rowland.harvard.edu>

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to