HID-sensor(-hub.c) and device IDs

linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* HID-sensor(-hub.c) and device IDs
@ 2012-12-05 18:44 Alexander Holler
  2012-12-05 19:21 ` Pandruvada, Srinivas
  0 siblings, 1 reply; 7+ messages in thread
From: Alexander Holler @ 2012-12-05 18:44 UTC (permalink / raw)
  To: linux-iio, linux-input, srinivas pandruvada

Hello,

is there any special reason why vendor and devices IDs for HID-sensors 
must be listed in drivers/hid/hid-sensor-hub.c and drivers/hid/hid-core.c?

Maybe I'm missing something, but I don't think this is how HID is 
supposed to work.

Regards,

Alexander

^ permalink raw reply	[flat|nested] 7+ messages in thread

* RE: HID-sensor(-hub.c) and device IDs
  2012-12-05 18:44 HID-sensor(-hub.c) and device IDs Alexander Holler
@ 2012-12-05 19:21 ` Pandruvada, Srinivas
  2012-12-05 20:15   ` Alexander Holler
  0 siblings, 1 reply; 7+ messages in thread
From: Pandruvada, Srinivas @ 2012-12-05 19:21 UTC (permalink / raw)
  To: Alexander Holler, linux-iio@vger.kernel.org,
	linux-input@vger.kernel.org

We have not tested on sensor hubs from different vendors except few. It is possible that some vendors are using sensor hub as raw devices . We can remove vendor product ids once we have a good list for support. 

Thanks,
Srinivas

-----Original Message-----
From: Alexander Holler [mailto:holler@ahsoftware.de] 
Sent: Wednesday, December 05, 2012 10:45 AM
To: linux-iio@vger.kernel.org; linux-input@vger.kernel.org; Pandruvada, Srinivas
Subject: HID-sensor(-hub.c) and device IDs

Hello,

is there any special reason why vendor and devices IDs for HID-sensors must be listed in drivers/hid/hid-sensor-hub.c and drivers/hid/hid-core.c?

Maybe I'm missing something, but I don't think this is how HID is supposed to work.

Regards,

Alexander

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: HID-sensor(-hub.c) and device IDs
  2012-12-05 19:21 ` Pandruvada, Srinivas
@ 2012-12-05 20:15   ` Alexander Holler
  2012-12-05 20:24     ` Pandruvada, Srinivas
  0 siblings, 1 reply; 7+ messages in thread
From: Alexander Holler @ 2012-12-05 20:15 UTC (permalink / raw)
  To: Pandruvada, Srinivas
  Cc: linux-iio@vger.kernel.org, linux-input@vger.kernel.org

Am 05.12.2012 20:21, schrieb Pandruvada, Srinivas:
> We have not tested on sensor hubs from different vendors except few. It is possible that some vendors are using sensor hub as raw devices . We can remove vendor product ids once we have a good list for support.

Hmm, sorry, I don't understand that fully. Does that mean that the list 
is there to use the driver only with a small group of devices because 
otherwise some HID-sensors aren't usable as raw devices?

I don't know how many HID-sensor devices are already out in the wild, 
but because the standard looks fairly new, I don't think there are that 
many. And I wonder how you want to get the list larger, if nobody 
notifies that they should use hid-sensor-hub.c.

I think the correct way would be to build a blacklist instead of a 
whitelist.

Regards,

Alexander

> Thanks,
> Srinivas
>
> -----Original Message-----
> From: Alexander Holler [mailto:holler@ahsoftware.de]
> Sent: Wednesday, December 05, 2012 10:45 AM
> To: linux-iio@vger.kernel.org; linux-input@vger.kernel.org; Pandruvada, Srinivas
> Subject: HID-sensor(-hub.c) and device IDs
>
> Hello,
>
> is there any special reason why vendor and devices IDs for HID-sensors must be listed in drivers/hid/hid-sensor-hub.c and drivers/hid/hid-core.c?
>
> Maybe I'm missing something, but I don't think this is how HID is supposed to work.
>
> Regards,
>
> Alexander
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>


^ permalink raw reply	[flat|nested] 7+ messages in thread

