[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-iio
Subject:    Re: [PATCH v1] iio: accel: st_accel: Use st_sensors_dev_name_probe()
From:       Jonathan Cameron <jic23 () kernel ! org>
Date:       2020-01-29 20:38:48
Message-ID: 20200129203848.73dd3a9b () archlinux
[Download RAW message or body]

On Mon, 20 Jan 2020 11:25:50 +0200
Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:

> Use st_sensors_dev_name_probe() instead open coded variant.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Applied to the togreg branch of iio.git and pushed out as testing for
the autobuilders to start poking at it.

Thanks,

Jonathan

> ---
>  drivers/iio/accel/st_accel_i2c.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/drivers/iio/accel/st_accel_i2c.c b/drivers/iio/accel/st_accel_i2c.c
> index 633955d764cc..8c489312f668 100644
> --- a/drivers/iio/accel/st_accel_i2c.c
> +++ b/drivers/iio/accel/st_accel_i2c.c
> @@ -147,12 +147,9 @@ static int st_accel_i2c_probe(struct i2c_client *client)
>  	const struct st_sensor_settings *settings;
>  	struct st_sensor_data *adata;
>  	struct iio_dev *indio_dev;
> -	const char *match;
>  	int ret;
>  
> -	match = device_get_match_data(&client->dev);
> -	if (match)
> -		strlcpy(client->name, match, sizeof(client->name));
> +	st_sensors_dev_name_probe(&client->dev, client->name, sizeof(client->name));
>  
>  	settings = st_accel_get_settings(client->name);
>  	if (!settings) {

[prev in list] [next in list] [prev in thread] [next in thread]