linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] power: tps65217_charger: Add properties like voltage and current charge
@ 2017-04-21 15:50 Enric Balletbo i Serra
  2017-04-28 16:49 ` Rob Herring
  2017-05-01 15:41 ` Sebastian Reichel
  0 siblings, 2 replies; 3+ messages in thread
From: Enric Balletbo i Serra @ 2017-04-21 15:50 UTC (permalink / raw)
  To: Sebastian Reichel, Rob Herring, Mark Rutland
  Cc: linux-pm, devicetree, linux-kernel

Allow the possibility to configure the charge and the current voltage of
the charger and also the NTC type for battery temperature measurement.

Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
---
Changes since v1:
 - Requested by Rob Herring
   - Rename ti,charge-* to charge-* to be standard properties.
   - Use unit suffixes as per bindings/property-units.txt
---
 .../bindings/power/supply/tps65217_charger.txt     |  15 ++
 drivers/power/supply/tps65217_charger.c            | 187 +++++++++++++++++++--
 include/linux/mfd/tps65217.h                       |   2 +
 3 files changed, 192 insertions(+), 12 deletions(-)

diff --git a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
index a11072c..4415618 100644
--- a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
+++ b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
@@ -6,6 +6,18 @@ Required Properties:
              Should be <0> for the USB charger and <1> for the AC adapter.
 -interrupt-names: Should be "USB" and "AC"
 
+Optional properties:
+-charge-voltage-microvolt: set the charge voltage. The value can be: 4100000,
+	4150000, 4200000, 4250000; default: 4100000
+
+-charge-current-microamp: set the charging current. The value can be: 300000,
+	400000, 500000, 700000; default: 500000
+
+-ti,ntc-type: set the NTC type for battery temperature measurement. The value
+	must be 0 or 1, where:
+	  0 – 100k (curve 1, B = 3960)
+	  1 – 10k  (curve 2, B = 3480) (default)
+
 This node is a subnode of the tps65217 PMIC.
 
 Example:
@@ -14,4 +26,7 @@ Example:
 		compatible = "ti,tps65217-charger";
 		interrupts = <0>, <1>;
 		interrupt-names = "USB", "AC";
+		charge-voltage-microvolt = <4100000>;
+		charge-current-microamp = <500000>;
+		ti,ntc-type = <1>;
 	};
diff --git a/drivers/power/supply/tps65217_charger.c b/drivers/power/supply/tps65217_charger.c
index 1f52340..087f29c 100644
--- a/drivers/power/supply/tps65217_charger.c
+++ b/drivers/power/supply/tps65217_charger.c
@@ -39,6 +39,12 @@
 #define NUM_CHARGER_IRQS	2
 #define POLL_INTERVAL		(HZ * 2)
 
