[v2,09/25] iio: adc: axp20x_adc: map acin_i and acin_v

Message ID 20170127085458.18270-10-quentin.schulz@free-electrons.com
State New
Headers show
Series
  • add support for AXP20X and AXP22X power supply drivers
Related show

Commit Message

Quentin Schulz Jan. 27, 2017, 8:54 a.m.
This maps the IIO channels acin_i and acin_v (respectively exposing the
current current and voltage measures of the AC power supply) to the AC
power supply driver.

Only the AXP20X PMICs have these ADC channels and thus they are only
mapped for this version of the PMIC.

Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com>

---

added in v2

 drivers/iio/adc/axp20x_adc.c | 8 ++++++++
 1 file changed, 8 insertions(+)

-- 
2.9.3


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Comments

Jonathan Cameron Jan. 28, 2017, 2:51 p.m. | #1
On 27/01/17 08:54, Quentin Schulz wrote:
> This maps the IIO channels acin_i and acin_v (respectively exposing the

> current current and voltage measures of the AC power supply) to the AC

> power supply driver.

> 

> Only the AXP20X PMICs have these ADC channels and thus they are only

> mapped for this version of the PMIC.

> 

> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com>

Acked-by: Jonathan Cameron <jic23@kernel.org>

> ---

> 

> added in v2

> 

>  drivers/iio/adc/axp20x_adc.c | 8 ++++++++

>  1 file changed, 8 insertions(+)

> 

> diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c

> index bacde92..14f4ec0 100644

> --- a/drivers/iio/adc/axp20x_adc.c

> +++ b/drivers/iio/adc/axp20x_adc.c

> @@ -104,6 +104,14 @@ static struct iio_map axp20x_maps[] = {

>  		.consumer_dev_name = "axp20x-usb-power-supply",

>  		.consumer_channel = "vbus_i",

>  		.adc_channel_label = "vbus_i",

> +	}, {

> +		.consumer_dev_name = "axp20x-ac-power-supply",

> +		.consumer_channel = "acin_v",

> +		.adc_channel_label = "acin_v",

> +	}, {

> +		.consumer_dev_name = "axp20x-ac-power-supply",

> +		.consumer_channel = "acin_i",

> +		.adc_channel_label = "acin_i",

>  	}, { /* sentinel */ }

>  };

>  

> 



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Sebastian Reichel Jan. 29, 2017, 4:31 p.m. | #2
Hi,

On Fri, Jan 27, 2017 at 09:54:42AM +0100, Quentin Schulz wrote:
> This maps the IIO channels acin_i and acin_v (respectively exposing the

> current current and voltage measures of the AC power supply) to the AC

> power supply driver.

> 

> Only the AXP20X PMICs have these ADC channels and thus they are only

> mapped for this version of the PMIC.

> 

> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com>

> ---

> 

> added in v2

> 

>  drivers/iio/adc/axp20x_adc.c | 8 ++++++++

>  1 file changed, 8 insertions(+)

> 

> diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c

> index bacde92..14f4ec0 100644

> --- a/drivers/iio/adc/axp20x_adc.c

> +++ b/drivers/iio/adc/axp20x_adc.c

> @@ -104,6 +104,14 @@ static struct iio_map axp20x_maps[] = {

>  		.consumer_dev_name = "axp20x-usb-power-supply",

>  		.consumer_channel = "vbus_i",

>  		.adc_channel_label = "vbus_i",

> +	}, {

> +		.consumer_dev_name = "axp20x-ac-power-supply",

> +		.consumer_channel = "acin_v",

> +		.adc_channel_label = "acin_v",

> +	}, {

> +		.consumer_dev_name = "axp20x-ac-power-supply",

> +		.consumer_channel = "acin_i",

> +		.adc_channel_label = "acin_i",

>  	}, { /* sentinel */ }

>  };


I suggest to merge this into Patch 4 before resending the series.

-- Sebastian
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Patch hide | download patch | download mbox

diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c
index bacde92..14f4ec0 100644
--- a/drivers/iio/adc/axp20x_adc.c
+++ b/drivers/iio/adc/axp20x_adc.c
@@ -104,6 +104,14 @@  static struct iio_map axp20x_maps[] = {
 		.consumer_dev_name = "axp20x-usb-power-supply",
 		.consumer_channel = "vbus_i",
 		.adc_channel_label = "vbus_i",
+	}, {
+		.consumer_dev_name = "axp20x-ac-power-supply",
+		.consumer_channel = "acin_v",
+		.adc_channel_label = "acin_v",
+	}, {
+		.consumer_dev_name = "axp20x-ac-power-supply",
+		.consumer_channel = "acin_i",
+		.adc_channel_label = "acin_i",
 	}, { /* sentinel */ }
 };