From: Jonathan Cameron <jic23@kernel.org>
To: linux-iio@vger.kernel.org
Cc: Jonathan Cameron <jic23@kernel.org>
Subject: [PATCH 0/5] staging:iio: drop sw_ring buffer implementation.
Date: Fri, 30 Nov 2012 14:22:47 +0000 [thread overview]
Message-ID: <1354285372-11779-1-git-send-email-jic23@kernel.org> (raw)
sw_ring was the original buffer implementation for IIO. It kind of evolved
over a couple of late nights and even though I wrote it I was never
convinced it was right. Then a nice kfifo implementation came along
and in the vast majority of IIO use cases whether we have a kfifo or
a ring buffer is not as it turns out that important.
We have been telling people not to use this for quite some time now.
Lets make that finally a really hard rule by dropping it.
There are 4 remaining users. This patch drops support form lis3l02dq
which had support for both this and kfifo. For the other 3 users
it simply switches over to kfifo. We haven't seen any real problems
doing this with any other driver so I can't see why these would cause
trouble!
Note my aim for this cycle ist to greatly cut down what we have left
in staging by removing legacy cruft and getting other stuff cleaned up and
out!
Jonathan
Jonathan Cameron (5):
staging:iio:accel:lis3l02dq remove sw_ring support.
staging:iio:impedance-analyzer switch from sw_ring to kfifo.
staging:iio:meter:ade7758 switch from sw_ring to kfifo
staging:iio:imu: adis16400 switch from sw_ring to kfifo
staging:iio: drop sw_ring buffer implementation.
drivers/staging/iio/Kconfig | 13 -
drivers/staging/iio/Makefile | 2 -
drivers/staging/iio/accel/Kconfig | 23 +-
drivers/staging/iio/accel/lis3l02dq.h | 8 -
drivers/staging/iio/accel/lis3l02dq_ring.c | 7 +-
drivers/staging/iio/impedance-analyzer/Kconfig | 2 +-
drivers/staging/iio/impedance-analyzer/ad5933.c | 8 +-
drivers/staging/iio/imu/Kconfig | 2 +-
drivers/staging/iio/imu/adis16400_ring.c | 12 +-
drivers/staging/iio/meter/Kconfig | 2 +-
drivers/staging/iio/meter/ade7758_ring.c | 12 +-
drivers/staging/iio/ring_sw.c | 366 ------------------------
drivers/staging/iio/ring_sw.h | 30 --
13 files changed, 23 insertions(+), 464 deletions(-)
delete mode 100644 drivers/staging/iio/ring_sw.c
delete mode 100644 drivers/staging/iio/ring_sw.h
--
1.8.0.1
next reply other threads:[~2012-11-30 14:22 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-30 14:22 Jonathan Cameron [this message]
2012-11-30 14:22 ` [PATCH 1/5] staging:iio:accel:lis3l02dq remove sw_ring support Jonathan Cameron
2012-11-30 14:22 ` [PATCH 2/5] staging:iio:impedance-analyzer switch from sw_ring to kfifo Jonathan Cameron
2012-11-30 14:22 ` [PATCH 3/5] staging:iio:meter:ade7758 " Jonathan Cameron
2012-11-30 14:22 ` [PATCH 4/5] staging:iio:imu: adis16400 " Jonathan Cameron
2012-11-30 14:35 ` Lars-Peter Clausen
2012-11-30 14:58 ` Jonathan Cameron
2012-11-30 14:22 ` [PATCH 5/5] staging:iio: drop sw_ring buffer implementation Jonathan Cameron
2013-01-19 13:49 ` [PATCH 0/5] " Jonathan Cameron
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=1354285372-11779-1-git-send-email-jic23@kernel.org \
--to=jic23@kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).