+struct tps65217_charger_platform_data {
+	u32	charge_current_uamp;
+	u32	charge_voltage_uvolt;
+	int	ntc_type;
+};
+
 struct tps65217_charger {
 	struct tps65217 *tps;
 	struct device *dev;
@@ -48,16 +54,82 @@ struct tps65217_charger {
 	int	prev_online;
 
 	struct task_struct	*poll_task;
+	struct tps65217_charger_platform_data *pdata;
 };
 
 static enum power_supply_property tps65217_charger_props[] = {
 	POWER_SUPPLY_PROP_ONLINE,
 };
 
-static int tps65217_config_charger(struct tps65217_charger *charger)
+static int tps65217_set_charge_current(struct tps65217_charger *charger,
+				       unsigned int uamp)
+{
+	int ret, val;
+
+	dev_dbg(charger->dev, "setting charge current to %d uA\n", uamp);
+
+	if (uamp == 300000)
+		val = 0x00;
+	else if (uamp == 400000)
+		val = 0x01;
+	else if (uamp == 500000)
+		val = 0x02;
+	else if (uamp == 700000)
+		val = 0x03;
+	else
+		return -EINVAL;
+
+	ret = tps65217_set_bits(charger->tps, TPS65217_REG_CHGCONFIG3,
+				TPS65217_CHGCONFIG3_ICHRG_MASK,
+				val << TPS65217_CHGCONFIG3_ICHRG_SHIFT,
+				TPS65217_PROTECT_NONE);
+	if (ret) {
+		dev_err(charger->dev,
+			"failed to set ICHRG setting to 0x%02x (err: %d)\n",
+			val, ret);
+		return ret;
+	}
+
+	return 0;
+}
+
+static int tps65217_set_charge_voltage(struct tps65217_charger *charger,
+				       unsigned int uvolt)
+{
+	int ret, val;
+
+	dev_dbg(charger->dev, "setting charge voltage to %d uV\n", uvolt);
+
+	if (uvolt != 4100000 && uvolt != 4150000 &&
+	    uvolt != 4200000 && uvolt != 4250000)
+		return -EINVAL;
+
+	val = (uvolt - 4100000) / 50000;
+
+	ret = tps65217_set_bits(charger->tps, TPS65217_REG_CHGCONFIG2,
+				TPS65217_CHGCONFIG2_VOREG_MASK,
+				val << TPS65217_CHGCONFIG2_VOREG_SHIFT,
+				TPS65217_PROTECT_NONE);
+	if (ret) {
+		dev_err(charger->dev,
+			"failed to set VOCHG setting to 0x%02x (err: %d)\n",
+			val, ret);
+		return ret;
+	}
+
+	return 0;
+}
+
+static int tps65217_set_ntc_type(struct tps65217_charger *charger,
+				 unsigned int ntc)
 {
 	int ret;
 
+	dev_dbg(charger->dev, "setting NTC type to %d\n", ntc);
+
+	if (ntc != 0 && ntc != 1)
+		return -EINVAL;
+
 	/*
 	 * tps65217 rev. G, p. 31 (see p. 32 for NTC schematic)
 	 *
@@ -74,14 +146,57 @@ static int tps65217_config_charger(struct tps65217_charger *charger)
 	 * NTC TYPE (for battery temperature measurement)
 	 *   0 – 100k (curve 1, B = 3960)
 	 *   1 – 10k  (curve 2, B = 3480) (default on reset)
-	 *
 	 */
-	ret = tps65217_clear_bits(charger->tps, TPS65217_REG_CHGCONFIG1,
-				  TPS65217_CHGCONFIG1_NTC_TYPE,
-				  TPS65217_PROTECT_NONE);
+	if (ntc) {
+		ret = tps65217_set_bits(charger->tps, TPS65217_REG_CHGCONFIG1,
+					TPS65217_CHGCONFIG1_NTC_TYPE,
+					TPS65217_CHGCONFIG1_NTC_TYPE,
+					TPS65217_PROTECT_NONE);
+		if (ret) {
+			dev_err(charger->dev,
+				"failed to set NTC type to 10K: %d\n", ret);
+			return ret;
+		}
+	} else {
+		ret = tps65217_clear_bits(charger->tps, TPS65217_REG_CHGCONFIG1,
+					  TPS65217_CHGCONFIG1_NTC_TYPE,
+					  TPS65217_PROTECT_NONE);
+		if (ret) {
+			dev_err(charger->dev,
+				"failed to set NTC type to 100K: %d\n", ret);
+			return ret;
+		}
+	}
+
+	return 0;
+}
+
+static int tps65217_config_charger(struct tps65217_charger *charger)
+{
+	int ret;
+	struct tps65217_charger_platform_data *pdata = charger->pdata;
+
+	if (!charger->pdata)
+		return -EINVAL;
+
+	ret = tps65217_set_charge_voltage(charger, pdata->charge_voltage_uvolt);
 	if (ret) {
 		dev_err(charger->dev,
-			"failed to set 100k NTC setting: %d\n", ret);
+			"failed to set charge voltage setting: %d\n", ret);
+		return ret;
+	}
+
+	ret = tps65217_set_charge_current(charger, pdata->charge_current_uamp);
+	if (ret) {
+		dev_err(charger->dev,
+			"failed to set charge current setting: %d\n", ret);
+		return ret;
+	}
+
+	ret = tps65217_set_ntc_type(charger, pdata->ntc_type);
+	if (ret) {
+		dev_err(charger->dev,
+			"failed to set NTC type setting: %d\n", ret);
 		return ret;
 	}
 
@@ -185,6 +300,48 @@ static int tps65217_charger_poll_task(void *data)
 	return 0;
 }
 
