diff mbox series

[v2] power: supply: bq2415x_charger: report online status

Message ID 20240228065840.2395243-2-absicsz@gmail.com
State Accepted
Commit 9a451f1b028e116d037a93bf13eb8f8620994205
Headers show
Series [v2] power: supply: bq2415x_charger: report online status | expand

Commit Message

Sicelo A. Mhlongo Feb. 28, 2024, 6:58 a.m. UTC 
Provide the Online property. This chip does not have specific flags to
indicate the presence of an input voltage, but this is implied by all valid
charging states. Fault states also imply the presence of VBUS, therefore
set Online true for those as well, taking care to not set it when i2c
reads have failed.

Signed-off-by: Sicelo A. Mhlongo <absicsz@gmail.com>
---
 drivers/power/supply/bq2415x_charger.c | 9 +++++++++
 1 file changed, 9 insertions(+)
diff mbox series

Patch

diff --git a/drivers/power/supply/bq2415x_charger.c b/drivers/power/supply/bq2415x_charger.c
index 6a4798a62588..8f24a33f0338 100644
--- a/drivers/power/supply/bq2415x_charger.c
+++ b/drivers/power/supply/bq2415x_charger.c
@@ -991,6 +991,7 @@  static enum power_supply_property bq2415x_power_supply_props[] = {
 	/* TODO: maybe add more power supply properties */
 	POWER_SUPPLY_PROP_STATUS,
 	POWER_SUPPLY_PROP_MODEL_NAME,
+	POWER_SUPPLY_PROP_ONLINE,
 };
 
 static int bq2415x_power_supply_get_property(struct power_supply *psy,
@@ -1017,6 +1018,14 @@  static int bq2415x_power_supply_get_property(struct power_supply *psy,
 	case POWER_SUPPLY_PROP_MODEL_NAME:
 		val->strval = bq->model;
 		break;
+	case POWER_SUPPLY_PROP_ONLINE:
+		/* VBUS is present for all charging and fault states,
+		 * except the 'Ready' state. On all i2c read errors, the
+		 * Online state cannot be determined.
+		 */
+		ret = bq2415x_exec_command(bq, BQ2415X_CHARGE_STATUS);
+		val->intval = ret > 0;
+		break;
 	default:
 		return -EINVAL;
 	}