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


Groups > linux.kernel > #1230129 > unrolled thread

Re: [PATCH v3 1/2] iio: resistance: Document that resistance can be output

Started byDaniel Baluta <daniel.baluta@gmail.com>
First post2015-09-22 14:50 +0200
Last post2015-09-22 15:00 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v3 1/2] iio: resistance: Document that resistance can be output Daniel Baluta <daniel.baluta@gmail.com> - 2015-09-22 14:50 +0200
    Re: [PATCH v3 1/2] iio: resistance: Document that resistance can be output Daniel Baluta <daniel.baluta@gmail.com> - 2015-09-22 15:00 +0200
    Re: [PATCH v3 1/2] iio: resistance: Document that resistance can be  output Peter Rosin <peda@lysator.liu.se> - 2015-09-22 15:00 +0200

#1230129 — Re: [PATCH v3 1/2] iio: resistance: Document that resistance can be output

FromDaniel Baluta <daniel.baluta@gmail.com>
Date2015-09-22 14:50 +0200
SubjectRe: [PATCH v3 1/2] iio: resistance: Document that resistance can be output
Message-ID<qbvbb-5BU-11@gated-at.bofh.it>
On Tue, Sep 22, 2015 at 3:39 PM, Peter Rosin <peda@lysator.liu.se> wrote:
> From: Peter Rosin <peda@axentia.se>
>
> Signed-off-by: Peter Rosin <peda@axentia.se>
> ---
>  Documentation/ABI/testing/sysfs-bus-iio |    2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> index 9398484196c4..2eea468f704d 100644
> --- a/Documentation/ABI/testing/sysfs-bus-iio
> +++ b/Documentation/ABI/testing/sysfs-bus-iio
> @@ -1475,6 +1475,8 @@ Description:
>
>  What:          /sys/bus/iio/devices/iio:deviceX/in_resistance_raw
>  What:          /sys/bus/iio/devices/iio:deviceX/in_resistanceX_raw
> +What:          /sys/bus/iio/devices/iio:deviceX/out_resistance_raw
> +What:          /sys/bus/iio/devices/iio:deviceX/out_resistanceX_raw
>  KernelVersion: 4.3
>  Contact:       linux-iio@vger.kernel.org
>  Description:

One more thing. The description for this sections says:

»       »       Raw (unscaled no offset etc.) resistance reading that
can be processed
»       »       into an ohm value.

I'm not sure this still holds for out_resistanceX_raw. We have two options here:

* remove 'reading' because with the out_ file we are basically setting
the resistance
raw value OR
* add a separate section for out_ files, with proper description.

Any other comments?

thanks,
Daniel.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1230138

FromDaniel Baluta <daniel.baluta@gmail.com>
Date2015-09-22 15:00 +0200
Message-ID<qbvkR-5Nf-3@gated-at.bofh.it>
In reply to#1230129
On Tue, Sep 22, 2015 at 3:55 PM, Peter Rosin <peda@lysator.liu.se> wrote:
> On 2015-09-22 14:48, Daniel Baluta wrote:
>> On Tue, Sep 22, 2015 at 3:39 PM, Peter Rosin <peda@lysator.liu.se> wrote:
>>> From: Peter Rosin <peda@axentia.se>
>>>
>>> Signed-off-by: Peter Rosin <peda@axentia.se>
>>> ---
>>>  Documentation/ABI/testing/sysfs-bus-iio |    2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
>>> index 9398484196c4..2eea468f704d 100644
>>> --- a/Documentation/ABI/testing/sysfs-bus-iio
>>> +++ b/Documentation/ABI/testing/sysfs-bus-iio
>>> @@ -1475,6 +1475,8 @@ Description:
>>>
>>>  What:          /sys/bus/iio/devices/iio:deviceX/in_resistance_raw
>>>  What:          /sys/bus/iio/devices/iio:deviceX/in_resistanceX_raw
>>> +What:          /sys/bus/iio/devices/iio:deviceX/out_resistance_raw
>>> +What:          /sys/bus/iio/devices/iio:deviceX/out_resistanceX_raw
>>>  KernelVersion: 4.3
>>>  Contact:       linux-iio@vger.kernel.org
>>>  Description:
>>
>> One more thing. The description for this sections says:
>>
>> »       »       Raw (unscaled no offset etc.) resistance reading that
>> can be processed
>> »       »       into an ohm value.
>>
>> I'm not sure this still holds for out_resistanceX_raw. We have two options here:
>
> I figured it did hold, since you effectively select what reading the
> resistance inside the potentiometer should have. If it's really
> important I can of course make another spin, but is this somthing
> that is likely to be misunderstood?

Now, with your explanation it makes more sense. We can leave it like
that.

thanks,
Daniel.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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


#1230150 — Re: [PATCH v3 1/2] iio: resistance: Document that resistance can be output

FromPeter Rosin <peda@lysator.liu.se>
Date2015-09-22 15:00 +0200
SubjectRe: [PATCH v3 1/2] iio: resistance: Document that resistance can be output
Message-ID<qbvkR-5Nf-5@gated-at.bofh.it>
In reply to#1230129
On 2015-09-22 14:48, Daniel Baluta wrote:
> On Tue, Sep 22, 2015 at 3:39 PM, Peter Rosin <peda@lysator.liu.se> wrote:
>> From: Peter Rosin <peda@axentia.se>
>>
>> Signed-off-by: Peter Rosin <peda@axentia.se>
>> ---
>>  Documentation/ABI/testing/sysfs-bus-iio |    2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
>> index 9398484196c4..2eea468f704d 100644
>> --- a/Documentation/ABI/testing/sysfs-bus-iio
>> +++ b/Documentation/ABI/testing/sysfs-bus-iio
>> @@ -1475,6 +1475,8 @@ Description:
>>
>>  What:          /sys/bus/iio/devices/iio:deviceX/in_resistance_raw
>>  What:          /sys/bus/iio/devices/iio:deviceX/in_resistanceX_raw
>> +What:          /sys/bus/iio/devices/iio:deviceX/out_resistance_raw
>> +What:          /sys/bus/iio/devices/iio:deviceX/out_resistanceX_raw
>>  KernelVersion: 4.3
>>  Contact:       linux-iio@vger.kernel.org
>>  Description:
> 
> One more thing. The description for this sections says:
> 
> »       »       Raw (unscaled no offset etc.) resistance reading that
> can be processed
> »       »       into an ohm value.
> 
> I'm not sure this still holds for out_resistanceX_raw. We have two options here:

I figured it did hold, since you effectively select what reading the
resistance inside the potentiometer should have. If it's really
important I can of course make another spin, but is this somthing
that is likely to be misunderstood?

> * remove 'reading' because with the out_ file we are basically setting
> the resistance
> raw value OR
> * add a separate section for out_ files, with proper description.
> 
> Any other comments?
> 
> thanks,
> Daniel.
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web