Re: [PATCH 03/15] iio: adc: at91: cleanup platform_data

[Jonathan Cameron <jic23@kernel.org>]

On 05/03/14 17:35, Alexandre Belloni wrote:
> num_channels and registers are not used anymore since they are defined inside
> the driver and assigned by matching the id_table.
>
> Also, struct at91_adc_reg_desc is now only used inside the driver.
>
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Acked-by: Jonathan Cameron <jic23@kernel.org>

Hmm. Looking on in the series, I'm guessing taking the lot through IIO
once everyone is happy is probably the way to go.

I'll be wanting acks from Atmel maintainers for pretty much all of them though.

Jonathan
> ---
>   drivers/iio/adc/at91_adc.c             | 19 +++++++++++++++++++
>   include/linux/platform_data/at91_adc.h | 23 -----------------------
>   2 files changed, 19 insertions(+), 23 deletions(-)
>
> diff --git a/drivers/iio/adc/at91_adc.c b/drivers/iio/adc/at91_adc.c
> index 89777ed9abd8..1beae65aef2c 100644
> --- a/drivers/iio/adc/at91_adc.c
> +++ b/drivers/iio/adc/at91_adc.c
> @@ -46,6 +46,25 @@
>   #define TOUCH_SAMPLE_PERIOD_US		2000	/* 2ms */
>   #define TOUCH_PEN_DETECT_DEBOUNCE_US	200
>
> +/**
> + * struct at91_adc_reg_desc - Various informations relative to registers
> + * @channel_base:	Base offset for the channel data registers
> + * @drdy_mask:		Mask of the DRDY field in the relevant registers
> +			(Interruptions registers mostly)
> + * @status_register:	Offset of the Interrupt Status Register
> + * @trigger_register:	Offset of the Trigger setup register
> + * @mr_prescal_mask:	Mask of the PRESCAL field in the adc MR register
> + * @mr_startup_mask:	Mask of the STARTUP field in the adc MR register
> + */
> +struct at91_adc_reg_desc {
> +	u8	channel_base;
> +	u32	drdy_mask;
> +	u8	status_register;
> +	u8	trigger_register;
> +	u32	mr_prescal_mask;
> +	u32	mr_startup_mask;
> +};
> +
>   struct at91_adc_caps {
>   	bool	has_ts;		/* Support touch screen */
>   	bool	has_tsmr;	/* only at91sam9x5, sama5d3 have TSMR reg */
> diff --git a/include/linux/platform_data/at91_adc.h b/include/linux/platform_data/at91_adc.h
> index b3ca1e94e0c8..fcf73879dbfe 100644
> --- a/include/linux/platform_data/at91_adc.h
> +++ b/include/linux/platform_data/at91_adc.h
> @@ -8,25 +8,6 @@
>   #define _AT91_ADC_H_
>
>   /**
> - * struct at91_adc_reg_desc - Various informations relative to registers
> - * @channel_base:	Base offset for the channel data registers
> - * @drdy_mask:		Mask of the DRDY field in the relevant registers
> -			(Interruptions registers mostly)
> - * @status_register:	Offset of the Interrupt Status Register
> - * @trigger_register:	Offset of the Trigger setup register
> - * @mr_prescal_mask:	Mask of the PRESCAL field in the adc MR register
> - * @mr_startup_mask:	Mask of the STARTUP field in the adc MR register
> - */
> -struct at91_adc_reg_desc {
> -	u8	channel_base;
> -	u32	drdy_mask;
> -	u8	status_register;
> -	u8	trigger_register;
> -	u32	mr_prescal_mask;
> -	u32	mr_startup_mask;
> -};
> -
> -/**
>    * struct at91_adc_trigger - description of triggers
>    * @name:		name of the trigger advertised to the user
>    * @value:		value to set in the ADC's trigger setup register
> @@ -42,8 +23,6 @@ struct at91_adc_trigger {
>   /**
>    * struct at91_adc_data - platform data for ADC driver
>    * @channels_used:		channels in use on the board as a bitmask
> - * @num_channels:		global number of channels available on the board
> - * @registers:			Registers definition on the board
>    * @startup_time:		startup time of the ADC in microseconds
>    * @trigger_list:		Triggers available in the ADC
>    * @trigger_number:		Number of triggers available in the ADC
> @@ -52,8 +31,6 @@ struct at91_adc_trigger {
>    */
>   struct at91_adc_data {
>   	unsigned long			channels_used;
> -	u8				num_channels;
> -	struct at91_adc_reg_desc	*registers;
>   	u8				startup_time;
>   	struct at91_adc_trigger		*trigger_list;
>   	u8				trigger_number;
>