All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mathieu Othacehe <m.othacehe@gmail.com>
To: jic23@kernel.org, knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net
Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
	Mathieu Othacehe <m.othacehe@gmail.com>
Subject: [PATCH v2 0/4] iio: vcnl: Add interrupts support for VCNL4010/20.
Date: Mon,  6 Apr 2020 16:53:52 +0200	[thread overview]
Message-ID: <20200406145356.25883-1-m.othacehe@gmail.com> (raw)

Hello,

Thanks for reviewing Jonathan! I did split the original patch into four
parts. I should have done it in the first place, sorry about that.

I did drop the ALS data from the buffer, as they are not really useful
anyway. I also used iio_device_claim_direct_mode as requested. But making sure
buffer and event capture are not done simultaneously is quite tricky as you
mentionned.

Thanks,

Mathieu

Changes from v1:
* Split into four different patches.
* Use iio_device_claim_direct_mode to protect
raw access from buffer capture.
* Requesting a sampling frequency above the limit is no longer possible.
* Inline read_isr and write_isr functions.
* Remove IIO_LIGHT data from buffer capture.
* Make sure postenable and predisable functions respect the common form.
* Do not set the trigger by default.
* Remove the devm_iio_triggered_buffer_setup top half.

Mathieu Othacehe (4):
  iio: vcnl4000: Factorize data reading and writing.
  iio: vcnl4000: Add event support for VCNL4010/20.
  iio: vcnl4000: Add sampling frequency support for VCNL4010/20.
  iio: vcnl4000: Add buffer support for VCNL4010/20.

 drivers/iio/light/Kconfig    |   2 +
 drivers/iio/light/vcnl4000.c | 856 +++++++++++++++++++++++++++++++++--
 2 files changed, 813 insertions(+), 45 deletions(-)

-- 
2.26.0


             reply	other threads:[~2020-04-06 14:54 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-06 14:53 Mathieu Othacehe [this message]
2020-04-06 14:53 ` [PATCH v2 1/4] iio: vcnl4000: Factorize data reading and writing Mathieu Othacehe
2020-04-06 14:53 ` [PATCH v2 2/4] iio: vcnl4000: Add event support for VCNL4010/20 Mathieu Othacehe
2020-04-06 14:53 ` [PATCH v2 3/4] iio: vcnl4000: Add sampling frequency " Mathieu Othacehe
2020-04-06 14:53 ` [PATCH v2 4/4] iio: vcnl4000: Add buffer " Mathieu Othacehe
2020-04-12 13:46   ` Jonathan Cameron
2020-04-14 14:56     ` Mathieu Othacehe

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=20200406145356.25883-1-m.othacehe@gmail.com \
    --to=m.othacehe@gmail.com \
    --cc=jic23@kernel.org \
    --cc=knaack.h@gmx.de \
    --cc=lars@metafoo.de \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pmeerw@pmeerw.net \
    /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.