[haiku-commits] Re: haiku: hrev52924 - src/add-ons/kernel/busses/usb

  • From: waddlesplash <waddlesplash@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 24 Feb 2019 14:58:59 -0500

On Sun, Feb 24, 2019 at 3:31 AM Jérôme Duval <jerome.duval@xxxxxxxxx> wrote:

-               // Status Stage is an OUT transfer when the device sent data
-               // (see XHCI 1.1 § 4.11.2.2 Table 4-6)
+               // Status Stage is an OUT transfer when the device is 
sending data.
+               // (XHCI 1.1 § 4.11.2.2 Table 4-6 p205.)

Augustin, nice work on the XHCI driver and thank you for reviewing my commits 
:)

:)

I wish I understood all this better. I'm learning quickly, but it's
extremely frustrating to just bang your head against the specs or
other drivers until something falls out of them...

About the verb tense, I used the simple past tense, because when this
status descriptor is processed by the host controller, the data
transmission already happened. The present progressive tense doesn't
make much sense to me in this context but I'm not native English as
you know.

Right, but these are all run as a unit, and so if we think of them as
a logical group, then they are "one action" and so the present
progressive makes more sense to me at least. But maybe that's just my
preference...

About the page number, well, the table actually starts at page 204,
you might want to tweak again.

Yes, but the line for this specific case is on p205. :)

-waddlesplash

Other related posts:

  1. Home
  2. haiku-commits
  3. Archive
  4. 02-2019