From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: Re: [PATCH v1] i2c-hid: introduce HID over i2c specification implementation Date: Sat, 6 Oct 2012 23:11:11 +0200 Message-ID: <20121006231111.6350c014@endymion.delvare> References: <1347630103-4105-1-git-send-email-benjamin.tissoires@gmail.com> <20121006220421.47f5fd56@endymion.delvare> <6F756B9E-DC87-4EF6-BB09-8A69A5F8C999@enac.fr> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <6F756B9E-DC87-4EF6-BB09-8A69A5F8C999@enac.fr> Sender: linux-input-owner@vger.kernel.org To: =?ISO-8859-1?B?U3TpcGhhbmU=?= Chatty Cc: "benjamin.tissoires" , Dmitry Torokhov , Jiri Kosina , Fabien =?UTF-8?B?QW5kcsOp?= , =?UTF-8?B?5YqJ5ZiJ6ae/?= , Ben Dooks , Wolfram Sang , linux-i2c@vger.kernel.org, USB list , linux-kernel@vger.kernel.org, Marcel Holtmann List-Id: linux-i2c@vger.kernel.org On Sat, 6 Oct 2012 22:30:00 +0200, St=E9phane Chatty wrote: > Le 6 oct. 2012 =E0 22:04, Jean Delvare a =E9crit : > > Looks like the wrong place for this driver. HID-over-USB support li= ves > > under drivers/hid, so your driver should go there as well. Not only > > this will be more consistent, but it also makes more sense: your dr= iver > > is a user, not an implementer, of the I2C layer, so it doesn't belo= ng > > to drivers/i2c. >=20 > This is a question I asked a few months back, but apparently not all = points of views had been expressed at the time. Currently, HID-over-USB= lives in drivers/hid, but HID-over-BT lives in drivers/bluetooth. When= I asked, Jiri explained that he maintained HID-over-USB and Marcel mai= ntained HID-over-BT, which explained the choices made. Let's try to sum= marize what we know now: >=20 > The question is what drives the choice of where to put HID-over-XXX, = among the following > 1- who the maintainer is. Here, Benjamin will probably maintain this= so it does not help. > 2- dependencies. HID-over-XXX depends on HID as much as it depends o= n XXX, so it does not help. > 3- data flow. Indeed, HID is a client of HID-over-XXX which is a cli= ent of XXX. Are there other parts of the kernel where this drives the c= hoice of where YYY-over-XXX lives? >=20 > Jiri, Marcel, Greg, others, any opinions? My vote is a clear 3. It took me a few years to kick all users (as opposed to implementers) of i2c from drivers/i2c and finding them a proper home, I'm not going to accept new intruders. Grouping drivers according to what they implement makes it a lot easier to share code and ideas between related drivers. If you want to convince yourself, just imagine the mess it would be if all drivers for PCI devices lived under drivers/pci. --=20 Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-input" = in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html