* RE: HID-sensor(-hub.c) and device IDs
  2012-12-05 20:15   ` Alexander Holler
@ 2012-12-05 20:24     ` Pandruvada, Srinivas
       [not found]       ` <4FA419E87744DF4DAECD5BCE1214B7A91C95FEB6-P5GAC/sN6hk8Ug9VwtkbtrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
  0 siblings, 1 reply; 7+ messages in thread
From: Pandruvada, Srinivas @ 2012-12-05 20:24 UTC (permalink / raw)
  To: Alexander Holler; +Cc: linux-iio@vger.kernel.org, linux-input@vger.kernel.org

Yes, We can have a black list instead of white list. Are you planning to submit changes?
Thanks,
Srinivas


-----Original Message-----
From: Alexander Holler [mailto:holler@ahsoftware.de] 
Sent: Wednesday, December 05, 2012 12:15 PM
To: Pandruvada, Srinivas
Cc: linux-iio@vger.kernel.org; linux-input@vger.kernel.org
Subject: Re: HID-sensor(-hub.c) and device IDs

Am 05.12.2012 20:21, schrieb Pandruvada, Srinivas:
> We have not tested on sensor hubs from different vendors except few. It is possible that some vendors are using sensor hub as raw devices . We can remove vendor product ids once we have a good list for support.

Hmm, sorry, I don't understand that fully. Does that mean that the list is there to use the driver only with a small group of devices because otherwise some HID-sensors aren't usable as raw devices?

I don't know how many HID-sensor devices are already out in the wild, but because the standard looks fairly new, I don't think there are that many. And I wonder how you want to get the list larger, if nobody notifies that they should use hid-sensor-hub.c.


I think the correct way would be to build a blacklist instead of a whitelist.

Regards,

Alexander

> Thanks,
> Srinivas
>
> -----Original Message-----
> From: Alexander Holler [mailto:holler@ahsoftware.de]
> Sent: Wednesday, December 05, 2012 10:45 AM
> To: linux-iio@vger.kernel.org; linux-input@vger.kernel.org; 
> Pandruvada, Srinivas
> Subject: HID-sensor(-hub.c) and device IDs
>
> Hello,
>
> is there any special reason why vendor and devices IDs for HID-sensors must be listed in drivers/hid/hid-sensor-hub.c and drivers/hid/hid-core.c?
>
> Maybe I'm missing something, but I don't think this is how HID is supposed to work.
>
> Regards,
>
> Alexander
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" 
> in the body of a message to majordomo@vger.kernel.org More majordomo 
> info at  http://vger.kernel.org/majordomo-info.html
>


^ permalink raw reply	[flat|nested] 7+ messages in thread

[parent not found: <4FA419E87744DF4DAECD5BCE1214B7A91C95FEB6-P5GAC/sN6hk8Ug9VwtkbtrfspsVTdybXVpNB7YpNyf8@public.gmane.org>]

* Re: HID-sensor(-hub.c) and device IDs
       [not found]       ` <4FA419E87744DF4DAECD5BCE1214B7A91C95FEB6-P5GAC/sN6hk8Ug9VwtkbtrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
