public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Éric Piel" <Eric.Piel@tremplin-utc.net>
To: Samu Onkalo <samu.p.onkalo@nokia.com>, Andrew Morton <akpm@osdl.org>
Cc: linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org
Subject: Re: [PATCH v4 0/5] LIS3 feature update
Date: Tue, 01 Dec 2009 09:55:44 +0100	[thread overview]
Message-ID: <4B14DA10.8040106@tremplin-utc.net> (raw)
In-Reply-To: <1259656422-7697-1-git-send-email-samu.p.onkalo@nokia.com>

Op 01-12-09 09:33, Samu Onkalo schreef:
Hi,
All the patches look good to me :-)

Andrew, would you mind queuing these 5 patches for 2.6.33, in addition 
the ones already there?

Thanks,
Eric


> Changes since v3 patch set:
>
> 0001: two if statements combined to one
>
> 0002: added "Acked by Eric Piel" to description
>
> 0003: Replaced with edited version by Eric Piel. Tested on 8 bit device
> by me.
>
> 0004: Constant names changed to shorter ones. Shorter names allowed some clean
> up. No functional changes
>
> 0005: added "Acked by Eric Piel" to description
>
> Description from version 3 patch set:
>
> This patch set is made to top of 7 previously accepted lis3 related patches in
> mm-tree:
> - I2C bus support for lis3lv02d and variant accelerometer chips...
> - Add the possibility to remap axes via platform data...
> - Send input_sync after each measurement round...
> - polled input device itself was not free'd....
> - riginally the driver was only targeted to 12bits sensors...
> - Most of the documentation and comments were written when the driver ..
> - Lis3 accelerometer sensors have quite long power on delay ..
>
> Tested with 2.6.32-RC7 which was patched with above 7 lis3 related patches.
> Tested only with 8 bit lis3 device since I don't have possibility to test other chips.
>
> 0001:
> Implement selftest feature for HW diagnostic purposes. This feature can be used to detect if the sensor is electrically or mechanically damaged.
>
> 0002:
> Calibration functionality is removed from the driver. Chip is allready calibrated by the manufacturer. SW calibration doesn't improve the situation.
>
> 0003:
> Added possibility to set chip sampling rate. Position entry in sysfs allows reading at sampling rate.
>
> 0004:
> Scale output values to mg (1/1000 of earth gravity).
> Behaviour of the joystick interface is not changed. All other interfaces will have different scale after this patch. See further information from patch itself.
>
> 0005:
> Update documentation to cover changes which were made by these patches.
>
> Samu Onkalo (5):
>    lis3: Selftest support
>    LIS3LV02D: Remove calibaration functionality
>    lis3: Sysfs entry for setting chip measurement rate
>    lis3: Scale output values to mg
>    LIS3: Update documentation to match latest changes
>
>   Documentation/hwmon/lis3lv02d |   31 ++++++--
>   drivers/hwmon/lis3lv02d.c     |  174 ++++++++++++++++++++++++++++++++---------
>   drivers/hwmon/lis3lv02d.h     |   23 ++++-
>   include/linux/lis3lv02d.h     |    3 +
>   4 files changed, 182 insertions(+), 49 deletions(-)
>


      parent reply	other threads:[~2009-12-01  8:55 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-01  8:33 [PATCH v4 0/5] LIS3 feature update Samu Onkalo
2009-12-01  8:33 ` [PATCH v4 1/5] lis3: Selftest support Samu Onkalo
2009-12-01  8:33   ` [PATCH v4 2/5] LIS3LV02D: Remove calibaration functionality Samu Onkalo
2009-12-01  8:33     ` [PATCH v4 3/5] lis3: Sysfs entry for setting chip measurement rate Samu Onkalo
2009-12-01  8:33       ` [PATCH v4 4/5] lis3: Scale output values to mg Samu Onkalo
2009-12-01  8:33         ` [PATCH v4 5/5] LIS3: Update documentation to match latest changes Samu Onkalo
2009-12-02 17:40           ` Randy Dunlap
2009-12-01  8:53         ` [PATCH v4 4/5] lis3: Scale output values to mg Éric Piel
2009-12-01  8:53   ` [PATCH v4 1/5] lis3: Selftest support Éric Piel
2009-12-01  8:55 ` Éric Piel [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4B14DA10.8040106@tremplin-utc.net \
    --to=eric.piel@tremplin-utc.net \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lm-sensors@lm-sensors.org \
    --cc=samu.p.onkalo@nokia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox