Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752171Ab2KPO4F (ORCPT ); Fri, 16 Nov 2012 09:56:05 -0500 Received: from mail-lb0-f174.google.com ([209.85.217.174]:42140 "EHLO mail-lb0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751799Ab2KPO4D (ORCPT ); Fri, 16 Nov 2012 09:56:03 -0500 MIME-Version: 1.0 In-Reply-To: References: <1352731379-24683-1-git-send-email-benjamin.tissoires@gmail.com> Date: Fri, 16 Nov 2012 15:56:01 +0100 Message-ID: Subject: Re: [PATCH v3] i2c-hid: introduce HID over i2c specification implementation From: Benjamin Tissoires To: Jiri Kosina , ramalingamc@nvidia.com Cc: Dmitry Torokhov , Stephane Chatty , fabien.andre@gmail.com, scott.liu@emc.com.tw, Jean Delvare , JJ Ding , Jiri Slaby , Shubhrajyoti Datta , linux-i2c@vger.kernel.org, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2343 Lines: 62 On Thu, Nov 15, 2012 at 3:04 PM, Benjamin Tissoires wrote: > On Thu, Nov 15, 2012 at 2:51 PM, Jiri Kosina wrote: >> On Mon, 12 Nov 2012, Benjamin Tissoires wrote: >> >>> Microsoft published the protocol specification of HID over i2c: >>> http://msdn.microsoft.com/en-us/library/windows/hardware/hh852380.aspx >>> >>> This patch introduces an implementation of this protocol. >>> >>> This implementation does not includes the ACPI part of the specification. >>> This will come when ACPI 5.0 devices enumeration will be available. >>> >>> Once the ACPI part is done, OEM will not have to declare HID over I2C >>> devices in their platform specific driver. >>> >>> Signed-off-by: Benjamin Tissoires >> >> Out of curiosity -- has this been tested on a real device (is there any >> such device available anyway?), or is that just the implementation of the >> defined protocol? > > It has been tested on an ELAN microelectronics device (a prototype), > on an odroid-x board. That's how we figure out the bug in the > set_report command. > I think we manage to test all main features of the protocol > (get_report, irqs, hid descriptor, report descriptors, set_report). > > I'm currently waiting for a Synaptics touchpad to check if it's also > working with their devices. > > The thing is that HID over i2c for x86 platform will presumably > require the Haswell platform from Intel (we need ACPI 5 for > enumeration), but it would be very nice to get this in the kernel just > before hardware arrive on the market :) > However, I won't be surprise if android OEMs also start using this > specification because it won't force them to write kernel drivers... And as a complement, Ramalingam tested it for Nvidia on an early NVIDIA's Tegra reference board for PISMO which is registered at http://www.arm.linux.org.uk/developer/machines/list.php?id=4439 , with a HID over i2c keyboard. He is up to test also his Synaptics touchpad. Cheers, Benjamin > > Cheers, > Benjamin > >> >> Thanks, >> >> -- >> Jiri Kosina >> SUSE Labs -- 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/