+#ifdef CONFIG_OF
+static struct tps65217_charger_platform_data *tps65217_charger_pdata_init(
+		struct platform_device *pdev)
+{
+	struct tps65217_charger_platform_data *pdata;
+	struct device_node *np = pdev->dev.of_node;
+	int ret;
+
+	if (!np) {
+		dev_err(&pdev->dev, "No charger OF node\n");
+		return ERR_PTR(-EINVAL);
+	}
+
+	pdata = devm_kzalloc(&pdev->dev, sizeof(*pdata), GFP_KERNEL);
+	if (!pdata)
+		return ERR_PTR(-ENOMEM);
+
+	ret = of_property_read_u32(np, "charge-voltage-microvolt",
+				   &pdata->charge_voltage_uvolt);
+	if (ret)
+		pdata->charge_voltage_uvolt = 4100000;
+
+	ret = of_property_read_u32(np, "charge-current-microamp",
+				   &pdata->charge_current_uamp);
+	if (ret)
+		pdata->charge_current_uamp = 500000;
+
+	ret = of_property_read_u32(np, "ti,ntc-type",
+				   &pdata->ntc_type);
+	if (ret)
+		pdata->ntc_type = 1;	/* 10k  (curve 2, B = 3480) */
+
+	return pdata;
+}
+#else /* CONFIG_OF */
+static struct tps65217_charger_platform_data *tps65217_charger_pdata_init(
+		struct platform_device *pdev)
+{
+	return NULL;
+}
+#endif /* CONFIG_OF */
+
 static const struct power_supply_desc tps65217_charger_desc = {
 	.name			= "tps65217-charger",
 	.type			= POWER_SUPPLY_TYPE_MAINS,
@@ -214,6 +371,18 @@ static int tps65217_charger_probe(struct platform_device *pdev)
 	cfg.of_node = pdev->dev.of_node;
 	cfg.drv_data = charger;
 
+	charger->pdata = tps65217_charger_pdata_init(pdev);
+	if (IS_ERR(charger->pdata)) {
+		dev_err(charger->dev, "failed: getting platform data\n");
+		return PTR_ERR(charger->pdata);
+	}
+
+	ret = tps65217_config_charger(charger);
+	if (ret < 0) {
+		dev_err(charger->dev, "charger config failed, err %d\n", ret);
+		return ret;
+	}
+
 	charger->psy = devm_power_supply_register(&pdev->dev,
 						  &tps65217_charger_desc,
 						  &cfg);
@@ -225,12 +394,6 @@ static int tps65217_charger_probe(struct platform_device *pdev)
 	irq[0] = platform_get_irq_byname(pdev, "USB");
 	irq[1] = platform_get_irq_byname(pdev, "AC");
 
-	ret = tps65217_config_charger(charger);
-	if (ret < 0) {
-		dev_err(charger->dev, "charger config failed, err %d\n", ret);
-		return ret;
-	}
-
 	/* Create a polling thread if an interrupt is invalid */
 	if (irq[0] < 0 || irq[1] < 0) {
 		poll_task = kthread_run(tps65217_charger_poll_task,
diff --git a/include/linux/mfd/tps65217.h b/include/linux/mfd/tps65217.h
index eac2857..d040062 100644
--- a/include/linux/mfd/tps65217.h
+++ b/include/linux/mfd/tps65217.h
@@ -103,8 +103,10 @@
 #define TPS65217_CHGCONFIG2_DYNTMR	BIT(7)
 #define TPS65217_CHGCONFIG2_VPREGHG	BIT(6)
 #define TPS65217_CHGCONFIG2_VOREG_MASK	0x30
+#define TPS65217_CHGCONFIG2_VOREG_SHIFT	4
 
 #define TPS65217_CHGCONFIG3_ICHRG_MASK	0xC0
+#define TPS65217_CHGCONFIG3_ICHRG_SHIFT	6
 #define TPS65217_CHGCONFIG3_DPPMTH_MASK	0x30
 #define TPS65217_CHGCONFIG2_PCHRGT	BIT(3)
 #define TPS65217_CHGCONFIG2_TERMIF	0x06
-- 
2.9.3

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] power: tps65217_charger: Add properties like voltage and current charge
  2017-04-21 15:50 [PATCH v2] power: tps65217_charger: Add properties like voltage and current charge Enric Balletbo i Serra
@ 2017-04-28 16:49 ` Rob Herring
  2017-05-01 15:41 ` Sebastian Reichel
  1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring @ 2017-04-28 16:49 UTC (permalink / raw)
  To: Enric Balletbo i Serra
  Cc: Sebastian Reichel, Mark Rutland, linux-pm, devicetree, linux-kernel

On Fri, Apr 21, 2017 at 05:50:32PM +0200, Enric Balletbo i Serra wrote:
> Allow the possibility to configure the charge and the current voltage of
> the charger and also the NTC type for battery temperature measurement.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> ---
> Changes since v1:
>  - Requested by Rob Herring
>    - Rename ti,charge-* to charge-* to be standard properties.
>    - Use unit suffixes as per bindings/property-units.txt
> ---
>  .../bindings/power/supply/tps65217_charger.txt     |  15 ++

Acked-by: Rob Herring <robh@kernel.org>

>  drivers/power/supply/tps65217_charger.c            | 187 +++++++++++++++++++--
>  include/linux/mfd/tps65217.h                       |   2 +
>  3 files changed, 192 insertions(+), 12 deletions(-)

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] power: tps65217_charger: Add properties like voltage and current charge
  2017-04-21 15:50 [PATCH v2] power: tps65217_charger: Add properties like voltage and current charge Enric Balletbo i Serra
  2017-04-28 16:49 ` Rob Herring
@ 2017-05-01 15:41 ` Sebastian Reichel
  1 sibling, 0 replies; 3+ messages in thread
From: Sebastian Reichel @ 2017-05-01 15:41 UTC (permalink / raw)
  To: Enric Balletbo i Serra
  Cc: Rob Herring, Mark Rutland, linux-pm, devicetree, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 10834 bytes --]

