| Message ID | 20180802182729.2061830-1-pn@denx.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v4,1/3] iio: Add modifier for white light | expand |
Hello, it is not clear to me why 'white' is needed; isn't that the default, i.e. unfiltered light? thanks, regards, p. > Signed-off-by: Parthiban Nallathambi <pn@denx.de> > --- > Documentation/ABI/testing/sysfs-bus-iio | 7 +++++++ > drivers/iio/industrialio-core.c | 1 + > include/uapi/linux/iio/types.h | 1 + > tools/iio/iio_event_monitor.c | 2 ++ > 4 files changed, 11 insertions(+) > > diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio > index 731146c3b138..43e481aed5b2 100644 > --- a/Documentation/ABI/testing/sysfs-bus-iio > +++ b/Documentation/ABI/testing/sysfs-bus-iio > @@ -1312,6 +1312,13 @@ Description: > standardised CIE Erythemal Action Spectrum. UV index values range > from 0 (low) to >=11 (extreme). > > +What: /sys/.../iio:deviceX/in_intensityY_white_raw > +KernelVersion: 4.18 > +Contact: linux-iio@vger.kernel.org > +Description: > + Modifier white indicates that measurements contain white LED > + component. > + > What: /sys/.../iio:deviceX/in_intensity_red_integration_time > What: /sys/.../iio:deviceX/in_intensity_green_integration_time > What: /sys/.../iio:deviceX/in_intensity_blue_integration_time > diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c > index 19bdf3d2962a..cb939b9fad16 100644 > --- a/drivers/iio/industrialio-core.c > +++ b/drivers/iio/industrialio-core.c > @@ -108,6 +108,7 @@ static const char * const iio_modifier_names[] = { > [IIO_MOD_LIGHT_GREEN] = "green", > [IIO_MOD_LIGHT_BLUE] = "blue", > [IIO_MOD_LIGHT_UV] = "uv", > + [IIO_MOD_LIGHT_WHITE] = "white", > [IIO_MOD_QUATERNION] = "quaternion", > [IIO_MOD_TEMP_AMBIENT] = "ambient", > [IIO_MOD_TEMP_OBJECT] = "object", > diff --git a/include/uapi/linux/iio/types.h b/include/uapi/linux/iio/types.h > index 4213cdf88e3c..de87a6c7e6de 100644 > --- a/include/uapi/linux/iio/types.h > +++ b/include/uapi/linux/iio/types.h > @@ -84,6 +84,7 @@ enum iio_modifier { > IIO_MOD_CO2, > IIO_MOD_VOC, > IIO_MOD_LIGHT_UV, > + IIO_MOD_LIGHT_WHITE, > }; > > enum iio_event_type { > diff --git a/tools/iio/iio_event_monitor.c b/tools/iio/iio_event_monitor.c > index b61245e1181d..a2f9c62a79dd 100644 > --- a/tools/iio/iio_event_monitor.c > +++ b/tools/iio/iio_event_monitor.c > @@ -96,6 +96,7 @@ static const char * const iio_modifier_names[] = { > [IIO_MOD_LIGHT_GREEN] = "green", > [IIO_MOD_LIGHT_BLUE] = "blue", > [IIO_MOD_LIGHT_UV] = "uv", > + [IIO_MOD_LIGHT_WHITE] = "white", > [IIO_MOD_QUATERNION] = "quaternion", > [IIO_MOD_TEMP_AMBIENT] = "ambient", > [IIO_MOD_TEMP_OBJECT] = "object", > @@ -178,6 +179,7 @@ static bool event_is_known(struct iio_event_data *event) > case IIO_MOD_LIGHT_GREEN: > case IIO_MOD_LIGHT_BLUE: > case IIO_MOD_LIGHT_UV: > + case IIO_MOD_LIGHT_WHITE: > case IIO_MOD_QUATERNION: > case IIO_MOD_TEMP_AMBIENT: > case IIO_MOD_TEMP_OBJECT: >
Hello Peter, On 08/02/2018 09:30 PM, Peter Meerwald-Stadler wrote: > Hello, > > it is not clear to me why 'white' is needed; > isn't that the default, i.e. unfiltered light? Yes, it is. But devices like vcnl4035 veml7700, White LED data one register and all other sources of light (like fluorescent, incandescent ,sunlight) in separate register. So in such cases this WHITE modifier is needed. Should it needs to come under IIO_MOD_LIGHT_CLEAR? > > thanks, regards, p. > >> Signed-off-by: Parthiban Nallathambi <pn@denx.de> >> --- >> Documentation/ABI/testing/sysfs-bus-iio | 7 +++++++ >> drivers/iio/industrialio-core.c | 1 + >> include/uapi/linux/iio/types.h | 1 + >> tools/iio/iio_event_monitor.c | 2 ++ >> 4 files changed, 11 insertions(+) >> >> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio >> index 731146c3b138..43e481aed5b2 100644 >> --- a/Documentation/ABI/testing/sysfs-bus-iio >> +++ b/Documentation/ABI/testing/sysfs-bus-iio >> @@ -1312,6 +1312,13 @@ Description: >> standardised CIE Erythemal Action Spectrum. UV index values range >> from 0 (low) to >=11 (extreme). >> >> +What: /sys/.../iio:deviceX/in_intensityY_white_raw >> +KernelVersion: 4.18 >> +Contact: linux-iio@vger.kernel.org >> +Description: >> + Modifier white indicates that measurements contain white LED >> + component. >> + >> What: /sys/.../iio:deviceX/in_intensity_red_integration_time >> What: /sys/.../iio:deviceX/in_intensity_green_integration_time >> What: /sys/.../iio:deviceX/in_intensity_blue_integration_time >> diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c >> index 19bdf3d2962a..cb939b9fad16 100644 >> --- a/drivers/iio/industrialio-core.c >> +++ b/drivers/iio/industrialio-core.c >> @@ -108,6 +108,7 @@ static const char * const iio_modifier_names[] = { >> [IIO_MOD_LIGHT_GREEN] = "green", >> [IIO_MOD_LIGHT_BLUE] = "blue", >> [IIO_MOD_LIGHT_UV] = "uv", >> + [IIO_MOD_LIGHT_WHITE] = "white", >> [IIO_MOD_QUATERNION] = "quaternion", >> [IIO_MOD_TEMP_AMBIENT] = "ambient", >> [IIO_MOD_TEMP_OBJECT] = "object", >> diff --git a/include/uapi/linux/iio/types.h b/include/uapi/linux/iio/types.h >> index 4213cdf88e3c..de87a6c7e6de 100644 >> --- a/include/uapi/linux/iio/types.h >> +++ b/include/uapi/linux/iio/types.h >> @@ -84,6 +84,7 @@ enum iio_modifier { >> IIO_MOD_CO2, >> IIO_MOD_VOC, >> IIO_MOD_LIGHT_UV, >> + IIO_MOD_LIGHT_WHITE, >> }; >> >> enum iio_event_type { >> diff --git a/tools/iio/iio_event_monitor.c b/tools/iio/iio_event_monitor.c >> index b61245e1181d..a2f9c62a79dd 100644 >> --- a/tools/iio/iio_event_monitor.c >> +++ b/tools/iio/iio_event_monitor.c >> @@ -96,6 +96,7 @@ static const char * const iio_modifier_names[] = { >> [IIO_MOD_LIGHT_GREEN] = "green", >> [IIO_MOD_LIGHT_BLUE] = "blue", >> [IIO_MOD_LIGHT_UV] = "uv", >> + [IIO_MOD_LIGHT_WHITE] = "white", >> [IIO_MOD_QUATERNION] = "quaternion", >> [IIO_MOD_TEMP_AMBIENT] = "ambient", >> [IIO_MOD_TEMP_OBJECT] = "object", >> @@ -178,6 +179,7 @@ static bool event_is_known(struct iio_event_data *event) >> case IIO_MOD_LIGHT_GREEN: >> case IIO_MOD_LIGHT_BLUE: >> case IIO_MOD_LIGHT_UV: >> + case IIO_MOD_LIGHT_WHITE: >> case IIO_MOD_QUATERNION: >> case IIO_MOD_TEMP_AMBIENT: >> case IIO_MOD_TEMP_OBJECT: >> >
Hello, > > it is not clear to me why 'white' is needed; > > isn't that the default, i.e. unfiltered light? > > Yes, it is. But devices like vcnl4035 veml7700, White LED data one > register and all other sources of light (like fluorescent, > incandescent ,sunlight) in separate register. > > So in such cases this WHITE modifier is needed. Should it needs to > come under IIO_MOD_LIGHT_CLEAR? it is a mess already :), there is _intensity _intensity_ir _intensity_both _intensity_uv _intensity_red _intensity_green _intensity_blue _intensity_clear I think that ir, uv, red, green, blue are filters for particular ranges of the spectrum _intensity_clear might be unfiltered and _intensity might relate to the human eye photopic curve, I think the proposed white might be the same as clear? 'both' means ir + _intensity (not clearly specified) regards, p. > > > +KernelVersion: 4.18 > > > +Contact: linux-iio@vger.kernel.org > > > +Description: > > > + Modifier white indicates that measurements contain white LED > > > + component. > > > + > > > What: /sys/.../iio:deviceX/in_intensity_red_integration_time > > > What: > > > /sys/.../iio:deviceX/in_intensity_green_integration_time > > > What: > > > /sys/.../iio:deviceX/in_intensity_blue_integration_time > > > diff --git a/drivers/iio/industrialio-core.c > > > b/drivers/iio/industrialio-core.c > > > index 19bdf3d2962a..cb939b9fad16 100644 > > > --- a/drivers/iio/industrialio-core.c > > > +++ b/drivers/iio/industrialio-core.c > > > @@ -108,6 +108,7 @@ static const char * const iio_modifier_names[] = { > > > [IIO_MOD_LIGHT_GREEN] = "green", > > > [IIO_MOD_LIGHT_BLUE] = "blue", > > > [IIO_MOD_LIGHT_UV] = "uv", > > > + [IIO_MOD_LIGHT_WHITE] = "white", > > > [IIO_MOD_QUATERNION] = "quaternion", > > > [IIO_MOD_TEMP_AMBIENT] = "ambient", > > > [IIO_MOD_TEMP_OBJECT] = "object", > > > diff --git a/include/uapi/linux/iio/types.h > > > b/include/uapi/linux/iio/types.h > > > index 4213cdf88e3c..de87a6c7e6de 100644 > > > --- a/include/uapi/linux/iio/types.h > > > +++ b/include/uapi/linux/iio/types.h > > > @@ -84,6 +84,7 @@ enum iio_modifier { > > > IIO_MOD_CO2, > > > IIO_MOD_VOC, > > > IIO_MOD_LIGHT_UV, > > > + IIO_MOD_LIGHT_WHITE, > > > }; > > > enum iio_event_type { > > > diff --git a/tools/iio/iio_event_monitor.c b/tools/iio/iio_event_monitor.c > > > index b61245e1181d..a2f9c62a79dd 100644 > > > --- a/tools/iio/iio_event_monitor.c > > > +++ b/tools/iio/iio_event_monitor.c > > > @@ -96,6 +96,7 @@ static const char * const iio_modifier_names[] = { > > > [IIO_MOD_LIGHT_GREEN] = "green", > > > [IIO_MOD_LIGHT_BLUE] = "blue", > > > [IIO_MOD_LIGHT_UV] = "uv", > > > + [IIO_MOD_LIGHT_WHITE] = "white", > > > [IIO_MOD_QUATERNION] = "quaternion", > > > [IIO_MOD_TEMP_AMBIENT] = "ambient", > > > [IIO_MOD_TEMP_OBJECT] = "object", > > > @@ -178,6 +179,7 @@ static bool event_is_known(struct iio_event_data > > > *event) > > > case IIO_MOD_LIGHT_GREEN: > > > case IIO_MOD_LIGHT_BLUE: > > > case IIO_MOD_LIGHT_UV: > > > + case IIO_MOD_LIGHT_WHITE: > > > case IIO_MOD_QUATERNION: > > > case IIO_MOD_TEMP_AMBIENT: > > > case IIO_MOD_TEMP_OBJECT: > > > > > > >
On Fri, 3 Aug 2018 13:38:17 +0200 (CEST) Peter Meerwald-Stadler <pmeerw@pmeerw.net> wrote: > Hello, > > > > it is not clear to me why 'white' is needed; > > > isn't that the default, i.e. unfiltered light? > > > > Yes, it is. But devices like vcnl4035 veml7700, White LED data one > > register and all other sources of light (like fluorescent, > > incandescent ,sunlight) in separate register. > > > > So in such cases this WHITE modifier is needed. Should it needs to > > come under IIO_MOD_LIGHT_CLEAR? This wins the aware for odd. The sensor is drawn as only have two sensing elements, one for Ambient light sensing and one for proximity. A particular award should be given for using a different axis for the white channel in which normalized output is decimal vs all the others where it is a percentage ;) Looking closely at those graphs I think this is actually a standard sensor combination job. So 'white' is what we have always called _both in that it reads both visible and infrared. The other sensor is infrared. Ambient is illuminance not intensity (approximate human eye response). The only dubious bit about this is that I can see how they managed to cancel out some of the signal around 700nm as there is nothing much on the infrared and lots on the white. Alternative is that they actually have a filter that really approximates the human eye and the white channel is the sum of the other two? > > it is a mess already :), there is > _intensity > _intensity_ir > _intensity_both > _intensity_uv > _intensity_red > _intensity_green > _intensity_blue > _intensity_clear > > I think that ir, uv, red, green, blue are filters for particular ranges > of the spectrum That is the intent certainly. > > _intensity_clear might be unfiltered and _intensity might relate to the > human eye photopic curve, I think the proposed white might be the same as > clear? > > 'both' means ir + _intensity (not clearly specified) Yup, that's the power of initial assumptions. There was a time when the only light sensors you tended to get were ir and both ir and visible. Visible was obtained by taking one from the other. One of those things we'd change if doing it again but it's a bit late now.. Jonathan > > regards, p. > > > > > +KernelVersion: 4.18 > > > > +Contact: linux-iio@vger.kernel.org > > > > +Description: > > > > + Modifier white indicates that measurements contain white LED > > > > + component. > > > > + > > > > What: /sys/.../iio:deviceX/in_intensity_red_integration_time > > > > What: > > > > /sys/.../iio:deviceX/in_intensity_green_integration_time > > > > What: > > > > /sys/.../iio:deviceX/in_intensity_blue_integration_time > > > > diff --git a/drivers/iio/industrialio-core.c > > > > b/drivers/iio/industrialio-core.c > > > > index 19bdf3d2962a..cb939b9fad16 100644 > > > > --- a/drivers/iio/industrialio-core.c > > > > +++ b/drivers/iio/industrialio-core.c > > > > @@ -108,6 +108,7 @@ static const char * const iio_modifier_names[] = { > > > > [IIO_MOD_LIGHT_GREEN] = "green", > > > > [IIO_MOD_LIGHT_BLUE] = "blue", > > > > [IIO_MOD_LIGHT_UV] = "uv", > > > > + [IIO_MOD_LIGHT_WHITE] = "white", > > > > [IIO_MOD_QUATERNION] = "quaternion", > > > > [IIO_MOD_TEMP_AMBIENT] = "ambient", > > > > [IIO_MOD_TEMP_OBJECT] = "object", > > > > diff --git a/include/uapi/linux/iio/types.h > > > > b/include/uapi/linux/iio/types.h > > > > index 4213cdf88e3c..de87a6c7e6de 100644 > > > > --- a/include/uapi/linux/iio/types.h > > > > +++ b/include/uapi/linux/iio/types.h > > > > @@ -84,6 +84,7 @@ enum iio_modifier { > > > > IIO_MOD_CO2, > > > > IIO_MOD_VOC, > > > > IIO_MOD_LIGHT_UV, > > > > + IIO_MOD_LIGHT_WHITE, > > > > }; > > > > enum iio_event_type { > > > > diff --git a/tools/iio/iio_event_monitor.c b/tools/iio/iio_event_monitor.c > > > > index b61245e1181d..a2f9c62a79dd 100644 > > > > --- a/tools/iio/iio_event_monitor.c > > > > +++ b/tools/iio/iio_event_monitor.c > > > > @@ -96,6 +96,7 @@ static const char * const iio_modifier_names[] = { > > > > [IIO_MOD_LIGHT_GREEN] = "green", > > > > [IIO_MOD_LIGHT_BLUE] = "blue", > > > > [IIO_MOD_LIGHT_UV] = "uv", > > > > + [IIO_MOD_LIGHT_WHITE] = "white", > > > > [IIO_MOD_QUATERNION] = "quaternion", > > > > [IIO_MOD_TEMP_AMBIENT] = "ambient", > > > > [IIO_MOD_TEMP_OBJECT] = "object", > > > > @@ -178,6 +179,7 @@ static bool event_is_known(struct iio_event_data > > > > *event) > > > > case IIO_MOD_LIGHT_GREEN: > > > > case IIO_MOD_LIGHT_BLUE: > > > > case IIO_MOD_LIGHT_UV: > > > > + case IIO_MOD_LIGHT_WHITE: > > > > case IIO_MOD_QUATERNION: > > > > case IIO_MOD_TEMP_AMBIENT: > > > > case IIO_MOD_TEMP_OBJECT: > > > > > > > > > > > > -- 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
diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio index 731146c3b138..43e481aed5b2 100644 --- a/Documentation/ABI/testing/sysfs-bus-iio +++ b/Documentation/ABI/testing/sysfs-bus-iio @@ -1312,6 +1312,13 @@ Description: standardised CIE Erythemal Action Spectrum. UV index values range from 0 (low) to >=11 (extreme). +What: /sys/.../iio:deviceX/in_intensityY_white_raw +KernelVersion: 4.18 +Contact: linux-iio@vger.kernel.org +Description: + Modifier white indicates that measurements contain white LED + component. + What: /sys/.../iio:deviceX/in_intensity_red_integration_time What: /sys/.../iio:deviceX/in_intensity_green_integration_time What: /sys/.../iio:deviceX/in_intensity_blue_integration_time diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c index 19bdf3d2962a..cb939b9fad16 100644 --- a/drivers/iio/industrialio-core.c +++ b/drivers/iio/industrialio-core.c @@ -108,6 +108,7 @@ static const char * const iio_modifier_names[] = { [IIO_MOD_LIGHT_GREEN] = "green", [IIO_MOD_LIGHT_BLUE] = "blue", [IIO_MOD_LIGHT_UV] = "uv", + [IIO_MOD_LIGHT_WHITE] = "white", [IIO_MOD_QUATERNION] = "quaternion", [IIO_MOD_TEMP_AMBIENT] = "ambient", [IIO_MOD_TEMP_OBJECT] = "object", diff --git a/include/uapi/linux/iio/types.h b/include/uapi/linux/iio/types.h index 4213cdf88e3c..de87a6c7e6de 100644 --- a/include/uapi/linux/iio/types.h +++ b/include/uapi/linux/iio/types.h @@ -84,6 +84,7 @@ enum iio_modifier { IIO_MOD_CO2, IIO_MOD_VOC, IIO_MOD_LIGHT_UV, + IIO_MOD_LIGHT_WHITE, }; enum iio_event_type { diff --git a/tools/iio/iio_event_monitor.c b/tools/iio/iio_event_monitor.c index b61245e1181d..a2f9c62a79dd 100644 --- a/tools/iio/iio_event_monitor.c +++ b/tools/iio/iio_event_monitor.c @@ -96,6 +96,7 @@ static const char * const iio_modifier_names[] = { [IIO_MOD_LIGHT_GREEN] = "green", [IIO_MOD_LIGHT_BLUE] = "blue", [IIO_MOD_LIGHT_UV] = "uv", + [IIO_MOD_LIGHT_WHITE] = "white", [IIO_MOD_QUATERNION] = "quaternion", [IIO_MOD_TEMP_AMBIENT] = "ambient", [IIO_MOD_TEMP_OBJECT] = "object", @@ -178,6 +179,7 @@ static bool event_is_known(struct iio_event_data *event) case IIO_MOD_LIGHT_GREEN: case IIO_MOD_LIGHT_BLUE: case IIO_MOD_LIGHT_UV: + case IIO_MOD_LIGHT_WHITE: case IIO_MOD_QUATERNION: case IIO_MOD_TEMP_AMBIENT: case IIO_MOD_TEMP_OBJECT:
Signed-off-by: Parthiban Nallathambi <pn@denx.de> --- Documentation/ABI/testing/sysfs-bus-iio | 7 +++++++ drivers/iio/industrialio-core.c | 1 + include/uapi/linux/iio/types.h | 1 + tools/iio/iio_event_monitor.c | 2 ++ 4 files changed, 11 insertions(+)