From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754691AbbBBQVa (ORCPT ); Mon, 2 Feb 2015 11:21:30 -0500 Received: from bh-25.webhostbox.net ([208.91.199.152]:53453 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754227AbbBBQV3 (ORCPT ); Mon, 2 Feb 2015 11:21:29 -0500 Date: Mon, 2 Feb 2015 08:21:18 -0800 From: Guenter Roeck To: Robert Rosengren Cc: Johan Adolfsson , "linux-kernel@vger.kernel.org" , "lm-sensors@lm-sensors.org" Subject: Re: [lm-sensors] [PATCH] hwmon: (ads7828) Make sample interval configurable Message-ID: <20150202162118.GA14642@roeck-us.net> References: <20150127233455.0c62457e@endymion.delvare> <54C86035.9080605@roeck-us.net> <54C87F3C.2020105@axis.com> <54C8F73D.4040808@roeck-us.net> <54C9DA9D.3010802@axis.com> <54C9DBD5.5030808@roeck-us.net> <54CA226E.6080505@axis.com> <20150129193040.GA21937@roeck-us.net> <54CD36DF.7010602@roeck-us.net> <54CF3189.8050504@axis.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <54CF3189.8050504@axis.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-Authenticated_sender: guenter@roeck-us.net X-OutGoing-Spam-Status: No, score=-1.0 X-CTCH-PVer: 0000001 X-CTCH-Spam: Unknown X-CTCH-VOD: Unknown X-CTCH-Flags: 0 X-CTCH-RefID: str=0001.0A020205.54CFA408.026D,ss=1,re=0.001,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0 X-CTCH-Score: 0.001 X-CTCH-ScoreCust: 0.000 X-CTCH-Rules: C_4847, X-CTCH-SenderID: linux@roeck-us.net X-CTCH-SenderID-Flags: 0 X-CTCH-SenderID-TotalMessages: 3 X-CTCH-SenderID-TotalSpam: 0 X-CTCH-SenderID-TotalSuspected: 0 X-CTCH-SenderID-TotalConfirmed: 0 X-CTCH-SenderID-TotalBulk: 0 X-CTCH-SenderID-TotalVirus: 0 X-CTCH-SenderID-TotalRecipients: 0 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - bh-25.webhostbox.net X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - roeck-us.net X-Get-Message-Sender-Via: bh-25.webhostbox.net: mailgid no entry from get_relayhosts_entry X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Mon, 02 Feb 2015 16:21:18 +0000 Subject: Re: [lm-sensors] [PATCH] hwmon: (ads7828) Make sample interval configurable Message-Id: <20150202162118.GA14642@roeck-us.net> List-Id: References: <20150127233455.0c62457e@endymion.delvare> <54C86035.9080605@roeck-us.net> <54C87F3C.2020105@axis.com> <54C8F73D.4040808@roeck-us.net> <54C9DA9D.3010802@axis.com> <54C9DBD5.5030808@roeck-us.net> <54CA226E.6080505@axis.com> <20150129193040.GA21937@roeck-us.net> <54CD36DF.7010602@roeck-us.net> <54CF3189.8050504@axis.com> In-Reply-To: <54CF3189.8050504@axis.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Robert Rosengren Cc: Johan Adolfsson , "linux-kernel@vger.kernel.org" , "lm-sensors@lm-sensors.org" 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 _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors