Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1573886 > unrolled thread

[PATCH 1/4] iio: chemical: add particle number concentration channel type

Started byTomasz Duszynski <tduszyns@gmail.com>
First post2017-02-05 16:10 +0100
Last post2017-02-11 11:40 +0100
Articles 4 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/4] iio: chemical: add particle number concentration channel type Tomasz Duszynski <tduszyns@gmail.com> - 2017-02-05 16:10 +0100
    Re: [PATCH 1/4] iio: chemical: add particle number concentration  channel type Peter Meerwald-Stadler <pmeerw@pmeerw.net> - 2017-02-05 16:30 +0100
      Re: [PATCH 1/4] iio: chemical: add particle number concentration  channel type Tomasz Duszynski <tduszyns@gmail.com> - 2017-02-05 17:10 +0100
        Re: [PATCH 1/4] iio: chemical: add particle number concentration  channel type Jonathan Cameron <jic23@kernel.org> - 2017-02-11 11:40 +0100

#1573886 — [PATCH 1/4] iio: chemical: add particle number concentration channel type

FromTomasz Duszynski <tduszyns@gmail.com>
Date2017-02-05 16:10 +0100
Subject[PATCH 1/4] iio: chemical: add particle number concentration channel type
Message-ID<t7wBY-xf-23@gated-at.bofh.it>
This patch adds channel type for expressing particle number
concentration.

Signed-off-by: Tomasz Duszynski <tduszyns@gmail.com>
---
 Documentation/ABI/testing/sysfs-bus-iio | 7 +++++++
 drivers/iio/industrialio-core.c         | 1 +
 include/uapi/linux/iio/types.h          | 1 +
 3 files changed, 9 insertions(+)

diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
index 530809ccfacf..481771e585ee 100644
--- a/Documentation/ABI/testing/sysfs-bus-iio
+++ b/Documentation/ABI/testing/sysfs-bus-iio
@@ -1608,3 +1608,10 @@ Description:
 		provides an absolute positional reference (e.g. a pulse once per
 		revolution) which may be used to home positional systems as
 		required.
