Re: [PATCH v1 1/1] iio: ssp_sensors: Make use of device properties

[Jonathan Cameron <jic23@kernel.org>]

On Thu, 10 Feb 2022 15:57:24 +0200
Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:

> Convert the module to be property provider agnostic and allow
> it to be used on non-OF platforms.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Applied to the togreg branch of iio.git and pushed out as testing.

Thanks,

Jonathan

> ---
>  drivers/iio/common/ssp_sensors/ssp_dev.c | 22 +++++-----------------
>  1 file changed, 5 insertions(+), 17 deletions(-)
> 
> diff --git a/drivers/iio/common/ssp_sensors/ssp_dev.c b/drivers/iio/common/ssp_sensors/ssp_dev.c
> index eafaf4529df5..c1abd20ff808 100644
> --- a/drivers/iio/common/ssp_sensors/ssp_dev.c
> +++ b/drivers/iio/common/ssp_sensors/ssp_dev.c
> @@ -7,9 +7,10 @@
>  #include <linux/interrupt.h>
>  #include <linux/io.h>
>  #include <linux/mfd/core.h>
> +#include <linux/mod_devicetable.h>
>  #include <linux/module.h>
> -#include <linux/of.h>
> -#include <linux/of_platform.h>
> +#include <linux/property.h>
> +
>  #include "ssp.h"
>  
>  #define SSP_WDT_TIME			10000
> @@ -425,7 +426,6 @@ int ssp_queue_ssp_refresh_task(struct ssp_data *data, unsigned int delay)
>  				  msecs_to_jiffies(delay));
>  }
>  
> -#ifdef CONFIG_OF
>  static const struct of_device_id ssp_of_match[] = {
>  	{
>  		.compatible	= "samsung,sensorhub-rinato",
> @@ -441,8 +441,6 @@ MODULE_DEVICE_TABLE(of, ssp_of_match);
>  static struct ssp_data *ssp_parse_dt(struct device *dev)
>  {
>  	struct ssp_data *data;
> -	struct device_node *node = dev->of_node;
> -	const struct of_device_id *match;
>  
>  	data = devm_kzalloc(dev, sizeof(*data), GFP_KERNEL);
>  	if (!data)
> @@ -461,22 +459,12 @@ static struct ssp_data *ssp_parse_dt(struct device *dev)
>  	if (IS_ERR(data->mcu_reset_gpiod))
>  		return NULL;
>  
> -	match = of_match_node(ssp_of_match, node);
> -	if (!match)
> -		return NULL;
> -
> -	data->sensorhub_info = match->data;
> +	data->sensorhub_info = device_get_match_data(dev);
>  
>  	dev_set_drvdata(dev, data);
>  
>  	return data;
>  }
> -#else
> -static struct ssp_data *ssp_parse_dt(struct device *pdev)
> -{
> -	return NULL;
> -}
> -#endif
>  
>  /**
>   * ssp_register_consumer() - registers iio consumer in ssp framework
> @@ -670,7 +658,7 @@ static struct spi_driver ssp_driver = {
>  	.remove = ssp_remove,
>  	.driver = {
>  		.pm = &ssp_pm_ops,
> -		.of_match_table = of_match_ptr(ssp_of_match),
> +		.of_match_table = ssp_of_match,
>  		.name = "sensorhub"
>  	},
>  };