Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-09-23 Thread Hector Palacios
Hi Alexandre, On 09/23/2013 02:47 PM, Alexandre Belloni wrote: Hector, I'd like to go forward with this and have a chance to see that included in 3.13. Do you mind if I rework your patch series by removing the DT part and then resubmit ? I don't mind at all. Please go ahead. Regards, On

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-09-23 Thread Alexandre Belloni
Hector, I'd like to go forward with this and have a chance to see that included in 3.13. Do you mind if I rework your patch series by removing the DT part and then resubmit ? Regards, On 24/08/2013 01:00, Jonathan Cameron wrote: > On 08/22/13 17:51, Pawel Moll wrote: >> On Thu, 2013-08-22 at

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-09-23 Thread Alexandre Belloni
Hector, I'd like to go forward with this and have a chance to see that included in 3.13. Do you mind if I rework your patch series by removing the DT part and then resubmit ? Regards, On 24/08/2013 01:00, Jonathan Cameron wrote: On 08/22/13 17:51, Pawel Moll wrote: On Thu, 2013-08-22 at

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-09-23 Thread Hector Palacios
Hi Alexandre, On 09/23/2013 02:47 PM, Alexandre Belloni wrote: Hector, I'd like to go forward with this and have a chance to see that included in 3.13. Do you mind if I rework your patch series by removing the DT part and then resubmit ? I don't mind at all. Please go ahead. Regards, On

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-23 Thread Jonathan Cameron
On 08/22/13 17:51, Pawel Moll wrote: > On Thu, 2013-08-22 at 07:17 +0100, Jonathan Cameron wrote: >> I would favour option 2 though some of the discussions going on at the >> moment about >> bindings might result in a generic description of this and any other bits of >> analog front end. > >

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-23 Thread Jonathan Cameron
On 08/22/13 17:51, Pawel Moll wrote: On Thu, 2013-08-22 at 07:17 +0100, Jonathan Cameron wrote: I would favour option 2 though some of the discussions going on at the moment about bindings might result in a generic description of this and any other bits of analog front end. Any link to

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Lars-Peter Clausen
On 08/22/2013 06:41 PM, Pawel Moll wrote: > > On Wed, 2013-08-21 at 23:13 +0100, Alexandre Belloni wrote: >> You are not so wrong. There is indeed actually only one reference >> voltage (and that is 1.85V). But, before feeding the voltage to the ADC >> channels, you sometimes have a divider.

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Pawel Moll
On Thu, 2013-08-22 at 07:17 +0100, Jonathan Cameron wrote: > I would favour option 2 though some of the discussions going on at the moment > about > bindings might result in a generic description of this and any other bits of > analog front end. Any link to this discussion?

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Pawel Moll
On Thu, 2013-08-22 at 09:05 +0100, Hector Palacios wrote: > This was what I originally submitted but it then looked like it would better > fit in > the DeviceTree. The spear-adc seemed to use a similar approach: I'm not sure if it's the best example... I much more prefer the AD7303's way:

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Pawel Moll
On Wed, 2013-08-21 at 23:13 +0100, Alexandre Belloni wrote: > You are not so wrong. There is indeed actually only one reference > voltage (and that is 1.85V). But, before feeding the voltage to the ADC > channels, you sometimes have a divider. Then, after the channel muxing, > you can add a by 2

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Hector Palacios
Dear Alexandre, On 08/22/2013 12:13 AM, Alexandre Belloni wrote: Hi Pawel, On 14/08/2013 16:44, Pawel Moll wrote: On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Jonathan Cameron
Sorry accidental HTML message hence resend Alexandre Belloni wrote: >Hi Pawel, > >On 14/08/2013 16:44, Pawel Moll wrote: >> On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: >>> On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Jonathan Cameron
Sorry accidental HTML message hence resend Alexandre Belloni alexandre.bell...@free-electrons.com wrote: Hi Pawel, On 14/08/2013 16:44, Pawel Moll wrote: On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Hector Palacios
Dear Alexandre, On 08/22/2013 12:13 AM, Alexandre Belloni wrote: Hi Pawel, On 14/08/2013 16:44, Pawel Moll wrote: On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Pawel Moll
On Wed, 2013-08-21 at 23:13 +0100, Alexandre Belloni wrote: You are not so wrong. There is indeed actually only one reference voltage (and that is 1.85V). But, before feeding the voltage to the ADC channels, you sometimes have a divider. Then, after the channel muxing, you can add a by 2

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Pawel Moll
On Thu, 2013-08-22 at 09:05 +0100, Hector Palacios wrote: This was what I originally submitted but it then looked like it would better fit in the DeviceTree. The spear-adc seemed to use a similar approach: I'm not sure if it's the best example... I much more prefer the AD7303's way:

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Pawel Moll
On Thu, 2013-08-22 at 07:17 +0100, Jonathan Cameron wrote: I would favour option 2 though some of the discussions going on at the moment about bindings might result in a generic description of this and any other bits of analog front end. Any link to this discussion?

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Lars-Peter Clausen
On 08/22/2013 06:41 PM, Pawel Moll wrote: On Wed, 2013-08-21 at 23:13 +0100, Alexandre Belloni wrote: You are not so wrong. There is indeed actually only one reference voltage (and that is 1.85V). But, before feeding the voltage to the ADC channels, you sometimes have a divider. Then, after

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-21 Thread Alexandre Belloni
Hi Pawel, On 14/08/2013 16:44, Pawel Moll wrote: > On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: >> On 07/22/13 15:04, Hector Palacios wrote: >>> Some LRADC channels have fixed pre-dividers so they can measure >>> different voltages at full scale. The reference voltage allows to >>>

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-21 Thread Alexandre Belloni
Hi Pawel, On 14/08/2013 16:44, Pawel Moll wrote: On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-14 Thread Pawel Moll
On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: > On 07/22/13 15:04, Hector Palacios wrote: > > Some LRADC channels have fixed pre-dividers so they can measure > > different voltages at full scale. The reference voltage allows to > > expose a scaling attribute through the IIO sysfs so

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-14 Thread Pawel Moll
On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-13 Thread Jonathan Cameron
On 07/22/13 15:04, Hector Palacios wrote: > Some LRADC channels have fixed pre-dividers so they can measure > different voltages at full scale. The reference voltage allows to > expose a scaling attribute through the IIO sysfs so that a user can > compute the real voltage out of a measured sample

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-13 Thread Jonathan Cameron
On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real voltage out of a measured sample

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-26 Thread Alexandre Belloni
On 23/07/2013 00:06, Marek Vasut wrote: > Dear Lars-Peter Clausen, > >> On 07/22/2013 04:04 PM, Hector Palacios wrote: >>> Some LRADC channels have fixed pre-dividers so they can measure >>> different voltages at full scale. The reference voltage allows to >>> expose a scaling attribute through

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-26 Thread Alexandre Belloni
On 23/07/2013 00:06, Marek Vasut wrote: Dear Lars-Peter Clausen, On 07/22/2013 04:04 PM, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-22 Thread Marek Vasut
Dear Lars-Peter Clausen, > On 07/22/2013 04:04 PM, Hector Palacios wrote: > > Some LRADC channels have fixed pre-dividers so they can measure > > different voltages at full scale. The reference voltage allows to > > expose a scaling attribute through the IIO sysfs so that a user can > > compute

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-22 Thread Lars-Peter Clausen
On 07/22/2013 04:04 PM, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real voltage out of a measured sample

[PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-22 Thread Hector Palacios
Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real voltage out of a measured sample value. Signed-off-by: Hector Palacios Acked-by:

[PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-22 Thread Hector Palacios
Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real voltage out of a measured sample value. Signed-off-by: Hector Palacios

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-22 Thread Lars-Peter Clausen
On 07/22/2013 04:04 PM, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real voltage out of a measured sample

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-22 Thread Marek Vasut
Dear Lars-Peter Clausen, On 07/22/2013 04:04 PM, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real