Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752660Ab2JGQVM (ORCPT ); Sun, 7 Oct 2012 12:21:12 -0400 Received: from smtp.intuilab.com ([93.93.42.171]:38046 "EHLO zimbra.intuilab.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751593Ab2JGQVD convert rfc822-to-8bit (ORCPT ); Sun, 7 Oct 2012 12:21:03 -0400 Subject: Re: [PATCH v1] i2c-hid: introduce HID over i2c specification implementation Mime-Version: 1.0 (Apple Message framework v1084) Content-Type: text/plain; charset=iso-8859-1 From: =?iso-8859-1?Q?St=E9phane_Chatty?= In-Reply-To: Date: Sun, 7 Oct 2012 18:20:54 +0200 Cc: Jiri Kosina , Jean Delvare , Dmitry Torokhov , =?iso-8859-1?Q?Fabien_Andr=E9?= , =?utf-8?B?5YqJ5ZiJ6ae/?= , Ben Dooks , Wolfram Sang , linux-i2c@vger.kernel.org, USB list , linux-kernel@vger.kernel.org, Marcel Holtmann Content-Transfer-Encoding: 8BIT Message-Id: <91E2364F-9A56-4222-A876-BEC6D474EC75@enac.fr> References: <1347630103-4105-1-git-send-email-benjamin.tissoires@gmail.com>< 20121006220421.47f5fd56@endymion.delvare><6F756B9E-DC87-4EF6-BB09-8A69A5F8C 999@enac.fr><20121006231111.6350c014@endymion.delvare><914C6C58-5A3B-4F7D-94BE-EF6868ED5D9D@enac.fr> To: Benjamin Tissoires X-Mailer: Apple Mail (2.1084) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1208 Lines: 41 Le 7 oct. 2012 ? 18:07, Benjamin Tissoires a ?crit : >>> >>> Basically, to me this all boils down to the question -- what is more >>> important: low-level transport being used, or the general function of the >>> device? >>> >>> To me, it's the latter, and as such, everything would belong under >>> drivers/hid. >> >> Then shouldn't is be drivers/input, rather? > > Ouch, it will introduce more and more complexity. Purely rhetorical question, I agree. But still. > > It seems that hid transport layers should go in drivers/hid. > However, I don't like mixing the transport layer and the final > drivers. Maybe this is the time to rework a little bit the tree. > To minimize the moves, we could introduce: > drivers/hid/busses/usb > drivers/hid/busses/i2c > drivers/hid/busses/bluetooth What about creating drivers/hid/core and move all generic stuff there? That is: drivers/hid/core drivers/hid/usb drivers/hid/i2c drivers/hid/bluetooth Cheers, St. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/