Re: kern/50249 (XHCI driver missing delay after address set)

[Sprow <webpages%sprow.co.uk@localhost>]

The following reply was made to PR kern/50249; it has been noted by GNATS.

From: Sprow <webpages%sprow.co.uk@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: kern/50249 (XHCI driver missing delay after address set)
Date: Wed, 23 Sep 2015 08:30:21 +0100

 On 23 Sep, <skrll%NetBSD.org@localhost> wrote:
 > Active development of xhci is happening on the nick-nhusb branch which
 > has the delay after address set already.
 
 I've been following the branch with interest, but assumed the trunk was still
 open to submissions since only the bold/brave would be using the experimental
 version. Hence adding the SET_ADDRESS_SETTLE to the trunk benefits those
 users.
 
 > The field rename is not required, imo.
 
 My aim was to (try to) make the code clearer, because it's pretty opaque at
 the moment. If I search the XHCI spec for "CS" I get false hits for the
 command stop flag throughout the document (and the same register!) whereas
 "RCS" is unique and describes what the variable holds.