Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-14 Thread Gergely Imreh
On 14 June 2015 at 23:55, Jonathan Cameron wrote: > On 14/06/15 16:45, Lars-Peter Clausen wrote: >> On 06/14/2015 05:15 AM, Gergely Imreh wrote: >>> On 14 June 2015 at 02:30, Jonathan Cameron wrote: On 11/06/15 09:22, Gergely Imreh wrote: > The Xilinx XADC driver has both a temperature

Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-14 Thread Jonathan Cameron
On 14/06/15 16:45, Lars-Peter Clausen wrote: > On 06/14/2015 05:15 AM, Gergely Imreh wrote: >> On 14 June 2015 at 02:30, Jonathan Cameron wrote: >>> On 11/06/15 09:22, Gergely Imreh wrote: The Xilinx XADC driver has both a temperature channel and 8 voltage channels. The voltage channels

Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-14 Thread Lars-Peter Clausen
On 06/14/2015 05:15 AM, Gergely Imreh wrote: On 14 June 2015 at 02:30, Jonathan Cameron wrote: On 11/06/15 09:22, Gergely Imreh wrote: The Xilinx XADC driver has both a temperature channel and 8 voltage channels. The voltage channels have no offset, but actually were still set the same offset

Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-14 Thread Lars-Peter Clausen
On 06/14/2015 05:15 AM, Gergely Imreh wrote: On 14 June 2015 at 02:30, Jonathan Cameron ji...@kernel.org wrote: On 11/06/15 09:22, Gergely Imreh wrote: The Xilinx XADC driver has both a temperature channel and 8 voltage channels. The voltage channels have no offset, but actually were still set

Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-14 Thread Jonathan Cameron
On 14/06/15 16:45, Lars-Peter Clausen wrote: On 06/14/2015 05:15 AM, Gergely Imreh wrote: On 14 June 2015 at 02:30, Jonathan Cameron ji...@kernel.org wrote: On 11/06/15 09:22, Gergely Imreh wrote: The Xilinx XADC driver has both a temperature channel and 8 voltage channels. The voltage

Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-14 Thread Gergely Imreh
On 14 June 2015 at 23:55, Jonathan Cameron ji...@kernel.org wrote: On 14/06/15 16:45, Lars-Peter Clausen wrote: On 06/14/2015 05:15 AM, Gergely Imreh wrote: On 14 June 2015 at 02:30, Jonathan Cameron ji...@kernel.org wrote: On 11/06/15 09:22, Gergely Imreh wrote: The Xilinx XADC driver has

Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-13 Thread Gergely Imreh
On 14 June 2015 at 02:30, Jonathan Cameron wrote: > On 11/06/15 09:22, Gergely Imreh wrote: >> The Xilinx XADC driver has both a temperature channel and 8 voltage >> channels. The voltage channels have no offset, but actually were still >> set the same offset as the temperature channel. This did

Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-13 Thread Jonathan Cameron
On 11/06/15 09:22, Gergely Imreh wrote: > The Xilinx XADC driver has both a temperature channel and 8 voltage > channels. The voltage channels have no offset, but actually were still > set the same offset as the temperature channel. This did not cause > problems in /sys/bus/iio/ but can cause

Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-13 Thread Gergely Imreh
On 14 June 2015 at 02:30, Jonathan Cameron ji...@kernel.org wrote: On 11/06/15 09:22, Gergely Imreh wrote: The Xilinx XADC driver has both a temperature channel and 8 voltage channels. The voltage channels have no offset, but actually were still set the same offset as the temperature channel.

Re: [PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-13 Thread Jonathan Cameron
On 11/06/15 09:22, Gergely Imreh wrote: The Xilinx XADC driver has both a temperature channel and 8 voltage channels. The voltage channels have no offset, but actually were still set the same offset as the temperature channel. This did not cause problems in /sys/bus/iio/ but can cause problems

[PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-11 Thread Gergely Imreh
The Xilinx XADC driver has both a temperature channel and 8 voltage channels. The voltage channels have no offset, but actually were still set the same offset as the temperature channel. This did not cause problems in /sys/bus/iio/ but can cause problems with other drivers using iio data. For

[PATCH 1/1] iio:adc: XADC: Set offset explicitly to zero on voltage channels

2015-06-11 Thread Gergely Imreh
The Xilinx XADC driver has both a temperature channel and 8 voltage channels. The voltage channels have no offset, but actually were still set the same offset as the temperature channel. This did not cause problems in /sys/bus/iio/ but can cause problems with other drivers using iio data. For