Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Sun, 9 Aug 2020 12:43:29 +0100
From:   Jonathan Cameron <jic23@kernel.org>
To:     Crt Mori <cmo@melexis.com>
Cc:     linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] iio:temperature:mlx90632: Some stylefixing leftovers
Message-ID: <20200809124329.5ed66c0b@archlinux>
In-Reply-To: <20200806212139.923270-1-cmo@melexis.com>
References: <20200806212139.923270-1-cmo@melexis.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu,  6 Aug 2020 23:21:39 +0200
Crt Mori <cmo@melexis.com> wrote:

> There is some inconsistency and whitespace cleanup performed in this
> patch. It was done on top of my other patches, but I can rebase to head
> of the togreg branch if it would go in sooner.
> 
> Signed-off-by: Crt Mori <cmo@melexis.com>
If not already done so, probably just add this to the other series.

If you prefer to keep it separate then remind me if I seem to have
lost this one after those patches are in place.

Thanks,

Jonathan

> ---
>  drivers/iio/temperature/mlx90632.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/iio/temperature/mlx90632.c b/drivers/iio/temperature/mlx90632.c
> index bb35a65bb9f0..d966e5387c48 100644
> --- a/drivers/iio/temperature/mlx90632.c
> +++ b/drivers/iio/temperature/mlx90632.c
> @@ -100,10 +100,10 @@
>  #define MLX90632_DSP_VERSION	5 /* DSP version */
>  #define MLX90632_DSP_MASK	GENMASK(7, 0) /* DSP version in EE_VERSION */
>  #define MLX90632_RESET_CMD	0x0006 /* Reset sensor (address or global) */
> -#define MLX90632_REF_12		12LL /**< ResCtrlRef value of Ch 1 or Ch 2 */
> -#define MLX90632_REF_3		12LL /**< ResCtrlRef value of Channel 3 */
> -#define MLX90632_MAX_MEAS_NUM	31 /**< Maximum measurements in list */
> -#define MLX90632_SLEEP_DELAY_MS 3000 /**< Autosleep delay */
> +#define MLX90632_REF_12 	12LL /* ResCtrlRef value of Ch 1 or Ch 2 */
> +#define MLX90632_REF_3		12LL /* ResCtrlRef value of Channel 3 */
> +#define MLX90632_MAX_MEAS_NUM	31 /* Maximum measurements in list */
> +#define MLX90632_SLEEP_DELAY_MS 3000 /* Autosleep delay */
>  #define MLX90632_EXTENDED_LIMIT 27000 /* Extended mode raw value limit */
>  
>  struct mlx90632_data {
> @@ -884,7 +884,7 @@ static int mlx90632_probe(struct i2c_client *client,
>  		mlx90632->mtyp = MLX90632_MTYP_EXTENDED;
>  	} else if ((read & MLX90632_DSP_MASK) == MLX90632_DSP_VERSION) {
>  		dev_dbg(&client->dev,
> -			"Detected Unknown EEPROM calibration %x\n", read);	
> +			"Detected Unknown EEPROM calibration %x\n", read);
>  	} else {
>  		dev_err(&client->dev,
>  			"Wrong DSP version %x (expected %x)\n",