From: Stephen Warren <swarren@wwwdotorg.org>
To: Jonathan Cameron <jic23@kernel.org>
Cc: Oleksandr Kravchenko <o.v.kravchenko@globallogic.com>,
devicetree@vger.kernel.org, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org,
rob.herring@calxeda.com, pawel.moll@arm.com,
mark.rutland@arm.com, ian.campbell@citrix.com, rob@landley.net,
jic23@cam.ac.uk, denis.ciocca@st.com,
srinivas.pandruvada@linux.intel.com, lars@metafoo.de
Subject: Re: [PATCH v4 2/2] iio: add Bosch BMA180 acceleration sensor driver
Date: Mon, 19 Aug 2013 15:22:28 -0600 [thread overview]
Message-ID: <52128C94.2070402@wwwdotorg.org> (raw)
In-Reply-To: <520FDF09.2000507@kernel.org>
On 08/17/2013 02:37 PM, Jonathan Cameron wrote:
> On 08/16/13 13:12, Oleksandr Kravchenko wrote:
>> This patch adds IIO driver for Bosch BMA180 triaxial
>> acceleration sensor.
>> http://dlnmh9ip6v2uc.cloudfront.net/datasheets/
>> Sensors/Accelerometers/BST-BMA180-DS000-07_2.pdf
>>
>> Signed-off-by: Oleksandr Kravchenko <o.v.kravchenko@globallogic.com>
>
> Surely your power_mode ext_info entry wants to be shared?
> It is the same control for all 3 accelerometer channels...
>
> Jonathan
Admittedly you put a signature here so it was a little bit clearer than
some cases, but it'd help greatly if all irrelevant context was deleted
from emails when replying so people reading the reply didn't have to
scroll through hundreds of lines of patch just to see if anything else
was said.
> [entire original patch quoted]
next prev parent reply other threads:[~2013-08-19 21:22 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-16 12:12 [PATCH v4 1/2] staging: iio: Documentation sysfs-bus-iio add power_mode Oleksandr Kravchenko
2013-08-16 12:12 ` [PATCH v4 2/2] iio: add Bosch BMA180 acceleration sensor driver Oleksandr Kravchenko
2013-08-17 20:37 ` Jonathan Cameron
2013-08-17 19:48 ` Jonathan Cameron
2013-08-19 21:22 ` Stephen Warren [this message]
2013-08-19 21:24 ` Stephen Warren
2013-08-17 20:29 ` [PATCH v4 1/2] staging: iio: Documentation sysfs-bus-iio add power_mode 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=52128C94.2070402@wwwdotorg.org \
--to=swarren@wwwdotorg.org \
--cc=denis.ciocca@st.com \
--cc=devicetree@vger.kernel.org \
--cc=ian.campbell@citrix.com \
--cc=jic23@cam.ac.uk \
--cc=jic23@kernel.org \
--cc=lars@metafoo.de \
--cc=linux-doc@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=o.v.kravchenko@globallogic.com \
--cc=pawel.moll@arm.com \
--cc=rob.herring@calxeda.com \
--cc=rob@landley.net \
--cc=srinivas.pandruvada@linux.intel.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