Presenting duplicate channels for hid sensor light

Presenting duplicate channels for hid sensor light

[Pandruvada, Srinivas]

SGkgSm9uYXRoYW4sDQoNCkhJRCBzZW5zb3IgZGVmaW5lcyBjb21tb24gaWxsdW1pbmFuY2UgZmll
bGQgd2hpY2ggY2FuIGJlIG91dHB1dCBmcm9tDQpsaWdodCBzZW5zb3IgYW5kIGNvbG9yIHNlbnNv
cnMuIEN1cnJlbnRseSB3ZSBhcmUgcHJlc2VudGluZyB0aGlzIGFzDQppbnRlbnNpdHkuDQpUaGVy
ZSBhcmUgc29tZSB1c2VyIHNwYWNlcyBleHBlY3Rpbmcgb25seSBpbGx1bWluYW5jZSBhdHRyaWJ1
dGUgZm9yDQpyZWFkaW5nIHRoaXMgYW5kIGdldHRpbmcgcmVxdWVzdCB0byBhZGQgdGhpcy7CoA0K
VGhlIG9ubHkgb3B0aW9uIEkgc2VlIGlzIHRvIGFkZCBhIGR1cGxpY2F0ZSBjaGFubmVsLg0KDQpX
aGF0IGRvIHlvdSB0aGluaz8NCsKgDQoNClRoYW5rcywNClNyaW5pdmFz

Re: Presenting duplicate channels for hid sensor light

[Jonathan Cameron]

On 11/12/16 04:48, Pandruvada, Srinivas wrote:
> Hi Jonathan,
> 
> HID sensor defines common illuminance field which can be output from
> light sensor and color sensors. Currently we are presenting this as
> intensity.
> There are some user spaces expecting only illuminance attribute for
> reading this and getting request to add this. 
> The only option I see is to add a duplicate channel.
> 
> What do you think?
Hi Srinivas,

This isn't the first time we have run into this sort of issue.

As you say, presenting a duplicate channel is pretty much the only option as we can't break
userspace by removing the old one.

As long as it is actually in the right units etc for illuminance, feel free to add the
channel.  If things get 'interesting' for buffered output it may be necessary to
only have one of them available via that interface.  Obviously this could be worked around
but it is probably more trouble that it is worth to do so!

Jonathan

>  
> 
> Thanks,
> SrinivasN�����r��y���b�X��ǧv�^�)޺{.n�+����{��*"��^n�r���z�\x1a��h����&��\x1e�G���h�\x03(�階�ݢj"��\x1a�^[m�����z�ޖ���f���h���~�mml==
>