Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cf5aac99 authored by Ashay Jaiswal's avatar Ashay Jaiswal
Browse files

power: smb358-charger: suspend USB path for fake battery or debug board 



Battery with ID resistance of 7.5 Kohm is considered as a fake-battery or
debug board. Force the charger to suspend when this is detected.

Change-Id: I5e28699d871872f92fffc7b04ac93d33a5bb610c
Signed-off-by: default avatarAshay Jaiswal <ashayj@codeaurora.org>
parent f68526e9

Documentation/devicetree/bindings/power/smb358-charger.txt

+7 −0
Original line number Diff line number Diff line
@@ -66,6 +66,13 @@ Optional Properties: 
- qcom,warm-bat-mv:		Maximum warm battery target voltage in milli-volts.

- qcom,cool-bat-mv:		Maximum cool battery target voltage in milli-volts.

- qcom,chg-vadc			Corresponding VADC device's phandle.

- qcom,skip-usb-suspend-for-fake-battery:	A boolean property to skip suspending USB path for fake

				battery. If this property is not present then 'qcom,batt-id-vref-uv' and

				'qcom,batt-id-rpullup-kohm' should be present.

- qcom,batt-id-vref-uv		The reference voltage on the battery-ID line

				specified in micro-volts.

- qcom,batt-id-rpullup-kohm	The pull-up resistor connected on the battery-ID

				(vref) line.



Example:

	i2c@f9967000 {

drivers/power/smb358-charger.c

+138 −50
Original line number Diff line number Diff line 
/* Copyright (c) 2014 The Linux Foundation. All rights reserved.

/* Copyright (c) 2014-2015 The Linux Foundation. All rights reserved.

 *

 * This program is free software; you can redistribute it and/or modify

 * it under the terms of the GNU General Public License version 2 and
@@ -183,6 +183,7 @@ enum { 
	THERMAL		= BIT(1),

	CURRENT		= BIT(2),

	SOC		= BIT(3),

	FAKE_BATTERY	= BIT(4),

};



struct smb358_regulator {
@@ -212,6 +213,7 @@ struct smb358_charger { 
	bool			resume_completed;

	bool			irq_waiting;

	bool			bms_controlled_charging;

	bool			skip_usb_suspend_for_fake_battery;

	struct mutex		read_write_lock;

	struct mutex		path_suspend_lock;

	struct mutex		irq_complete;
@@ -223,6 +225,7 @@ struct smb358_charger { 
	unsigned int		warm_bat_ma;

	unsigned int		cool_bat_mv;

	unsigned int		warm_bat_mv;

	unsigned int		connected_rid;



	/* debugfs related */

#if defined(CONFIG_DEBUG_FS)
@@ -642,6 +645,53 @@ static int smb358_regulator_init(struct smb358_charger *chip) 
	return rc;

}



static int __smb358_path_suspend(struct smb358_charger *chip, bool suspend)

{

	int rc;



	rc = smb358_masked_write(chip, CMD_A_REG, CMD_A_CHG_SUSP_EN_MASK,

					suspend ? CMD_A_CHG_SUSP_EN_BIT : 0);

	if (rc < 0)

		dev_err(chip->dev, "Couldn't set CMD_A reg, rc = %d\n", rc);



	return rc;

}



static int smb358_path_suspend(struct smb358_charger *chip, int reason,

								bool suspend)

{

	int rc = 0;

	int suspended;



	mutex_lock(&chip->path_suspend_lock);

	suspended = chip->usb_suspended;



	if (suspend == false)

		suspended &= ~reason;

	else

		suspended |= reason;



	if (!chip->usb_suspended && suspended) {

		rc = __smb358_path_suspend(chip, true);

		chip->usb_suspended = suspended;

		power_supply_set_online(chip->usb_psy, !chip->usb_suspended);

		power_supply_changed(chip->usb_psy);

	} else if (chip->usb_suspended && !suspended) {

		rc = __smb358_path_suspend(chip, false);

		chip->usb_suspended = suspended;

		power_supply_set_online(chip->usb_psy, !chip->usb_suspended);

		power_supply_changed(chip->usb_psy);

	}



	if (rc)

		dev_err(chip->dev, "Couldn't set/unset suspend rc = %d\n", rc);



	mutex_unlock(&chip->path_suspend_lock);



	return rc;

}





static int __smb358_charging_disable(struct smb358_charger *chip, bool disable)

{

	int rc;
@@ -687,6 +737,8 @@ skip: 
	return rc;

}



#define MAX_INV_BATT_ID		7700

#define MIN_INV_BATT_ID		7300

static int smb358_hw_init(struct smb358_charger *chip)

