From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: Two regressions on BYT/T ASUS T100TA 4.12-rc: #2 Keyboard no longer works Date: Fri, 9 Jun 2017 11:00:28 +0200 Message-ID: <20170609090028.GA30604@kroah.com> References: <20170608203904.73abb5e1@lxorguk.ukuu.org.uk> <0d3a087b-ee57-05be-a32a-a5610127db71@redhat.com> <20170609082703.GA29271@kroah.com> <8760g5ilut.fsf@miraculix.mork.no> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Return-path: Received: from out1-smtp.messagingengine.com ([66.111.4.25]:41937 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751671AbdFIJAg (ORCPT ); Fri, 9 Jun 2017 05:00:36 -0400 Content-Disposition: inline In-Reply-To: <8760g5ilut.fsf@miraculix.mork.no> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: =?iso-8859-1?Q?Bj=F8rn?= Mork Cc: Jiri Kosina , Hans de Goede , Linus Torvalds , Alan Stern , Benjamin Tissoires , Alan Cox , USB list , "linux-input@vger.kernel.org" On Fri, Jun 09, 2017 at 10:52:42AM +0200, Bjørn Mork wrote: > Greg KH writes: > > > On Fri, Jun 09, 2017 at 10:21:47AM +0200, Jiri Kosina wrote: > >> Longer-term, we'd ideally make 'generic' driver special and let it attach > >> as a 'last resort driver' if none of the specific driver picked the device > >> up during probe. But I don't think our current driver model allows this > >> easily ... or is there way I am not seeing? > > > > Nope, the driver model does not provide a way for this, sorry, it's been > > a complaint from the very beginning that we don't really know how to > > handle, except to make sure the order of the drivers loaded are the > > priority list, and that's not really a solution at all. > > Add a 'priority' field to struct device_driver and sort the list by this > field instead of load order? We've been over this before :) What happens when a new module is loaded, are you supposed to then disconnect a device from an existing driver, recan all of the existing drivers, and then bind the "correct" one then? sorry, greg k-h