From: "Trisal, Kalhan" <kalhan.trisal@intel.com>
To: "samu.p.onkalo@nokia.com" <samu.p.onkalo@nokia.com>,
       =?utf-8?B?ZXh0IMOJcmljIFBpZWw=?= <eric.piel@tremplin-utc.net>
CC: Jonathan Cameron <Jonathan.Cameron@gmail.com>,
       "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
       LM Sensors <lm-sensors@lm-sensors.org>,
       Jean Delvare <khali@linux-fr.org>,
       Andrew Morton <akpm@linux-foundation.org>
Date: Tue, 13 Oct 2009 15:50:10 +0530
Subject: RE: [RFC][PATCH 0/2] LIS3LV02D I2C driver
Thread-Topic: [RFC][PATCH 0/2] LIS3LV02D I2C driver
Thread-Index: AcpL7d4MWRwB4D9IRJCg98SNk/pU8QAAGLUw
Message-ID: <3CA6C6D9F70D314CA34352990B57DA1539AE7E1D@bgsmsx502.gar.corp.intel.com>
References: <1254391591-9696-1-git-send-email-samu.p.onkalo@nokia.com>
	 <4ACCC273.1030308@gmail.com> <4ACCC957.3060802@tremplin-utc.net>
	 <4ACCCF48.5090005@tremplin-utc.net> <1255428813.3974.9.camel@4fid08082>
In-Reply-To: <1255428813.3974.9.camel@4fid08082>
Accept-Language: en-US
Content-Language: en-US
acceptlanguage: en-US
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 2555
Lines: 67


-----Original Message-----
From: Onkalo Samu [mailto:samu.p.onkalo@nokia.com] 
Sent: Tuesday, October 13, 2009 3:44 PM
To: ext Éric Piel; Trisal, Kalhan
Cc: Jonathan Cameron; linux-kernel@vger.kernel.org; LM Sensors; Jean Delvare; Andrew Morton
Subject: Re: [RFC][PATCH 0/2] LIS3LV02D I2C driver

On Wed, 2009-10-07 at 19:26 +0200, ext Éric Piel wrote:
> Op 07-10-09 19:01, Éric Piel schreef:
> > Op 07-10-09 18:31, Jonathan Cameron schreef:
> >> Just a quick heads up wrt overlapping work.
> >>
> >> For the i2c support Kalhan Trisal has been posting patches for i2c
> >> support for this
> >> driver to the lm-sensors list for some time and the latest version of that
> >> set is also pretty clean.
> > You mean this post, right?
> > http://lists.lm-sensors.org/pipermail/lm-sensors/2009-August/026505.html
> > "Accelerometer driver for STMicroeletronics-LIS331DL-three-axis-digital"
> > Thanks for the heads up, I had never heard of this driver before.
> 
> :
> > Kalhan, would
> > you mind having a look at the patch from Samu, and see what is required
> > to get your LIS331DL working with the lis3lv02d driver? It should be
> > very little and avoid duplication of efforts :-)
> Ah, bah,
> I see you have actually already more or less done this!
> http://lists.lm-sensors.org/pipermail/lm-sensors/2009-October/026840.html
> "I2C glue layer for lis3lv02d STMicroelectronics digital accelerometer"
> 
> The main thing you forgot to do with this patch was to CC the maintainer
> of the driver (AKA me) ;-)
> 
> Anyway, this version is _very_ close to the one from Samu. It has just
> all the basics, really clean! Samu has already added some nifty things
> (like changing the axis conversion, support for suspend...). So it
> should _really_ easy to converge :-) As Samu's patch is already in
> Andrew's queue, I think it's simpler to leave it as it, and just to keep
> my request to you, Kalhan, to check that it works for your hardware as
> well.

Hi Kalhan,

Have you been able to test my patch if it works also for your HW?

http://marc.info/?l=linux-kernel&m=125472767421625&w=2

http://marc.info/?l=linux-kernel&m=125472767421629&w=2

http://marc.info/?l=linux-kernel&m=125472767421639&w=2
 
Br,
Samu


Hi Samu, 
  I haven't tested your patch yet but it will work.
   I have seen the patch, the glue layer is exactly same 
   what I have also tested part of my driver.

Br
Kalhan

????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?