+
+What:		/sys/bus/iio/devices/iio:deviceX/in_numberconcentration_input
+KernelVersion: 	4.12
+Contact:	linux-iio@vger.kernel.org
+Description:
+		Reading of particulates number in a volume of a mixture.
+		Unit is in 1/m3.
diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
index d18ded45bedd..451e3bb8d623 100644
--- a/drivers/iio/industrialio-core.c
+++ b/drivers/iio/industrialio-core.c
@@ -85,6 +85,7 @@ static const char * const iio_chan_type_name_spec[] = {
 	[IIO_COUNT] = "count",
 	[IIO_INDEX] = "index",
 	[IIO_GRAVITY]  = "gravity",
+	[IIO_NUMBERCONCENTRATION] = "numberconcentration",
 };
 
 static const char * const iio_modifier_names[] = {
diff --git a/include/uapi/linux/iio/types.h b/include/uapi/linux/iio/types.h
index ffafd6c25a48..52696e251c4d 100644
--- a/include/uapi/linux/iio/types.h
+++ b/include/uapi/linux/iio/types.h
@@ -43,6 +43,7 @@ enum iio_chan_type {
 	IIO_COUNT,
 	IIO_INDEX,
 	IIO_GRAVITY,
+	IIO_NUMBERCONCENTRATION,
 };
 
 enum iio_modifier {
-- 
2.11.1

[toc] | [next] | [standalone]


#1573896 — Re: [PATCH 1/4] iio: chemical: add particle number concentration channel type

FromPeter Meerwald-Stadler <pmeerw@pmeerw.net>
Date2017-02-05 16:30 +0100
SubjectRe: [PATCH 1/4] iio: chemical: add particle number concentration channel type
Message-ID<t7wVk-DW-9@gated-at.bofh.it>
In reply to#1573886
> This patch adds channel type for expressing particle number
> concentration.

numberconcentration sound a bit strange to be, more like a number 
theoretic concept

how about particlecontentration? or numberparticles?
 
> Signed-off-by: Tomasz Duszynski <tduszyns@gmail.com>
> ---
>  Documentation/ABI/testing/sysfs-bus-iio | 7 +++++++
>  drivers/iio/industrialio-core.c         | 1 +
>  include/uapi/linux/iio/types.h          | 1 +
>  3 files changed, 9 insertions(+)
> 
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> index 530809ccfacf..481771e585ee 100644
> --- a/Documentation/ABI/testing/sysfs-bus-iio
> +++ b/Documentation/ABI/testing/sysfs-bus-iio
> @@ -1608,3 +1608,10 @@ Description:
>  		provides an absolute positional reference (e.g. a pulse once per
>  		revolution) which may be used to home positional systems as
>  		required.
> +
> +What:		/sys/bus/iio/devices/iio:deviceX/in_numberconcentration_input
> +KernelVersion: 	4.12
> +Contact:	linux-iio@vger.kernel.org
> +Description:
> +		Reading of particulates number in a volume of a mixture.
> +		Unit is in 1/m3.
> diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
> index d18ded45bedd..451e3bb8d623 100644
> --- a/drivers/iio/industrialio-core.c
> +++ b/drivers/iio/industrialio-core.c
> @@ -85,6 +85,7 @@ static const char * const iio_chan_type_name_spec[] = {
>  	[IIO_COUNT] = "count",
>  	[IIO_INDEX] = "index",
>  	[IIO_GRAVITY]  = "gravity",
> +	[IIO_NUMBERCONCENTRATION] = "numberconcentration",
>  };
>  
>  static const char * const iio_modifier_names[] = {
> diff --git a/include/uapi/linux/iio/types.h b/include/uapi/linux/iio/types.h
> index ffafd6c25a48..52696e251c4d 100644
> --- a/include/uapi/linux/iio/types.h
> +++ b/include/uapi/linux/iio/types.h
> @@ -43,6 +43,7 @@ enum iio_chan_type {
>  	IIO_COUNT,
>  	IIO_INDEX,
>  	IIO_GRAVITY,
> +	IIO_NUMBERCONCENTRATION,
>  };
>  
>  enum iio_modifier {
> 

-- 

Peter Meerwald-Stadler
+43-664-2444418 (mobile)

[toc] | [prev] | [next] | [standalone]


#1573908 — Re: [PATCH 1/4] iio: chemical: add particle number concentration channel type

FromTomasz Duszynski <tduszyns@gmail.com>
Date2017-02-05 17:10 +0100
SubjectRe: [PATCH 1/4] iio: chemical: add particle number concentration channel type
Message-ID<t7xy2-16Q-13@gated-at.bofh.it>
In reply to#1573896
On Sun, Feb 05, 2017 at 04:21:27PM +0100, Peter Meerwald-Stadler wrote:
>
> > This patch adds channel type for expressing particle number
> > concentration.
>
> numberconcentration sound a bit strange to be, more like a number
> theoretic concept
>
> how about particlecontentration? or numberparticles?
>
Agree it may sound a bit strange but in fact it is a formal physical quantity
with SI unit. Perhaps I can leave it as is to stay consistent with
science world?
> > Signed-off-by: Tomasz Duszynski <tduszyns@gmail.com>
> > ---
> >  Documentation/ABI/testing/sysfs-bus-iio | 7 +++++++
> >  drivers/iio/industrialio-core.c         | 1 +
> >  include/uapi/linux/iio/types.h          | 1 +
> >  3 files changed, 9 insertions(+)
> >
> > diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> > index 530809ccfacf..481771e585ee 100644
> > --- a/Documentation/ABI/testing/sysfs-bus-iio
> > +++ b/Documentation/ABI/testing/sysfs-bus-iio
> > @@ -1608,3 +1608,10 @@ Description:
> >  		provides an absolute positional reference (e.g. a pulse once per
> >  		revolution) which may be used to home positional systems as
> >  		required.
> > +
> > +What:		/sys/bus/iio/devices/iio:deviceX/in_numberconcentration_input
> > +KernelVersion: 	4.12
> > +Contact:	linux-iio@vger.kernel.org
> > +Description:
> > +		Reading of particulates number in a volume of a mixture.
> > +		Unit is in 1/m3.
> > diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
> > index d18ded45bedd..451e3bb8d623 100644
> > --- a/drivers/iio/industrialio-core.c
> > +++ b/drivers/iio/industrialio-core.c
> > @@ -85,6 +85,7 @@ static const char * const iio_chan_type_name_spec[] = {
> >  	[IIO_COUNT] = "count",
> >  	[IIO_INDEX] = "index",
> >  	[IIO_GRAVITY]  = "gravity",
> > +	[IIO_NUMBERCONCENTRATION] = "numberconcentration",
> >  };
> >
> >  static const char * const iio_modifier_names[] = {
> > diff --git a/include/uapi/linux/iio/types.h b/include/uapi/linux/iio/types.h
> > index ffafd6c25a48..52696e251c4d 100644
> > --- a/include/uapi/linux/iio/types.h
> > +++ b/include/uapi/linux/iio/types.h
> > @@ -43,6 +43,7 @@ enum iio_chan_type {
> >  	IIO_COUNT,
> >  	IIO_INDEX,
> >  	IIO_GRAVITY,
> > +	IIO_NUMBERCONCENTRATION,
> >  };
> >
> >  enum iio_modifier {
> >
>
> --
>
> Peter Meerwald-Stadler
> +43-664-2444418 (mobile)

[toc] | [prev] | [next] | [standalone]


#1579015 — Re: [PATCH 1/4] iio: chemical: add particle number concentration channel type

FromJonathan Cameron <jic23@kernel.org>
Date2017-02-11 11:40 +0100
SubjectRe: [PATCH 1/4] iio: chemical: add particle number concentration channel type
Message-ID<t9DfX-7ZL-11@gated-at.bofh.it>
In reply to#1573908
On 05/02/17 16:06, Tomasz Duszynski wrote:
> On Sun, Feb 05, 2017 at 04:21:27PM +0100, Peter Meerwald-Stadler wrote:
>>
>>> This patch adds channel type for expressing particle number
>>> concentration.
>>
>> numberconcentration sound a bit strange to be, more like a number
>> theoretic concept
>>
>> how about particlecontentration? or numberparticles?
>>
> Agree it may sound a bit strange but in fact it is a formal physical quantity
> with SI unit. Perhaps I can leave it as is to stay consistent with
> science world?
Wikipedia seems to suggest that there are several choices, but this is
a common one in Chemistry so fair enough.

Jonathan
>>> Signed-off-by: Tomasz Duszynski <tduszyns@gmail.com>
>>> ---
>>>  Documentation/ABI/testing/sysfs-bus-iio | 7 +++++++
>>>  drivers/iio/industrialio-core.c         | 1 +
>>>  include/uapi/linux/iio/types.h          | 1 +
>>>  3 files changed, 9 insertions(+)
>>>
>>> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
>>> index 530809ccfacf..481771e585ee 100644
>>> --- a/Documentation/ABI/testing/sysfs-bus-iio
>>> +++ b/Documentation/ABI/testing/sysfs-bus-iio
>>> @@ -1608,3 +1608,10 @@ Description:
>>>  		provides an absolute positional reference (e.g. a pulse once per
>>>  		revolution) which may be used to home positional systems as
>>>  		required.
>>> +
>>> +What:		/sys/bus/iio/devices/iio:deviceX/in_numberconcentration_input
>>> +KernelVersion: 	4.12
>>> +Contact:	linux-iio@vger.kernel.org
>>> +Description:
>>> +		Reading of particulates number in a volume of a mixture.
>>> +		Unit is in 1/m3.
>>> diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
>>> index d18ded45bedd..451e3bb8d623 100644
>>> --- a/drivers/iio/industrialio-core.c
>>> +++ b/drivers/iio/industrialio-core.c
>>> @@ -85,6 +85,7 @@ static const char * const iio_chan_type_name_spec[] = {
>>>  	[IIO_COUNT] = "count",
>>>  	[IIO_INDEX] = "index",
>>>  	[IIO_GRAVITY]  = "gravity",
>>> +	[IIO_NUMBERCONCENTRATION] = "numberconcentration",
>>>  };
>>>
>>>  static const char * const iio_modifier_names[] = {
>>> diff --git a/include/uapi/linux/iio/types.h b/include/uapi/linux/iio/types.h
>>> index ffafd6c25a48..52696e251c4d 100644
>>> --- a/include/uapi/linux/iio/types.h
>>> +++ b/include/uapi/linux/iio/types.h
>>> @@ -43,6 +43,7 @@ enum iio_chan_type {
>>>  	IIO_COUNT,
>>>  	IIO_INDEX,
>>>  	IIO_GRAVITY,
>>> +	IIO_NUMBERCONCENTRATION,
>>>  };
>>>
>>>  enum iio_modifier {
>>>
>>
>> --
>>
>> Peter Meerwald-Stadler
>> +43-664-2444418 (mobile)

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web