From: denis.ciocca@st.com (Denis CIOCCA)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 14/38] iio: accel: st: Append _accel to accelerator sensor device names
Date: Mon, 16 Sep 2013 11:21:12 +0200 [thread overview]
Message-ID: <5236CD88.10809@st.com> (raw)
In-Reply-To: <5236C259.6050701@metafoo.de>
Hi Lars,
> On 09/14/2013 02:27 PM, Jonathan Cameron wrote:
>> On 09/14/13 13:14, Jonathan Cameron wrote:
>>> On 09/10/13 13:49, Lee Jones wrote:
>>>> Some of ST's sensors are appended with their sensor type and some
>>>> are not. For consistency we're extending the same naming convention
>>>> throughout.
>>>>
>>>> Signed-off-by: Lee Jones <lee.jones@linaro.org>
>>> Honestly I don't care either way on these, but consistency would definitely
>>> be good so applied to the togreg branch of iio.git
>>>
>>> Thanks,
>> Actually change of plan. I'm going to hold off on these as this an ABI change.
>> Iritating though having these as completely inconsistent is, changing this
>> will change device identification from userspace which is not a good idea.
>>
>> Sorry Lee, but we really shouldn't do this. I should have picked up on this
>> in the original driver reviews but that's hindsight for you.
> I think those with the suffix are the ones where you have accel and gyro in
> the same package, the ones without the suffix are pure accelerometers.
Yes You are right. I've used this kind of logic... What do you think about?
Denis
next prev parent reply other threads:[~2013-09-16 9:21 UTC|newest]
Thread overview: 109+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-10 12:49 [PATCH v3 00/38] iio: ST clean-ups and new sensor support Lee Jones
2013-09-10 12:49 ` [PATCH 01/38] ARM: ux500: Remove PrimeCell IDs from Nomadik I2C DT nodes Lee Jones
2013-09-10 15:23 ` Olof Johansson
2013-09-10 15:30 ` Lee Jones
2013-09-10 16:57 ` Olof Johansson
2013-09-11 8:06 ` Linus Walleij
2013-09-11 8:19 ` Lee Jones
2013-09-11 9:33 ` Linus Walleij
[not found] ` <CAF2Aj3iPdqM-+FQ7R9MZ-6jV1Q1me9TG1P4cEGA7fSBBYH92zA@mail.gmail.com>
2013-09-11 11:13 ` Linus Walleij
2013-09-11 20:36 ` Olof Johansson
2013-09-12 12:23 ` Linus Walleij
2013-09-12 12:32 ` Russell King - ARM Linux
2013-09-11 8:17 ` Lee Jones
2013-09-10 12:49 ` [PATCH 02/38] ARM: ux500: Enable the LPS001WP Pressure & Temperature sensor from DT Lee Jones
2013-09-10 12:49 ` [PATCH 03/38] ARM: ux500: Enable the LSM303DLH Accelerator " Lee Jones
2013-09-10 12:49 ` [PATCH 04/38] ARM: ux500: Enable the LSM303DLH Magnetometer " Lee Jones
2013-09-10 12:49 ` [PATCH 05/38] ARM: ux500: Enable the L3G4200D Gyroscope " Lee Jones
2013-09-10 12:49 ` [PATCH 06/38] ARM: ux500: CONFIG: Enable ST's IIO Pressure Sensors by default Lee Jones
2013-09-10 12:49 ` [PATCH 07/38] ARM: ux500: CONFIG: Enable ST's IIO Accelerometer " Lee Jones
2013-09-10 15:20 ` Olof Johansson
2013-09-10 15:35 ` Lee Jones
2013-09-10 16:51 ` Olof Johansson
2013-09-11 7:20 ` Lee Jones
2013-09-10 12:49 ` [PATCH 08/38] ARM: ux500: CONFIG: Enable ST's IIO Magnetometer " Lee Jones
2013-09-10 12:49 ` [PATCH 09/38] ARM: ux500: CONFIG: Enable ST's IIO Gyroscope " Lee Jones
2013-09-10 12:49 ` [PATCH 10/38] Documentation: dt: iio: Add binding for LPS001WP Lee Jones
2013-09-14 12:08 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 11/38] Documentation: dt: iio: Add binding for LSM303DLH Lee Jones
2013-09-14 12:09 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 12/38] Documentation: dt: iio: Add binding for L3G4200D Lee Jones
2013-09-14 12:09 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 13/38] Documentation: dt: iio: Add binding for LSM303DLH Lee Jones
2013-09-14 12:09 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 14/38] iio: accel: st: Append _accel to accelerator sensor device names Lee Jones
2013-09-14 12:14 ` Jonathan Cameron
2013-09-14 12:27 ` Jonathan Cameron
2013-09-16 8:33 ` Lars-Peter Clausen
2013-09-16 9:21 ` Denis CIOCCA [this message]
2013-09-16 12:03 ` Lars-Peter Clausen
2013-09-16 19:15 ` Jonathan Cameron
2013-09-16 13:31 ` Lee Jones
2013-09-16 13:43 ` Lars-Peter Clausen
2013-09-16 19:17 ` Jonathan Cameron
2013-09-17 8:05 ` Lee Jones
2013-09-10 12:49 ` [PATCH 15/38] iio: gyro: st: Append _gyro to gyroscope " Lee Jones
2013-09-10 12:49 ` [PATCH 16/38] iio: magn: st: Append _magn to magnetometer " Lee Jones
2013-09-10 12:49 ` [PATCH 17/38] iio: press: st: Append _press to pressure " Lee Jones
2013-09-10 12:49 ` [PATCH 18/38] iio: sensors-core: st: Allow full-scale to be an optional feature Lee Jones
2013-09-14 16:45 ` Jonathan Cameron
2013-09-14 17:07 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 19/38] iio: sensors-core: st: Support sensors which don't have a Data Ready pin Lee Jones
2013-09-14 16:53 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 20/38] iio: sensors-core: st: Clean-up error handling in st_sensors_init_sensor() Lee Jones
2013-09-14 16:54 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 21/38] iio: sensors-core: st: Clean-up error handling in st_sensors_read_axis_data() Lee Jones
2013-09-14 16:58 ` Jonathan Cameron
2013-09-16 8:22 ` Lee Jones
2013-09-10 12:49 ` [PATCH 22/38] iio: sensors-core: st: Clean-up error handling in st_sensors_read_info_raw() Lee Jones
2013-09-14 17:01 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 23/38] iio: pressure-core: st: Describe LPS331AP defines by name Lee Jones
2013-09-14 17:09 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 24/38] iio: pressure-core: st: Expand and rename LPS331AP's channel descriptor Lee Jones
2013-09-14 17:14 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 25/38] iio: pressure-core: st: Allow for number of channels to vary Lee Jones
2013-09-14 17:18 ` Jonathan Cameron
2013-09-16 8:17 ` Lee Jones
2013-09-16 15:32 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 26/38] iio: pressure-core: st: Clean-up probe() function Lee Jones
2013-09-10 16:23 ` Jonathan Cameron
2013-09-11 7:19 ` Lee Jones
2013-09-14 17:21 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 27/38] iio: pressure-core: st: Give some indication if device probing was successful Lee Jones
2013-09-10 16:25 ` Jonathan Cameron
2013-09-11 7:10 ` Lee Jones
2013-09-11 7:18 ` Lars-Peter Clausen
2013-09-11 7:29 ` Lee Jones
2013-09-11 15:35 ` Mark Brown
2013-09-13 20:57 ` Getz, Robin
2013-09-16 9:47 ` Lee Jones
2013-09-11 18:30 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 28/38] iio: pressure: st: Add support for new LPS001WP pressure sensor Lee Jones
2013-09-14 17:26 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 29/38] iio: pressure-core: st: Provide support for the Vdd power supply Lee Jones
2013-09-14 17:28 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 30/38] iio: pressure-core: st: Provide support for the Vdd_IO " Lee Jones
2013-09-10 12:49 ` [PATCH 31/38] iio: accel-core: st: Clean up error handling in probe() Lee Jones
2013-09-14 17:31 ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 32/38] iio: accel-core: st: Move LSM303DLH into correct group Lee Jones
2013-09-14 17:51 ` Jonathan Cameron
2013-09-16 8:05 ` Lee Jones
2013-09-16 9:19 ` Denis CIOCCA
2013-09-16 9:38 ` Lee Jones
2013-09-16 9:57 ` Denis CIOCCA
2013-09-16 10:19 ` Lee Jones
2013-09-16 10:23 ` Denis CIOCCA
2013-09-16 11:08 ` Lee Jones
2013-09-10 12:49 ` [PATCH 33/38] iio: accel-core: st: Give some indication if device probing was successful Lee Jones
2013-09-10 12:49 ` [PATCH 34/38] iio: gyro-core: st: Clean up error handling in probe() Lee Jones
2013-09-10 12:49 ` [PATCH 35/38] iio: gyro-core: st: Give some indication if device probing was successful Lee Jones
2013-09-10 12:49 ` [PATCH 36/38] iio: magn-core: st: Clean up error handling in probe() Lee Jones
2013-09-10 12:49 ` [PATCH 37/38] iio: magn-core: st: Give some indication if device probing was successful Lee Jones
2013-09-10 12:49 ` [PATCH 38/38] iio: magn-core: st: Provide support for the LSM303DLH Lee Jones
2013-09-14 17:53 ` Jonathan Cameron
2013-09-16 7:31 ` Lee Jones
2013-09-16 9:32 ` Denis CIOCCA
2013-09-16 10:08 ` Lee Jones
2013-09-16 10:18 ` Denis CIOCCA
2013-09-16 11:11 ` Lee Jones
2013-09-16 11:26 ` Lee Jones
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=5236CD88.10809@st.com \
--to=denis.ciocca@st.com \
--cc=linux-arm-kernel@lists.infradead.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).