{

	int rc;
@@ -800,6 +852,17 @@ static int smb358_hw_init(struct smb358_charger *chip) 
	if (rc)

		dev_err(chip->dev, "Couldn't set recharge para rc=%d\n", rc);



	/* suspend USB path for fake battery */

	if (!chip->skip_usb_suspend_for_fake_battery) {

		if ((chip->connected_rid >= MIN_INV_BATT_ID) &&

				(chip->connected_rid <= MAX_INV_BATT_ID)) {

			rc = smb358_path_suspend(chip, FAKE_BATTERY, true);

			if (!rc)

				dev_info(chip->dev,

					"Suspended USB path reason FAKE_BATTERY\n");

		}

	}



	/* enable/disable charging */

	if (chip->charging_disabled) {

		rc = smb358_charging_disable(chip, USER, 1);
@@ -975,51 +1038,6 @@ smb358_get_prop_battery_voltage_now(struct smb358_charger *chip) 
	return results.physical;

}



static int __smb358_path_suspend(struct smb358_charger *chip, bool suspend)

{

	int rc;



	rc = smb358_masked_write(chip, CMD_A_REG,

			CMD_A_CHG_SUSP_EN_MASK,

				suspend ? CMD_A_CHG_SUSP_EN_BIT : 0);

	if (rc < 0)

		dev_err(chip->dev, "Couldn't set CMD_A reg, rc = %d\n", rc);

	return rc;

}



static int smb358_path_suspend(struct smb358_charger *chip, int reason,

							bool suspend)

{

	int rc = 0;

	int suspended;



	mutex_lock(&chip->path_suspend_lock);

	suspended = chip->usb_suspended;



	if (suspend == false)

		suspended &= ~reason;

	else

		suspended |= reason;



	if (!chip->usb_suspended && suspended) {

		rc = __smb358_path_suspend(chip, true);

		chip->usb_suspended = suspended;

		power_supply_set_online(chip->usb_psy, !chip->usb_suspended);

		power_supply_changed(chip->usb_psy);

	} else if (chip->usb_suspended && !suspended) {

		rc = __smb358_path_suspend(chip, false);

		chip->usb_suspended = suspended;

		power_supply_set_online(chip->usb_psy, !chip->usb_suspended);

		power_supply_changed(chip->usb_psy);

	}



	if (rc)

		dev_err(chip->dev, "Couldn't set/unset suspend rc = %d\n", rc);



	mutex_unlock(&chip->path_suspend_lock);

	return rc;

}



static int smb358_set_usb_chg_current(struct smb358_charger *chip,

		int current_ma)

{
@@ -2096,6 +2114,59 @@ static void dump_regs(struct smb358_charger *chip) 
}

#endif



static int smb_parse_batt_id(struct smb358_charger *chip)

{

	int rc = 0, rpull = 0, vref = 0;

	int64_t denom, batt_id_uv, numerator;

	struct device_node *node = chip->dev->of_node;

	struct qpnp_vadc_result result;



	rc = of_property_read_u32(node, "qcom,batt-id-vref-uv", &vref);

	if (rc < 0) {

		dev_err(chip->dev,

			"Couldn't read batt-id-vref-uv rc=%d\n", rc);

		return rc;

	}



	rc = of_property_read_u32(node, "qcom,batt-id-rpullup-kohm", &rpull);

	if (rc < 0) {

		dev_err(chip->dev,

			"Couldn't read batt-id-rpullup-kohm rc=%d\n", rc);

		return rc;

	}



	/* read battery ID */

	rc = qpnp_vadc_read(chip->vadc_dev, LR_MUX2_BAT_ID, &result);

	if (rc) {

		dev_err(chip->dev,

			"Couldn't read batt id channel=%d, rc=%d\n",

			LR_MUX2_BAT_ID, rc);

		return rc;

	}

	batt_id_uv = result.physical;



	if (batt_id_uv == 0) {

		/*vadc not correct or batt id line grounded, report 0 kohms */

		dev_warn(chip->dev, "batt_id_uv=0, batt-id grounded\n");

		return 0;

	}



	numerator = batt_id_uv * rpull * 1000;

	denom = vref  - batt_id_uv;



	/* batt id connector might be open, return 0 kohms */

	if (denom == 0)

		return 0;



	chip->connected_rid = div64_s64(numerator, denom);



	dev_dbg(chip->dev,

		"batt_id_voltage=%lld numerator=%lld denom=%lld connected_rid=%d\n",

		batt_id_uv, numerator, denom, chip->connected_rid);



	return 0;

}



static int smb_parse_dt(struct smb358_charger *chip)

{

	int rc;
@@ -2216,6 +2287,23 @@ static int smb_parse_dt(struct smb358_charger *chip) 
		}

	}



	chip->skip_usb_suspend_for_fake_battery = of_property_read_bool(node,

				"qcom,skip-usb-suspend-for-fake-battery");

	if (!chip->skip_usb_suspend_for_fake_battery) {

		if (!chip->vadc_dev) {

			dev_err(chip->dev,

				"VADC device not present with usb suspend on fake battery\n");

			return -EINVAL;

		}



		rc = smb_parse_batt_id(chip);

		if (rc) {

			dev_err(chip->dev,

				"failed to read batt-id rc=%d\n", rc);

			return rc;

		}

	}



	pr_debug("inhibit-disabled = %d, recharge-disabled = %d, recharge-mv = %d,",

		chip->inhibit_disabled, chip->recharge_disabled,

						chip->recharge_mv);