Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
Date:   Mon, 28 Mar 2022 18:06:23 +0100
From:   Jonathan Cameron <jic23@kernel.org>
To:     Paul Lemmermann <thepaulodoom@thepaulodoom.com>
Cc:     linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] IIO: accel: fixed coding style issues
Message-ID: <20220328180623.1380d23e@jic23-huawei>
In-Reply-To: <YkHOpCT2Gad1YaxN@hp-amd-paul>
References: <YkHOpCT2Gad1YaxN@hp-amd-paul>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Mon, 28 Mar 2022 10:05:08 -0500
Paul Lemmermann <thepaulodoom@thepaulodoom.com> wrote:

> Fixed case statement issues and spacing issues.
> 
> Signed-off-by: Paul Lemmermann <thepaulodoom@thepaulodoom.com>
Hi Paul,

Thanks for the patch.

These are drivers written / maintained by different authors, so one
patch per driver preferred.  Particularly handy as people might not
agree with all of them so separate patches would allow me to pick
and choose which ones to pick up.

Comments inline.

Please state how you identified the changes btw.  Script?

Jonathan

> ---
>  drivers/iio/accel/bmc150-accel-core.c | 15 +++++++++++----
>  drivers/iio/accel/dmard09.c           |  2 +-
>  drivers/iio/accel/kxsd9-spi.c         |  4 ++--
>  3 files changed, 14 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/iio/accel/bmc150-accel-core.c b/drivers/iio/accel/bmc150-accel-core.c
> index d11f66801..f7dd7ec2c 100644
> --- a/drivers/iio/accel/bmc150-accel-core.c
> +++ b/drivers/iio/accel/bmc150-accel-core.c
> @@ -432,10 +432,17 @@ static bool bmc150_apply_bosc0200_acpi_orientation(struct device *dev,
>  
>  		for (j = 0; j < 3; j++) {
>  			switch (val[j]) {
> -			case -1: str = "-1"; break;
> -			case 0:  str = "0";  break;
> -			case 1:  str = "1";  break;
> -			default: goto unknown_format;
> +			case -1:
> +				str = "-1";
> +				break;
> +			case 0:
> +				str = "0";
> +				break;
> +			case 1:
> +				str = "1";
> +				break;
> +			default:
> +				goto unknown_format;

I'm not seeing any clear advantage to this change.


>  			}
>  			orientation->rotation[i * 3 + j] = str;
>  		}
> diff --git a/drivers/iio/accel/dmard09.c b/drivers/iio/accel/dmard09.c
> index e6e28c964..87bc38d4d 100644
> --- a/drivers/iio/accel/dmard09.c
> +++ b/drivers/iio/accel/dmard09.c
> @@ -24,7 +24,7 @@
>  #define DMARD09_AXIS_Y 1
>  #define DMARD09_AXIS_Z 2
>  #define DMARD09_AXIS_X_OFFSET ((DMARD09_AXIS_X + 1) * 2)
> -#define DMARD09_AXIS_Y_OFFSET ((DMARD09_AXIS_Y + 1 )* 2)
> +#define DMARD09_AXIS_Y_OFFSET ((DMARD09_AXIS_Y + 1) * 2)

This is one is good.

>  #define DMARD09_AXIS_Z_OFFSET ((DMARD09_AXIS_Z + 1) * 2)
>  
>  struct dmard09_data {
> diff --git a/drivers/iio/accel/kxsd9-spi.c b/drivers/iio/accel/kxsd9-spi.c
> index 57c451cfb..989f53fb0 100644
> --- a/drivers/iio/accel/kxsd9-spi.c
> +++ b/drivers/iio/accel/kxsd9-spi.c
> @@ -44,8 +44,8 @@ static const struct spi_device_id kxsd9_spi_id[] = {
>  MODULE_DEVICE_TABLE(spi, kxsd9_spi_id);
>  
>  static const struct of_device_id kxsd9_of_match[] = {
> -        { .compatible = "kionix,kxsd9" },
> -        { },
> +	{ .compatible = "kionix,kxsd9" },

Also good to clean up.

Thanks,

Jonathan


> +	{ },
>  };
>  MODULE_DEVICE_TABLE(of, kxsd9_of_match);
>