Hi Enric,

On Fri, Apr 21, 2017 at 05:50:32PM +0200, Enric Balletbo i Serra wrote:
> Allow the possibility to configure the charge and the current voltage of
> the charger and also the NTC type for battery temperature measurement.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> ---
> Changes since v1:
>  - Requested by Rob Herring
>    - Rename ti,charge-* to charge-* to be standard properties.
>    - Use unit suffixes as per bindings/property-units.txt
> ---
>  .../bindings/power/supply/tps65217_charger.txt     |  15 ++
>  drivers/power/supply/tps65217_charger.c            | 187 +++++++++++++++++++--
>  include/linux/mfd/tps65217.h                       |   2 +
>  3 files changed, 192 insertions(+), 12 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
> index a11072c..4415618 100644
> --- a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
> +++ b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
> @@ -6,6 +6,18 @@ Required Properties:
>               Should be <0> for the USB charger and <1> for the AC adapter.
>  -interrupt-names: Should be "USB" and "AC"
>  
> +Optional properties:
> +-charge-voltage-microvolt: set the charge voltage. The value can be: 4100000,
> +	4150000, 4200000, 4250000; default: 4100000
> +
> +-charge-current-microamp: set the charging current. The value can be: 300000,
> +	400000, 500000, 700000; default: 500000

These two are battery specific. Please use newly introduced
"simple-battery" instead. See 

https://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-power-supply.git/commit/?h=for-next&id=ecc931a585b3b2567da772233215f59b56fb42f9
https://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-power-supply.git/commit/?h=for-next&id=fb38342a5ae6e4f438256e98e4c29a7916195125

