Re: [PATCH 2/3] lp8788-charger: fix ADC channel names

[Jonathan Cameron <jic23@kernel.org>]

On 10/18/2012 07:11 AM, Kim, Milo wrote:
>  LP8788 IIO ADC driver and platform data have specific naming convention
>  for ADC channels. That is using prefix 'lp8788_'.
>  To keep this rule, ADC channel names are changed.
> 
It's a little unusual to name the consumer side of the map so specifically.
If you want to do so it's up to you though (and it obviously has to match)
that provided in platform_data including the default map in the iio driver.
> Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
> ---
>  drivers/power/lp8788-charger.c |   14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/power/lp8788-charger.c b/drivers/power/lp8788-charger.c
> index 614e6aa..02fc9ab 100644
> --- a/drivers/power/lp8788-charger.c
> +++ b/drivers/power/lp8788-charger.c
> @@ -600,13 +600,13 @@ static void lp8788_setup_adc_channel(const char *consumer_name,
>  	struct iio_channel *chan;
>  	enum lp8788_adc_id id;
>  	const char *chan_name[LPADC_MAX] = {
> -		[LPADC_VBATT_5P5] = "vbatt-5p5",
> -		[LPADC_VBATT_6P0] = "vbatt-6p0",
> -		[LPADC_VBATT_5P0] = "vbatt-5p0",
> -		[LPADC_ADC1]      = "adc1",
> -		[LPADC_ADC2]      = "adc2",
> -		[LPADC_ADC3]      = "adc3",
> -		[LPADC_ADC4]      = "adc4",
> +		[LPADC_VBATT_5P5] = "lp8788_vbatt_5p5",
> +		[LPADC_VBATT_6P0] = "lp8788_vbatt_6p0",
> +		[LPADC_VBATT_5P0] = "lp8788_vbatt_5p0",
> +		[LPADC_ADC1]      = "lp8788_adc1",
> +		[LPADC_ADC2]      = "lp8788_adc2",
> +		[LPADC_ADC3]      = "lp8788_adc3",
> +		[LPADC_ADC4]      = "lp8788_adc4",
>  	};
>  
>  	if (!pdata)
>