On Thu, 22 Jun 2017 19:57:42 +0200
Lorenzo Bianconi <lorenzo.bianconi83@xxxxxxxxx> wrote:
> Add device tree support for LPS22HB pressure sensor
>
> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@xxxxxx>
> ---
> drivers/iio/pressure/st_pressure_spi.c | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/drivers/iio/pressure/st_pressure_spi.c b/drivers/iio/pressure/st_pressure_spi.c
> index 550508025af1..6dde7cc3e999 100644
> --- a/drivers/iio/pressure/st_pressure_spi.c
> +++ b/drivers/iio/pressure/st_pressure_spi.c
> @@ -18,6 +18,19 @@
> #include <linux/iio/common/st_sensors_spi.h>
> #include "st_pressure.h"
>
> +#ifdef CONFIG_OF
> +static const struct of_device_id st_press_of_match[] = {
> + {
> + .compatible = "st,lps22hb-press",
Rob requested the other day that we don't do this sort
of naming. That particular discussion was about a dual
chip package. Here as far as I can tell the lsp22hb is
only ever a simple pressure sensor. Hence can we drop the
-press?
If we need to have it for backwards compatibility then
perhaps both is the way to go with the press version clearly
marked as deprecated.
> + .data = LPS22HB_PRESS_DEV_NAME,
> + },
> + {},
> +};
> +MODULE_DEVICE_TABLE(of, st_press_of_match);
> +#else
> +#define st_press_of_match NULL
> +#endif
> +
> static int st_press_spi_probe(struct spi_device *spi)
> {
> struct iio_dev *indio_dev;
> @@ -30,6 +43,8 @@ static int st_press_spi_probe(struct spi_device *spi)
>
> press_data = iio_priv(indio_dev);
>
> + st_sensors_of_name_probe(&spi->dev, st_press_of_match,
> + spi->modalias, sizeof(spi->modalias));
> st_sensors_spi_configure(indio_dev, spi, press_data);
>
> err = st_press_common_probe(indio_dev);
> @@ -58,6 +73,7 @@ MODULE_DEVICE_TABLE(spi, st_press_id_table);
> static struct spi_driver st_press_driver = {
> .driver = {
> .name = "st-press-spi",
> + .of_match_table = of_match_ptr(st_press_of_match),
> },
> .probe = st_press_spi_probe,
> .remove = st_press_spi_remove,
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
