Re: [lm-sensors] [PATCH] hwmon: (ads7828) Make sample interval configurable

From: Robert Rosengren <robert.rosengren@axis.com>
To: Guenter Roeck <linux@roeck-us.net>
Cc: Johan Adolfsson <johana@axis.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"lm-sensors@lm-sensors.org" <lm-sensors@lm-sensors.org>
Subject: Re: [lm-sensors] [PATCH] hwmon: (ads7828) Make sample interval configurable
Date: Mon, 2 Feb 2015 09:12:57 +0100	[thread overview]
Message-ID: <54CF3189.8050504@axis.com> (raw)
In-Reply-To: <54CD36DF.7010602@roeck-us.net>

On 01/31/2015 09:11 PM, Guenter Roeck wrote:
> On 01/29/2015 11:30 AM, Guenter Roeck wrote:
>> On Thu, Jan 29, 2015 at 01:07:10PM +0100, Robert Rosengren wrote:
>>>> Is your hardware big endian or little endian ?
>>> CONFIG_CPU_LITTLE_ENDIAN=y
>>>
>> Hi Robert,
>>
>> I have another question: What is your i2c controller type ?
>>
> Robert,
>
> I now tried both 3.15 and 3.19-rc6. Both kernels require the byte swap code.

Agrees, no difference for me when trying with 3.15 and 3.18 version.

The i2c is a bit-bang driver, specific for the hardware I am running on.

>
> The only reason I can imagine why you don't need it in your code would be
> that your i2c controller does not support i2c functionality but only SMBus.
> This would be the case, for example, with the i801 controller. So it would
> be quite helpful to know which controller your system uses to access the
> ads7828.
>
> Thanks,
> Guenter
>

I tried your latest patch set (the 3 patches submitted last night) and 
it works fine for me. Great work!

Please change my Cc-tag in the patches to Reviewed-by or Tested-by (I 
have done both, but one tag is enough). Thanks :-)

BR,
Robert