From: Jonathan Cameron <jic23@kernel.org>
To: Denis CIOCCA <denis.ciocca@st.com>
Cc: lars@metafoo.de, linux-iio@vger.kernel.org
Subject: Re: iio: STMicroelectronics iio drivers bugfix
Date: Sat, 12 Jan 2013 22:02:43 +0000 [thread overview]
Message-ID: <50F1DD83.2080301@kernel.org> (raw)
In-Reply-To: <1357733221-6248-1-git-send-email-denis.ciocca@st.com>
On 01/09/2013 12:06 PM, Denis CIOCCA wrote:
> Hi guys,
>
> sorry but yesterday I sended to you the wrong patch! I attach the last (correct) patchs for the linux-next kernel and I have modified the config names in according with Alessandro suggestion.
> Thanks
>
> Denis
>
Hi Denis,
Just a quick email to say I'm very happy with the state these patches are
now in. I have one utterly trivial point on patch 1 but am happy to fix
that up during the merge. (Basically you type cast a parameter to u16 instead
of __le16).
However, given the size of the series I would like to leave them sitting
on the mailing list for a little while (maybe until next Saturday given
my usual timetable for such things) in case anyone else wants to review
them. Also right now I've only looked at 1 and 2 (assuming the others are
very similar for obvious reasons!) I will give them a look before merging
just in case anything slipped by.
Thanks for all your work on these. When I get some time I'll see what is needed
to add support for the lis3l02dq to this driver. From what I recall that
chip is a little dumber than some of the newer ones and needs some uggly
interrupt handling trickery. Given it's an ancient chip I doubt it is
terribly high on your list to support. I just happen to have one on my
test board so it's always been of inordinate interest to me ;)
Jonathan
next prev parent reply other threads:[~2013-01-12 22:02 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-09 12:06 iio: STMicroelectronics iio drivers bugfix Denis CIOCCA
2013-01-09 12:06 ` [PATCH 1/4] iio:common: Add STMicroelectronics common library Denis CIOCCA
2013-01-12 21:59 ` Jonathan Cameron
2013-01-09 12:06 ` [PATCH 2/4] iio:accel: Add STMicroelectronics accelerometers driver Denis CIOCCA
2013-01-09 12:07 ` [PATCH 3/4] iio:gyro: Add STMicroelectronics gyroscopes driver Denis CIOCCA
2013-01-09 12:07 ` [PATCH 4/4] iio:magnetometer: Add STMicroelectronics magnetometers driver Denis CIOCCA
2013-01-12 22:02 ` Jonathan Cameron [this message]
2013-01-12 22:20 ` iio: STMicroelectronics iio drivers bugfix Denis Ciocca
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=50F1DD83.2080301@kernel.org \
--to=jic23@kernel.org \
--cc=denis.ciocca@st.com \
--cc=lars@metafoo.de \
--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).