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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Mon, 2 Feb 2015 08:21:18 -0800
From:	Guenter Roeck <linux@...ck-us.net>
To:	Robert Rosengren <robert.rosengren@...s.com>
Cc:	Johan Adolfsson <johana@...s.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"lm-sensors@...sensors.org" <lm-sensors@...sensors.org>
Subject: Re: [lm-sensors] [PATCH] hwmon: (ads7828) Make sample interval
 configurable

On Mon, Feb 02, 2015 at 09:12:57AM +0100, Robert Rosengren wrote:
> 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 :-)
> 
I made it Reviewed-and-Tested-by.

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