All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Cameron <jic23@cam.ac.uk>
To: Barry Song <21cnbao@gmail.com>
Cc: linux-iio@vger.kernel.org, Barry.Song@analog.com
Subject: Re: [PATCH 0/6] Various cleanups.
Date: Mon, 28 Jun 2010 10:50:31 +0100	[thread overview]
Message-ID: <4C287067.6080805@cam.ac.uk> (raw)
In-Reply-To: <AANLkTil0gGAy1qJFPizZyl0AvmUCMLMVOoaXGXXBNzOI@mail.gmail.com>

On 06/28/10 09:16, Barry Song wrote:
> On Fri, Jun 25, 2010 at 11:55 PM, Jonathan Cameron <jic23@cam.ac.uk> wrote:
>> This patch set is the result of taking a look at Barry Song's
>> RFC.  The idea was to get the simple uncontroversial elements
>> of that in place before we move on to discussion of the more
>> involved elements.
> Thanks a lot for your great help.  I acked the three patches
> abstracted from the discussion.
Thanks, though for some reason I only got the ack emails on patches
3 and 4.  Which other one were you happy to ack? If you have a chane
to cast your eyes over 6 (the be16 one) just to check I got that the
right way round that would be great!

Thanks,

Jonathan
> 
>>
>> One or two general cleanups have slipped in here as well.
>>
>> Jonathan Cameron (6):
>>  staging:iio: Use kasprintf to allocate and fill trig->name
>>  staging:iio:lis3l02dq remove unused set_ring_length function
>>  staging:iio: Add and convert drivers to use iio_alloc_pollfunc
>>  staging:iio: Add iio_triggered_ring postenable and predisable + use
>>    in drivers
>>  staging:iio: remove timestamp field from trigger and pass instead
>>    through pollfuncs
>>  staging:iio: replace combine_8_to_16 with be16_to_cpup where
>>    possible.
>>
>>  drivers/staging/iio/accel/adis16209_ring.c         |   51 ++++---------------
>>  drivers/staging/iio/accel/adis16209_trigger.c      |   10 ++--
>>  drivers/staging/iio/accel/adis16240_ring.c         |   51 ++++---------------
>>  drivers/staging/iio/accel/adis16240_trigger.c      |   10 ++--
>>  drivers/staging/iio/accel/lis3l02dq_ring.c         |   53 +++++---------------
>>  drivers/staging/iio/adc/max1363_ring.c             |   45 ++---------------
>>  drivers/staging/iio/gyro/adis16260_ring.c          |   51 ++++---------------
>>  drivers/staging/iio/gyro/adis16260_trigger.c       |   10 ++--
>>  drivers/staging/iio/imu/adis16300_ring.c           |   51 ++++---------------
>>  drivers/staging/iio/imu/adis16300_trigger.c        |   10 ++--
>>  drivers/staging/iio/imu/adis16350_ring.c           |   51 ++++---------------
>>  drivers/staging/iio/imu/adis16350_trigger.c        |   10 ++--
>>  drivers/staging/iio/imu/adis16400_ring.c           |   51 ++++---------------
>>  drivers/staging/iio/imu/adis16400_trigger.c        |   10 ++--
>>  drivers/staging/iio/industrialio-trigger.c         |   48 ++++++++++++++----
>>  drivers/staging/iio/trigger.h                      |   24 ++++++---
>>  drivers/staging/iio/trigger/iio-trig-gpio.c        |   12 ++---
>>  .../staging/iio/trigger/iio-trig-periodic-rtc.c    |   15 ++----
>>  18 files changed, 164 insertions(+), 399 deletions(-)
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
> 


  reply	other threads:[~2010-06-28  9:47 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-25 15:55 [PATCH 0/6] Various cleanups Jonathan Cameron
2010-06-25 15:55 ` [PATCH 1/6] staging:iio: Use kasprintf to allocate and fill trig->name Jonathan Cameron
2010-06-25 15:55   ` [PATCH 2/6] staging:iio:lis3l02dq remove unused set_ring_length function Jonathan Cameron
2010-06-25 15:55     ` [PATCH 3/6] staging:iio: Add and convert drivers to use iio_alloc_pollfunc Jonathan Cameron
2010-06-25 15:55       ` [PATCH 4/6] staging:iio: Add iio_triggered_ring postenable and predisable + use in drivers Jonathan Cameron
2010-06-25 15:55         ` [PATCH 5/6] staging:iio: remove timestamp field from trigger and pass instead through pollfuncs Jonathan Cameron
2010-06-25 15:55           ` [PATCH 6/6] staging:iio: replace combine_8_to_16 with be16_to_cpup where possible Jonathan Cameron
2010-06-30  4:18             ` Barry Song
2010-06-29 10:30           ` [PATCH 5/6] staging:iio: remove timestamp field from trigger and pass instead through pollfuncs Barry Song
2010-06-28  6:35         ` [PATCH 4/6] staging:iio: Add iio_triggered_ring postenable and predisable + use in drivers Barry Song
2010-06-28  6:35       ` [PATCH 3/6] staging:iio: Add and convert drivers to use iio_alloc_pollfunc Barry Song
2010-06-30  4:21     ` [PATCH 2/6] staging:iio:lis3l02dq remove unused set_ring_length function Barry Song
2010-06-29 10:18   ` [PATCH 1/6] staging:iio: Use kasprintf to allocate and fill trig->name Barry Song
2010-06-28  8:16 ` [PATCH 0/6] Various cleanups Barry Song
2010-06-28  9:50   ` Jonathan Cameron [this message]
  -- strict thread matches above, loose matches on Subject: below --
2025-06-27  1:11 Damien Le Moal
2009-11-27 10:57 Philippe Seewer
     [not found] ` <1259319436-14925-1-git-send-email-philippe.seewer-omB+W0Dpw2o@public.gmane.org>
2009-11-27 13:08   ` Harald Hoyer
2007-04-04  1:06 [patch " Jeremy Fitzhardinge
2007-04-04  1:06 ` Jeremy Fitzhardinge

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=4C287067.6080805@cam.ac.uk \
    --to=jic23@cam.ac.uk \
    --cc=21cnbao@gmail.com \
    --cc=Barry.Song@analog.com \
    --cc=linux-iio@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.