Re: [PATCH 1/6] iio: mxs-lradc: clarify supported devices

[Jonathan Cameron <jic23@kernel.org>]

On 18/07/15 13:30, Stefan Wahren wrote:
> At the beginning the driver supported only i.MX28 SoC, but now the
> whole MXS platform. So remove any confusing comments which apply
> only to i.MX28.
> 
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Applied to the togreg branch of iio.git. Initially pushed out as
testing for the autobuilders to play with it.

Thanks,

Jonathan
> ---
>  .../bindings/staging/iio/adc/mxs-lradc.txt         |    2 +-
>  drivers/staging/iio/adc/mxs-lradc.c                |   18 +++++++++---------
>  2 files changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/staging/iio/adc/mxs-lradc.txt b/Documentation/devicetree/bindings/staging/iio/adc/mxs-lradc.txt
> index 3075377..555fb11 100644
> --- a/Documentation/devicetree/bindings/staging/iio/adc/mxs-lradc.txt
> +++ b/Documentation/devicetree/bindings/staging/iio/adc/mxs-lradc.txt
> @@ -1,4 +1,4 @@
> -* Freescale i.MX28 LRADC device driver
> +* Freescale MXS LRADC device driver
>  
>  Required properties:
>  - compatible: Should be "fsl,imx23-lradc" for i.MX23 SoC and "fsl,imx28-lradc"
> diff --git a/drivers/staging/iio/adc/mxs-lradc.c b/drivers/staging/iio/adc/mxs-lradc.c
> index d7c5223..1ccb367 100644
> --- a/drivers/staging/iio/adc/mxs-lradc.c
> +++ b/drivers/staging/iio/adc/mxs-lradc.c
> @@ -1,5 +1,5 @@
>  /*
> - * Freescale i.MX28 LRADC driver
> + * Freescale MXS LRADC driver
>   *
>   * Copyright (c) 2012 DENX Software Engineering, GmbH.
>   * Marek Vasut <marex@denx.de>
> @@ -1392,7 +1392,7 @@ static const struct iio_chan_spec mxs_lradc_chan_spec[] = {
>  	MXS_ADC_CHAN(4, IIO_VOLTAGE),
>  	MXS_ADC_CHAN(5, IIO_VOLTAGE),
>  	MXS_ADC_CHAN(6, IIO_VOLTAGE),
> -	MXS_ADC_CHAN(7, IIO_VOLTAGE),	/* VBATT */
> +	MXS_ADC_CHAN(7, IIO_VOLTAGE),
>  	/* Combined Temperature sensors */
>  	{
>  		.type = IIO_TEMP,
> @@ -1411,12 +1411,12 @@ static const struct iio_chan_spec mxs_lradc_chan_spec[] = {
>  		.scan_index = -1,
>  		.channel = 9,
>  	},
> -	MXS_ADC_CHAN(10, IIO_VOLTAGE),	/* VDDIO */
> -	MXS_ADC_CHAN(11, IIO_VOLTAGE),	/* VTH */
> -	MXS_ADC_CHAN(12, IIO_VOLTAGE),	/* VDDA */
> -	MXS_ADC_CHAN(13, IIO_VOLTAGE),	/* VDDD */
> -	MXS_ADC_CHAN(14, IIO_VOLTAGE),	/* VBG */
> -	MXS_ADC_CHAN(15, IIO_VOLTAGE),	/* VDD5V */
> +	MXS_ADC_CHAN(10, IIO_VOLTAGE),
> +	MXS_ADC_CHAN(11, IIO_VOLTAGE),
> +	MXS_ADC_CHAN(12, IIO_VOLTAGE),
> +	MXS_ADC_CHAN(13, IIO_VOLTAGE),
> +	MXS_ADC_CHAN(14, IIO_VOLTAGE),
> +	MXS_ADC_CHAN(15, IIO_VOLTAGE),
>  };
>  
>  static int mxs_lradc_hw_init(struct mxs_lradc *lradc)
> @@ -1707,6 +1707,6 @@ static struct platform_driver mxs_lradc_driver = {
>  module_platform_driver(mxs_lradc_driver);
>  
>  MODULE_AUTHOR("Marek Vasut <marex@denx.de>");
> -MODULE_DESCRIPTION("Freescale i.MX28 LRADC driver");
> +MODULE_DESCRIPTION("Freescale MXS LRADC driver");
>  MODULE_LICENSE("GPL v2");
>  MODULE_ALIAS("platform:" DRIVER_NAME);
>