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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux