From: Jonathan Cameron <jic23@kernel.org>
To: Scott Matheina <scott@matheina.com>, linux-kernel@vger.kernel.org
Cc: Hartmut Knaack <knaack.h@gmx.de>,
Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
linux-iio@vger.kernel.org, devel@driverdev.osuosl.org
Subject: Re: [PATCHv4 3/8] Fix camel case issues
Date: Fri, 30 Dec 2016 18:45:19 +0000 [thread overview]
Message-ID: <8d349688-4be7-01be-66b9-f2fdcbe301ce@kernel.org> (raw)
In-Reply-To: <1482696216-8013-4-git-send-email-scott@matheina.com>
On 25/12/16 19:56, Scott Matheina wrote:
> Cases of camel case were fixed by making variables lower case throughout
> the file.
>
> Signed-off-by: Scott Matheina <scott@matheina.com>
So of these cases might arguably be worth doing, but extreme care is needed to ensure
that the code is not broken.
Also, you are modifiying userspace ABI here so another level of care is needed.
To approach this sort of change I would suggest an RFC for the proposed ABI
changes.
In this particular case the driver is so far from standard ABI that you would
likely get the response that it needs to conform to standard ABI rather than
fixing it's own random choices of what sysfs files should be called.
Jonathan
> ---
> drivers/staging/iio/addac/adt7316.c | 82 ++++++++++++++++++-------------------
> 1 file changed, 41 insertions(+), 41 deletions(-)
>
> diff --git a/drivers/staging/iio/addac/adt7316.c b/drivers/staging/iio/addac/adt7316.c
> index 13d9ed2..2b584a0 100644
> --- a/drivers/staging/iio/addac/adt7316.c
> +++ b/drivers/staging/iio/addac/adt7316.c
> @@ -1485,7 +1485,7 @@ static ssize_t adt7316_show_DAC_A(struct device *dev,
> struct iio_dev *dev_info = dev_to_iio_dev(dev);
> struct adt7316_chip_info *chip = iio_priv(dev_info);
>
> - return adt7316_show_DAC(chip, 0, buf);
> + return adt7316_show_dac(chip, 0, buf);
> }
>
> static ssize_t adt7316_store_DAC_A(struct device *dev,
> @@ -1496,7 +1496,7 @@ static ssize_t adt7316_store_DAC_A(struct device *dev,
> struct iio_dev *dev_info = dev_to_iio_dev(dev);
> struct adt7316_chip_info *chip = iio_priv(dev_info);
>
> - return adt7316_store_DAC(chip, 0, buf, len);
> + return adt7316_store_dac(chip, 0, buf, len);
> }
>
> static IIO_DEVICE_ATTR(DAC_A, S_IRUGO | S_IWUSR, adt7316_show_DAC_A,
> @@ -1509,7 +1509,7 @@ static ssize_t adt7316_show_DAC_B(struct device *dev,
> struct iio_dev *dev_info = dev_to_iio_dev(dev);
> struct adt7316_chip_info *chip = iio_priv(dev_info);
>
> - return adt7316_show_DAC(chip, 1, buf);
> + return adt7316_show_dac(chip, 1, buf);
> }
>
> static ssize_t adt7316_store_DAC_B(struct device *dev,
> @@ -1520,7 +1520,7 @@ static ssize_t adt7316_store_DAC_B(struct device *dev,
> struct iio_dev *dev_info = dev_to_iio_dev(dev);
> struct adt7316_chip_info *chip = iio_priv(dev_info);
>
> - return adt7316_store_DAC(chip, 1, buf, len);
> + return adt7316_store_dac(chip, 1, buf, len);
> }
>
> static IIO_DEVICE_ATTR(DAC_B, S_IRUGO | S_IWUSR, adt7316_show_DAC_B,
> @@ -1533,7 +1533,7 @@ static ssize_t adt7316_show_DAC_C(struct device *dev,
> struct iio_dev *dev_info = dev_to_iio_dev(dev);
> struct adt7316_chip_info *chip = iio_priv(dev_info);
>
> - return adt7316_show_DAC(chip, 2, buf);
> + return adt7316_show_dac(chip, 2, buf);
> }
>
> static ssize_t adt7316_store_DAC_C(struct device *dev,
> @@ -1544,7 +1544,7 @@ static ssize_t adt7316_store_DAC_C(struct device *dev,
> struct iio_dev *dev_info = dev_to_iio_dev(dev);
> struct adt7316_chip_info *chip = iio_priv(dev_info);
>
> - return adt7316_store_DAC(chip, 2, buf, len);
> + return adt7316_store_dac(chip, 2, buf, len);
> }
>
> static IIO_DEVICE_ATTR(DAC_C, S_IRUGO | S_IWUSR, adt7316_show_DAC_C,
> @@ -1557,7 +1557,7 @@ static ssize_t adt7316_show_DAC_D(struct device *dev,
> struct iio_dev *dev_info = dev_to_iio_dev(dev);
> struct adt7316_chip_info *chip = iio_priv(dev_info);
>
> - return adt7316_show_DAC(chip, 3, buf);
> + return adt7316_show_dac(chip, 3, buf);
> }
>
> static ssize_t adt7316_store_DAC_D(struct device *dev,
> @@ -1568,7 +1568,7 @@ static ssize_t adt7316_store_DAC_D(struct device *dev,
> struct iio_dev *dev_info = dev_to_iio_dev(dev);
> struct adt7316_chip_info *chip = iio_priv(dev_info);
>
> - return adt7316_store_DAC(chip, 3, buf, len);
> + return adt7316_store_dac(chip, 3, buf, len);
> }
>
> static IIO_DEVICE_ATTR(DAC_D, S_IRUGO | S_IWUSR, adt7316_show_DAC_D,
> @@ -1661,26 +1661,26 @@ static struct attribute *adt7316_attributes[] = {
> &iio_dev_attr_powerdown.dev_attr.attr,
> &iio_dev_attr_fast_ad_clock.dev_attr.attr,
> &iio_dev_attr_da_high_resolution.dev_attr.attr,
> - &iio_dev_attr_enable_proportion_DACA.dev_attr.attr,
> - &iio_dev_attr_enable_proportion_DACB.dev_attr.attr,
> - &iio_dev_attr_DAC_2Vref_channels_mask.dev_attr.attr,
> - &iio_dev_attr_DAC_update_mode.dev_attr.attr,
> - &iio_dev_attr_all_DAC_update_modes.dev_attr.attr,
> - &iio_dev_attr_update_DAC.dev_attr.attr,
> - &iio_dev_attr_DA_AB_Vref_bypass.dev_attr.attr,
> - &iio_dev_attr_DA_CD_Vref_bypass.dev_attr.attr,
> - &iio_dev_attr_DAC_internal_Vref.dev_attr.attr,
> - &iio_dev_attr_VDD.dev_attr.attr,
> + &iio_dev_attr_enable_proportion_daca.dev_attr.attr,
> + &iio_dev_attr_enable_proportion_dacb.dev_attr.attr,
> + &iio_dev_attr_dac_2vref_channels_mask.dev_attr.attr,
> + &iio_dev_attr_dac_internal_vref.dev_attr.attr,
> + &iio_dev_attr_all_dac_update_modes.dev_attr.attr,
> + &iio_dev_attr_update_dac.dev_attr.attr,
> + &iio_dev_attr_da_ab_vref_bypass.dev_attr.attr,
> + &iio_dev_attr_da_cd_vref_bypass.dev_attr.attr,
> + &iio_dev_attr_dac_internal_vref.dev_attr.attr,
> + &iio_dev_attr_vdd.dev_attr.attr,
> &iio_dev_attr_in_temp.dev_attr.attr,
> &iio_dev_attr_ex_temp.dev_attr.attr,
> &iio_dev_attr_in_temp_offset.dev_attr.attr,
> &iio_dev_attr_ex_temp_offset.dev_attr.attr,
> &iio_dev_attr_in_analog_temp_offset.dev_attr.attr,
> &iio_dev_attr_ex_analog_temp_offset.dev_attr.attr,
> - &iio_dev_attr_DAC_A.dev_attr.attr,
> - &iio_dev_attr_DAC_B.dev_attr.attr,
> - &iio_dev_attr_DAC_C.dev_attr.attr,
> - &iio_dev_attr_DAC_D.dev_attr.attr,
> + &iio_dev_attr_dac_a.dev_attr.attr,
> + &iio_dev_attr_dac_b.dev_attr.attr,
> + &iio_dev_attr_dac_c.dev_attr.attr,
> + &iio_dev_attr_dac_d.dev_attr.attr,
> &iio_dev_attr_device_id.dev_attr.attr,
> &iio_dev_attr_manufactorer_id.dev_attr.attr,
> &iio_dev_attr_device_rev.dev_attr.attr,
> @@ -1703,31 +1703,31 @@ static struct attribute *adt7516_attributes[] = {
> &iio_dev_attr_enable_smbus_timeout.dev_attr.attr,
> &iio_dev_attr_powerdown.dev_attr.attr,
> &iio_dev_attr_fast_ad_clock.dev_attr.attr,
> - &iio_dev_attr_AIN_internal_Vref.dev_attr.attr,
> + &iio_dev_attr_ain_internal_vref.dev_attr.attr,
> &iio_dev_attr_da_high_resolution.dev_attr.attr,
> - &iio_dev_attr_enable_proportion_DACA.dev_attr.attr,
> - &iio_dev_attr_enable_proportion_DACB.dev_attr.attr,
> - &iio_dev_attr_DAC_2Vref_channels_mask.dev_attr.attr,
> - &iio_dev_attr_DAC_update_mode.dev_attr.attr,
> - &iio_dev_attr_all_DAC_update_modes.dev_attr.attr,
> - &iio_dev_attr_update_DAC.dev_attr.attr,
> - &iio_dev_attr_DA_AB_Vref_bypass.dev_attr.attr,
> - &iio_dev_attr_DA_CD_Vref_bypass.dev_attr.attr,
> - &iio_dev_attr_DAC_internal_Vref.dev_attr.attr,
> - &iio_dev_attr_VDD.dev_attr.attr,
> + &iio_dev_attr_enable_proportion_daca.dev_attr.attr,
> + &iio_dev_attr_enable_proportion_dacb.dev_attr.attr,
> + &iio_dev_attr_dac_2vref_channels_mask.dev_attr.attr,
> + &iio_dev_attr_dac_update_mode.dev_attr.attr,
> + &iio_dev_attr_all_dac_update_modes.dev_attr.attr,
> + &iio_dev_attr_update_dac.dev_attr.attr,
> + &iio_dev_attr_da_ab_vref_bypass.dev_attr.attr,
> + &iio_dev_attr_da_cd_vref_bypass.dev_attr.attr,
> + &iio_dev_attr_dac_internal_vref.dev_attr.attr,
> + &iio_dev_attr_vdd.dev_attr.attr,
> &iio_dev_attr_in_temp.dev_attr.attr,
> - &iio_dev_attr_ex_temp_AIN1.dev_attr.attr,
> - &iio_dev_attr_AIN2.dev_attr.attr,
> - &iio_dev_attr_AIN3.dev_attr.attr,
> - &iio_dev_attr_AIN4.dev_attr.attr,
> + &iio_dev_attr_ex_temp_ain1.dev_attr.attr,
> + &iio_dev_attr_ain2.dev_attr.attr,
> + &iio_dev_attr_ain3.dev_attr.attr,
> + &iio_dev_attr_ain4.dev_attr.attr,
> &iio_dev_attr_in_temp_offset.dev_attr.attr,
> &iio_dev_attr_ex_temp_offset.dev_attr.attr,
> &iio_dev_attr_in_analog_temp_offset.dev_attr.attr,
> &iio_dev_attr_ex_analog_temp_offset.dev_attr.attr,
> - &iio_dev_attr_DAC_A.dev_attr.attr,
> - &iio_dev_attr_DAC_B.dev_attr.attr,
> - &iio_dev_attr_DAC_C.dev_attr.attr,
> - &iio_dev_attr_DAC_D.dev_attr.attr,
> + &iio_dev_attr_dac_a.dev_attr.attr,
> + &iio_dev_attr_dac_b.dev_attr.attr,
> + &iio_dev_attr_dac_c.dev_attr.attr,
> + &iio_dev_attr_dac_d.dev_attr.attr,
> &iio_dev_attr_device_id.dev_attr.attr,
> &iio_dev_attr_manufactorer_id.dev_attr.attr,
> &iio_dev_attr_device_rev.dev_attr.attr,
>
next prev parent reply other threads:[~2016-12-30 18:45 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-25 19:56 [PATCHv4 0/8] Checkpatch fixes to driver/staging/iio/addac Scott Matheina
2016-12-25 19:56 ` [PATCHv4 1/8] fixed long description text exceeding 80 characters Scott Matheina
2016-12-30 18:40 ` Jonathan Cameron
2016-12-25 19:56 ` [PATCHv4 2/8] Fixed variables not being consistently lower case Scott Matheina
2016-12-25 20:31 ` kbuild test robot
2016-12-25 19:56 ` [PATCHv4 3/8] Fix camel case issues Scott Matheina
2016-12-25 20:54 ` kbuild test robot
2016-12-30 18:45 ` Jonathan Cameron [this message]
2016-12-25 19:56 ` [PATCHv4 4/8] Fix braces not present on all arms of if else statement Scott Matheina
2016-12-30 18:46 ` Jonathan Cameron
2016-12-30 18:48 ` Jonathan Cameron
2016-12-25 19:56 ` [PATCHv4 5/8] Remove line after closing braces Scott Matheina
2016-12-25 19:56 ` [PATCHv4 6/8] Fixed code wrap alignment with preceding ( Scott Matheina
2016-12-30 18:51 ` Jonathan Cameron
2016-12-25 19:56 ` [PATCHv4 7/8] Changed code to align with coding style of using octat Scott Matheina
2016-12-25 20:41 ` kbuild test robot
2016-12-30 18:53 ` Jonathan Cameron
2016-12-25 19:56 ` [PATCHv4 8/8] Fixes style issues due to mis-aligned carry over lines Scott Matheina
2016-12-25 21:18 ` kbuild test robot
2016-12-30 18:56 ` Jonathan Cameron
-- strict thread matches above, loose matches on Subject: below --
2016-12-25 19:41 [PATCHv4 0/8] Checkpatch fixes to driver/staging/iio/addac Scott Matheina
2016-12-25 19:41 ` [PATCHv4 3/8] Fix camel case issues Scott Matheina
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=8d349688-4be7-01be-66b9-f2fdcbe301ce@kernel.org \
--to=jic23@kernel.org \
--cc=devel@driverdev.osuosl.org \
--cc=knaack.h@gmx.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pmeerw@pmeerw.net \
--cc=scott@matheina.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
all inboxes | Powered by JetHome®