> +-ti,ntc-type: set the NTC type for battery temperature measurement. The value
> +	must be 0 or 1, where:
> +	  0 – 100k (curve 1, B = 3960)
> +	  1 – 10k  (curve 2, B = 3480) (default)
> +

This looks fine. Might make sense to move this into its own patch.

>  This node is a subnode of the tps65217 PMIC.
>  
>  Example:
> @@ -14,4 +26,7 @@ Example:
>  		compatible = "ti,tps65217-charger";
>  		interrupts = <0>, <1>;
>  		interrupt-names = "USB", "AC";
> +		charge-voltage-microvolt = <4100000>;
> +		charge-current-microamp = <500000>;
> +		ti,ntc-type = <1>;
>  	};
> diff --git a/drivers/power/supply/tps65217_charger.c b/drivers/power/supply/tps65217_charger.c
> index 1f52340..087f29c 100644
> --- a/drivers/power/supply/tps65217_charger.c
> +++ b/drivers/power/supply/tps65217_charger.c
> @@ -39,6 +39,12 @@
>  #define NUM_CHARGER_IRQS	2
>  #define POLL_INTERVAL		(HZ * 2)
>  
> +struct tps65217_charger_platform_data {
> +	u32	charge_current_uamp;
> +	u32	charge_voltage_uvolt;
> +	int	ntc_type;
> +};
> +
>  struct tps65217_charger {
>  	struct tps65217 *tps;
>  	struct device *dev;
> @@ -48,16 +54,82 @@ struct tps65217_charger {
>  	int	prev_online;
>  
>  	struct task_struct	*poll_task;
> +	struct tps65217_charger_platform_data *pdata;
>  };
>  
>  static enum power_supply_property tps65217_charger_props[] = {
>  	POWER_SUPPLY_PROP_ONLINE,
>  };
>  
> -static int tps65217_config_charger(struct tps65217_charger *charger)
> +static int tps65217_set_charge_current(struct tps65217_charger *charger,
> +				       unsigned int uamp)
> +{
> +	int ret, val;
> +
> +	dev_dbg(charger->dev, "setting charge current to %d uA\n", uamp);
> +
> +	if (uamp == 300000)
> +		val = 0x00;
> +	else if (uamp == 400000)
> +		val = 0x01;
> +	else if (uamp == 500000)
> +		val = 0x02;
> +	else if (uamp == 700000)
> +		val = 0x03;
> +	else
> +		return -EINVAL;
> +
> +	ret = tps65217_set_bits(charger->tps, TPS65217_REG_CHGCONFIG3,
> +				TPS65217_CHGCONFIG3_ICHRG_MASK,
> +				val << TPS65217_CHGCONFIG3_ICHRG_SHIFT,
> +				TPS65217_PROTECT_NONE);
> +	if (ret) {
> +		dev_err(charger->dev,
> +			"failed to set ICHRG setting to 0x%02x (err: %d)\n",
> +			val, ret);
> +		return ret;
> +	}
> +
> +	return 0;
> +}
> +
> +static int tps65217_set_charge_voltage(struct tps65217_charger *charger,
> +				       unsigned int uvolt)
> +{
> +	int ret, val;
> +
> +	dev_dbg(charger->dev, "setting charge voltage to %d uV\n", uvolt);
> +
> +	if (uvolt != 4100000 && uvolt != 4150000 &&
> +	    uvolt != 4200000 && uvolt != 4250000)
> +		return -EINVAL;
> +
> +	val = (uvolt - 4100000) / 50000;
> +
> +	ret = tps65217_set_bits(charger->tps, TPS65217_REG_CHGCONFIG2,
> +				TPS65217_CHGCONFIG2_VOREG_MASK,
> +				val << TPS65217_CHGCONFIG2_VOREG_SHIFT,
> +				TPS65217_PROTECT_NONE);
> +	if (ret) {
> +		dev_err(charger->dev,
> +			"failed to set VOCHG setting to 0x%02x (err: %d)\n",
> +			val, ret);
> +		return ret;
> +	}
> +
> +	return 0;
> +}
> +
> +static int tps65217_set_ntc_type(struct tps65217_charger *charger,
> +				 unsigned int ntc)
>  {
>  	int ret;
>  
> +	dev_dbg(charger->dev, "setting NTC type to %d\n", ntc);
> +
> +	if (ntc != 0 && ntc != 1)
> +		return -EINVAL;
> +
>  	/*
>  	 * tps65217 rev. G, p. 31 (see p. 32 for NTC schematic)
>  	 *
> @@ -74,14 +146,57 @@ static int tps65217_config_charger(struct tps65217_charger *charger)
>  	 * NTC TYPE (for battery temperature measurement)
>  	 *   0 – 100k (curve 1, B = 3960)
>  	 *   1 – 10k  (curve 2, B = 3480) (default on reset)
> -	 *
>  	 */
> -	ret = tps65217_clear_bits(charger->tps, TPS65217_REG_CHGCONFIG1,
> -				  TPS65217_CHGCONFIG1_NTC_TYPE,
> -				  TPS65217_PROTECT_NONE);
> +	if (ntc) {
> +		ret = tps65217_set_bits(charger->tps, TPS65217_REG_CHGCONFIG1,
> +					TPS65217_CHGCONFIG1_NTC_TYPE,
> +					TPS65217_CHGCONFIG1_NTC_TYPE,
> +					TPS65217_PROTECT_NONE);
> +		if (ret) {
> +			dev_err(charger->dev,
> +				"failed to set NTC type to 10K: %d\n", ret);
> +			return ret;
> +		}
> +	} else {
> +		ret = tps65217_clear_bits(charger->tps, TPS65217_REG_CHGCONFIG1,
> +					  TPS65217_CHGCONFIG1_NTC_TYPE,
> +					  TPS65217_PROTECT_NONE);
> +		if (ret) {
> +			dev_err(charger->dev,
> +				"failed to set NTC type to 100K: %d\n", ret);
> +			return ret;
> +		}
> +	}
> +
> +	return 0;
> +}
> +
> +static int tps65217_config_charger(struct tps65217_charger *charger)
> +{
> +	int ret;
> +	struct tps65217_charger_platform_data *pdata = charger->pdata;
> +
> +	if (!charger->pdata)
> +		return -EINVAL;
> +
> +	ret = tps65217_set_charge_voltage(charger, pdata->charge_voltage_uvolt);
>  	if (ret) {
>  		dev_err(charger->dev,
> -			"failed to set 100k NTC setting: %d\n", ret);
> +			"failed to set charge voltage setting: %d\n", ret);
> +		return ret;
> +	}
> +
> +	ret = tps65217_set_charge_current(charger, pdata->charge_current_uamp);
> +	if (ret) {
> +		dev_err(charger->dev,
> +			"failed to set charge current setting: %d\n", ret);
> +		return ret;
> +	}
> +
> +	ret = tps65217_set_ntc_type(charger, pdata->ntc_type);
> +	if (ret) {
> +		dev_err(charger->dev,
> +			"failed to set NTC type setting: %d\n", ret);
>  		return ret;
>  	}
>  
> @@ -185,6 +300,48 @@ static int tps65217_charger_poll_task(void *data)
>  	return 0;
>  }
>  
> +#ifdef CONFIG_OF
> +static struct tps65217_charger_platform_data *tps65217_charger_pdata_init(
> +		struct platform_device *pdev)
> +{
> +	struct tps65217_charger_platform_data *pdata;
> +	struct device_node *np = pdev->dev.of_node;
> +	int ret;
> +
> +	if (!np) {
> +		dev_err(&pdev->dev, "No charger OF node\n");
> +		return ERR_PTR(-EINVAL);
> +	}
> +
> +	pdata = devm_kzalloc(&pdev->dev, sizeof(*pdata), GFP_KERNEL);
> +	if (!pdata)
> +		return ERR_PTR(-ENOMEM);
> +
> +	ret = of_property_read_u32(np, "charge-voltage-microvolt",
> +				   &pdata->charge_voltage_uvolt);
> +	if (ret)
> +		pdata->charge_voltage_uvolt = 4100000;
> +
> +	ret = of_property_read_u32(np, "charge-current-microamp",
> +				   &pdata->charge_current_uamp);
> +	if (ret)
> +		pdata->charge_current_uamp = 500000;

struct power_supply_battery_info info = {};
power_supply_get_battery_info(tps65217, &info);

The simple-battery framework does not yet provide charge
current/voltage, so you will need to add that first. Please
Cc Liam on that patch.

> +	ret = of_property_read_u32(np, "ti,ntc-type",
> +				   &pdata->ntc_type);
> +	if (ret)
> +		pdata->ntc_type = 1;	/* 10k  (curve 2, B = 3480) */

Please use device_property_read_u32(dev, ...) instead.

> +
> +	return pdata;
> +}
> +#else /* CONFIG_OF */
> +static struct tps65217_charger_platform_data *tps65217_charger_pdata_init(
> +		struct platform_device *pdev)
> +{
> +	return NULL;
> +}
> +#endif /* CONFIG_OF */
> +
>  static const struct power_supply_desc tps65217_charger_desc = {
>  	.name			= "tps65217-charger",
>  	.type			= POWER_SUPPLY_TYPE_MAINS,
> @@ -214,6 +371,18 @@ static int tps65217_charger_probe(struct platform_device *pdev)
>  	cfg.of_node = pdev->dev.of_node;
>  	cfg.drv_data = charger;
>  
> +	charger->pdata = tps65217_charger_pdata_init(pdev);
> +	if (IS_ERR(charger->pdata)) {
> +		dev_err(charger->dev, "failed: getting platform data\n");
> +		return PTR_ERR(charger->pdata);
> +	}
> +
> +	ret = tps65217_config_charger(charger);
> +	if (ret < 0) {
> +		dev_err(charger->dev, "charger config failed, err %d\n", ret);
> +		return ret;
> +	}
> +
>  	charger->psy = devm_power_supply_register(&pdev->dev,
>  						  &tps65217_charger_desc,
>  						  &cfg);
> @@ -225,12 +394,6 @@ static int tps65217_charger_probe(struct platform_device *pdev)
>  	irq[0] = platform_get_irq_byname(pdev, "USB");
>  	irq[1] = platform_get_irq_byname(pdev, "AC");
>  
> -	ret = tps65217_config_charger(charger);
> -	if (ret < 0) {
> -		dev_err(charger->dev, "charger config failed, err %d\n", ret);
> -		return ret;
> -	}
> -
>  	/* Create a polling thread if an interrupt is invalid */
>  	if (irq[0] < 0 || irq[1] < 0) {
>  		poll_task = kthread_run(tps65217_charger_poll_task,
> diff --git a/include/linux/mfd/tps65217.h b/include/linux/mfd/tps65217.h
> index eac2857..d040062 100644
> --- a/include/linux/mfd/tps65217.h
> +++ b/include/linux/mfd/tps65217.h
> @@ -103,8 +103,10 @@
>  #define TPS65217_CHGCONFIG2_DYNTMR	BIT(7)
>  #define TPS65217_CHGCONFIG2_VPREGHG	BIT(6)
>  #define TPS65217_CHGCONFIG2_VOREG_MASK	0x30
> +#define TPS65217_CHGCONFIG2_VOREG_SHIFT	4
>  
>  #define TPS65217_CHGCONFIG3_ICHRG_MASK	0xC0
> +#define TPS65217_CHGCONFIG3_ICHRG_SHIFT	6
>  #define TPS65217_CHGCONFIG3_DPPMTH_MASK	0x30
>  #define TPS65217_CHGCONFIG2_PCHRGT	BIT(3)
>  #define TPS65217_CHGCONFIG2_TERMIF	0x06

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-05-01 15:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-21 15:50 [PATCH v2] power: tps65217_charger: Add properties like voltage and current charge Enric Balletbo i Serra
2017-04-28 16:49 ` Rob Herring
2017-05-01 15:41 ` Sebastian Reichel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).