@ 2012-12-05 21:08         ` Alexander Holler
  0 siblings, 0 replies; 7+ messages in thread
From: Alexander Holler @ 2012-12-05 21:08 UTC (permalink / raw)
  To: Pandruvada, Srinivas
  Cc: linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org

Am 05.12.2012 21:24, schrieb Pandruvada, Srinivas:
> Yes, We can have a black list instead of white list. Are you planning to submit changes?

I've just have implemented my first version of a time sensor. So yes (at 
least the time sensor).

That's why I stumbled over that. I had to add a vendor and device ID to 
those two files. I also noticed that hid-sensors(-hub) doesn't work with 
combined HID devices (e.g. mouse/keyboard/sensor) and as a first 
workaround I've just modified the HID-tree of my device. As I'm fairly 
new to the HID stuff, I'm still in the evaluation phase (reading source 
and such).

In regard to a blacklist, I don't know how to fill it. I only have the 
one sensor device I've build myself (it's a spare-time project).

I think the first step would be to identify devices usable by 
hid-sensor-hub, sensor_hub_check_for_sensor_page() looks like the way to 
go, at least for the current state of the hid-sensor-framework.

Regards,

Alexander

> Thanks,
> Srinivas
>
>
> -----Original Message-----
> From: Alexander Holler [mailto:holler-SXC+2es9fhnfWeYVQQPykw@public.gmane.org]
> Sent: Wednesday, December 05, 2012 12:15 PM
> To: Pandruvada, Srinivas
> Cc: linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> Subject: Re: HID-sensor(-hub.c) and device IDs
>
> Am 05.12.2012 20:21, schrieb Pandruvada, Srinivas:
>> We have not tested on sensor hubs from different vendors except few. It is possible that some vendors are using sensor hub as raw devices . We can remove vendor product ids once we have a good list for support.
>
> Hmm, sorry, I don't understand that fully. Does that mean that the list is there to use the driver only with a small group of devices because otherwise some HID-sensors aren't usable as raw devices?
>
> I don't know how many HID-sensor devices are already out in the wild, but because the standard looks fairly new, I don't think there are that many. And I wonder how you want to get the list larger, if nobody notifies that they should use hid-sensor-hub.c.
>
>
> I think the correct way would be to build a blacklist instead of a whitelist.
>
> Regards,
>
> Alexander
>
>> Thanks,
>> Srinivas
>>
>> -----Original Message-----
>> From: Alexander Holler [mailto:holler-SXC+2es9fhnfWeYVQQPykw@public.gmane.org]
>> Sent: Wednesday, December 05, 2012 10:45 AM
>> To: linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org;
>> Pandruvada, Srinivas
>> Subject: HID-sensor(-hub.c) and device IDs
>>
>> Hello,
>>
>> is there any special reason why vendor and devices IDs for HID-sensors must be listed in drivers/hid/hid-sensor-hub.c and drivers/hid/hid-core.c?
>>
>> Maybe I'm missing something, but I don't think this is how HID is supposed to work.
>>
>> Regards,
>>
>> Alexander
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-iio"
>> in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo
>> info at  http://vger.kernel.org/majordomo-info.html
>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: HID-sensor(-hub.c) and device IDs
@ 2012-12-06 14:32 Henrik Rydberg
  2012-12-06 19:58 ` Alexander Holler
  0 siblings, 1 reply; 7+ messages in thread
From: Henrik Rydberg @ 2012-12-06 14:32 UTC (permalink / raw)
  To: Alexander Holler; +Cc: linux-iio, linux-input

Hi Alexander,

> I think the first step would be to identify devices usable by
> hid-sensor-hub, sensor_hub_check_for_sensor_page() looks like the
> way to go, at least for the current state of the hid-sensor-framework.

If you can get the gist of that function into hid_scan_usage(), you should be able to drop both the list in hid-core.c and the list in hid-sensor-hub.c

Henrik

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: HID-sensor(-hub.c) and device IDs
  2012-12-06 14:32 Henrik Rydberg
@ 2012-12-06 19:58 ` Alexander Holler
  0 siblings, 0 replies; 7+ messages in thread
From: Alexander Holler @ 2012-12-06 19:58 UTC (permalink / raw)
  To: Henrik Rydberg; +Cc: linux-iio, linux-input

Am 06.12.2012 15:32, schrieb Henrik Rydberg:
> Hi Alexander,
>
>> I think the first step would be to identify devices usable by
>> hid-sensor-hub, sensor_hub_check_for_sensor_page() looks like the
>> way to go, at least for the current state of the hid-sensor-framework.
>
> If you can get the gist of that function into hid_scan_usage(), you should be able to drop both the list in hid-core.c and the list in hid-sensor-hub.c

Thats what I wanted to suggest. ;)

It's just that I only have one self-made HID sensor device and 
absolutely no knowledge about what HID sensor devices are in the wild 
and how they are used.

Regards,

Alexander

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2012-12-06 19:58 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-05 18:44 HID-sensor(-hub.c) and device IDs Alexander Holler
2012-12-05 19:21 ` Pandruvada, Srinivas
2012-12-05 20:15   ` Alexander Holler
2012-12-05 20:24     ` Pandruvada, Srinivas
     [not found]       ` <4FA419E87744DF4DAECD5BCE1214B7A91C95FEB6-P5GAC/sN6hk8Ug9VwtkbtrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2012-12-05 21:08         ` Alexander Holler
  -- strict thread matches above, loose matches on Subject: below --
2012-12-06 14:32 Henrik Rydberg
2012-12-06 19:58 ` Alexander Holler

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).