* [PATCH] HID: i2c-hid: add platform data for quirks @ 2013-10-03 8:48 Bibek Basu 2013-10-09 10:26 ` Jiri Kosina 0 siblings, 1 reply; 4+ messages in thread From: Bibek Basu @ 2013-10-03 8:48 UTC (permalink / raw) To: benjamin.tissoires; +Cc: linux-tegra, jkosina, kadduri, linux-input, Bibek Basu Some HID device does not responds to INPUT/FEATURE report query during initialization. As a result HID driver throws error even when its not mandatory according to the HID specification. Add platform data to provide quirk information to the driver so that init query is not done on such devices. Signed-off-by: Bibek Basu <bbasu@nvidia.com> --- drivers/hid/i2c-hid/i2c-hid.c | 1 + include/linux/i2c/i2c-hid.h | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c index fd7ce37..a7aec2c 100644 --- a/drivers/hid/i2c-hid/i2c-hid.c +++ b/drivers/hid/i2c-hid/i2c-hid.c @@ -1017,6 +1017,7 @@ static int i2c_hid_probe(struct i2c_client *client, hid->version = le16_to_cpu(ihid->hdesc.bcdVersion); hid->vendor = le16_to_cpu(ihid->hdesc.wVendorID); hid->product = le16_to_cpu(ihid->hdesc.wProductID); + hid->quirks = platform_data->quirks; snprintf(hid->name, sizeof(hid->name), "%s %04hX:%04hX", client->name, hid->vendor, hid->product); diff --git a/include/linux/i2c/i2c-hid.h b/include/linux/i2c/i2c-hid.h index 7aa901d..12e682d 100644 --- a/include/linux/i2c/i2c-hid.h +++ b/include/linux/i2c/i2c-hid.h @@ -17,7 +17,7 @@ /** * struct i2chid_platform_data - used by hid over i2c implementation. * @hid_descriptor_address: i2c register where the HID descriptor is stored. - * + * @quirks: quirks, if any for i2c-hid devices * Note that it is the responsibility of the platform driver (or the acpi 5.0 * driver, or the flattened device tree) to setup the irq related to the gpio in * the struct i2c_board_info. @@ -31,6 +31,7 @@ */ struct i2c_hid_platform_data { u16 hid_descriptor_address; + u32 quirks; }; #endif /* __LINUX_I2C_HID_H */ -- 1.8.1.5 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] HID: i2c-hid: add platform data for quirks 2013-10-03 8:48 [PATCH] HID: i2c-hid: add platform data for quirks Bibek Basu @ 2013-10-09 10:26 ` Jiri Kosina [not found] ` <alpine.LNX.2.00.1310091225230.13289-ztGlSCb7Y1iN3ZZ/Hiejyg@public.gmane.org> 0 siblings, 1 reply; 4+ messages in thread From: Jiri Kosina @ 2013-10-09 10:26 UTC (permalink / raw) To: Bibek Basu; +Cc: benjamin.tissoires, linux-tegra, kadduri, linux-input On Thu, 3 Oct 2013, Bibek Basu wrote: > Some HID device does not responds to INPUT/FEATURE > report query during initialization. As a result HID driver > throws error even when its not mandatory according to the > HID specification. Add platform data to provide > quirk information to the driver so that init query is not > done on such devices. > > Signed-off-by: Bibek Basu <bbasu@nvidia.com> > --- > drivers/hid/i2c-hid/i2c-hid.c | 1 + > include/linux/i2c/i2c-hid.h | 3 ++- > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c > index fd7ce37..a7aec2c 100644 > --- a/drivers/hid/i2c-hid/i2c-hid.c > +++ b/drivers/hid/i2c-hid/i2c-hid.c > @@ -1017,6 +1017,7 @@ static int i2c_hid_probe(struct i2c_client *client, > hid->version = le16_to_cpu(ihid->hdesc.bcdVersion); > hid->vendor = le16_to_cpu(ihid->hdesc.wVendorID); > hid->product = le16_to_cpu(ihid->hdesc.wProductID); > + hid->quirks = platform_data->quirks; > > snprintf(hid->name, sizeof(hid->name), "%s %04hX:%04hX", > client->name, hid->vendor, hid->product); > diff --git a/include/linux/i2c/i2c-hid.h b/include/linux/i2c/i2c-hid.h > index 7aa901d..12e682d 100644 > --- a/include/linux/i2c/i2c-hid.h > +++ b/include/linux/i2c/i2c-hid.h > @@ -17,7 +17,7 @@ > /** > * struct i2chid_platform_data - used by hid over i2c implementation. > * @hid_descriptor_address: i2c register where the HID descriptor is stored. > - * > + * @quirks: quirks, if any for i2c-hid devices > * Note that it is the responsibility of the platform driver (or the acpi 5.0 > * driver, or the flattened device tree) to setup the irq related to the gpio in > * the struct i2c_board_info. > @@ -31,6 +31,7 @@ > */ > struct i2c_hid_platform_data { > u16 hid_descriptor_address; > + u32 quirks; > }; Umm, and where does the quirks field of platform data actually get set? I don't seem to see any followup patchset doing that? Thanks, -- Jiri Kosina SUSE Labs ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <alpine.LNX.2.00.1310091225230.13289-ztGlSCb7Y1iN3ZZ/Hiejyg@public.gmane.org>]
* Re: [PATCH] HID: i2c-hid: add platform data for quirks [not found] ` <alpine.LNX.2.00.1310091225230.13289-ztGlSCb7Y1iN3ZZ/Hiejyg@public.gmane.org> @ 2013-10-09 10:59 ` Benjamin Tissoires 2013-10-15 5:21 ` Bibek Basu 0 siblings, 1 reply; 4+ messages in thread From: Benjamin Tissoires @ 2013-10-09 10:59 UTC (permalink / raw) To: Jiri Kosina Cc: Bibek Basu, linux-tegra-u79uwXL29TY76Z2rM5mHXA, Kiran Adduri, linux-input On Wed, Oct 9, 2013 at 12:26 PM, Jiri Kosina <jkosina-AlSwsSmVLrQ@public.gmane.org> wrote: > On Thu, 3 Oct 2013, Bibek Basu wrote: > >> Some HID device does not responds to INPUT/FEATURE >> report query during initialization. As a result HID driver >> throws error even when its not mandatory according to the >> HID specification. Add platform data to provide >> quirk information to the driver so that init query is not >> done on such devices. >> >> Signed-off-by: Bibek Basu <bbasu-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> >> --- >> drivers/hid/i2c-hid/i2c-hid.c | 1 + >> include/linux/i2c/i2c-hid.h | 3 ++- >> 2 files changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c >> index fd7ce37..a7aec2c 100644 >> --- a/drivers/hid/i2c-hid/i2c-hid.c >> +++ b/drivers/hid/i2c-hid/i2c-hid.c >> @@ -1017,6 +1017,7 @@ static int i2c_hid_probe(struct i2c_client *client, >> hid->version = le16_to_cpu(ihid->hdesc.bcdVersion); >> hid->vendor = le16_to_cpu(ihid->hdesc.wVendorID); >> hid->product = le16_to_cpu(ihid->hdesc.wProductID); >> + hid->quirks = platform_data->quirks; >> >> snprintf(hid->name, sizeof(hid->name), "%s %04hX:%04hX", >> client->name, hid->vendor, hid->product); >> diff --git a/include/linux/i2c/i2c-hid.h b/include/linux/i2c/i2c-hid.h >> index 7aa901d..12e682d 100644 >> --- a/include/linux/i2c/i2c-hid.h >> +++ b/include/linux/i2c/i2c-hid.h >> @@ -17,7 +17,7 @@ >> /** >> * struct i2chid_platform_data - used by hid over i2c implementation. >> * @hid_descriptor_address: i2c register where the HID descriptor is stored. >> - * >> + * @quirks: quirks, if any for i2c-hid devices >> * Note that it is the responsibility of the platform driver (or the acpi 5.0 >> * driver, or the flattened device tree) to setup the irq related to the gpio in >> * the struct i2c_board_info. >> @@ -31,6 +31,7 @@ >> */ >> struct i2c_hid_platform_data { >> u16 hid_descriptor_address; >> + u32 quirks; >> }; > Hi Jiri, > Umm, and where does the quirks field of platform data actually get set? I > don't seem to see any followup patchset doing that? The platform data is filled by specific march-* board definition data. In this case some I would say a tegra board. So it's fine not having a patchset setting the quirk (as it is per board, and maybe per version/manufacturer). Other than that I would love seeing the same for the device tree binding if we add this to the struct i2c_hid_platform_data. BTW, Jiri, the aim of the patch was to be able to add the quirk HID_QUIRK_NO_INIT_INPUT_REPORTS to some of the i2c-hid devices. However, the Windows driver uses this by default, so I already told to Bibek that we could just drop the related calls in i2c_hid_init_reports(). Cheers, Benjamin ^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH] HID: i2c-hid: add platform data for quirks 2013-10-09 10:59 ` Benjamin Tissoires @ 2013-10-15 5:21 ` Bibek Basu 0 siblings, 0 replies; 4+ messages in thread From: Bibek Basu @ 2013-10-15 5:21 UTC (permalink / raw) To: Benjamin Tissoires, Jiri Kosina Cc: linux-tegra@vger.kernel.org, Kiran Adduri, linux-input > -----Original Message----- > From: Benjamin Tissoires [mailto:benjamin.tissoires@gmail.com] > Sent: Wednesday, October 09, 2013 4:30 PM > To: Jiri Kosina > Cc: Bibek Basu; linux-tegra@vger.kernel.org; Kiran Adduri; linux-input > Subject: Re: [PATCH] HID: i2c-hid: add platform data for quirks > > On Wed, Oct 9, 2013 at 12:26 PM, Jiri Kosina <jkosina@suse.cz> wrote: > > On Thu, 3 Oct 2013, Bibek Basu wrote: > > > >> Some HID device does not responds to INPUT/FEATURE report query > >> during initialization. As a result HID driver throws error even when > >> its not mandatory according to the HID specification. Add platform > >> data to provide quirk information to the driver so that init query is > >> not done on such devices. > >> > >> Signed-off-by: Bibek Basu <bbasu@nvidia.com> > >> --- > >> drivers/hid/i2c-hid/i2c-hid.c | 1 + > >> include/linux/i2c/i2c-hid.h | 3 ++- > >> 2 files changed, 3 insertions(+), 1 deletion(-) > >> > >> diff --git a/drivers/hid/i2c-hid/i2c-hid.c > >> b/drivers/hid/i2c-hid/i2c-hid.c index fd7ce37..a7aec2c 100644 > >> --- a/drivers/hid/i2c-hid/i2c-hid.c > >> +++ b/drivers/hid/i2c-hid/i2c-hid.c > >> @@ -1017,6 +1017,7 @@ static int i2c_hid_probe(struct i2c_client *client, > >> hid->version = le16_to_cpu(ihid->hdesc.bcdVersion); > >> hid->vendor = le16_to_cpu(ihid->hdesc.wVendorID); > >> hid->product = le16_to_cpu(ihid->hdesc.wProductID); > >> + hid->quirks = platform_data->quirks; > >> > >> snprintf(hid->name, sizeof(hid->name), "%s %04hX:%04hX", > >> client->name, hid->vendor, hid->product); diff --git > >> a/include/linux/i2c/i2c-hid.h b/include/linux/i2c/i2c-hid.h index > >> 7aa901d..12e682d 100644 > >> --- a/include/linux/i2c/i2c-hid.h > >> +++ b/include/linux/i2c/i2c-hid.h > >> @@ -17,7 +17,7 @@ > >> /** > >> * struct i2chid_platform_data - used by hid over i2c implementation. > >> * @hid_descriptor_address: i2c register where the HID descriptor is > stored. > >> - * > >> + * @quirks: quirks, if any for i2c-hid devices > >> * Note that it is the responsibility of the platform driver (or the acpi 5.0 > >> * driver, or the flattened device tree) to setup the irq related to the gpio > in > >> * the struct i2c_board_info. > >> @@ -31,6 +31,7 @@ > >> */ > >> struct i2c_hid_platform_data { > >> u16 hid_descriptor_address; > >> + u32 quirks; > >> }; > > > > Hi Jiri, > > > Umm, and where does the quirks field of platform data actually get > > set? I don't seem to see any followup patchset doing that? > > The platform data is filled by specific march-* board definition data. > In this case some I would say a tegra board. So it's fine not having a patchset > setting the quirk (as it is per board, and maybe per version/manufacturer). > > Other than that I would love seeing the same for the device tree binding if > we add this to the struct i2c_hid_platform_data. > > BTW, Jiri, the aim of the patch was to be able to add the quirk > HID_QUIRK_NO_INIT_INPUT_REPORTS to some of the i2c-hid devices. > However, the Windows driver uses this by default, so I already told to Bibek > that we could just drop the related calls in i2c_hid_init_reports(). > Hi, Sorry, I was OOO. OK, I am repushing a new patch to just drop the i2c_hid_init_reports calls regards Bibek > Cheers, > Benjamin ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-10-15 5:21 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-03 8:48 [PATCH] HID: i2c-hid: add platform data for quirks Bibek Basu
2013-10-09 10:26 ` Jiri Kosina
[not found] ` <alpine.LNX.2.00.1310091225230.13289-ztGlSCb7Y1iN3ZZ/Hiejyg@public.gmane.org>
2013-10-09 10:59 ` Benjamin Tissoires
2013-10-15 5:21 ` Bibek Basu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).