On Thu, 4 Oct 2018 20:06:55 -0300
Renato Lui Geh <renatogeh@xxxxxxxxx> wrote:
> This patch fixes all "Alignment should match open parenthesis" check
> warnings found by checkpatch.pl in the addac/adt7316 driver.
>
> Signed-off-by: Renato Lui Geh <renatogeh@xxxxxxxxx>
Hi,
Mostly good, but alignment doesn't override other considerations such
as consistency in argument naming.
Thanks,
Jonathan
> ---
> drivers/staging/iio/addac/adt7316.c | 462 ++++++++++++++--------------
> drivers/staging/iio/addac/adt7316.h | 2 +-
> 2 files changed, 233 insertions(+), 231 deletions(-)
>
...
> @@ -1363,9 +1365,9 @@ static ssize_t adt7316_show_in_analog_temp_offset(struct device *dev,
> }
>
> static ssize_t adt7316_store_in_analog_temp_offset(struct device *dev,
> - struct device_attribute *attr,
> - const char *buf,
> - size_t len)
> + struct device_attribute *att,
I wasn't expecting to see a rename in here. I know it's not a big thing
as the parameter isn't even used, but it is better to have consistency
of naming than a minor improvement in alignment.
> + const char *buf,
> + size_t len)
> {
>
> static ssize_t adt7316_store_ex_analog_temp_offset(struct device *dev,
> - struct device_attribute *attr,
> - const char *buf,
> - size_t len)
> + struct device_attribute *att,
Same here...
> + const char *buf,
> + size_t len)
> {
> struct iio_dev *dev_info = dev_to_iio_dev(dev);
> struct adt7316_chip_info *chip = iio_priv(dev_info);
...
