Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756382Ab3INL1P (ORCPT ); Sat, 14 Sep 2013 07:27:15 -0400 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:40724 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752516Ab3INL1O (ORCPT ); Sat, 14 Sep 2013 07:27:14 -0400 Message-ID: <5234562D.5050803@kernel.org> Date: Sat, 14 Sep 2013 13:27:25 +0100 From: Jonathan Cameron User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130806 Thunderbird/17.0.8 MIME-Version: 1.0 To: Lee Jones CC: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, jic23@cam.ac.uk, arnd@arndb.de, linus.walleij@linaro.org, denis.ciocca@st.com, linux-iio@vger.kernel.org Subject: Re: [PATCH 14/38] iio: accel: st: Append _accel to accelerator sensor device names References: <1378817379-8238-1-git-send-email-lee.jones@linaro.org> <1378817379-8238-15-git-send-email-lee.jones@linaro.org> <52345330.4050707@kernel.org> In-Reply-To: <52345330.4050707@kernel.org> X-Enigmail-Version: 1.5.2 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2197 Lines: 52 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 > 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. >> --- >> drivers/iio/accel/st_accel.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/iio/accel/st_accel.h b/drivers/iio/accel/st_accel.h >> index c387763..d8d22e5 100644 >> --- a/drivers/iio/accel/st_accel.h >> +++ b/drivers/iio/accel/st_accel.h >> @@ -15,11 +15,11 @@ >> #include >> >> #define LSM303DLHC_ACCEL_DEV_NAME "lsm303dlhc_accel" >> -#define LIS3DH_ACCEL_DEV_NAME "lis3dh" >> +#define LIS3DH_ACCEL_DEV_NAME "lis3dh_accel" >> #define LSM330D_ACCEL_DEV_NAME "lsm330d_accel" >> #define LSM330DL_ACCEL_DEV_NAME "lsm330dl_accel" >> #define LSM330DLC_ACCEL_DEV_NAME "lsm330dlc_accel" >> -#define LIS331DLH_ACCEL_DEV_NAME "lis331dlh" >> +#define LIS331DLH_ACCEL_DEV_NAME "lis331dlh_accel" >> #define LSM303DL_ACCEL_DEV_NAME "lsm303dl_accel" >> #define LSM303DLH_ACCEL_DEV_NAME "lsm303dlh_accel" >> #define LSM303DLM_ACCEL_DEV_NAME "lsm303dlm_accel" >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/