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

From: Jonathan Cameron
Date: Fri Oct 19 2012 - 13:43:43 EST


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@xxxxxx>
> ---
> 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)
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/