From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933936AbcCISiG (ORCPT ); Wed, 9 Mar 2016 13:38:06 -0500 Received: from mail-ig0-f171.google.com ([209.85.213.171]:34513 "EHLO mail-ig0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933893AbcCIShx (ORCPT ); Wed, 9 Mar 2016 13:37:53 -0500 MIME-Version: 1.0 In-Reply-To: <20160308192539.GA23955@dtor-ws> References: <20160308192539.GA23955@dtor-ws> Date: Wed, 9 Mar 2016 10:37:51 -0800 Message-ID: Subject: Re: [PATCH] HID: i2c-hid: Add hid-over-i2c name to i2c id table From: Andrew Duggan To: Dmitry Torokhov Cc: Jiri Kosina , Benjamin Tissoires , Mika Westerberg , Benson Leung , linux-input , linux-kernel@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 8, 2016 at 11:25 AM, Dmitry Torokhov wrote: > When using the device tree binding OF compatible = "hid-over-i2c" the > i2c id table also needs to have that name in order to auto load this > driver, since i2c core reports module alias as i2c: where > is compatible string of OF binding stripped of manufacturer's > prefix. > > Signed-off-by: Benson Leung > Signed-off-by: Dmitry Torokhov Tested-by: Andrew Duggan > --- > drivers/hid/i2c-hid/i2c-hid.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c > index b921693..a2c6c98 100644 > --- a/drivers/hid/i2c-hid/i2c-hid.c > +++ b/drivers/hid/i2c-hid/i2c-hid.c > @@ -1191,6 +1191,7 @@ static const struct dev_pm_ops i2c_hid_pm = { > > static const struct i2c_device_id i2c_hid_id_table[] = { > { "hid", 0 }, > + { "hid-over-i2c", 0 }, > { }, > }; > MODULE_DEVICE_TABLE(i2c, i2c_hid_id_table); > -- > 2.7.0.rc3.207.g0ac5344 > > > -- > Dmitry > -- > 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