From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751825AbaEUGbv (ORCPT ); Wed, 21 May 2014 02:31:51 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:38882 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750858AbaEUGbu (ORCPT ); Wed, 21 May 2014 02:31:50 -0400 Date: Wed, 21 May 2014 15:31:38 +0900 From: Greg KH To: Dan Williams Cc: Takashi Iwai , Mathias Nyman , USB list , "linux-kernel@vger.kernel.org" , Sarah Sharp , Holger Hans Peter Freyther , Oliver Neukum Subject: Re: [PATCH 02/10] xhci: 'noxhci_port_switch' kernel parameter Message-ID: <20140521063138.GA5952@kroah.com> References: <1399566363-25837-1-git-send-email-mathias.nyman@linux.intel.com> <1399566363-25837-3-git-send-email-mathias.nyman@linux.intel.com> <20140520010137.GA8146@kroah.com> <537B24B8.9050303@linux.intel.com> <20140520203432.GB30178@kroah.com> <20140521002720.GC997@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 20, 2014 at 11:21:03PM -0700, Dan Williams wrote: > On Tue, May 20, 2014 at 5:27 PM, Greg KH wrote: > >> Greg, > >> > >> Sorry, I don't think it is fair to users to force them to re-compile > >> their kernel to get their device to work. > > > > I totally agree. > > > >> Granted, I'm new to USB > >> development, but the rate of reports of endpoint devices that mess up > >> and require quirks in the hcd-driver or usb-core seems un-ending to > >> me. So, I don't think it is fair to expect that the tide of quirky > >> devices will be stemmed in any reasonable amount of time. Having a > >> "works with noxhci_port_switch" report from users is good data (hmm, I > >> think a printk to tell users to file a report upstream if the option > >> resolves their issue is needed). > > > > How about just adding a debugfs file instead? That way, once you fix > > this, we can then remove it and no one will care. > > The only thing stopping me from saying "deal." is that this darn > things is presently a pci quirk. So it happens well before the user > has a chance to manually override it with a debugfs file. Then have the debugfs file disconnect the device and reconnect it. > Let me look into delaying the quirk until the driver loads, because > ideally the right interface for this is "blacklist xhci_hcd" in > /etc/modprobe.conf. Which really doesn't work for systems / distros that build the driver into the kernel. thanks, greg k-h