Re: [PATCH] xhci: Add BayTrail to list of Intel switchable hosts

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, May 25, 2013 at 03:00:28AM +0400, Sergei Shtylyov wrote:
> Hello.
> 
> On 05/25/2013 01:54 AM, Sarah Sharp wrote:
> 
> >>>>>From: "Chew, Chiau Ee" <chiau.ee.chew@xxxxxxxxx>
> >>>>>Like the xHCI controller on Intel Panther Point and Lynx Point
> >>>>>chipsets, the xHCI controller on Intel BayTrail has also ports that
> >>>>>can be switched between the EHCI host controller.
> >>>>    s/between/to/
> >>>Sergei, DO NOT REVIEW PATCHES TO THE XHCI DRIVER.
> >>    OK, I'll try to follow another rule from you, though I'm getting
> >>tired of you telling me what not to do.
> >I've told you not to review patches to the xHCI driver three times now.
> >This is not "another rule", this is something I've asked you to avoid
> >already.
> 
>    Strange, I would have remembered if you told me *thrice*, yet
> what I clearly
> remember you told me is not to comment on your own patches.

Our recollections differ.  Fine.

I am telling you now, do not provide reviews for ANY xHCI driver
patches where the number of "noise" comments (grammar, spelling fixes,
extra parens, etc) outnumber the "signal" comments (bugs in the patch,
missing some idea, etc).

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




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]