diff mbox series

[v4,2/2] power: supply: bq25790: Introduce the BQ25790 charger driver

Message ID 20201009144112.3007-2-dmurphy@ti.com
State New
Headers show
Series None | expand

Commit Message

Dan Murphy Oct. 9, 2020, 2:41 p.m. UTC
BQ25790 is a highly integrated switch-mode buck-boost charger
for 1-4 cell Li-ion battery and Li-polymer battery.

Signed-off-by: Ricardo Rivera-Matos <r-rivera-matos@ti.com>
Signed-off-by: Dan Murphy <dmurphy@ti.com>
---
 drivers/power/supply/Kconfig           |    8 +
 drivers/power/supply/Makefile          |    1 +
 drivers/power/supply/bq25790_charger.c | 1121 ++++++++++++++++++++++++
 drivers/power/supply/bq25790_charger.h |  150 ++++
 4 files changed, 1280 insertions(+)
 create mode 100644 drivers/power/supply/bq25790_charger.c
 create mode 100644 drivers/power/supply/bq25790_charger.h

Comments

Sebastian Reichel Dec. 27, 2020, 6:46 p.m. UTC | #1
Hi,

Sorry, took me a lot longer than expected...

On Fri, Oct 09, 2020 at 09:41:12AM -0500, Dan Murphy wrote:
> BQ25790 is a highly integrated switch-mode buck-boost charger
> for 1-4 cell Li-ion battery and Li-polymer battery.
> 
> Signed-off-by: Ricardo Rivera-Matos <r-rivera-matos@ti.com>
> Signed-off-by: Dan Murphy <dmurphy@ti.com>
> ---
>  drivers/power/supply/Kconfig           |    8 +
>  drivers/power/supply/Makefile          |    1 +
>  drivers/power/supply/bq25790_charger.c | 1121 ++++++++++++++++++++++++
>  drivers/power/supply/bq25790_charger.h |  150 ++++
>  4 files changed, 1280 insertions(+)
>  create mode 100644 drivers/power/supply/bq25790_charger.c
>  create mode 100644 drivers/power/supply/bq25790_charger.h
> 
> diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig
> index eec646c568b7..1cc361238d9a 100644
> --- a/drivers/power/supply/Kconfig
> +++ b/drivers/power/supply/Kconfig
> @@ -628,6 +628,14 @@ config CHARGER_BQ2515X
>  	  rail, ADC for battery and system monitoring, and push-button
>  	  controller.
>  
> +config CHARGER_BQ25790
> +	tristate "TI BQ25790 battery charger driver"
> +	depends on I2C
> +	depends on GPIOLIB || COMPILE_TEST
> +	select REGMAP_I2C
> +	help
> +	  Say Y to enable support for the TI BQ25790 battery charger.
> +
>  config CHARGER_BQ25890
>  	tristate "TI BQ25890 battery charger driver"
>  	depends on I2C
> diff --git a/drivers/power/supply/Makefile b/drivers/power/supply/Makefile
> index dd4b86318cd9..29b71cc92550 100644
> --- a/drivers/power/supply/Makefile
> +++ b/drivers/power/supply/Makefile
> @@ -83,6 +83,7 @@ obj-$(CONFIG_CHARGER_BQ24190)	+= bq24190_charger.o
>  obj-$(CONFIG_CHARGER_BQ24257)	+= bq24257_charger.o
>  obj-$(CONFIG_CHARGER_BQ24735)	+= bq24735-charger.o
>  obj-$(CONFIG_CHARGER_BQ2515X)	+= bq2515x_charger.o
> +obj-$(CONFIG_CHARGER_BQ25790)	+= bq25790_charger.o
>  obj-$(CONFIG_CHARGER_BQ25890)	+= bq25890_charger.o
>  obj-$(CONFIG_CHARGER_BQ25980)	+= bq25980_charger.o
>  obj-$(CONFIG_CHARGER_SMB347)	+= smb347-charger.o
> diff --git a/drivers/power/supply/bq25790_charger.c b/drivers/power/supply/bq25790_charger.c
> new file mode 100644
> index 000000000000..838a49c2f9f0
> --- /dev/null
> +++ b/drivers/power/supply/bq25790_charger.c
> @@ -0,0 +1,1121 @@
> +// SPDX-License-Identifier: GPL-2.0
> +// BQ25790 driver
> +// Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/
> +
> +#include <linux/module.h>
> +#include <linux/i2c.h>
> +#include <linux/power_supply.h>
> +#include <linux/regmap.h>
> +#include <linux/types.h>
> +#include <linux/gpio/consumer.h>
> +#include <linux/interrupt.h>
> +#include <linux/delay.h>
> +#include <linux/usb/phy.h>
> +
> +#include <linux/acpi.h>
> +#include <linux/of.h>
> +
> +#include "bq25790_charger.h"
> +
> +#define BQ25790_NUM_WD_VAL	8
> +
> +struct bq25790_init_data {
> +	u32 ichg;
> +	u32 ilim;
> +	u32 vreg;
> +	u32 iterm;
> +	u32 iprechg;
> +	u32 vlim;
> +	u32 ichg_max;
> +	u32 vreg_max;
> +};
> +
> +struct bq25790_state {
> +	bool online;
> +	u8 chrg_status;
> +	u8 chrg_type;
> +	u8 health;
> +	u8 chrg_fault;
> +	u8 vsys_status;
> +	u8 vbus_status;
> +	u8 fault_0;
> +	u8 fault_1;
> +	u32 vbat_adc;
> +	u32 vbus_adc;
> +	u32 ibat_adc;
> +};
> +
> +struct bq25790_device {
> +	struct i2c_client *client;
> +	struct device *dev;
> +	struct power_supply *charger;
> +	struct power_supply *battery;
> +	struct mutex lock;
> +
> +	struct usb_phy *usb2_phy;
> +	struct usb_phy *usb3_phy;
> +	struct notifier_block usb_nb;
> +	struct work_struct usb_work;
> +	unsigned long usb_event;
> +	struct regmap *regmap;
> +
> +	char model_name[I2C_NAME_SIZE];
> +	int device_id;
> +
> +	struct bq25790_init_data init_data;
> +	struct bq25790_state state;
> +	int watchdog_timer;
> +};
> +
> +static struct reg_default bq25790_reg_defs[] = {
> +	{BQ25790_INPUT_V_LIM, 0x24},
> +	{BQ25790_INPUT_I_LIM_MSB, 0x01},
> +	{BQ25790_INPUT_I_LIM_LSB, 0x2c},
> +	{BQ25790_PRECHRG_CTRL, 0xc3},
> +	{BQ25790_TERM_CTRL, 0x5},
> +	{BQ25790_VOTG_REG, 0xdc},
> +	{BQ25790_IOTG_REG, 0x4b},
> +	{BQ25790_TIMER_CTRL, 0x3d},
> +	{BQ25790_CHRG_CTRL_0, 0xa2},
> +	{BQ25790_CHRG_CTRL_1, 0x85},
> +	{BQ25790_CHRG_CTRL_2, 0x40},
> +	{BQ25790_CHRG_CTRL_3, 0x12},
> +	{BQ25790_CHRG_CTRL_5, 0x16},
> +	{BQ25790_MPPT_CTRL, 0xaa},
> +	{BQ25790_TEMP_CTRL, 0xc0},
> +	{BQ25790_NTC_CTRL_0, 0x7a},
> +	{BQ25790_NTC_CTRL_1, 0x54},
> +	{BQ25790_ICO_I_LIM, 0x0},
> +	{BQ25790_CHRG_STAT_0, 0x0},
> +	{BQ25790_CHRG_STAT_1, 0x0},
> +	{BQ25790_CHRG_STAT_2, 0x0},
> +	{BQ25790_CHRG_STAT_3, 0x0},
> +	{BQ25790_CHRG_STAT_4, 0x0},
> +	{BQ25790_FAULT_STAT_0, 0x0},
> +	{BQ25790_FAULT_STAT_1, 0x0},
> +	{BQ25790_CHRG_FLAG_0, 0x0},
> +	{BQ25790_CHRG_FLAG_1, 0x0},
> +	{BQ25790_CHRG_FLAG_2, 0x0},
> +	{BQ25790_CHRG_FLAG_3, 0x0},
> +	{BQ25790_FAULT_FLAG_0, 0x0},
> +	{BQ25790_FAULT_FLAG_1, 0x0},
> +	{BQ25790_CHRG_MSK_0, 0x0},
> +	{BQ25790_CHRG_MSK_1, 0x0},
> +	{BQ25790_CHRG_MSK_2, 0x0},
> +	{BQ25790_CHRG_MSK_3, 0x0},
> +	{BQ25790_FAULT_MSK_0, 0x0},
> +	{BQ25790_FAULT_MSK_1, 0x0},
> +	{BQ25790_ADC_CTRL, 0x30},
> +	{BQ25790_FN_DISABE_0, 0x0},
> +	{BQ25790_FN_DISABE_1, 0x0},
> +	{BQ25790_ADC_IBUS, 0x0},
> +	{BQ25790_ADC_IBAT_MSB, 0x0},
> +	{BQ25790_ADC_IBAT_LSB, 0x0},
> +	{BQ25790_ADC_VBUS_MSB, 0x0},
> +	{BQ25790_ADC_VBUS_LSB, 0x0},
> +	{BQ25790_ADC_VAC1, 0x0},
> +	{BQ25790_ADC_VAC2, 0x0},
> +	{BQ25790_ADC_VBAT_MSB, 0x0},
> +	{BQ25790_ADC_VBAT_LSB, 0x0},
> +	{BQ25790_ADC_VBUS_MSB, 0x0},
> +	{BQ25790_ADC_VBUS_LSB, 0x0},
> +	{BQ25790_ADC_TS, 0x0},
> +	{BQ25790_ADC_TDIE, 0x0},
> +	{BQ25790_ADC_DP, 0x0},
> +	{BQ25790_ADC_DM, 0x0},
> +	{BQ25790_DPDM_DRV, 0x0},
> +	{BQ25790_PART_INFO, 0x0},
> +};
> +
> +static int bq25790_watchdog_time[BQ25790_NUM_WD_VAL] = {0, 500, 1000, 2000,
> +							20000, 40000, 80000,
> +							160000};
> +
> +static enum power_supply_usb_type bq25790_usb_type[] = {
> +	POWER_SUPPLY_USB_TYPE_PD_DRP,
> +	POWER_SUPPLY_USB_TYPE_SDP,
> +	POWER_SUPPLY_USB_TYPE_CDP,
> +	POWER_SUPPLY_USB_TYPE_DCP,
> +	POWER_SUPPLY_USB_TYPE_UNKNOWN,
> +};
> +
> +static int bq25790_usb_notifier(struct notifier_block *nb, unsigned long val,
> +				void *priv)
> +{
> +	struct bq25790_device *bq =
> +			container_of(nb, struct bq25790_device, usb_nb);
> +
> +	bq->usb_event = val;
> +	queue_work(system_power_efficient_wq, &bq->usb_work);
> +
> +	return NOTIFY_OK;
> +}
> +
> +static void bq25790_usb_work(struct work_struct *data)
> +{
> +	struct bq25790_device *bq =
> +			container_of(data, struct bq25790_device, usb_work);
> +
> +	switch (bq->usb_event) {
> +	case USB_EVENT_ID:
> +		break;
> +
> +	case USB_EVENT_NONE:
> +		power_supply_changed(bq->charger);
> +		break;
> +	}
> +
> +	return;
> +}
> +
> +static int bq25790_get_vbat_adc(struct bq25790_device *bq)
> +{
> +	int ret;
> +	int vbat_adc_lsb, vbat_adc_msb;
> +	u16 vbat_adc;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_ADC_VBAT_MSB, &vbat_adc_msb);
> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_ADC_VBAT_LSB, &vbat_adc_lsb);
> +	if (ret)
> +		return ret;
> +
> +	vbat_adc = (vbat_adc_msb << 8) | vbat_adc_lsb;
> +
> +	return vbat_adc * BQ25790_ADC_VOLT_STEP_uV;
> +}
> +
> +static int bq25790_get_vbus_adc(struct bq25790_device *bq)
> +{
> +	int ret;
> +	int vbus_adc_lsb, vbus_adc_msb;
> +	u16 vbus_adc;
> +
> +	ret = regmap_update_bits(bq->regmap, BQ25790_ADC_CTRL,
> +				 BQ25790_ADC_EN, BQ25790_ADC_EN);
> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_ADC_VBUS_MSB, &vbus_adc_msb);
> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_ADC_VBUS_LSB, &vbus_adc_lsb);
> +	if (ret)
> +		return ret;
> +
> +	vbus_adc = (vbus_adc_msb << 8) | vbus_adc_lsb;
> +
> +	return vbus_adc * BQ25790_ADC_VOLT_STEP_uV;
> +}
> +
> +static int bq25790_get_ibat_adc(struct bq25790_device *bq)
> +{
> +	int ret;
> +	int ibat_adc_lsb, ibat_adc_msb;
> +	u16 ibat_adc;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_ADC_IBAT_MSB, &ibat_adc_msb);
> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_ADC_IBAT_LSB, &ibat_adc_lsb);
> +	if (ret)
> +		return ret;
> +
> +	ibat_adc = (ibat_adc_msb << 8) | ibat_adc_lsb;
> +
> +	return ibat_adc * BQ25790_ADC_CURR_STEP_uA;
> +}
> +
> +static int bq25790_get_term_curr(struct bq25790_device *bq)
> +{
> +	int ret;
> +	int reg_val;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_TERM_CTRL, &reg_val);
> +	if (ret)
> +		return ret;
> +
> +	reg_val &= BQ25790_TERMCHRG_CUR_MASK;
> +
> +	return reg_val * BQ25790_TERMCHRG_CURRENT_STEP_uA;
> +}
> +
> +static int bq25790_get_prechrg_curr(struct bq25790_device *bq)
> +{
> +	int ret;
> +	int reg_val;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_PRECHRG_CTRL, &reg_val);
> +	if (ret)
> +		return ret;
> +
> +	reg_val &= BQ25790_PRECHRG_CUR_MASK;
> +
> +	return reg_val * BQ25790_PRECHRG_CURRENT_STEP_uA;
> +}
> +
> +static int bq25790_get_ichg_curr(struct bq25790_device *bq)
> +{
> +	int ret;
> +	int ichg, ichg_lsb, ichg_msb;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_I_LIM_LSB, &ichg_lsb);
> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_I_LIM_MSB, &ichg_msb);
> +	if (ret)
> +		return ret;
> +
> +	ichg = (ichg_msb << 8) | ichg_lsb;
> +
> +	return ichg * BQ25790_ICHRG_CURRENT_STEP_uA;
> +}
> +
> +static int bq25790_set_term_curr(struct bq25790_device *bq, int term_current)
> +{
> +	int reg_val;
> +
> +	if (term_current < BQ25790_TERMCHRG_I_MIN_uA ||
> +	    term_current > BQ25790_TERMCHRG_I_MAX_uA)
> +		return -EINVAL;
> +
> +	reg_val = term_current / BQ25790_TERMCHRG_CURRENT_STEP_uA;
> +
> +	return regmap_update_bits(bq->regmap, BQ25790_TERM_CTRL,
> +				  BQ25790_TERMCHRG_CUR_MASK, reg_val);
> +}
> +
> +static int bq25790_set_prechrg_curr(struct bq25790_device *bq, int pre_current)
> +{
> +	int reg_val;
> +
> +	if (pre_current < BQ25790_PRECHRG_I_MIN_uA ||
> +	    pre_current > BQ25790_PRECHRG_I_MAX_uA)
> +		return -EINVAL;
> +
> +	reg_val = pre_current / BQ25790_PRECHRG_CURRENT_STEP_uA;
> +
> +	return regmap_update_bits(bq->regmap, BQ25790_PRECHRG_CTRL,
> +				  BQ25790_PRECHRG_CUR_MASK, reg_val);
> +}
> +
> +static int bq25790_set_ichrg_curr(struct bq25790_device *bq, int chrg_curr)
> +{
> +	int ret;
> +	int ichg, ichg_msb, ichg_lsb;
> +
> +	if (chrg_curr < BQ25790_ICHRG_I_MIN_uA ||
> +	    chrg_curr > BQ25790_ICHRG_I_MAX_uA)
> +		return -EINVAL;
> +
> +	ichg = chrg_curr / BQ25790_ICHRG_CURRENT_STEP_uA;
> +	ichg_msb = (ichg >> 8) & 0xff;
> +	ret = regmap_write(bq->regmap, BQ25790_CHRG_I_LIM_MSB, ichg_msb);
> +	if (ret)
> +		return ret;
> +
> +	ichg_lsb = ichg & 0xff;
> +
> +	return regmap_write(bq->regmap, BQ25790_CHRG_I_LIM_LSB, ichg_lsb);
> +}
> +
> +static int bq25790_set_chrg_volt(struct bq25790_device *bq, int chrg_volt)
> +{
> +	int vlim_lsb, vlim_msb, vlim;
> +	int ret;
> +
> +	if (chrg_volt < BQ25790_VREG_V_MIN_uV ||
> +	    chrg_volt > BQ25790_VREG_V_MAX_uV)
> +		return -EINVAL;
> +
> +	vlim = chrg_volt / BQ25790_VREG_V_STEP_uV;
> +	vlim_msb = (vlim >> 8) & 0xff;
> +	ret = regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_MSB, vlim_msb);
> +	if (ret)
> +		return ret;
> +
> +	vlim_lsb = vlim & 0xff;
> +
> +	return regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_LSB, vlim_lsb);
> +}
> +
> +static int bq25790_get_chrg_volt(struct bq25790_device *bq)
> +{
> +	int ret;
> +	int vlim_lsb, vlim_msb, chrg_volt;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_V_LIM_MSB, &vlim_msb);
> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_V_LIM_LSB, &vlim_lsb);
> +	if (ret)
> +		return ret;
> +
> +	chrg_volt = (vlim_msb << 8) | vlim_lsb;
> +
> +	return chrg_volt * BQ25790_VREG_V_STEP_uV;
> +}
> +
> +static int bq25790_set_input_volt_lim(struct bq25790_device *bq, int vindpm)
> +{
> +	int ret;
> +	int vlim_lsb, vlim_msb;
> +	int vlim;
> +
> +	if (vindpm < BQ25790_VINDPM_V_MIN_uV ||
> +	    vindpm > BQ25790_VINDPM_V_MAX_uV)
> +		return -EINVAL;
> +
> +	vlim = vindpm / BQ25790_VINDPM_STEP_uV;
> +
> +	vlim_msb = (vlim >> 8) & 0xff;
> +
> +	ret = regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_MSB, vlim_msb);
> +	if (ret)
> +		return ret;
> +
> +	vlim_lsb = vlim & 0xff;
> +
> +	return regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_LSB, vlim_lsb);
> +}
> +
> +static int bq25790_get_input_volt_lim(struct bq25790_device *bq)
> +{
> +	int ret;
> +	int vlim;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_INPUT_V_LIM, &vlim);
> +	if (ret)
> +		return ret;
> +
> +	return vlim * BQ25790_VINDPM_STEP_uV;
> +}
> +
> +static int bq25790_set_input_curr_lim(struct bq25790_device *bq, int iindpm)
> +{
> +	int ret;
> +	int ilim, ilim_lsb, ilim_msb;
> +
> +	if (iindpm < BQ25790_IINDPM_I_MIN_uA ||
> +	    iindpm > BQ25790_IINDPM_I_MAX_uA)
> +		return -EINVAL;
> +
> +	ilim = iindpm / BQ25790_IINDPM_STEP_uA;
> +	ilim_msb = (ilim >> 8) & 0xff;
> +
> +	ret = regmap_write(bq->regmap, BQ25790_INPUT_I_LIM_MSB, ilim_msb);
> +	if (ret)
> +		return ret;
> +
> +	ilim_lsb = ilim & 0xff;
> +
> +	return regmap_write(bq->regmap, BQ25790_INPUT_I_LIM_LSB, ilim_lsb);
> +}
> +
> +static int bq25790_get_input_curr_lim(struct bq25790_device *bq)
> +{
> +	int ret;
> +	int ilim_msb, ilim_lsb;
> +	u16 ilim;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_INPUT_I_LIM_MSB, &ilim_msb);
> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_INPUT_I_LIM_LSB, &ilim_lsb);
> +	if (ret)
> +		return ret;
> +
> +	ilim = (ilim_msb << 8) | ilim_lsb;
> +
> +	return ilim * BQ25790_IINDPM_STEP_uA;
> +}
> +
> +static int bq25790_get_state(struct bq25790_device *bq,
> +			     struct bq25790_state *state)
> +{
> +	int chrg_stat_0, chrg_stat_1, chrg_stat_3, chrg_stat_4;
> +	int chrg_ctrl_0, fault_0, fault_1;
> +	int ret;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_0, &chrg_stat_0);
> +	if (ret)
> +		return ret;
> +
> +	state->vbus_status = chrg_stat_0 & BQ25790_VBUS_PRESENT;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_1, &chrg_stat_1);
> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_CTRL_0, &chrg_ctrl_0);
> +	if (ret)
> +		return ret;
> +
> +	if (chrg_ctrl_0 & BQ25790_CHRG_EN)
> +		state->chrg_status = chrg_stat_1 & BQ25790_CHG_STAT_MSK;
> +	else
> +		state->chrg_status = BQ25790_NOT_CHRGING;
> +
> +	state->chrg_type = chrg_stat_1 & BQ25790_VBUS_STAT_MSK;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_4, &chrg_stat_4);
> +	if (ret)
> +		return ret;
> +
> +	state->health = chrg_stat_4 & BQ25790_TEMP_MASK;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_FAULT_STAT_0, &fault_0);
> +	if (ret)
> +		return ret;
> +
> +	state->fault_0 = fault_0;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_FAULT_STAT_1, &fault_1);
> +	if (ret)
> +		return ret;
> +
> +	state->fault_1 = fault_1;
> +
> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_3, &chrg_stat_3);
> +	if (ret)
> +		return ret;
> +
> +	state->online = chrg_stat_3 & BQ25790_VSYS_STAT;
> +
> +	state->vbat_adc = bq25790_get_vbat_adc(bq);
> +
> +	state->vbus_adc = bq25790_get_vbus_adc(bq);
> +
> +	state->ibat_adc = bq25790_get_ibat_adc(bq);
> +
> +	return 0;
> +}
> +
> +static int bq25790_set_property(struct power_supply *psy,
> +		enum power_supply_property prop,
> +		const union power_supply_propval *val)
> +{
> +	struct bq25790_device *bq = power_supply_get_drvdata(psy);
> +	int ret = -EINVAL;
> +
> +	switch (prop) {
> +	case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
> +		ret = bq25790_set_input_curr_lim(bq, val->intval);
> +		break;
> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
> +		ret = bq25790_set_chrg_volt(bq, val->intval);
> +		break;
> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
> +		ret = bq25790_set_ichrg_curr(bq, val->intval);
> +		break;
> +	case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:
> +		ret = bq25790_set_prechrg_curr(bq, val->intval);
> +		break;
> +	case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:
> +		ret = bq25790_set_term_curr(bq, val->intval);
> +		break;
> +	case POWER_SUPPLY_PROP_STATUS:
> +		break;

what's the point of exposing PROP_STATUS and always returning
EINVAL?

> +	case POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT:
> +		ret = bq25790_set_input_volt_lim(bq, val->intval);
> +		break;
> +	default:
> +		return -EINVAL;
> +	}
> +
> +	return ret;
> +}
> +
> +static int bq25790_get_property(struct power_supply *psy,
> +				enum power_supply_property psp,
> +				union power_supply_propval *val)
> +{
> +	struct bq25790_device *bq = power_supply_get_drvdata(psy);
> +	struct bq25790_state state;
> +	int ret = 0;
> +
> +	mutex_lock(&bq->lock);
> +	ret = bq25790_get_state(bq, &state);
> +	mutex_unlock(&bq->lock);

useless lock, since state is a function local variable.

> +	if (ret)
> +		return ret;
> +
> +	switch (psp) {

please be consistent with newlines after break :)

> +	case POWER_SUPPLY_PROP_STATUS:
> +		if (!state.chrg_type || (state.chrg_type == BQ25790_OTG_MODE))
> +			val->intval = POWER_SUPPLY_STATUS_DISCHARGING;
> +		else if (!state.chrg_status)
> +			val->intval = POWER_SUPPLY_STATUS_NOT_CHARGING;
> +		else if (state.chrg_status == BQ25790_TERM_CHRG)
> +			val->intval = POWER_SUPPLY_STATUS_FULL;
> +		else
> +			val->intval = POWER_SUPPLY_STATUS_CHARGING;
> +		break;
> +	case POWER_SUPPLY_PROP_CHARGE_TYPE:
> +		switch (state.chrg_status) {
> +		case BQ25790_TRICKLE_CHRG:
> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE;
> +			break;
> +		case BQ25790_PRECHRG:
> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE;
> +			break;
> +		case BQ25790_FAST_CHRG:
> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_FAST;
> +			break;
> +		case BQ25790_TAPER_CHRG:
> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_STANDARD;
> +			break;
> +		case BQ25790_TOP_OFF_CHRG:
> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE;
> +			break;
> +		case BQ25790_NOT_CHRGING:
> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_NONE;
> +			break;
> +		default:
> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_UNKNOWN;
> +		}
> +		break;
> +	case POWER_SUPPLY_PROP_MANUFACTURER:
> +		val->strval = BQ25790_MANUFACTURER;
> +		break;
> +
> +	case POWER_SUPPLY_PROP_MODEL_NAME:
> +		val->strval = BQ25790_NAME;

I think it's better to report bq->model_name (and otherwise that
field is unused :))

> +		break;
> +
> +	case POWER_SUPPLY_PROP_ONLINE:
> +		val->intval = state.online;
> +		break;
> +	case POWER_SUPPLY_PROP_USB_TYPE:
> +		if (!state.chrg_type) {
> +			val->intval = POWER_SUPPLY_USB_TYPE_UNKNOWN;
> +			break;
> +		}
> +		switch (state.chrg_type) {
> +		case BQ25790_USB_SDP:
> +			val->intval = POWER_SUPPLY_USB_TYPE_SDP;
> +			break;
> +		case BQ25790_USB_CDP:
> +			val->intval = POWER_SUPPLY_USB_TYPE_CDP;
> +			break;
> +		case BQ25790_USB_DCP:
> +			val->intval = POWER_SUPPLY_USB_TYPE_DCP;
> +			break;
> +		case BQ25790_OTG_MODE:
> +			val->intval = POWER_SUPPLY_USB_TYPE_PD_DRP;
> +			break;
> +
> +		default:
> +			val->intval = POWER_SUPPLY_USB_TYPE_UNKNOWN;
> +			break;
> +		}
> +		break;
> +
> +	case POWER_SUPPLY_PROP_HEALTH:
> +		if (state.fault_1 & (BQ25790_OTG_OVP | BQ25790_VSYS_OVP))
> +			val->intval = POWER_SUPPLY_HEALTH_OVERVOLTAGE;
> +		else
> +			val->intval = POWER_SUPPLY_HEALTH_GOOD;
> +
> +		switch (state.health) {
> +		case BQ25790_TEMP_HOT:
> +			val->intval = POWER_SUPPLY_HEALTH_HOT;
> +			break;
> +		case BQ25790_TEMP_WARM:
> +			val->intval = POWER_SUPPLY_HEALTH_WARM;
> +			break;
> +		case BQ25790_TEMP_COOL:
> +			val->intval = POWER_SUPPLY_HEALTH_COOL;
> +			break;
> +		case BQ25790_TEMP_COLD:
> +			val->intval = POWER_SUPPLY_HEALTH_COLD;
> +			break;
> +		}
> +		break;
> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
> +		ret = bq25790_get_ichg_curr(bq);
> +		if (ret < 0)
> +			return ret;
> +
> +		val->intval = ret;
> +		break;
> +
> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
> +		ret = bq25790_get_chrg_volt(bq);
> +		if (ret < 0)
> +			return ret;
> +
> +		val->intval = ret;
> +		break;
> +
> +	case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:
> +		ret = bq25790_get_prechrg_curr(bq);
> +		if (ret < 0)
> +			return ret;
> +
> +		val->intval = ret;
> +		break;
> +	case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:
> +		ret = bq25790_get_term_curr(bq);
> +		if (ret < 0)
> +			return ret;
> +
> +		val->intval = ret;
> +		break;
> +	case POWER_SUPPLY_PROP_VOLTAGE_NOW:
> +		val->intval = state.vbus_adc;
> +		break;
> +	case POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT:
> +		ret = bq25790_get_input_volt_lim(bq);
> +		if (ret < 0)
> +			return ret;
> +
> +		val->intval = ret;
> +		break;
> +	case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
> +		ret = bq25790_get_input_curr_lim(bq);
> +		if (ret < 0)
> +			return ret;
> +
> +		val->intval = ret;
> +		break;
> +	default:
> +		return -EINVAL;
> +	}
> +
> +	return ret;
> +}
> +
> +static int bq25790_battery_get_property(struct power_supply *psy,
> +				       enum power_supply_property psp,
> +				       union power_supply_propval *val)
> +{
> +	struct bq25790_device *bq = power_supply_get_drvdata(psy);
> +	struct bq25790_state state;
> +	int ret = 0;
> +
> +	mutex_lock(&bq->lock);
> +	ret = bq25790_get_state(bq, &state);
> +	mutex_unlock(&bq->lock);

another useless lock, since state is also a function local variable
in this place.

> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_update_bits(bq->regmap, BQ25790_ADC_CTRL,
> +				 BQ25790_ADC_EN, BQ25790_ADC_EN);
> +	if (ret)
> +		return ret;

why is ADC enabled here?

> +
> +	switch (psp) {

please also be consistent with newline after break in this
switch statement.

> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX:
> +		val->intval = bq->init_data.ichg_max;
> +		break;
> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX:
> +		val->intval = bq->init_data.vreg_max;
> +		break;
> +
> +	case POWER_SUPPLY_PROP_VOLTAGE_NOW:
> +		val->intval = state.vbat_adc;
> +		break;
> +	case POWER_SUPPLY_PROP_CURRENT_NOW:
> +		val->intval = state.ibat_adc;
> +		break;
> +	default:
> +		return -EINVAL;
> +	}
> +
> +	return ret;
> +}
> +
> +static bool bq25790_state_changed(struct bq25790_device *bq,
> +				  struct bq25790_state *new_state)
> +{
> +	struct bq25790_state old_state;
> +
> +	mutex_lock(&bq->lock);
> +	old_state = bq->state;
> +	mutex_unlock(&bq->lock);
> +
> +	return (old_state.chrg_status != new_state->chrg_status ||
> +		old_state.chrg_fault != new_state->chrg_fault	||
> +		old_state.online != new_state->online		||
> +		old_state.health != new_state->health	||
> +		old_state.fault_0 != new_state->fault_0 ||
> +		old_state.fault_1 != new_state->fault_1 ||
> +		old_state.chrg_type != new_state->chrg_type ||
> +		old_state.vsys_status != new_state->vsys_status ||
> +		old_state.vbat_adc != new_state->vbat_adc ||
> +		old_state.vbus_adc != new_state->vbus_adc ||
> +		old_state.ibat_adc != new_state->ibat_adc);
> +}
> +
> +static irqreturn_t bq25790_irq_handler_thread(int irq, void *private)
> +{
> +	struct bq25790_device *bq = private;
> +	struct bq25790_state state;
> +	int ret;
> +
> +	ret = bq25790_get_state(bq, &state);
> +	if (ret < 0)
> +		goto irq_out;
> +
> +	if (!bq25790_state_changed(bq, &state))
> +		goto irq_out;
> +
> +	mutex_lock(&bq->lock);
> +	bq->state = state;
> +	mutex_unlock(&bq->lock);
> +
> +	power_supply_changed(bq->charger);
> +
> +irq_out:
> +	return IRQ_HANDLED;
> +}
> +
> +static enum power_supply_property bq25790_power_supply_props[] = {
> +	POWER_SUPPLY_PROP_MANUFACTURER,
> +	POWER_SUPPLY_PROP_MODEL_NAME,
> +	POWER_SUPPLY_PROP_STATUS,
> +	POWER_SUPPLY_PROP_ONLINE,
> +	POWER_SUPPLY_PROP_HEALTH,
> +	POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT,
> +	POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE,
> +	POWER_SUPPLY_PROP_PRECHARGE_CURRENT,
> +	POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT,
> +	POWER_SUPPLY_PROP_VOLTAGE_NOW,
> +	POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT,
> +	POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT,
> +	POWER_SUPPLY_PROP_CHARGE_TYPE,
> +	POWER_SUPPLY_PROP_USB_TYPE,
> +};
> +
> +static enum power_supply_property bq25790_battery_props[] = {
> +	POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX,
> +	POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX,
> +	POWER_SUPPLY_PROP_VOLTAGE_NOW,
> +	POWER_SUPPLY_PROP_CURRENT_NOW,
> +};
> +
> +static char *bq25790_charger_supplied_to[] = {
> +	"main-battery",
> +};

why do you need this? Usually this information should come from
platform data (e.g. DT).

> +static int bq25790_property_is_writeable(struct power_supply *psy,
> +					 enum power_supply_property prop)
> +{
> +	switch (prop) {
> +	case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
> +	case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:
> +	case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:
> +	case POWER_SUPPLY_PROP_STATUS:
> +	case POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT:
> +		return true;
> +	default:
> +		return false;
> +	}
> +}
> +
> +static const struct power_supply_desc bq25790_power_supply_desc = {
> +	.name = "bq25790-charger",
> +	.type = POWER_SUPPLY_TYPE_USB,
> +	.usb_types = bq25790_usb_type,
> +	.num_usb_types = ARRAY_SIZE(bq25790_usb_type),
> +	.properties = bq25790_power_supply_props,
> +	.num_properties = ARRAY_SIZE(bq25790_power_supply_props),
> +	.get_property = bq25790_get_property,
> +	.set_property = bq25790_set_property,
> +	.property_is_writeable = bq25790_property_is_writeable,
> +};
> +
> +static const struct power_supply_desc bq25790_battery_desc = {
> +	.name = "bq25790-battery",
> +	.type = POWER_SUPPLY_TYPE_BATTERY,
> +	.get_property = bq25790_battery_get_property,
> +	.properties = bq25790_battery_props,
> +	.num_properties = ARRAY_SIZE(bq25790_battery_props),
> +};
> +
> +static bool bq25790_is_volatile_reg(struct device *dev, unsigned int reg)
> +{
> +	switch (reg) {
> +	case BQ25790_ICO_I_LIM...BQ25790_FAULT_FLAG_1:
> +	case BQ25790_ADC_IBUS...BQ25790_ADC_DM:
> +	case BQ25790_CHRG_CTRL_0:
> +		return true;
> +	default:
> +		return false;
> +	}
> +}
> +
> +static const struct regmap_config bq25790_regmap_config = {
> +	.reg_bits = 8,
> +	.val_bits = 8,
> +
> +	.max_register = BQ25790_PART_INFO,
> +	.reg_defaults	= bq25790_reg_defs,
> +	.num_reg_defaults = ARRAY_SIZE(bq25790_reg_defs),
> +	.cache_type = REGCACHE_RBTREE,

There are no holes in the register map, so REGCACHE_FLAT
should be enough.

> +	.volatile_reg = bq25790_is_volatile_reg,
> +};
> +
> +static int bq25790_power_supply_init(struct bq25790_device *bq,
> +							struct device *dev)
> +{
> +	struct power_supply_config psy_cfg = { .drv_data = bq,
> +						.of_node = dev->of_node, };
> +
> +	psy_cfg.supplied_to = bq25790_charger_supplied_to;
> +	psy_cfg.num_supplicants = ARRAY_SIZE(bq25790_charger_supplied_to);
> +
> +	bq->charger = devm_power_supply_register(bq->dev,
> +						 &bq25790_power_supply_desc,
> +						 &psy_cfg);
> +	if (IS_ERR(bq->charger))
> +		return -EINVAL;
> +
> +	bq->battery = devm_power_supply_register(bq->dev,
> +						      &bq25790_battery_desc,
> +						      &psy_cfg);
> +	if (IS_ERR(bq->battery))
> +		return -EINVAL;
> +	return 0;
> +}
> +
> +static int bq25790_hw_init(struct bq25790_device *bq)
> +{
> +	struct power_supply_battery_info bat_info = { };
> +	int wd_reg_val = BQ25790_WATCHDOG_DIS;
> +	int wd_max_val = BQ25790_NUM_WD_VAL - 1;
> +	int ret = 0;
> +	int i;
> +
> +	if (bq->watchdog_timer) {
> +		if (bq->watchdog_timer >= bq25790_watchdog_time[wd_max_val])
> +			wd_reg_val = wd_max_val;
> +		else {
> +			for (i = 0; i < wd_max_val; i++) {
> +				if (bq->watchdog_timer > bq25790_watchdog_time[i] &&
> +				    bq->watchdog_timer < bq25790_watchdog_time[i + 1]) {
> +					wd_reg_val = i;
> +					break;
> +				}
> +			}
> +		}
> +	}
> +
> +	ret = regmap_update_bits(bq->regmap, BQ25790_CHRG_CTRL_1,
> +				 BQ25790_WATCHDOG_MASK, wd_reg_val);
> +	if (ret)
> +		return ret;
> +
> +	ret = power_supply_get_battery_info(bq->charger, &bat_info);
> +	if (ret) {
> +		dev_warn(bq->dev, "battery info missing, default values will be applied\n");
> +
> +		bat_info.constant_charge_current_max_ua =
> +							BQ25790_ICHRG_I_DEF_uA;
> +
> +		bat_info.constant_charge_voltage_max_uv =
> +							BQ25790_VREG_V_DEF_uV;
> +
> +		bat_info.precharge_current_ua = BQ25790_PRECHRG_I_DEF_uA;
> +		bat_info.charge_term_current_ua = BQ25790_TERMCHRG_I_DEF_uA;
> +		bq->init_data.ichg_max = BQ25790_ICHRG_I_MAX_uA;
> +		bq->init_data.vreg_max = BQ25790_VREG_V_MAX_uV;
> +	} else {
> +		bq->init_data.ichg_max =
> +				bat_info.constant_charge_current_max_ua;
> +
> +		bq->init_data.vreg_max =
> +				bat_info.constant_charge_voltage_max_uv;
> +	}
> +
> +	ret = bq25790_set_ichrg_curr(bq,
> +				bat_info.constant_charge_current_max_ua);
> +	if (ret)
> +		goto err_out;
> +
> +	ret = bq25790_set_prechrg_curr(bq, bat_info.precharge_current_ua);
> +	if (ret)
> +		goto err_out;
> +
> +	ret = bq25790_set_chrg_volt(bq,
> +				bat_info.constant_charge_voltage_max_uv);
> +	if (ret)
> +		goto err_out;
> +
> +	ret = bq25790_set_term_curr(bq, bat_info.charge_term_current_ua);
> +	if (ret)
> +		goto err_out;
> +
> +	ret = bq25790_set_input_volt_lim(bq, bq->init_data.vlim);
> +	if (ret)
> +		goto err_out;
> +
> +	ret = bq25790_set_input_curr_lim(bq, bq->init_data.ilim);
> +	if (ret)
> +		goto err_out;
> +
> +err_out:
> +	return ret;

You need to call power_supply_put_battery_info(), since the
struct contains some pointers with allocated data.

> +}
> +
> +static int bq25790_parse_dt(struct bq25790_device *bq)
> +{
> +	int ret;
> +
> +	ret = device_property_read_u32(bq->dev, "ti,watchdog-timeout-ms",
> +				       &bq->watchdog_timer);
> +	if (ret)
> +		bq->watchdog_timer = BQ25790_WATCHDOG_DIS;
> +
> +	if (bq->watchdog_timer > BQ25790_WATCHDOG_MAX ||
> +	    bq->watchdog_timer < BQ25790_WATCHDOG_DIS)
> +		return -EINVAL;
> +
> +	ret = device_property_read_u32(bq->dev,
> +				       "input-voltage-limit-microvolt",
> +				       &bq->init_data.vlim);
> +	if (ret)
> +		bq->init_data.vlim = BQ25790_VINDPM_DEF_uV;
> +
> +	if (bq->init_data.vlim > BQ25790_VINDPM_V_MAX_uV ||
> +	    bq->init_data.vlim < BQ25790_VINDPM_V_MIN_uV)
> +		return -EINVAL;
> +
> +	ret = device_property_read_u32(bq->dev,
> +				       "input-current-limit-microamp",
> +				       &bq->init_data.ilim);
> +	if (ret)
> +		bq->init_data.ilim = BQ25790_IINDPM_DEF_uA;
> +
> +	if (bq->init_data.ilim > BQ25790_IINDPM_I_MAX_uA ||
> +	    bq->init_data.ilim < BQ25790_IINDPM_I_MIN_uA)
> +		return -EINVAL;
> +
> +	return 0;
> +}
> +
> +static int bq25790_probe(struct i2c_client *client,
> +			 const struct i2c_device_id *id)
> +{
> +	struct device *dev = &client->dev;
> +	struct bq25790_device *bq;
> +	int ret;
> +
> +	bq = devm_kzalloc(dev, sizeof(*bq), GFP_KERNEL);
> +	if (!bq)
> +		return -ENOMEM;
> +
> +	bq->client = client;
> +	bq->dev = dev;
> +
> +	mutex_init(&bq->lock);
> +
> +	strncpy(bq->model_name, id->name, I2C_NAME_SIZE);
> +
> +	bq->regmap = devm_regmap_init_i2c(client, &bq25790_regmap_config);
> +	if (IS_ERR(bq->regmap)) {
> +		dev_err(dev, "Failed to allocate register map\n");
> +		return PTR_ERR(bq->regmap);
> +	}
> +
> +	i2c_set_clientdata(client, bq);
> +
> +	ret = bq25790_parse_dt(bq);
> +	if (ret) {
> +		dev_err(dev, "Failed to read device tree properties%d\n", ret);
> +		return ret;
> +	}
> +
> +	/* OTG reporting */
> +	bq->usb2_phy = devm_usb_get_phy(dev, USB_PHY_TYPE_USB2);
> +	if (!IS_ERR_OR_NULL(bq->usb2_phy)) {
> +		INIT_WORK(&bq->usb_work, bq25790_usb_work);
> +		bq->usb_nb.notifier_call = bq25790_usb_notifier;
> +		usb_register_notifier(bq->usb2_phy, &bq->usb_nb);

You also need to unregister the usb notifiers on driver removal.
I suggest to use devm_add_action_or_reset():

static void bq25790_unregister_usb_notifier(void *data)
{
    struct bq25790_device *bq = (struct bq25790_device *) data;

    if (!IS_ERR_OR_NULL(bq->usb2_phy))
        usb_unregister_notifier(bq->usb2_phy, &bq->usb_nb);

    if (!IS_ERR_OR_NULL(bq->usb3_phy))
        usb_unregister_notifier(bq->usb3_phy, &bq->usb_nb);
}

ret = devm_add_action_or_reset(dev, bq25790_unregister_usb_notifier, bq)
if (ret)
    return ret;

-- Sebastian

> +	}
> +
> +	bq->usb3_phy = devm_usb_get_phy(dev, USB_PHY_TYPE_USB3);
> +	if (!IS_ERR_OR_NULL(bq->usb3_phy)) {
> +		INIT_WORK(&bq->usb_work, bq25790_usb_work);
> +		bq->usb_nb.notifier_call = bq25790_usb_notifier;
> +		usb_register_notifier(bq->usb3_phy, &bq->usb_nb);
> +	}
> +
> +	if (client->irq) {
> +		ret = devm_request_threaded_irq(dev, client->irq, NULL,
> +						bq25790_irq_handler_thread,
> +						IRQF_TRIGGER_FALLING |
> +						IRQF_ONESHOT,
> +						dev_name(&client->dev), bq);
> +		if (ret)
> +			goto error_out;
> +	}
> +
> +	ret = bq25790_power_supply_init(bq, dev);
> +	if (ret) {
> +		dev_err(dev, "Failed to register power supply\n");
> +		goto error_out;
> +	}
> +
> +	ret = bq25790_hw_init(bq);
> +	if (ret) {
> +		dev_err(dev, "Cannot initialize the chip.\n");
> +		goto error_out;
> +	}
> +
> +	return ret;
> +error_out:
> +	if (!IS_ERR_OR_NULL(bq->usb2_phy))
> +		usb_unregister_notifier(bq->usb2_phy, &bq->usb_nb);
> +
> +	if (!IS_ERR_OR_NULL(bq->usb3_phy))
> +		usb_unregister_notifier(bq->usb3_phy, &bq->usb_nb);
> +	return ret;
> +}
> +
> +static const struct i2c_device_id bq25790_i2c_ids[] = {
> +	{ BQ25790_NAME, 0 },
> +	{ BQ25792_NAME, 1 },
> +	{},
> +};
> +MODULE_DEVICE_TABLE(i2c, bq25790_i2c_ids);
> +
> +static const struct of_device_id bq25790_of_match[] = {
> +	{ .compatible = "ti,bq25790", },
> +	{ .compatible = "ti,bq25792", },
> +	{ },
> +};
> +MODULE_DEVICE_TABLE(of, bq25790_of_match);
> +
> +static const struct acpi_device_id bq25790_acpi_match[] = {
> +	{BQ25790_NAME, 0},
> +	{BQ25792_NAME, 0},
> +	{},
> +};
> +MODULE_DEVICE_TABLE(acpi, bq25790_acpi_match);
> +
> +static struct i2c_driver bq25790_driver = {
> +	.driver = {
> +		.name = "bq25790-charger",
> +		.of_match_table = bq25790_of_match,
> +		.acpi_match_table = ACPI_PTR(bq25790_acpi_match),
> +	},
> +	.probe = bq25790_probe,
> +	.id_table = bq25790_i2c_ids,
> +};
> +module_i2c_driver(bq25790_driver);
> +
> +MODULE_AUTHOR("Dan Murphy <dmurphy@ti.com>");
> +MODULE_AUTHOR("Ricardo Rivera-Matos <r-rivera-matos@ti.com>");
> +MODULE_DESCRIPTION("bq25790 charger driver");
> +MODULE_LICENSE("GPL v2");
> diff --git a/drivers/power/supply/bq25790_charger.h b/drivers/power/supply/bq25790_charger.h
> new file mode 100644
> index 000000000000..77234e116a6b
> --- /dev/null
> +++ b/drivers/power/supply/bq25790_charger.h
> @@ -0,0 +1,150 @@
> +/* SPDX-License-Identifier: GPL-2.0-only */
> +// BQ25790 Charger Driver
> +// Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/
> +
> +#ifndef _BQ25790_CHARGER_H
> +#define _BQ25790_CHARGER_H
> +
> +#define BQ25790_MANUFACTURER	"Texas Instruments"
> +#define BQ25790_NAME		"bq25790"
> +#define BQ25792_NAME		"bq25792"
> +
> +#define BQ25790_MIN_SYS_V	0x00
> +#define BQ25790_CHRG_V_LIM_MSB	0x01
> +#define BQ25790_CHRG_V_LIM_LSB	0x02
> +#define BQ25790_CHRG_I_LIM_MSB	0x03
> +#define BQ25790_CHRG_I_LIM_LSB	0x04
> +#define BQ25790_INPUT_V_LIM	0x05
> +#define BQ25790_INPUT_I_LIM_MSB	0x06
> +#define BQ25790_INPUT_I_LIM_LSB	0x07
> +#define BQ25790_PRECHRG_CTRL	0x08
> +#define BQ25790_TERM_CTRL	0x09
> +#define BQ25790_RECHRG_CTRL	0x0a
> +#define BQ25790_VOTG_REG	0x0b
> +#define BQ25790_IOTG_REG	0x0d
> +#define BQ25790_TIMER_CTRL	0x0e
> +#define BQ25790_CHRG_CTRL_0	0x0f
> +#define BQ25790_CHRG_CTRL_1	0x10
> +#define BQ25790_CHRG_CTRL_2	0x11
> +#define BQ25790_CHRG_CTRL_3	0x12
> +#define BQ25790_CHRG_CTRL_4	0x13
> +#define BQ25790_CHRG_CTRL_5	0x14
> +#define BQ25790_MPPT_CTRL	0x15
> +#define BQ25790_TEMP_CTRL	0x16
> +#define BQ25790_NTC_CTRL_0	0x17
> +#define BQ25790_NTC_CTRL_1	0x18
> +#define BQ25790_ICO_I_LIM	0x19
> +#define BQ25790_CHRG_STAT_0	0x1b
> +#define BQ25790_CHRG_STAT_1	0x1c
> +#define BQ25790_CHRG_STAT_2	0x1d
> +#define BQ25790_CHRG_STAT_3	0x1e
> +#define BQ25790_CHRG_STAT_4	0x1f
> +#define BQ25790_FAULT_STAT_0	0x20
> +#define BQ25790_FAULT_STAT_1	0x21
> +#define BQ25790_CHRG_FLAG_0	0x22
> +#define BQ25790_CHRG_FLAG_1	0x23
> +#define BQ25790_CHRG_FLAG_2	0x24
> +#define BQ25790_CHRG_FLAG_3	0x25
> +#define BQ25790_FAULT_FLAG_0	0x26
> +#define BQ25790_FAULT_FLAG_1	0x27
> +#define BQ25790_CHRG_MSK_0	0x28
> +#define BQ25790_CHRG_MSK_1	0x29
> +#define BQ25790_CHRG_MSK_2	0x2a
> +#define BQ25790_CHRG_MSK_3	0x2b
> +#define BQ25790_FAULT_MSK_0	0x2c
> +#define BQ25790_FAULT_MSK_1	0x2d
> +#define BQ25790_ADC_CTRL	0x2e
> +#define BQ25790_FN_DISABE_0	0x2f
> +#define BQ25790_FN_DISABE_1	0x30
> +#define BQ25790_ADC_IBUS	0x31
> +#define BQ25790_ADC_IBAT_MSB	0x33
> +#define BQ25790_ADC_IBAT_LSB	0x34
> +#define BQ25790_ADC_VBUS_MSB	0x35
> +#define BQ25790_ADC_VBUS_LSB	0x36
> +#define BQ25790_ADC_VAC1	0x37
> +#define BQ25790_ADC_VAC2	0x39
> +#define BQ25790_ADC_VBAT_MSB	0x3b
> +#define BQ25790_ADC_VBAT_LSB	0x3c
> +#define BQ25790_ADC_VSYS_MSB	0x3d
> +#define BQ25790_ADC_VSYS_LSB	0x3e
> +#define BQ25790_ADC_TS		0x3f
> +#define BQ25790_ADC_TDIE	0x41
> +#define BQ25790_ADC_DP		0x43
> +#define BQ25790_ADC_DM		0x45
> +#define BQ25790_DPDM_DRV	0x47
> +#define BQ25790_PART_INFO	0x48
> +
> +#define BQ25790_CHRG_EN		BIT(5)
> +#define BQ25790_ADC_EN		BIT(7)
> +
> +/* Charger Status 1 */
> +#define BQ25790_CHG_STAT_MSK	GENMASK(7, 5)
> +#define BQ25790_NOT_CHRGING	0
> +#define BQ25790_TRICKLE_CHRG	BIT(5)
> +#define BQ25790_PRECHRG		BIT(6)
> +#define BQ25790_FAST_CHRG	(BIT(5) | BIT(6))
> +#define BQ25790_TAPER_CHRG	BIT(7)
> +#define BQ25790_TOP_OFF_CHRG	(BIT(6) | BIT(7))
> +#define BQ25790_TERM_CHRG	(BIT(5) | BIT(6) | BIT(7))
> +#define BQ25790_VBUS_PRESENT	BIT(0)
> +
> +#define BQ25790_VBUS_STAT_MSK	GENMASK(4, 1)
> +#define BQ25790_USB_SDP		BIT(1)
> +#define BQ25790_USB_CDP		BIT(2)
> +#define BQ25790_USB_DCP		(BIT(1) | BIT(2))
> +#define BQ25790_HVDCP		BIT(3)
> +#define BQ25790_UNKNOWN_3A	(BIT(3) | BIT(1))
> +#define BQ25790_NON_STANDARD	(BIT(3) | BIT(2))
> +#define BQ25790_OTG_MODE	(BIT(3) | BIT(2) | BIT(1))
> +#define BQ25790_UNQUAL_ADAPT	BIT(4)
> +#define BQ25790_DIRECT_PWR	(BIT(4) | BIT(2) | BIT(1))
> +
> +/* Charger Status 4 */
> +#define BQ25790_TEMP_HOT	BIT(0)
> +#define BQ25790_TEMP_WARM	BIT(1)
> +#define BQ25790_TEMP_COOL	BIT(2)
> +#define BQ25790_TEMP_COLD	BIT(3)
> +#define BQ25790_TEMP_MASK	GENMASK(3, 0)
> +
> +#define BQ25790_OTG_OVP		BIT(5)
> +#define BQ25790_VSYS_OVP	BIT(6)
> +#define BQ25790_VSYS_STAT	BIT(4)
> +
> +#define BQ25790_PRECHRG_CUR_MASK		GENMASK(5, 0)
> +#define BQ25790_PRECHRG_CURRENT_STEP_uA		40000
> +#define BQ25790_PRECHRG_I_MIN_uA		40000
> +#define BQ25790_PRECHRG_I_MAX_uA		2000000
> +#define BQ25790_PRECHRG_I_DEF_uA		120000
> +#define BQ25790_TERMCHRG_CUR_MASK		GENMASK(4, 0)
> +#define BQ25790_TERMCHRG_CURRENT_STEP_uA	40000
> +#define BQ25790_TERMCHRG_I_MIN_uA		40000
> +#define BQ25790_TERMCHRG_I_MAX_uA		1000000
> +#define BQ25790_TERMCHRG_I_DEF_uA		200000
> +#define BQ25790_ICHRG_CURRENT_STEP_uA		10000
> +#define BQ25790_ICHRG_I_MIN_uA			50000
> +#define BQ25790_ICHRG_I_MAX_uA			5000000
> +#define BQ25790_ICHRG_I_DEF_uA			1000000
> +
> +#define BQ25790_VREG_V_MAX_uV	18800000
> +#define BQ25790_VREG_V_MIN_uV	3000000
> +#define BQ25790_VREG_V_DEF_uV	3600000
> +#define BQ25790_VREG_V_STEP_uV	10000
> +
> +#define BQ25790_IINDPM_I_MIN_uA	100000
> +#define BQ25790_IINDPM_I_MAX_uA	3300000
> +#define BQ25790_IINDPM_STEP_uA	10000
> +#define BQ25790_IINDPM_DEF_uA	1000000
> +
> +#define BQ25790_VINDPM_V_MIN_uV 3600000
> +#define BQ25790_VINDPM_V_MAX_uV 22000000
> +#define BQ25790_VINDPM_STEP_uV	100000
> +#define BQ25790_VINDPM_DEF_uV	3600000
> +
> +#define BQ25790_ADC_VOLT_STEP_uV	1000
> +#define BQ25790_ADC_CURR_STEP_uA	1000
> +
> +#define BQ25790_WATCHDOG_MASK	GENMASK(2, 0)
> +#define BQ25790_WATCHDOG_DIS	0
> +#define BQ25790_WATCHDOG_MAX	160000
> +
> +#endif /* _BQ25790_CHARGER_H */
> -- 
> 2.28.0.585.ge1cfff676549
>
Ricardo Rivera-Matos Feb. 1, 2021, 2:06 a.m. UTC | #2
Sebastian,

On 12/27/20 12:46 PM, Sebastian Reichel wrote:
> Hi,

>

> Sorry, took me a lot longer than expected...

>

> On Fri, Oct 09, 2020 at 09:41:12AM -0500, Dan Murphy wrote:

>> BQ25790 is a highly integrated switch-mode buck-boost charger

>> for 1-4 cell Li-ion battery and Li-polymer battery.

>>

>> Signed-off-by: Ricardo Rivera-Matos <r-rivera-matos@ti.com>

>> Signed-off-by: Dan Murphy <dmurphy@ti.com>

>> ---

>>   drivers/power/supply/Kconfig           |    8 +

>>   drivers/power/supply/Makefile          |    1 +

>>   drivers/power/supply/bq25790_charger.c | 1121 ++++++++++++++++++++++++

>>   drivers/power/supply/bq25790_charger.h |  150 ++++

>>   4 files changed, 1280 insertions(+)

>>   create mode 100644 drivers/power/supply/bq25790_charger.c

>>   create mode 100644 drivers/power/supply/bq25790_charger.h

>>

>> diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig

>> index eec646c568b7..1cc361238d9a 100644

>> --- a/drivers/power/supply/Kconfig

>> +++ b/drivers/power/supply/Kconfig

>> @@ -628,6 +628,14 @@ config CHARGER_BQ2515X

>>   	  rail, ADC for battery and system monitoring, and push-button

>>   	  controller.

>>   

>> +config CHARGER_BQ25790

>> +	tristate "TI BQ25790 battery charger driver"

>> +	depends on I2C

>> +	depends on GPIOLIB || COMPILE_TEST

>> +	select REGMAP_I2C

>> +	help

>> +	  Say Y to enable support for the TI BQ25790 battery charger.

>> +

>>   config CHARGER_BQ25890

>>   	tristate "TI BQ25890 battery charger driver"

>>   	depends on I2C

>> diff --git a/drivers/power/supply/Makefile b/drivers/power/supply/Makefile

>> index dd4b86318cd9..29b71cc92550 100644

>> --- a/drivers/power/supply/Makefile

>> +++ b/drivers/power/supply/Makefile

>> @@ -83,6 +83,7 @@ obj-$(CONFIG_CHARGER_BQ24190)	+= bq24190_charger.o

>>   obj-$(CONFIG_CHARGER_BQ24257)	+= bq24257_charger.o

>>   obj-$(CONFIG_CHARGER_BQ24735)	+= bq24735-charger.o

>>   obj-$(CONFIG_CHARGER_BQ2515X)	+= bq2515x_charger.o

>> +obj-$(CONFIG_CHARGER_BQ25790)	+= bq25790_charger.o

>>   obj-$(CONFIG_CHARGER_BQ25890)	+= bq25890_charger.o

>>   obj-$(CONFIG_CHARGER_BQ25980)	+= bq25980_charger.o

>>   obj-$(CONFIG_CHARGER_SMB347)	+= smb347-charger.o

>> diff --git a/drivers/power/supply/bq25790_charger.c b/drivers/power/supply/bq25790_charger.c

>> new file mode 100644

>> index 000000000000..838a49c2f9f0

>> --- /dev/null

>> +++ b/drivers/power/supply/bq25790_charger.c

>> @@ -0,0 +1,1121 @@

>> +// SPDX-License-Identifier: GPL-2.0

>> +// BQ25790 driver

>> +// Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/

>> +

>> +#include <linux/module.h>

>> +#include <linux/i2c.h>

>> +#include <linux/power_supply.h>

>> +#include <linux/regmap.h>

>> +#include <linux/types.h>

>> +#include <linux/gpio/consumer.h>

>> +#include <linux/interrupt.h>

>> +#include <linux/delay.h>

>> +#include <linux/usb/phy.h>

>> +

>> +#include <linux/acpi.h>

>> +#include <linux/of.h>

>> +

>> +#include "bq25790_charger.h"

>> +

>> +#define BQ25790_NUM_WD_VAL	8

>> +

>> +struct bq25790_init_data {

>> +	u32 ichg;

>> +	u32 ilim;

>> +	u32 vreg;

>> +	u32 iterm;

>> +	u32 iprechg;

>> +	u32 vlim;

>> +	u32 ichg_max;

>> +	u32 vreg_max;

>> +};

>> +

>> +struct bq25790_state {

>> +	bool online;

>> +	u8 chrg_status;

>> +	u8 chrg_type;

>> +	u8 health;

>> +	u8 chrg_fault;

>> +	u8 vsys_status;

>> +	u8 vbus_status;

>> +	u8 fault_0;

>> +	u8 fault_1;

>> +	u32 vbat_adc;

>> +	u32 vbus_adc;

>> +	u32 ibat_adc;

>> +};

>> +

>> +struct bq25790_device {

>> +	struct i2c_client *client;

>> +	struct device *dev;

>> +	struct power_supply *charger;

>> +	struct power_supply *battery;

>> +	struct mutex lock;

>> +

>> +	struct usb_phy *usb2_phy;

>> +	struct usb_phy *usb3_phy;

>> +	struct notifier_block usb_nb;

>> +	struct work_struct usb_work;

>> +	unsigned long usb_event;

>> +	struct regmap *regmap;

>> +

>> +	char model_name[I2C_NAME_SIZE];

>> +	int device_id;

>> +

>> +	struct bq25790_init_data init_data;

>> +	struct bq25790_state state;

>> +	int watchdog_timer;

>> +};

>> +

>> +static struct reg_default bq25790_reg_defs[] = {

>> +	{BQ25790_INPUT_V_LIM, 0x24},

>> +	{BQ25790_INPUT_I_LIM_MSB, 0x01},

>> +	{BQ25790_INPUT_I_LIM_LSB, 0x2c},

>> +	{BQ25790_PRECHRG_CTRL, 0xc3},

>> +	{BQ25790_TERM_CTRL, 0x5},

>> +	{BQ25790_VOTG_REG, 0xdc},

>> +	{BQ25790_IOTG_REG, 0x4b},

>> +	{BQ25790_TIMER_CTRL, 0x3d},

>> +	{BQ25790_CHRG_CTRL_0, 0xa2},

>> +	{BQ25790_CHRG_CTRL_1, 0x85},

>> +	{BQ25790_CHRG_CTRL_2, 0x40},

>> +	{BQ25790_CHRG_CTRL_3, 0x12},

>> +	{BQ25790_CHRG_CTRL_5, 0x16},

>> +	{BQ25790_MPPT_CTRL, 0xaa},

>> +	{BQ25790_TEMP_CTRL, 0xc0},

>> +	{BQ25790_NTC_CTRL_0, 0x7a},

>> +	{BQ25790_NTC_CTRL_1, 0x54},

>> +	{BQ25790_ICO_I_LIM, 0x0},

>> +	{BQ25790_CHRG_STAT_0, 0x0},

>> +	{BQ25790_CHRG_STAT_1, 0x0},

>> +	{BQ25790_CHRG_STAT_2, 0x0},

>> +	{BQ25790_CHRG_STAT_3, 0x0},

>> +	{BQ25790_CHRG_STAT_4, 0x0},

>> +	{BQ25790_FAULT_STAT_0, 0x0},

>> +	{BQ25790_FAULT_STAT_1, 0x0},

>> +	{BQ25790_CHRG_FLAG_0, 0x0},

>> +	{BQ25790_CHRG_FLAG_1, 0x0},

>> +	{BQ25790_CHRG_FLAG_2, 0x0},

>> +	{BQ25790_CHRG_FLAG_3, 0x0},

>> +	{BQ25790_FAULT_FLAG_0, 0x0},

>> +	{BQ25790_FAULT_FLAG_1, 0x0},

>> +	{BQ25790_CHRG_MSK_0, 0x0},

>> +	{BQ25790_CHRG_MSK_1, 0x0},

>> +	{BQ25790_CHRG_MSK_2, 0x0},

>> +	{BQ25790_CHRG_MSK_3, 0x0},

>> +	{BQ25790_FAULT_MSK_0, 0x0},

>> +	{BQ25790_FAULT_MSK_1, 0x0},

>> +	{BQ25790_ADC_CTRL, 0x30},

>> +	{BQ25790_FN_DISABE_0, 0x0},

>> +	{BQ25790_FN_DISABE_1, 0x0},

>> +	{BQ25790_ADC_IBUS, 0x0},

>> +	{BQ25790_ADC_IBAT_MSB, 0x0},

>> +	{BQ25790_ADC_IBAT_LSB, 0x0},

>> +	{BQ25790_ADC_VBUS_MSB, 0x0},

>> +	{BQ25790_ADC_VBUS_LSB, 0x0},

>> +	{BQ25790_ADC_VAC1, 0x0},

>> +	{BQ25790_ADC_VAC2, 0x0},

>> +	{BQ25790_ADC_VBAT_MSB, 0x0},

>> +	{BQ25790_ADC_VBAT_LSB, 0x0},

>> +	{BQ25790_ADC_VBUS_MSB, 0x0},

>> +	{BQ25790_ADC_VBUS_LSB, 0x0},

>> +	{BQ25790_ADC_TS, 0x0},

>> +	{BQ25790_ADC_TDIE, 0x0},

>> +	{BQ25790_ADC_DP, 0x0},

>> +	{BQ25790_ADC_DM, 0x0},

>> +	{BQ25790_DPDM_DRV, 0x0},

>> +	{BQ25790_PART_INFO, 0x0},

>> +};

>> +

>> +static int bq25790_watchdog_time[BQ25790_NUM_WD_VAL] = {0, 500, 1000, 2000,

>> +							20000, 40000, 80000,

>> +							160000};

>> +

>> +static enum power_supply_usb_type bq25790_usb_type[] = {

>> +	POWER_SUPPLY_USB_TYPE_PD_DRP,

>> +	POWER_SUPPLY_USB_TYPE_SDP,

>> +	POWER_SUPPLY_USB_TYPE_CDP,

>> +	POWER_SUPPLY_USB_TYPE_DCP,

>> +	POWER_SUPPLY_USB_TYPE_UNKNOWN,

>> +};

>> +

>> +static int bq25790_usb_notifier(struct notifier_block *nb, unsigned long val,

>> +				void *priv)

>> +{

>> +	struct bq25790_device *bq =

>> +			container_of(nb, struct bq25790_device, usb_nb);

>> +

>> +	bq->usb_event = val;

>> +	queue_work(system_power_efficient_wq, &bq->usb_work);

>> +

>> +	return NOTIFY_OK;

>> +}

>> +

>> +static void bq25790_usb_work(struct work_struct *data)

>> +{

>> +	struct bq25790_device *bq =

>> +			container_of(data, struct bq25790_device, usb_work);

>> +

>> +	switch (bq->usb_event) {

>> +	case USB_EVENT_ID:

>> +		break;

>> +

>> +	case USB_EVENT_NONE:

>> +		power_supply_changed(bq->charger);

>> +		break;

>> +	}

>> +

>> +	return;

>> +}

>> +

>> +static int bq25790_get_vbat_adc(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +	int vbat_adc_lsb, vbat_adc_msb;

>> +	u16 vbat_adc;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_ADC_VBAT_MSB, &vbat_adc_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_ADC_VBAT_LSB, &vbat_adc_lsb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	vbat_adc = (vbat_adc_msb << 8) | vbat_adc_lsb;

>> +

>> +	return vbat_adc * BQ25790_ADC_VOLT_STEP_uV;

>> +}

>> +

>> +static int bq25790_get_vbus_adc(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +	int vbus_adc_lsb, vbus_adc_msb;

>> +	u16 vbus_adc;

>> +

>> +	ret = regmap_update_bits(bq->regmap, BQ25790_ADC_CTRL,

>> +				 BQ25790_ADC_EN, BQ25790_ADC_EN);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_ADC_VBUS_MSB, &vbus_adc_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_ADC_VBUS_LSB, &vbus_adc_lsb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	vbus_adc = (vbus_adc_msb << 8) | vbus_adc_lsb;

>> +

>> +	return vbus_adc * BQ25790_ADC_VOLT_STEP_uV;

>> +}

>> +

>> +static int bq25790_get_ibat_adc(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +	int ibat_adc_lsb, ibat_adc_msb;

>> +	u16 ibat_adc;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_ADC_IBAT_MSB, &ibat_adc_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_ADC_IBAT_LSB, &ibat_adc_lsb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ibat_adc = (ibat_adc_msb << 8) | ibat_adc_lsb;

>> +

>> +	return ibat_adc * BQ25790_ADC_CURR_STEP_uA;

>> +}

>> +

>> +static int bq25790_get_term_curr(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +	int reg_val;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_TERM_CTRL, &reg_val);

>> +	if (ret)

>> +		return ret;

>> +

>> +	reg_val &= BQ25790_TERMCHRG_CUR_MASK;

>> +

>> +	return reg_val * BQ25790_TERMCHRG_CURRENT_STEP_uA;

>> +}

>> +

>> +static int bq25790_get_prechrg_curr(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +	int reg_val;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_PRECHRG_CTRL, &reg_val);

>> +	if (ret)

>> +		return ret;

>> +

>> +	reg_val &= BQ25790_PRECHRG_CUR_MASK;

>> +

>> +	return reg_val * BQ25790_PRECHRG_CURRENT_STEP_uA;

>> +}

>> +

>> +static int bq25790_get_ichg_curr(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +	int ichg, ichg_lsb, ichg_msb;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_I_LIM_LSB, &ichg_lsb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_I_LIM_MSB, &ichg_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ichg = (ichg_msb << 8) | ichg_lsb;

>> +

>> +	return ichg * BQ25790_ICHRG_CURRENT_STEP_uA;

>> +}

>> +

>> +static int bq25790_set_term_curr(struct bq25790_device *bq, int term_current)

>> +{

>> +	int reg_val;

>> +

>> +	if (term_current < BQ25790_TERMCHRG_I_MIN_uA ||

>> +	    term_current > BQ25790_TERMCHRG_I_MAX_uA)

>> +		return -EINVAL;

>> +

>> +	reg_val = term_current / BQ25790_TERMCHRG_CURRENT_STEP_uA;

>> +

>> +	return regmap_update_bits(bq->regmap, BQ25790_TERM_CTRL,

>> +				  BQ25790_TERMCHRG_CUR_MASK, reg_val);

>> +}

>> +

>> +static int bq25790_set_prechrg_curr(struct bq25790_device *bq, int pre_current)

>> +{

>> +	int reg_val;

>> +

>> +	if (pre_current < BQ25790_PRECHRG_I_MIN_uA ||

>> +	    pre_current > BQ25790_PRECHRG_I_MAX_uA)

>> +		return -EINVAL;

>> +

>> +	reg_val = pre_current / BQ25790_PRECHRG_CURRENT_STEP_uA;

>> +

>> +	return regmap_update_bits(bq->regmap, BQ25790_PRECHRG_CTRL,

>> +				  BQ25790_PRECHRG_CUR_MASK, reg_val);

>> +}

>> +

>> +static int bq25790_set_ichrg_curr(struct bq25790_device *bq, int chrg_curr)

>> +{

>> +	int ret;

>> +	int ichg, ichg_msb, ichg_lsb;

>> +

>> +	if (chrg_curr < BQ25790_ICHRG_I_MIN_uA ||

>> +	    chrg_curr > BQ25790_ICHRG_I_MAX_uA)

>> +		return -EINVAL;

>> +

>> +	ichg = chrg_curr / BQ25790_ICHRG_CURRENT_STEP_uA;

>> +	ichg_msb = (ichg >> 8) & 0xff;

>> +	ret = regmap_write(bq->regmap, BQ25790_CHRG_I_LIM_MSB, ichg_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ichg_lsb = ichg & 0xff;

>> +

>> +	return regmap_write(bq->regmap, BQ25790_CHRG_I_LIM_LSB, ichg_lsb);

>> +}

>> +

>> +static int bq25790_set_chrg_volt(struct bq25790_device *bq, int chrg_volt)

>> +{

>> +	int vlim_lsb, vlim_msb, vlim;

>> +	int ret;

>> +

>> +	if (chrg_volt < BQ25790_VREG_V_MIN_uV ||

>> +	    chrg_volt > BQ25790_VREG_V_MAX_uV)

>> +		return -EINVAL;

>> +

>> +	vlim = chrg_volt / BQ25790_VREG_V_STEP_uV;

>> +	vlim_msb = (vlim >> 8) & 0xff;

>> +	ret = regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_MSB, vlim_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	vlim_lsb = vlim & 0xff;

>> +

>> +	return regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_LSB, vlim_lsb);

>> +}

>> +

>> +static int bq25790_get_chrg_volt(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +	int vlim_lsb, vlim_msb, chrg_volt;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_V_LIM_MSB, &vlim_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_V_LIM_LSB, &vlim_lsb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	chrg_volt = (vlim_msb << 8) | vlim_lsb;

>> +

>> +	return chrg_volt * BQ25790_VREG_V_STEP_uV;

>> +}

>> +

>> +static int bq25790_set_input_volt_lim(struct bq25790_device *bq, int vindpm)

>> +{

>> +	int ret;

>> +	int vlim_lsb, vlim_msb;

>> +	int vlim;

>> +

>> +	if (vindpm < BQ25790_VINDPM_V_MIN_uV ||

>> +	    vindpm > BQ25790_VINDPM_V_MAX_uV)

>> +		return -EINVAL;

>> +

>> +	vlim = vindpm / BQ25790_VINDPM_STEP_uV;

>> +

>> +	vlim_msb = (vlim >> 8) & 0xff;

>> +

>> +	ret = regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_MSB, vlim_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	vlim_lsb = vlim & 0xff;

>> +

>> +	return regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_LSB, vlim_lsb);

>> +}

>> +

>> +static int bq25790_get_input_volt_lim(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +	int vlim;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_INPUT_V_LIM, &vlim);

>> +	if (ret)

>> +		return ret;

>> +

>> +	return vlim * BQ25790_VINDPM_STEP_uV;

>> +}

>> +

>> +static int bq25790_set_input_curr_lim(struct bq25790_device *bq, int iindpm)

>> +{

>> +	int ret;

>> +	int ilim, ilim_lsb, ilim_msb;

>> +

>> +	if (iindpm < BQ25790_IINDPM_I_MIN_uA ||

>> +	    iindpm > BQ25790_IINDPM_I_MAX_uA)

>> +		return -EINVAL;

>> +

>> +	ilim = iindpm / BQ25790_IINDPM_STEP_uA;

>> +	ilim_msb = (ilim >> 8) & 0xff;

>> +

>> +	ret = regmap_write(bq->regmap, BQ25790_INPUT_I_LIM_MSB, ilim_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ilim_lsb = ilim & 0xff;

>> +

>> +	return regmap_write(bq->regmap, BQ25790_INPUT_I_LIM_LSB, ilim_lsb);

>> +}

>> +

>> +static int bq25790_get_input_curr_lim(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +	int ilim_msb, ilim_lsb;

>> +	u16 ilim;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_INPUT_I_LIM_MSB, &ilim_msb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_INPUT_I_LIM_LSB, &ilim_lsb);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ilim = (ilim_msb << 8) | ilim_lsb;

>> +

>> +	return ilim * BQ25790_IINDPM_STEP_uA;

>> +}

>> +

>> +static int bq25790_get_state(struct bq25790_device *bq,

>> +			     struct bq25790_state *state)

>> +{

>> +	int chrg_stat_0, chrg_stat_1, chrg_stat_3, chrg_stat_4;

>> +	int chrg_ctrl_0, fault_0, fault_1;

>> +	int ret;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_0, &chrg_stat_0);

>> +	if (ret)

>> +		return ret;

>> +

>> +	state->vbus_status = chrg_stat_0 & BQ25790_VBUS_PRESENT;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_1, &chrg_stat_1);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_CTRL_0, &chrg_ctrl_0);

>> +	if (ret)

>> +		return ret;

>> +

>> +	if (chrg_ctrl_0 & BQ25790_CHRG_EN)

>> +		state->chrg_status = chrg_stat_1 & BQ25790_CHG_STAT_MSK;

>> +	else

>> +		state->chrg_status = BQ25790_NOT_CHRGING;

>> +

>> +	state->chrg_type = chrg_stat_1 & BQ25790_VBUS_STAT_MSK;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_4, &chrg_stat_4);

>> +	if (ret)

>> +		return ret;

>> +

>> +	state->health = chrg_stat_4 & BQ25790_TEMP_MASK;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_FAULT_STAT_0, &fault_0);

>> +	if (ret)

>> +		return ret;

>> +

>> +	state->fault_0 = fault_0;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_FAULT_STAT_1, &fault_1);

>> +	if (ret)

>> +		return ret;

>> +

>> +	state->fault_1 = fault_1;

>> +

>> +	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_3, &chrg_stat_3);

>> +	if (ret)

>> +		return ret;

>> +

>> +	state->online = chrg_stat_3 & BQ25790_VSYS_STAT;

>> +

>> +	state->vbat_adc = bq25790_get_vbat_adc(bq);

>> +

>> +	state->vbus_adc = bq25790_get_vbus_adc(bq);

>> +

>> +	state->ibat_adc = bq25790_get_ibat_adc(bq);

>> +

>> +	return 0;

>> +}

>> +

>> +static int bq25790_set_property(struct power_supply *psy,

>> +		enum power_supply_property prop,

>> +		const union power_supply_propval *val)

>> +{

>> +	struct bq25790_device *bq = power_supply_get_drvdata(psy);

>> +	int ret = -EINVAL;

>> +

>> +	switch (prop) {

>> +	case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:

>> +		ret = bq25790_set_input_curr_lim(bq, val->intval);

>> +		break;

>> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:

>> +		ret = bq25790_set_chrg_volt(bq, val->intval);

>> +		break;

>> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:

>> +		ret = bq25790_set_ichrg_curr(bq, val->intval);

>> +		break;

>> +	case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:

>> +		ret = bq25790_set_prechrg_curr(bq, val->intval);

>> +		break;

>> +	case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:

>> +		ret = bq25790_set_term_curr(bq, val->intval);

>> +		break;

>> +	case POWER_SUPPLY_PROP_STATUS:

>> +		break;

> what's the point of exposing PROP_STATUS and always returning

> EINVAL?

ACK It is no longer necessary.
>

>> +	case POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT:

>> +		ret = bq25790_set_input_volt_lim(bq, val->intval);

>> +		break;

>> +	default:

>> +		return -EINVAL;

>> +	}

>> +

>> +	return ret;

>> +}

>> +

>> +static int bq25790_get_property(struct power_supply *psy,

>> +				enum power_supply_property psp,

>> +				union power_supply_propval *val)

>> +{

>> +	struct bq25790_device *bq = power_supply_get_drvdata(psy);

>> +	struct bq25790_state state;

>> +	int ret = 0;

>> +

>> +	mutex_lock(&bq->lock);

>> +	ret = bq25790_get_state(bq, &state);

>> +	mutex_unlock(&bq->lock);

> useless lock, since state is a function local variable.

ACK
>

>> +	if (ret)

>> +		return ret;

>> +

>> +	switch (psp) {

> please be consistent with newlines after break :)

ACK
>

>> +	case POWER_SUPPLY_PROP_STATUS:

>> +		if (!state.chrg_type || (state.chrg_type == BQ25790_OTG_MODE))

>> +			val->intval = POWER_SUPPLY_STATUS_DISCHARGING;

>> +		else if (!state.chrg_status)

>> +			val->intval = POWER_SUPPLY_STATUS_NOT_CHARGING;

>> +		else if (state.chrg_status == BQ25790_TERM_CHRG)

>> +			val->intval = POWER_SUPPLY_STATUS_FULL;

>> +		else

>> +			val->intval = POWER_SUPPLY_STATUS_CHARGING;

>> +		break;

>> +	case POWER_SUPPLY_PROP_CHARGE_TYPE:

>> +		switch (state.chrg_status) {

>> +		case BQ25790_TRICKLE_CHRG:

>> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE;

>> +			break;

>> +		case BQ25790_PRECHRG:

>> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE;

>> +			break;

>> +		case BQ25790_FAST_CHRG:

>> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_FAST;

>> +			break;

>> +		case BQ25790_TAPER_CHRG:

>> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_STANDARD;

>> +			break;

>> +		case BQ25790_TOP_OFF_CHRG:

>> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE;

>> +			break;

>> +		case BQ25790_NOT_CHRGING:

>> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_NONE;

>> +			break;

>> +		default:

>> +			val->intval = POWER_SUPPLY_CHARGE_TYPE_UNKNOWN;

>> +		}

>> +		break;

>> +	case POWER_SUPPLY_PROP_MANUFACTURER:

>> +		val->strval = BQ25790_MANUFACTURER;

>> +		break;

>> +

>> +	case POWER_SUPPLY_PROP_MODEL_NAME:

>> +		val->strval = BQ25790_NAME;

> I think it's better to report bq->model_name (and otherwise that

> field is unused :))

ACK
>

>> +		break;

>> +

>> +	case POWER_SUPPLY_PROP_ONLINE:

>> +		val->intval = state.online;

>> +		break;

>> +	case POWER_SUPPLY_PROP_USB_TYPE:

>> +		if (!state.chrg_type) {

>> +			val->intval = POWER_SUPPLY_USB_TYPE_UNKNOWN;

>> +			break;

>> +		}

>> +		switch (state.chrg_type) {

>> +		case BQ25790_USB_SDP:

>> +			val->intval = POWER_SUPPLY_USB_TYPE_SDP;

>> +			break;

>> +		case BQ25790_USB_CDP:

>> +			val->intval = POWER_SUPPLY_USB_TYPE_CDP;

>> +			break;

>> +		case BQ25790_USB_DCP:

>> +			val->intval = POWER_SUPPLY_USB_TYPE_DCP;

>> +			break;

>> +		case BQ25790_OTG_MODE:

>> +			val->intval = POWER_SUPPLY_USB_TYPE_PD_DRP;

>> +			break;

>> +

>> +		default:

>> +			val->intval = POWER_SUPPLY_USB_TYPE_UNKNOWN;

>> +			break;

>> +		}

>> +		break;

>> +

>> +	case POWER_SUPPLY_PROP_HEALTH:

>> +		if (state.fault_1 & (BQ25790_OTG_OVP | BQ25790_VSYS_OVP))

>> +			val->intval = POWER_SUPPLY_HEALTH_OVERVOLTAGE;

>> +		else

>> +			val->intval = POWER_SUPPLY_HEALTH_GOOD;

>> +

>> +		switch (state.health) {

>> +		case BQ25790_TEMP_HOT:

>> +			val->intval = POWER_SUPPLY_HEALTH_HOT;

>> +			break;

>> +		case BQ25790_TEMP_WARM:

>> +			val->intval = POWER_SUPPLY_HEALTH_WARM;

>> +			break;

>> +		case BQ25790_TEMP_COOL:

>> +			val->intval = POWER_SUPPLY_HEALTH_COOL;

>> +			break;

>> +		case BQ25790_TEMP_COLD:

>> +			val->intval = POWER_SUPPLY_HEALTH_COLD;

>> +			break;

>> +		}

>> +		break;

>> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:

>> +		ret = bq25790_get_ichg_curr(bq);

>> +		if (ret < 0)

>> +			return ret;

>> +

>> +		val->intval = ret;

>> +		break;

>> +

>> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:

>> +		ret = bq25790_get_chrg_volt(bq);

>> +		if (ret < 0)

>> +			return ret;

>> +

>> +		val->intval = ret;

>> +		break;

>> +

>> +	case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:

>> +		ret = bq25790_get_prechrg_curr(bq);

>> +		if (ret < 0)

>> +			return ret;

>> +

>> +		val->intval = ret;

>> +		break;

>> +	case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:

>> +		ret = bq25790_get_term_curr(bq);

>> +		if (ret < 0)

>> +			return ret;

>> +

>> +		val->intval = ret;

>> +		break;

>> +	case POWER_SUPPLY_PROP_VOLTAGE_NOW:

>> +		val->intval = state.vbus_adc;

>> +		break;

>> +	case POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT:

>> +		ret = bq25790_get_input_volt_lim(bq);

>> +		if (ret < 0)

>> +			return ret;

>> +

>> +		val->intval = ret;

>> +		break;

>> +	case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:

>> +		ret = bq25790_get_input_curr_lim(bq);

>> +		if (ret < 0)

>> +			return ret;

>> +

>> +		val->intval = ret;

>> +		break;

>> +	default:

>> +		return -EINVAL;

>> +	}

>> +

>> +	return ret;

>> +}

>> +

>> +static int bq25790_battery_get_property(struct power_supply *psy,

>> +				       enum power_supply_property psp,

>> +				       union power_supply_propval *val)

>> +{

>> +	struct bq25790_device *bq = power_supply_get_drvdata(psy);

>> +	struct bq25790_state state;

>> +	int ret = 0;

>> +

>> +	mutex_lock(&bq->lock);

>> +	ret = bq25790_get_state(bq, &state);

>> +	mutex_unlock(&bq->lock);

> another useless lock, since state is also a function local variable

> in this place.

ACK
>

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = regmap_update_bits(bq->regmap, BQ25790_ADC_CTRL,

>> +				 BQ25790_ADC_EN, BQ25790_ADC_EN);

>> +	if (ret)

>> +		return ret;

> why is ADC enabled here?

This will occur in the bq25790_get_state() instead
>

>> +

>> +	switch (psp) {

> please also be consistent with newline after break in this

> switch statement.

ACK
>

>> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX:

>> +		val->intval = bq->init_data.ichg_max;

>> +		break;

>> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX:

>> +		val->intval = bq->init_data.vreg_max;

>> +		break;

>> +

>> +	case POWER_SUPPLY_PROP_VOLTAGE_NOW:

>> +		val->intval = state.vbat_adc;

>> +		break;

>> +	case POWER_SUPPLY_PROP_CURRENT_NOW:

>> +		val->intval = state.ibat_adc;

>> +		break;

>> +	default:

>> +		return -EINVAL;

>> +	}

>> +

>> +	return ret;

>> +}

>> +

>> +static bool bq25790_state_changed(struct bq25790_device *bq,

>> +				  struct bq25790_state *new_state)

>> +{

>> +	struct bq25790_state old_state;

>> +

>> +	mutex_lock(&bq->lock);

>> +	old_state = bq->state;

>> +	mutex_unlock(&bq->lock);

>> +

>> +	return (old_state.chrg_status != new_state->chrg_status ||

>> +		old_state.chrg_fault != new_state->chrg_fault	||

>> +		old_state.online != new_state->online		||

>> +		old_state.health != new_state->health	||

>> +		old_state.fault_0 != new_state->fault_0 ||

>> +		old_state.fault_1 != new_state->fault_1 ||

>> +		old_state.chrg_type != new_state->chrg_type ||

>> +		old_state.vsys_status != new_state->vsys_status ||

>> +		old_state.vbat_adc != new_state->vbat_adc ||

>> +		old_state.vbus_adc != new_state->vbus_adc ||

>> +		old_state.ibat_adc != new_state->ibat_adc);

>> +}

>> +

>> +static irqreturn_t bq25790_irq_handler_thread(int irq, void *private)

>> +{

>> +	struct bq25790_device *bq = private;

>> +	struct bq25790_state state;

>> +	int ret;

>> +

>> +	ret = bq25790_get_state(bq, &state);

>> +	if (ret < 0)

>> +		goto irq_out;

>> +

>> +	if (!bq25790_state_changed(bq, &state))

>> +		goto irq_out;

>> +

>> +	mutex_lock(&bq->lock);

>> +	bq->state = state;

>> +	mutex_unlock(&bq->lock);

>> +

>> +	power_supply_changed(bq->charger);

>> +

>> +irq_out:

>> +	return IRQ_HANDLED;

>> +}

>> +

>> +static enum power_supply_property bq25790_power_supply_props[] = {

>> +	POWER_SUPPLY_PROP_MANUFACTURER,

>> +	POWER_SUPPLY_PROP_MODEL_NAME,

>> +	POWER_SUPPLY_PROP_STATUS,

>> +	POWER_SUPPLY_PROP_ONLINE,

>> +	POWER_SUPPLY_PROP_HEALTH,

>> +	POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT,

>> +	POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE,

>> +	POWER_SUPPLY_PROP_PRECHARGE_CURRENT,

>> +	POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT,

>> +	POWER_SUPPLY_PROP_VOLTAGE_NOW,

>> +	POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT,

>> +	POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT,

>> +	POWER_SUPPLY_PROP_CHARGE_TYPE,

>> +	POWER_SUPPLY_PROP_USB_TYPE,

>> +};

>> +

>> +static enum power_supply_property bq25790_battery_props[] = {

>> +	POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX,

>> +	POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX,

>> +	POWER_SUPPLY_PROP_VOLTAGE_NOW,

>> +	POWER_SUPPLY_PROP_CURRENT_NOW,

>> +};

>> +

>> +static char *bq25790_charger_supplied_to[] = {

>> +	"main-battery",

>> +};

> why do you need this? Usually this information should come from

> platform data (e.g. DT).

ACK
>

>> +static int bq25790_property_is_writeable(struct power_supply *psy,

>> +					 enum power_supply_property prop)

>> +{

>> +	switch (prop) {

>> +	case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:

>> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:

>> +	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:

>> +	case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:

>> +	case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:

>> +	case POWER_SUPPLY_PROP_STATUS:

>> +	case POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT:

>> +		return true;

>> +	default:

>> +		return false;

>> +	}

>> +}

>> +

>> +static const struct power_supply_desc bq25790_power_supply_desc = {

>> +	.name = "bq25790-charger",

>> +	.type = POWER_SUPPLY_TYPE_USB,

>> +	.usb_types = bq25790_usb_type,

>> +	.num_usb_types = ARRAY_SIZE(bq25790_usb_type),

>> +	.properties = bq25790_power_supply_props,

>> +	.num_properties = ARRAY_SIZE(bq25790_power_supply_props),

>> +	.get_property = bq25790_get_property,

>> +	.set_property = bq25790_set_property,

>> +	.property_is_writeable = bq25790_property_is_writeable,

>> +};

>> +

>> +static const struct power_supply_desc bq25790_battery_desc = {

>> +	.name = "bq25790-battery",

>> +	.type = POWER_SUPPLY_TYPE_BATTERY,

>> +	.get_property = bq25790_battery_get_property,

>> +	.properties = bq25790_battery_props,

>> +	.num_properties = ARRAY_SIZE(bq25790_battery_props),

>> +};

>> +

>> +static bool bq25790_is_volatile_reg(struct device *dev, unsigned int reg)

>> +{

>> +	switch (reg) {

>> +	case BQ25790_ICO_I_LIM...BQ25790_FAULT_FLAG_1:

>> +	case BQ25790_ADC_IBUS...BQ25790_ADC_DM:

>> +	case BQ25790_CHRG_CTRL_0:

>> +		return true;

>> +	default:

>> +		return false;

>> +	}

>> +}

>> +

>> +static const struct regmap_config bq25790_regmap_config = {

>> +	.reg_bits = 8,

>> +	.val_bits = 8,

>> +

>> +	.max_register = BQ25790_PART_INFO,

>> +	.reg_defaults	= bq25790_reg_defs,

>> +	.num_reg_defaults = ARRAY_SIZE(bq25790_reg_defs),

>> +	.cache_type = REGCACHE_RBTREE,

> There are no holes in the register map, so REGCACHE_FLAT

> should be enough.

ACK
>

>> +	.volatile_reg = bq25790_is_volatile_reg,

>> +};

>> +

>> +static int bq25790_power_supply_init(struct bq25790_device *bq,

>> +							struct device *dev)

>> +{

>> +	struct power_supply_config psy_cfg = { .drv_data = bq,

>> +						.of_node = dev->of_node, };

>> +

>> +	psy_cfg.supplied_to = bq25790_charger_supplied_to;

>> +	psy_cfg.num_supplicants = ARRAY_SIZE(bq25790_charger_supplied_to);

>> +

>> +	bq->charger = devm_power_supply_register(bq->dev,

>> +						 &bq25790_power_supply_desc,

>> +						 &psy_cfg);

>> +	if (IS_ERR(bq->charger))

>> +		return -EINVAL;

>> +

>> +	bq->battery = devm_power_supply_register(bq->dev,

>> +						      &bq25790_battery_desc,

>> +						      &psy_cfg);

>> +	if (IS_ERR(bq->battery))

>> +		return -EINVAL;

>> +	return 0;

>> +}

>> +

>> +static int bq25790_hw_init(struct bq25790_device *bq)

>> +{

>> +	struct power_supply_battery_info bat_info = { };

>> +	int wd_reg_val = BQ25790_WATCHDOG_DIS;

>> +	int wd_max_val = BQ25790_NUM_WD_VAL - 1;

>> +	int ret = 0;

>> +	int i;

>> +

>> +	if (bq->watchdog_timer) {

>> +		if (bq->watchdog_timer >= bq25790_watchdog_time[wd_max_val])

>> +			wd_reg_val = wd_max_val;

>> +		else {

>> +			for (i = 0; i < wd_max_val; i++) {

>> +				if (bq->watchdog_timer > bq25790_watchdog_time[i] &&

>> +				    bq->watchdog_timer < bq25790_watchdog_time[i + 1]) {

>> +					wd_reg_val = i;

>> +					break;

>> +				}

>> +			}

>> +		}

>> +	}

>> +

>> +	ret = regmap_update_bits(bq->regmap, BQ25790_CHRG_CTRL_1,

>> +				 BQ25790_WATCHDOG_MASK, wd_reg_val);

>> +	if (ret)

>> +		return ret;

>> +

>> +	ret = power_supply_get_battery_info(bq->charger, &bat_info);

>> +	if (ret) {

>> +		dev_warn(bq->dev, "battery info missing, default values will be applied\n");

>> +

>> +		bat_info.constant_charge_current_max_ua =

>> +							BQ25790_ICHRG_I_DEF_uA;

>> +

>> +		bat_info.constant_charge_voltage_max_uv =

>> +							BQ25790_VREG_V_DEF_uV;

>> +

>> +		bat_info.precharge_current_ua = BQ25790_PRECHRG_I_DEF_uA;

>> +		bat_info.charge_term_current_ua = BQ25790_TERMCHRG_I_DEF_uA;

>> +		bq->init_data.ichg_max = BQ25790_ICHRG_I_MAX_uA;

>> +		bq->init_data.vreg_max = BQ25790_VREG_V_MAX_uV;

>> +	} else {

>> +		bq->init_data.ichg_max =

>> +				bat_info.constant_charge_current_max_ua;

>> +

>> +		bq->init_data.vreg_max =

>> +				bat_info.constant_charge_voltage_max_uv;

>> +	}

>> +

>> +	ret = bq25790_set_ichrg_curr(bq,

>> +				bat_info.constant_charge_current_max_ua);

>> +	if (ret)

>> +		goto err_out;

>> +

>> +	ret = bq25790_set_prechrg_curr(bq, bat_info.precharge_current_ua);

>> +	if (ret)

>> +		goto err_out;

>> +

>> +	ret = bq25790_set_chrg_volt(bq,

>> +				bat_info.constant_charge_voltage_max_uv);

>> +	if (ret)

>> +		goto err_out;

>> +

>> +	ret = bq25790_set_term_curr(bq, bat_info.charge_term_current_ua);

>> +	if (ret)

>> +		goto err_out;

>> +

>> +	ret = bq25790_set_input_volt_lim(bq, bq->init_data.vlim);

>> +	if (ret)

>> +		goto err_out;

>> +

>> +	ret = bq25790_set_input_curr_lim(bq, bq->init_data.ilim);

>> +	if (ret)

>> +		goto err_out;

>> +

>> +err_out:

>> +	return ret;

> You need to call power_supply_put_battery_info(), since the

> struct contains some pointers with allocated data.

ACK
>

>> +}

>> +

>> +static int bq25790_parse_dt(struct bq25790_device *bq)

>> +{

>> +	int ret;

>> +

>> +	ret = device_property_read_u32(bq->dev, "ti,watchdog-timeout-ms",

>> +				       &bq->watchdog_timer);

>> +	if (ret)

>> +		bq->watchdog_timer = BQ25790_WATCHDOG_DIS;

>> +

>> +	if (bq->watchdog_timer > BQ25790_WATCHDOG_MAX ||

>> +	    bq->watchdog_timer < BQ25790_WATCHDOG_DIS)

>> +		return -EINVAL;

>> +

>> +	ret = device_property_read_u32(bq->dev,

>> +				       "input-voltage-limit-microvolt",

>> +				       &bq->init_data.vlim);

>> +	if (ret)

>> +		bq->init_data.vlim = BQ25790_VINDPM_DEF_uV;

>> +

>> +	if (bq->init_data.vlim > BQ25790_VINDPM_V_MAX_uV ||

>> +	    bq->init_data.vlim < BQ25790_VINDPM_V_MIN_uV)

>> +		return -EINVAL;

>> +

>> +	ret = device_property_read_u32(bq->dev,

>> +				       "input-current-limit-microamp",

>> +				       &bq->init_data.ilim);

>> +	if (ret)

>> +		bq->init_data.ilim = BQ25790_IINDPM_DEF_uA;

>> +

>> +	if (bq->init_data.ilim > BQ25790_IINDPM_I_MAX_uA ||

>> +	    bq->init_data.ilim < BQ25790_IINDPM_I_MIN_uA)

>> +		return -EINVAL;

>> +

>> +	return 0;

>> +}

>> +

>> +static int bq25790_probe(struct i2c_client *client,

>> +			 const struct i2c_device_id *id)

>> +{

>> +	struct device *dev = &client->dev;

>> +	struct bq25790_device *bq;

>> +	int ret;

>> +

>> +	bq = devm_kzalloc(dev, sizeof(*bq), GFP_KERNEL);

>> +	if (!bq)

>> +		return -ENOMEM;

>> +

>> +	bq->client = client;

>> +	bq->dev = dev;

>> +

>> +	mutex_init(&bq->lock);

>> +

>> +	strncpy(bq->model_name, id->name, I2C_NAME_SIZE);

>> +

>> +	bq->regmap = devm_regmap_init_i2c(client, &bq25790_regmap_config);

>> +	if (IS_ERR(bq->regmap)) {

>> +		dev_err(dev, "Failed to allocate register map\n");

>> +		return PTR_ERR(bq->regmap);

>> +	}

>> +

>> +	i2c_set_clientdata(client, bq);

>> +

>> +	ret = bq25790_parse_dt(bq);

>> +	if (ret) {

>> +		dev_err(dev, "Failed to read device tree properties%d\n", ret);

>> +		return ret;

>> +	}

>> +

>> +	/* OTG reporting */

>> +	bq->usb2_phy = devm_usb_get_phy(dev, USB_PHY_TYPE_USB2);

>> +	if (!IS_ERR_OR_NULL(bq->usb2_phy)) {

>> +		INIT_WORK(&bq->usb_work, bq25790_usb_work);

>> +		bq->usb_nb.notifier_call = bq25790_usb_notifier;

>> +		usb_register_notifier(bq->usb2_phy, &bq->usb_nb);

> You also need to unregister the usb notifiers on driver removal.

> I suggest to use devm_add_action_or_reset():

>

> static void bq25790_unregister_usb_notifier(void *data)

> {

>      struct bq25790_device *bq = (struct bq25790_device *) data;

>

>      if (!IS_ERR_OR_NULL(bq->usb2_phy))

>          usb_unregister_notifier(bq->usb2_phy, &bq->usb_nb);

>

>      if (!IS_ERR_OR_NULL(bq->usb3_phy))

>          usb_unregister_notifier(bq->usb3_phy, &bq->usb_nb);

> }

>

> ret = devm_add_action_or_reset(dev, bq25790_unregister_usb_notifier, bq)

> if (ret)

>      return ret;

ACK
>

> -- Sebastian

>

>> +	}

>> +

>> +	bq->usb3_phy = devm_usb_get_phy(dev, USB_PHY_TYPE_USB3);

>> +	if (!IS_ERR_OR_NULL(bq->usb3_phy)) {

>> +		INIT_WORK(&bq->usb_work, bq25790_usb_work);

>> +		bq->usb_nb.notifier_call = bq25790_usb_notifier;

>> +		usb_register_notifier(bq->usb3_phy, &bq->usb_nb);

>> +	}

>> +

>> +	if (client->irq) {

>> +		ret = devm_request_threaded_irq(dev, client->irq, NULL,

>> +						bq25790_irq_handler_thread,

>> +						IRQF_TRIGGER_FALLING |

>> +						IRQF_ONESHOT,

>> +						dev_name(&client->dev), bq);

>> +		if (ret)

>> +			goto error_out;

>> +	}

>> +

>> +	ret = bq25790_power_supply_init(bq, dev);

>> +	if (ret) {

>> +		dev_err(dev, "Failed to register power supply\n");

>> +		goto error_out;

>> +	}

>> +

>> +	ret = bq25790_hw_init(bq);

>> +	if (ret) {

>> +		dev_err(dev, "Cannot initialize the chip.\n");

>> +		goto error_out;

>> +	}

>> +

>> +	return ret;

>> +error_out:

>> +	if (!IS_ERR_OR_NULL(bq->usb2_phy))

>> +		usb_unregister_notifier(bq->usb2_phy, &bq->usb_nb);

>> +

>> +	if (!IS_ERR_OR_NULL(bq->usb3_phy))

>> +		usb_unregister_notifier(bq->usb3_phy, &bq->usb_nb);

>> +	return ret;

>> +}

>> +

>> +static const struct i2c_device_id bq25790_i2c_ids[] = {

>> +	{ BQ25790_NAME, 0 },

>> +	{ BQ25792_NAME, 1 },

>> +	{},

>> +};

>> +MODULE_DEVICE_TABLE(i2c, bq25790_i2c_ids);

>> +

>> +static const struct of_device_id bq25790_of_match[] = {

>> +	{ .compatible = "ti,bq25790", },

>> +	{ .compatible = "ti,bq25792", },

>> +	{ },

>> +};

>> +MODULE_DEVICE_TABLE(of, bq25790_of_match);

>> +

>> +static const struct acpi_device_id bq25790_acpi_match[] = {

>> +	{BQ25790_NAME, 0},

>> +	{BQ25792_NAME, 0},

>> +	{},

>> +};

>> +MODULE_DEVICE_TABLE(acpi, bq25790_acpi_match);

>> +

>> +static struct i2c_driver bq25790_driver = {

>> +	.driver = {

>> +		.name = "bq25790-charger",

>> +		.of_match_table = bq25790_of_match,

>> +		.acpi_match_table = ACPI_PTR(bq25790_acpi_match),

>> +	},

>> +	.probe = bq25790_probe,

>> +	.id_table = bq25790_i2c_ids,

>> +};

>> +module_i2c_driver(bq25790_driver);

>> +

>> +MODULE_AUTHOR("Dan Murphy <dmurphy@ti.com>");

>> +MODULE_AUTHOR("Ricardo Rivera-Matos <r-rivera-matos@ti.com>");

>> +MODULE_DESCRIPTION("bq25790 charger driver");

>> +MODULE_LICENSE("GPL v2");

>> diff --git a/drivers/power/supply/bq25790_charger.h b/drivers/power/supply/bq25790_charger.h

>> new file mode 100644

>> index 000000000000..77234e116a6b

>> --- /dev/null

>> +++ b/drivers/power/supply/bq25790_charger.h

>> @@ -0,0 +1,150 @@

>> +/* SPDX-License-Identifier: GPL-2.0-only */

>> +// BQ25790 Charger Driver

>> +// Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/

>> +

>> +#ifndef _BQ25790_CHARGER_H

>> +#define _BQ25790_CHARGER_H

>> +

>> +#define BQ25790_MANUFACTURER	"Texas Instruments"

>> +#define BQ25790_NAME		"bq25790"

>> +#define BQ25792_NAME		"bq25792"

>> +

>> +#define BQ25790_MIN_SYS_V	0x00

>> +#define BQ25790_CHRG_V_LIM_MSB	0x01

>> +#define BQ25790_CHRG_V_LIM_LSB	0x02

>> +#define BQ25790_CHRG_I_LIM_MSB	0x03

>> +#define BQ25790_CHRG_I_LIM_LSB	0x04

>> +#define BQ25790_INPUT_V_LIM	0x05

>> +#define BQ25790_INPUT_I_LIM_MSB	0x06

>> +#define BQ25790_INPUT_I_LIM_LSB	0x07

>> +#define BQ25790_PRECHRG_CTRL	0x08

>> +#define BQ25790_TERM_CTRL	0x09

>> +#define BQ25790_RECHRG_CTRL	0x0a

>> +#define BQ25790_VOTG_REG	0x0b

>> +#define BQ25790_IOTG_REG	0x0d

>> +#define BQ25790_TIMER_CTRL	0x0e

>> +#define BQ25790_CHRG_CTRL_0	0x0f

>> +#define BQ25790_CHRG_CTRL_1	0x10

>> +#define BQ25790_CHRG_CTRL_2	0x11

>> +#define BQ25790_CHRG_CTRL_3	0x12

>> +#define BQ25790_CHRG_CTRL_4	0x13

>> +#define BQ25790_CHRG_CTRL_5	0x14

>> +#define BQ25790_MPPT_CTRL	0x15

>> +#define BQ25790_TEMP_CTRL	0x16

>> +#define BQ25790_NTC_CTRL_0	0x17

>> +#define BQ25790_NTC_CTRL_1	0x18

>> +#define BQ25790_ICO_I_LIM	0x19

>> +#define BQ25790_CHRG_STAT_0	0x1b

>> +#define BQ25790_CHRG_STAT_1	0x1c

>> +#define BQ25790_CHRG_STAT_2	0x1d

>> +#define BQ25790_CHRG_STAT_3	0x1e

>> +#define BQ25790_CHRG_STAT_4	0x1f

>> +#define BQ25790_FAULT_STAT_0	0x20

>> +#define BQ25790_FAULT_STAT_1	0x21

>> +#define BQ25790_CHRG_FLAG_0	0x22

>> +#define BQ25790_CHRG_FLAG_1	0x23

>> +#define BQ25790_CHRG_FLAG_2	0x24

>> +#define BQ25790_CHRG_FLAG_3	0x25

>> +#define BQ25790_FAULT_FLAG_0	0x26

>> +#define BQ25790_FAULT_FLAG_1	0x27

>> +#define BQ25790_CHRG_MSK_0	0x28

>> +#define BQ25790_CHRG_MSK_1	0x29

>> +#define BQ25790_CHRG_MSK_2	0x2a

>> +#define BQ25790_CHRG_MSK_3	0x2b

>> +#define BQ25790_FAULT_MSK_0	0x2c

>> +#define BQ25790_FAULT_MSK_1	0x2d

>> +#define BQ25790_ADC_CTRL	0x2e

>> +#define BQ25790_FN_DISABE_0	0x2f

>> +#define BQ25790_FN_DISABE_1	0x30

>> +#define BQ25790_ADC_IBUS	0x31

>> +#define BQ25790_ADC_IBAT_MSB	0x33

>> +#define BQ25790_ADC_IBAT_LSB	0x34

>> +#define BQ25790_ADC_VBUS_MSB	0x35

>> +#define BQ25790_ADC_VBUS_LSB	0x36

>> +#define BQ25790_ADC_VAC1	0x37

>> +#define BQ25790_ADC_VAC2	0x39

>> +#define BQ25790_ADC_VBAT_MSB	0x3b

>> +#define BQ25790_ADC_VBAT_LSB	0x3c

>> +#define BQ25790_ADC_VSYS_MSB	0x3d

>> +#define BQ25790_ADC_VSYS_LSB	0x3e

>> +#define BQ25790_ADC_TS		0x3f

>> +#define BQ25790_ADC_TDIE	0x41

>> +#define BQ25790_ADC_DP		0x43

>> +#define BQ25790_ADC_DM		0x45

>> +#define BQ25790_DPDM_DRV	0x47

>> +#define BQ25790_PART_INFO	0x48

>> +

>> +#define BQ25790_CHRG_EN		BIT(5)

>> +#define BQ25790_ADC_EN		BIT(7)

>> +

>> +/* Charger Status 1 */

>> +#define BQ25790_CHG_STAT_MSK	GENMASK(7, 5)

>> +#define BQ25790_NOT_CHRGING	0

>> +#define BQ25790_TRICKLE_CHRG	BIT(5)

>> +#define BQ25790_PRECHRG		BIT(6)

>> +#define BQ25790_FAST_CHRG	(BIT(5) | BIT(6))

>> +#define BQ25790_TAPER_CHRG	BIT(7)

>> +#define BQ25790_TOP_OFF_CHRG	(BIT(6) | BIT(7))

>> +#define BQ25790_TERM_CHRG	(BIT(5) | BIT(6) | BIT(7))

>> +#define BQ25790_VBUS_PRESENT	BIT(0)

>> +

>> +#define BQ25790_VBUS_STAT_MSK	GENMASK(4, 1)

>> +#define BQ25790_USB_SDP		BIT(1)

>> +#define BQ25790_USB_CDP		BIT(2)

>> +#define BQ25790_USB_DCP		(BIT(1) | BIT(2))

>> +#define BQ25790_HVDCP		BIT(3)

>> +#define BQ25790_UNKNOWN_3A	(BIT(3) | BIT(1))

>> +#define BQ25790_NON_STANDARD	(BIT(3) | BIT(2))

>> +#define BQ25790_OTG_MODE	(BIT(3) | BIT(2) | BIT(1))

>> +#define BQ25790_UNQUAL_ADAPT	BIT(4)

>> +#define BQ25790_DIRECT_PWR	(BIT(4) | BIT(2) | BIT(1))

>> +

>> +/* Charger Status 4 */

>> +#define BQ25790_TEMP_HOT	BIT(0)

>> +#define BQ25790_TEMP_WARM	BIT(1)

>> +#define BQ25790_TEMP_COOL	BIT(2)

>> +#define BQ25790_TEMP_COLD	BIT(3)

>> +#define BQ25790_TEMP_MASK	GENMASK(3, 0)

>> +

>> +#define BQ25790_OTG_OVP		BIT(5)

>> +#define BQ25790_VSYS_OVP	BIT(6)

>> +#define BQ25790_VSYS_STAT	BIT(4)

>> +

>> +#define BQ25790_PRECHRG_CUR_MASK		GENMASK(5, 0)

>> +#define BQ25790_PRECHRG_CURRENT_STEP_uA		40000

>> +#define BQ25790_PRECHRG_I_MIN_uA		40000

>> +#define BQ25790_PRECHRG_I_MAX_uA		2000000

>> +#define BQ25790_PRECHRG_I_DEF_uA		120000

>> +#define BQ25790_TERMCHRG_CUR_MASK		GENMASK(4, 0)

>> +#define BQ25790_TERMCHRG_CURRENT_STEP_uA	40000

>> +#define BQ25790_TERMCHRG_I_MIN_uA		40000

>> +#define BQ25790_TERMCHRG_I_MAX_uA		1000000

>> +#define BQ25790_TERMCHRG_I_DEF_uA		200000

>> +#define BQ25790_ICHRG_CURRENT_STEP_uA		10000

>> +#define BQ25790_ICHRG_I_MIN_uA			50000

>> +#define BQ25790_ICHRG_I_MAX_uA			5000000

>> +#define BQ25790_ICHRG_I_DEF_uA			1000000

>> +

>> +#define BQ25790_VREG_V_MAX_uV	18800000

>> +#define BQ25790_VREG_V_MIN_uV	3000000

>> +#define BQ25790_VREG_V_DEF_uV	3600000

>> +#define BQ25790_VREG_V_STEP_uV	10000

>> +

>> +#define BQ25790_IINDPM_I_MIN_uA	100000

>> +#define BQ25790_IINDPM_I_MAX_uA	3300000

>> +#define BQ25790_IINDPM_STEP_uA	10000

>> +#define BQ25790_IINDPM_DEF_uA	1000000

>> +

>> +#define BQ25790_VINDPM_V_MIN_uV 3600000

>> +#define BQ25790_VINDPM_V_MAX_uV 22000000

>> +#define BQ25790_VINDPM_STEP_uV	100000

>> +#define BQ25790_VINDPM_DEF_uV	3600000

>> +

>> +#define BQ25790_ADC_VOLT_STEP_uV	1000

>> +#define BQ25790_ADC_CURR_STEP_uA	1000

>> +

>> +#define BQ25790_WATCHDOG_MASK	GENMASK(2, 0)

>> +#define BQ25790_WATCHDOG_DIS	0

>> +#define BQ25790_WATCHDOG_MAX	160000

>> +

>> +#endif /* _BQ25790_CHARGER_H */

>> -- 

>> 2.28.0.585.ge1cfff676549

>>

Ricardo
diff mbox series

Patch

diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig
index eec646c568b7..1cc361238d9a 100644
--- a/drivers/power/supply/Kconfig
+++ b/drivers/power/supply/Kconfig
@@ -628,6 +628,14 @@  config CHARGER_BQ2515X
 	  rail, ADC for battery and system monitoring, and push-button
 	  controller.
 
+config CHARGER_BQ25790
+	tristate "TI BQ25790 battery charger driver"
+	depends on I2C
+	depends on GPIOLIB || COMPILE_TEST
+	select REGMAP_I2C
+	help
+	  Say Y to enable support for the TI BQ25790 battery charger.
+
 config CHARGER_BQ25890
 	tristate "TI BQ25890 battery charger driver"
 	depends on I2C
diff --git a/drivers/power/supply/Makefile b/drivers/power/supply/Makefile
index dd4b86318cd9..29b71cc92550 100644
--- a/drivers/power/supply/Makefile
+++ b/drivers/power/supply/Makefile
@@ -83,6 +83,7 @@  obj-$(CONFIG_CHARGER_BQ24190)	+= bq24190_charger.o
 obj-$(CONFIG_CHARGER_BQ24257)	+= bq24257_charger.o
 obj-$(CONFIG_CHARGER_BQ24735)	+= bq24735-charger.o
 obj-$(CONFIG_CHARGER_BQ2515X)	+= bq2515x_charger.o
+obj-$(CONFIG_CHARGER_BQ25790)	+= bq25790_charger.o
 obj-$(CONFIG_CHARGER_BQ25890)	+= bq25890_charger.o
 obj-$(CONFIG_CHARGER_BQ25980)	+= bq25980_charger.o
 obj-$(CONFIG_CHARGER_SMB347)	+= smb347-charger.o
diff --git a/drivers/power/supply/bq25790_charger.c b/drivers/power/supply/bq25790_charger.c
new file mode 100644
index 000000000000..838a49c2f9f0
--- /dev/null
+++ b/drivers/power/supply/bq25790_charger.c
@@ -0,0 +1,1121 @@ 
+// SPDX-License-Identifier: GPL-2.0
+// BQ25790 driver
+// Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/
+
+#include <linux/module.h>
+#include <linux/i2c.h>
+#include <linux/power_supply.h>
+#include <linux/regmap.h>
+#include <linux/types.h>
+#include <linux/gpio/consumer.h>
+#include <linux/interrupt.h>
+#include <linux/delay.h>
+#include <linux/usb/phy.h>
+
+#include <linux/acpi.h>
+#include <linux/of.h>
+
+#include "bq25790_charger.h"
+
+#define BQ25790_NUM_WD_VAL	8
+
+struct bq25790_init_data {
+	u32 ichg;
+	u32 ilim;
+	u32 vreg;
+	u32 iterm;
+	u32 iprechg;
+	u32 vlim;
+	u32 ichg_max;
+	u32 vreg_max;
+};
+
+struct bq25790_state {
+	bool online;
+	u8 chrg_status;
+	u8 chrg_type;
+	u8 health;
+	u8 chrg_fault;
+	u8 vsys_status;
+	u8 vbus_status;
+	u8 fault_0;
+	u8 fault_1;
+	u32 vbat_adc;
+	u32 vbus_adc;
+	u32 ibat_adc;
+};
+
+struct bq25790_device {
+	struct i2c_client *client;
+	struct device *dev;
+	struct power_supply *charger;
+	struct power_supply *battery;
+	struct mutex lock;
+
+	struct usb_phy *usb2_phy;
+	struct usb_phy *usb3_phy;
+	struct notifier_block usb_nb;
+	struct work_struct usb_work;
+	unsigned long usb_event;
+	struct regmap *regmap;
+
+	char model_name[I2C_NAME_SIZE];
+	int device_id;
+
+	struct bq25790_init_data init_data;
+	struct bq25790_state state;
+	int watchdog_timer;
+};
+
+static struct reg_default bq25790_reg_defs[] = {
+	{BQ25790_INPUT_V_LIM, 0x24},
+	{BQ25790_INPUT_I_LIM_MSB, 0x01},
+	{BQ25790_INPUT_I_LIM_LSB, 0x2c},
+	{BQ25790_PRECHRG_CTRL, 0xc3},
+	{BQ25790_TERM_CTRL, 0x5},
+	{BQ25790_VOTG_REG, 0xdc},
+	{BQ25790_IOTG_REG, 0x4b},
+	{BQ25790_TIMER_CTRL, 0x3d},
+	{BQ25790_CHRG_CTRL_0, 0xa2},
+	{BQ25790_CHRG_CTRL_1, 0x85},
+	{BQ25790_CHRG_CTRL_2, 0x40},
+	{BQ25790_CHRG_CTRL_3, 0x12},
+	{BQ25790_CHRG_CTRL_5, 0x16},
+	{BQ25790_MPPT_CTRL, 0xaa},
+	{BQ25790_TEMP_CTRL, 0xc0},
+	{BQ25790_NTC_CTRL_0, 0x7a},
+	{BQ25790_NTC_CTRL_1, 0x54},
+	{BQ25790_ICO_I_LIM, 0x0},
+	{BQ25790_CHRG_STAT_0, 0x0},
+	{BQ25790_CHRG_STAT_1, 0x0},
+	{BQ25790_CHRG_STAT_2, 0x0},
+	{BQ25790_CHRG_STAT_3, 0x0},
+	{BQ25790_CHRG_STAT_4, 0x0},
+	{BQ25790_FAULT_STAT_0, 0x0},
+	{BQ25790_FAULT_STAT_1, 0x0},
+	{BQ25790_CHRG_FLAG_0, 0x0},
+	{BQ25790_CHRG_FLAG_1, 0x0},
+	{BQ25790_CHRG_FLAG_2, 0x0},
+	{BQ25790_CHRG_FLAG_3, 0x0},
+	{BQ25790_FAULT_FLAG_0, 0x0},
+	{BQ25790_FAULT_FLAG_1, 0x0},
+	{BQ25790_CHRG_MSK_0, 0x0},
+	{BQ25790_CHRG_MSK_1, 0x0},
+	{BQ25790_CHRG_MSK_2, 0x0},
+	{BQ25790_CHRG_MSK_3, 0x0},
+	{BQ25790_FAULT_MSK_0, 0x0},
+	{BQ25790_FAULT_MSK_1, 0x0},
+	{BQ25790_ADC_CTRL, 0x30},
+	{BQ25790_FN_DISABE_0, 0x0},
+	{BQ25790_FN_DISABE_1, 0x0},
+	{BQ25790_ADC_IBUS, 0x0},
+	{BQ25790_ADC_IBAT_MSB, 0x0},
+	{BQ25790_ADC_IBAT_LSB, 0x0},
+	{BQ25790_ADC_VBUS_MSB, 0x0},
+	{BQ25790_ADC_VBUS_LSB, 0x0},
+	{BQ25790_ADC_VAC1, 0x0},
+	{BQ25790_ADC_VAC2, 0x0},
+	{BQ25790_ADC_VBAT_MSB, 0x0},
+	{BQ25790_ADC_VBAT_LSB, 0x0},
+	{BQ25790_ADC_VBUS_MSB, 0x0},
+	{BQ25790_ADC_VBUS_LSB, 0x0},
+	{BQ25790_ADC_TS, 0x0},
+	{BQ25790_ADC_TDIE, 0x0},
+	{BQ25790_ADC_DP, 0x0},
+	{BQ25790_ADC_DM, 0x0},
+	{BQ25790_DPDM_DRV, 0x0},
+	{BQ25790_PART_INFO, 0x0},
+};
+
+static int bq25790_watchdog_time[BQ25790_NUM_WD_VAL] = {0, 500, 1000, 2000,
+							20000, 40000, 80000,
+							160000};
+
+static enum power_supply_usb_type bq25790_usb_type[] = {
+	POWER_SUPPLY_USB_TYPE_PD_DRP,
+	POWER_SUPPLY_USB_TYPE_SDP,
+	POWER_SUPPLY_USB_TYPE_CDP,
+	POWER_SUPPLY_USB_TYPE_DCP,
+	POWER_SUPPLY_USB_TYPE_UNKNOWN,
+};
+
+static int bq25790_usb_notifier(struct notifier_block *nb, unsigned long val,
+				void *priv)
+{
+	struct bq25790_device *bq =
+			container_of(nb, struct bq25790_device, usb_nb);
+
+	bq->usb_event = val;
+	queue_work(system_power_efficient_wq, &bq->usb_work);
+
+	return NOTIFY_OK;
+}
+
+static void bq25790_usb_work(struct work_struct *data)
+{
+	struct bq25790_device *bq =
+			container_of(data, struct bq25790_device, usb_work);
+
+	switch (bq->usb_event) {
+	case USB_EVENT_ID:
+		break;
+
+	case USB_EVENT_NONE:
+		power_supply_changed(bq->charger);
+		break;
+	}
+
+	return;
+}
+
+static int bq25790_get_vbat_adc(struct bq25790_device *bq)
+{
+	int ret;
+	int vbat_adc_lsb, vbat_adc_msb;
+	u16 vbat_adc;
+
+	ret = regmap_read(bq->regmap, BQ25790_ADC_VBAT_MSB, &vbat_adc_msb);
+	if (ret)
+		return ret;
+
+	ret = regmap_read(bq->regmap, BQ25790_ADC_VBAT_LSB, &vbat_adc_lsb);
+	if (ret)
+		return ret;
+
+	vbat_adc = (vbat_adc_msb << 8) | vbat_adc_lsb;
+
+	return vbat_adc * BQ25790_ADC_VOLT_STEP_uV;
+}
+
+static int bq25790_get_vbus_adc(struct bq25790_device *bq)
+{
+	int ret;
+	int vbus_adc_lsb, vbus_adc_msb;
+	u16 vbus_adc;
+
+	ret = regmap_update_bits(bq->regmap, BQ25790_ADC_CTRL,
+				 BQ25790_ADC_EN, BQ25790_ADC_EN);
+	if (ret)
+		return ret;
+
+	ret = regmap_read(bq->regmap, BQ25790_ADC_VBUS_MSB, &vbus_adc_msb);
+	if (ret)
+		return ret;
+
+	ret = regmap_read(bq->regmap, BQ25790_ADC_VBUS_LSB, &vbus_adc_lsb);
+	if (ret)
+		return ret;
+
+	vbus_adc = (vbus_adc_msb << 8) | vbus_adc_lsb;
+
+	return vbus_adc * BQ25790_ADC_VOLT_STEP_uV;
+}
+
+static int bq25790_get_ibat_adc(struct bq25790_device *bq)
+{
+	int ret;
+	int ibat_adc_lsb, ibat_adc_msb;
+	u16 ibat_adc;
+
+	ret = regmap_read(bq->regmap, BQ25790_ADC_IBAT_MSB, &ibat_adc_msb);
+	if (ret)
+		return ret;
+
+	ret = regmap_read(bq->regmap, BQ25790_ADC_IBAT_LSB, &ibat_adc_lsb);
+	if (ret)
+		return ret;
+
+	ibat_adc = (ibat_adc_msb << 8) | ibat_adc_lsb;
+
+	return ibat_adc * BQ25790_ADC_CURR_STEP_uA;
+}
+
+static int bq25790_get_term_curr(struct bq25790_device *bq)
+{
+	int ret;
+	int reg_val;
+
+	ret = regmap_read(bq->regmap, BQ25790_TERM_CTRL, &reg_val);
+	if (ret)
+		return ret;
+
+	reg_val &= BQ25790_TERMCHRG_CUR_MASK;
+
+	return reg_val * BQ25790_TERMCHRG_CURRENT_STEP_uA;
+}
+
+static int bq25790_get_prechrg_curr(struct bq25790_device *bq)
+{
+	int ret;
+	int reg_val;
+
+	ret = regmap_read(bq->regmap, BQ25790_PRECHRG_CTRL, &reg_val);
+	if (ret)
+		return ret;
+
+	reg_val &= BQ25790_PRECHRG_CUR_MASK;
+
+	return reg_val * BQ25790_PRECHRG_CURRENT_STEP_uA;
+}
+
+static int bq25790_get_ichg_curr(struct bq25790_device *bq)
+{
+	int ret;
+	int ichg, ichg_lsb, ichg_msb;
+
+	ret = regmap_read(bq->regmap, BQ25790_CHRG_I_LIM_LSB, &ichg_lsb);
+	if (ret)
+		return ret;
+
+	ret = regmap_read(bq->regmap, BQ25790_CHRG_I_LIM_MSB, &ichg_msb);
+	if (ret)
+		return ret;
+
+	ichg = (ichg_msb << 8) | ichg_lsb;
+
+	return ichg * BQ25790_ICHRG_CURRENT_STEP_uA;
+}
+
+static int bq25790_set_term_curr(struct bq25790_device *bq, int term_current)
+{
+	int reg_val;
+
+	if (term_current < BQ25790_TERMCHRG_I_MIN_uA ||
+	    term_current > BQ25790_TERMCHRG_I_MAX_uA)
+		return -EINVAL;
+
+	reg_val = term_current / BQ25790_TERMCHRG_CURRENT_STEP_uA;
+
+	return regmap_update_bits(bq->regmap, BQ25790_TERM_CTRL,
+				  BQ25790_TERMCHRG_CUR_MASK, reg_val);
+}
+
+static int bq25790_set_prechrg_curr(struct bq25790_device *bq, int pre_current)
+{
+	int reg_val;
+
+	if (pre_current < BQ25790_PRECHRG_I_MIN_uA ||
+	    pre_current > BQ25790_PRECHRG_I_MAX_uA)
+		return -EINVAL;
+
+	reg_val = pre_current / BQ25790_PRECHRG_CURRENT_STEP_uA;
+
+	return regmap_update_bits(bq->regmap, BQ25790_PRECHRG_CTRL,
+				  BQ25790_PRECHRG_CUR_MASK, reg_val);
+}
+
+static int bq25790_set_ichrg_curr(struct bq25790_device *bq, int chrg_curr)
+{
+	int ret;
+	int ichg, ichg_msb, ichg_lsb;
+
+	if (chrg_curr < BQ25790_ICHRG_I_MIN_uA ||
+	    chrg_curr > BQ25790_ICHRG_I_MAX_uA)
+		return -EINVAL;
+
+	ichg = chrg_curr / BQ25790_ICHRG_CURRENT_STEP_uA;
+	ichg_msb = (ichg >> 8) & 0xff;
+	ret = regmap_write(bq->regmap, BQ25790_CHRG_I_LIM_MSB, ichg_msb);
+	if (ret)
+		return ret;
+
+	ichg_lsb = ichg & 0xff;
+
+	return regmap_write(bq->regmap, BQ25790_CHRG_I_LIM_LSB, ichg_lsb);
+}
+
+static int bq25790_set_chrg_volt(struct bq25790_device *bq, int chrg_volt)
+{
+	int vlim_lsb, vlim_msb, vlim;
+	int ret;
+
+	if (chrg_volt < BQ25790_VREG_V_MIN_uV ||
+	    chrg_volt > BQ25790_VREG_V_MAX_uV)
+		return -EINVAL;
+
+	vlim = chrg_volt / BQ25790_VREG_V_STEP_uV;
+	vlim_msb = (vlim >> 8) & 0xff;
+	ret = regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_MSB, vlim_msb);
+	if (ret)
+		return ret;
+
+	vlim_lsb = vlim & 0xff;
+
+	return regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_LSB, vlim_lsb);
+}
+
+static int bq25790_get_chrg_volt(struct bq25790_device *bq)
+{
+	int ret;
+	int vlim_lsb, vlim_msb, chrg_volt;
+
+	ret = regmap_read(bq->regmap, BQ25790_CHRG_V_LIM_MSB, &vlim_msb);
+	if (ret)
+		return ret;
+
+	ret = regmap_read(bq->regmap, BQ25790_CHRG_V_LIM_LSB, &vlim_lsb);
+	if (ret)
+		return ret;
+
+	chrg_volt = (vlim_msb << 8) | vlim_lsb;
+
+	return chrg_volt * BQ25790_VREG_V_STEP_uV;
+}
+
+static int bq25790_set_input_volt_lim(struct bq25790_device *bq, int vindpm)
+{
+	int ret;
+	int vlim_lsb, vlim_msb;
+	int vlim;
+
+	if (vindpm < BQ25790_VINDPM_V_MIN_uV ||
+	    vindpm > BQ25790_VINDPM_V_MAX_uV)
+		return -EINVAL;
+
+	vlim = vindpm / BQ25790_VINDPM_STEP_uV;
+
+	vlim_msb = (vlim >> 8) & 0xff;
+
+	ret = regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_MSB, vlim_msb);
+	if (ret)
+		return ret;
+
+	vlim_lsb = vlim & 0xff;
+
+	return regmap_write(bq->regmap, BQ25790_CHRG_V_LIM_LSB, vlim_lsb);
+}
+
+static int bq25790_get_input_volt_lim(struct bq25790_device *bq)
+{
+	int ret;
+	int vlim;
+
+	ret = regmap_read(bq->regmap, BQ25790_INPUT_V_LIM, &vlim);
+	if (ret)
+		return ret;
+
+	return vlim * BQ25790_VINDPM_STEP_uV;
+}
+
+static int bq25790_set_input_curr_lim(struct bq25790_device *bq, int iindpm)
+{
+	int ret;
+	int ilim, ilim_lsb, ilim_msb;
+
+	if (iindpm < BQ25790_IINDPM_I_MIN_uA ||
+	    iindpm > BQ25790_IINDPM_I_MAX_uA)
+		return -EINVAL;
+
+	ilim = iindpm / BQ25790_IINDPM_STEP_uA;
+	ilim_msb = (ilim >> 8) & 0xff;
+
+	ret = regmap_write(bq->regmap, BQ25790_INPUT_I_LIM_MSB, ilim_msb);
+	if (ret)
+		return ret;
+
+	ilim_lsb = ilim & 0xff;
+
+	return regmap_write(bq->regmap, BQ25790_INPUT_I_LIM_LSB, ilim_lsb);
+}
+
+static int bq25790_get_input_curr_lim(struct bq25790_device *bq)
+{
+	int ret;
+	int ilim_msb, ilim_lsb;
+	u16 ilim;
+
+	ret = regmap_read(bq->regmap, BQ25790_INPUT_I_LIM_MSB, &ilim_msb);
+	if (ret)
+		return ret;
+
+	ret = regmap_read(bq->regmap, BQ25790_INPUT_I_LIM_LSB, &ilim_lsb);
+	if (ret)
+		return ret;
+
+	ilim = (ilim_msb << 8) | ilim_lsb;
+
+	return ilim * BQ25790_IINDPM_STEP_uA;
+}
+
+static int bq25790_get_state(struct bq25790_device *bq,
+			     struct bq25790_state *state)
+{
+	int chrg_stat_0, chrg_stat_1, chrg_stat_3, chrg_stat_4;
+	int chrg_ctrl_0, fault_0, fault_1;
+	int ret;
+
+	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_0, &chrg_stat_0);
+	if (ret)
+		return ret;
+
+	state->vbus_status = chrg_stat_0 & BQ25790_VBUS_PRESENT;
+
+	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_1, &chrg_stat_1);
+	if (ret)
+		return ret;
+
+	ret = regmap_read(bq->regmap, BQ25790_CHRG_CTRL_0, &chrg_ctrl_0);
+	if (ret)
+		return ret;
+
+	if (chrg_ctrl_0 & BQ25790_CHRG_EN)
+		state->chrg_status = chrg_stat_1 & BQ25790_CHG_STAT_MSK;
+	else
+		state->chrg_status = BQ25790_NOT_CHRGING;
+
+	state->chrg_type = chrg_stat_1 & BQ25790_VBUS_STAT_MSK;
+
+	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_4, &chrg_stat_4);
+	if (ret)
+		return ret;
+
+	state->health = chrg_stat_4 & BQ25790_TEMP_MASK;
+
+	ret = regmap_read(bq->regmap, BQ25790_FAULT_STAT_0, &fault_0);
+	if (ret)
+		return ret;
+
+	state->fault_0 = fault_0;
+
+	ret = regmap_read(bq->regmap, BQ25790_FAULT_STAT_1, &fault_1);
+	if (ret)
+		return ret;
+
+	state->fault_1 = fault_1;
+
+	ret = regmap_read(bq->regmap, BQ25790_CHRG_STAT_3, &chrg_stat_3);
+	if (ret)
+		return ret;
+
+	state->online = chrg_stat_3 & BQ25790_VSYS_STAT;
+
+	state->vbat_adc = bq25790_get_vbat_adc(bq);
+
+	state->vbus_adc = bq25790_get_vbus_adc(bq);
+
+	state->ibat_adc = bq25790_get_ibat_adc(bq);
+
+	return 0;
+}
+
+static int bq25790_set_property(struct power_supply *psy,
+		enum power_supply_property prop,
+		const union power_supply_propval *val)
+{
+	struct bq25790_device *bq = power_supply_get_drvdata(psy);
+	int ret = -EINVAL;
+
+	switch (prop) {
+	case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
+		ret = bq25790_set_input_curr_lim(bq, val->intval);
+		break;
+	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
+		ret = bq25790_set_chrg_volt(bq, val->intval);
+		break;
+	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
+		ret = bq25790_set_ichrg_curr(bq, val->intval);
+		break;
+	case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:
+		ret = bq25790_set_prechrg_curr(bq, val->intval);
+		break;
+	case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:
+		ret = bq25790_set_term_curr(bq, val->intval);
+		break;
+	case POWER_SUPPLY_PROP_STATUS:
+		break;
+	case POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT:
+		ret = bq25790_set_input_volt_lim(bq, val->intval);
+		break;
+	default:
+		return -EINVAL;
+	}
+
+	return ret;
+}
+
+static int bq25790_get_property(struct power_supply *psy,
+				enum power_supply_property psp,
+				union power_supply_propval *val)
+{
+	struct bq25790_device *bq = power_supply_get_drvdata(psy);
+	struct bq25790_state state;
+	int ret = 0;
+
+	mutex_lock(&bq->lock);
+	ret = bq25790_get_state(bq, &state);
+	mutex_unlock(&bq->lock);
+	if (ret)
+		return ret;
+
+	switch (psp) {
+	case POWER_SUPPLY_PROP_STATUS:
+		if (!state.chrg_type || (state.chrg_type == BQ25790_OTG_MODE))
+			val->intval = POWER_SUPPLY_STATUS_DISCHARGING;
+		else if (!state.chrg_status)
+			val->intval = POWER_SUPPLY_STATUS_NOT_CHARGING;
+		else if (state.chrg_status == BQ25790_TERM_CHRG)
+			val->intval = POWER_SUPPLY_STATUS_FULL;
+		else
+			val->intval = POWER_SUPPLY_STATUS_CHARGING;
+		break;
+	case POWER_SUPPLY_PROP_CHARGE_TYPE:
+		switch (state.chrg_status) {
+		case BQ25790_TRICKLE_CHRG:
+			val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE;
+			break;
+		case BQ25790_PRECHRG:
+			val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE;
+			break;
+		case BQ25790_FAST_CHRG:
+			val->intval = POWER_SUPPLY_CHARGE_TYPE_FAST;
+			break;
+		case BQ25790_TAPER_CHRG:
+			val->intval = POWER_SUPPLY_CHARGE_TYPE_STANDARD;
+			break;
+		case BQ25790_TOP_OFF_CHRG:
+			val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE;
+			break;
+		case BQ25790_NOT_CHRGING:
+			val->intval = POWER_SUPPLY_CHARGE_TYPE_NONE;
+			break;
+		default:
+			val->intval = POWER_SUPPLY_CHARGE_TYPE_UNKNOWN;
+		}
+		break;
+	case POWER_SUPPLY_PROP_MANUFACTURER:
+		val->strval = BQ25790_MANUFACTURER;
+		break;
+
+	case POWER_SUPPLY_PROP_MODEL_NAME:
+		val->strval = BQ25790_NAME;
+		break;
+
+	case POWER_SUPPLY_PROP_ONLINE:
+		val->intval = state.online;
+		break;
+	case POWER_SUPPLY_PROP_USB_TYPE:
+		if (!state.chrg_type) {
+			val->intval = POWER_SUPPLY_USB_TYPE_UNKNOWN;
+			break;
+		}
+		switch (state.chrg_type) {
+		case BQ25790_USB_SDP:
+			val->intval = POWER_SUPPLY_USB_TYPE_SDP;
+			break;
+		case BQ25790_USB_CDP:
+			val->intval = POWER_SUPPLY_USB_TYPE_CDP;
+			break;
+		case BQ25790_USB_DCP:
+			val->intval = POWER_SUPPLY_USB_TYPE_DCP;
+			break;
+		case BQ25790_OTG_MODE:
+			val->intval = POWER_SUPPLY_USB_TYPE_PD_DRP;
+			break;
+
+		default:
+			val->intval = POWER_SUPPLY_USB_TYPE_UNKNOWN;
+			break;
+		}
+		break;
+
+	case POWER_SUPPLY_PROP_HEALTH:
+		if (state.fault_1 & (BQ25790_OTG_OVP | BQ25790_VSYS_OVP))
+			val->intval = POWER_SUPPLY_HEALTH_OVERVOLTAGE;
+		else
+			val->intval = POWER_SUPPLY_HEALTH_GOOD;
+
+		switch (state.health) {
+		case BQ25790_TEMP_HOT:
+			val->intval = POWER_SUPPLY_HEALTH_HOT;
+			break;
+		case BQ25790_TEMP_WARM:
+			val->intval = POWER_SUPPLY_HEALTH_WARM;
+			break;
+		case BQ25790_TEMP_COOL:
+			val->intval = POWER_SUPPLY_HEALTH_COOL;
+			break;
+		case BQ25790_TEMP_COLD:
+			val->intval = POWER_SUPPLY_HEALTH_COLD;
+			break;
+		}
+		break;
+	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
+		ret = bq25790_get_ichg_curr(bq);
+		if (ret < 0)
+			return ret;
+
+		val->intval = ret;
+		break;
+
+	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
+		ret = bq25790_get_chrg_volt(bq);
+		if (ret < 0)
+			return ret;
+
+		val->intval = ret;
+		break;
+
+	case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:
+		ret = bq25790_get_prechrg_curr(bq);
+		if (ret < 0)
+			return ret;
+
+		val->intval = ret;
+		break;
+	case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:
+		ret = bq25790_get_term_curr(bq);
+		if (ret < 0)
+			return ret;
+
+		val->intval = ret;
+		break;
+	case POWER_SUPPLY_PROP_VOLTAGE_NOW:
+		val->intval = state.vbus_adc;
+		break;
+	case POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT:
+		ret = bq25790_get_input_volt_lim(bq);
+		if (ret < 0)
+			return ret;
+
+		val->intval = ret;
+		break;
+	case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
+		ret = bq25790_get_input_curr_lim(bq);
+		if (ret < 0)
+			return ret;
+
+		val->intval = ret;
+		break;
+	default:
+		return -EINVAL;
+	}
+
+	return ret;
+}
+
+static int bq25790_battery_get_property(struct power_supply *psy,
+				       enum power_supply_property psp,
+				       union power_supply_propval *val)
+{
+	struct bq25790_device *bq = power_supply_get_drvdata(psy);
+	struct bq25790_state state;
+	int ret = 0;
+
+	mutex_lock(&bq->lock);
+	ret = bq25790_get_state(bq, &state);
+	mutex_unlock(&bq->lock);
+	if (ret)
+		return ret;
+
+	ret = regmap_update_bits(bq->regmap, BQ25790_ADC_CTRL,
+				 BQ25790_ADC_EN, BQ25790_ADC_EN);
+	if (ret)
+		return ret;
+
+	switch (psp) {
+	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX:
+		val->intval = bq->init_data.ichg_max;
+		break;
+	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX:
+		val->intval = bq->init_data.vreg_max;
+		break;
+
+	case POWER_SUPPLY_PROP_VOLTAGE_NOW:
+		val->intval = state.vbat_adc;
+		break;
+	case POWER_SUPPLY_PROP_CURRENT_NOW:
+		val->intval = state.ibat_adc;
+		break;
+	default:
+		return -EINVAL;
+	}
+
+	return ret;
+}
+
+static bool bq25790_state_changed(struct bq25790_device *bq,
+				  struct bq25790_state *new_state)
+{
+	struct bq25790_state old_state;
+
+	mutex_lock(&bq->lock);
+	old_state = bq->state;
+	mutex_unlock(&bq->lock);
+
+	return (old_state.chrg_status != new_state->chrg_status ||
+		old_state.chrg_fault != new_state->chrg_fault	||
+		old_state.online != new_state->online		||
+		old_state.health != new_state->health	||
+		old_state.fault_0 != new_state->fault_0 ||
+		old_state.fault_1 != new_state->fault_1 ||
+		old_state.chrg_type != new_state->chrg_type ||
+		old_state.vsys_status != new_state->vsys_status ||
+		old_state.vbat_adc != new_state->vbat_adc ||
+		old_state.vbus_adc != new_state->vbus_adc ||
+		old_state.ibat_adc != new_state->ibat_adc);
+}
+
+static irqreturn_t bq25790_irq_handler_thread(int irq, void *private)
+{
+	struct bq25790_device *bq = private;
+	struct bq25790_state state;
+	int ret;
+
+	ret = bq25790_get_state(bq, &state);
+	if (ret < 0)
+		goto irq_out;
+
+	if (!bq25790_state_changed(bq, &state))
+		goto irq_out;
+
+	mutex_lock(&bq->lock);
+	bq->state = state;
+	mutex_unlock(&bq->lock);
+
+	power_supply_changed(bq->charger);
+
+irq_out:
+	return IRQ_HANDLED;
+}
+
+static enum power_supply_property bq25790_power_supply_props[] = {
+	POWER_SUPPLY_PROP_MANUFACTURER,
+	POWER_SUPPLY_PROP_MODEL_NAME,
+	POWER_SUPPLY_PROP_STATUS,
+	POWER_SUPPLY_PROP_ONLINE,
+	POWER_SUPPLY_PROP_HEALTH,
+	POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT,
+	POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE,
+	POWER_SUPPLY_PROP_PRECHARGE_CURRENT,
+	POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT,
+	POWER_SUPPLY_PROP_VOLTAGE_NOW,
+	POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT,
+	POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT,
+	POWER_SUPPLY_PROP_CHARGE_TYPE,
+	POWER_SUPPLY_PROP_USB_TYPE,
+};
+
+static enum power_supply_property bq25790_battery_props[] = {
+	POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX,
+	POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX,
+	POWER_SUPPLY_PROP_VOLTAGE_NOW,
+	POWER_SUPPLY_PROP_CURRENT_NOW,
+};
+
+static char *bq25790_charger_supplied_to[] = {
+	"main-battery",
+};
+
+static int bq25790_property_is_writeable(struct power_supply *psy,
+					 enum power_supply_property prop)
+{
+	switch (prop) {
+	case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
+	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
+	case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
+	case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:
+	case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:
+	case POWER_SUPPLY_PROP_STATUS:
+	case POWER_SUPPLY_PROP_INPUT_VOLTAGE_LIMIT:
+		return true;
+	default:
+		return false;
+	}
+}
+
+static const struct power_supply_desc bq25790_power_supply_desc = {
+	.name = "bq25790-charger",
+	.type = POWER_SUPPLY_TYPE_USB,
+	.usb_types = bq25790_usb_type,
+	.num_usb_types = ARRAY_SIZE(bq25790_usb_type),
+	.properties = bq25790_power_supply_props,
+	.num_properties = ARRAY_SIZE(bq25790_power_supply_props),
+	.get_property = bq25790_get_property,
+	.set_property = bq25790_set_property,
+	.property_is_writeable = bq25790_property_is_writeable,
+};
+
+static const struct power_supply_desc bq25790_battery_desc = {
+	.name = "bq25790-battery",
+	.type = POWER_SUPPLY_TYPE_BATTERY,
+	.get_property = bq25790_battery_get_property,
+	.properties = bq25790_battery_props,
+	.num_properties = ARRAY_SIZE(bq25790_battery_props),
+};
+
+static bool bq25790_is_volatile_reg(struct device *dev, unsigned int reg)
+{
+	switch (reg) {
+	case BQ25790_ICO_I_LIM...BQ25790_FAULT_FLAG_1:
+	case BQ25790_ADC_IBUS...BQ25790_ADC_DM:
+	case BQ25790_CHRG_CTRL_0:
+		return true;
+	default:
+		return false;
+	}
+}
+
+static const struct regmap_config bq25790_regmap_config = {
+	.reg_bits = 8,
+	.val_bits = 8,
+
+	.max_register = BQ25790_PART_INFO,
+	.reg_defaults	= bq25790_reg_defs,
+	.num_reg_defaults = ARRAY_SIZE(bq25790_reg_defs),
+	.cache_type = REGCACHE_RBTREE,
+	.volatile_reg = bq25790_is_volatile_reg,
+};
+
+static int bq25790_power_supply_init(struct bq25790_device *bq,
+							struct device *dev)
+{
+	struct power_supply_config psy_cfg = { .drv_data = bq,
+						.of_node = dev->of_node, };
+
+	psy_cfg.supplied_to = bq25790_charger_supplied_to;
+	psy_cfg.num_supplicants = ARRAY_SIZE(bq25790_charger_supplied_to);
+
+	bq->charger = devm_power_supply_register(bq->dev,
+						 &bq25790_power_supply_desc,
+						 &psy_cfg);
+	if (IS_ERR(bq->charger))
+		return -EINVAL;
+
+	bq->battery = devm_power_supply_register(bq->dev,
+						      &bq25790_battery_desc,
+						      &psy_cfg);
+	if (IS_ERR(bq->battery))
+		return -EINVAL;
+	return 0;
+}
+
+static int bq25790_hw_init(struct bq25790_device *bq)
+{
+	struct power_supply_battery_info bat_info = { };
+	int wd_reg_val = BQ25790_WATCHDOG_DIS;
+	int wd_max_val = BQ25790_NUM_WD_VAL - 1;
+	int ret = 0;
+	int i;
+
+	if (bq->watchdog_timer) {
+		if (bq->watchdog_timer >= bq25790_watchdog_time[wd_max_val])
+			wd_reg_val = wd_max_val;
+		else {
+			for (i = 0; i < wd_max_val; i++) {
+				if (bq->watchdog_timer > bq25790_watchdog_time[i] &&
+				    bq->watchdog_timer < bq25790_watchdog_time[i + 1]) {
+					wd_reg_val = i;
+					break;
+				}
+			}
+		}
+	}
+
+	ret = regmap_update_bits(bq->regmap, BQ25790_CHRG_CTRL_1,
+				 BQ25790_WATCHDOG_MASK, wd_reg_val);
+	if (ret)
+		return ret;
+
+	ret = power_supply_get_battery_info(bq->charger, &bat_info);
+	if (ret) {
+		dev_warn(bq->dev, "battery info missing, default values will be applied\n");
+
+		bat_info.constant_charge_current_max_ua =
+							BQ25790_ICHRG_I_DEF_uA;
+
+		bat_info.constant_charge_voltage_max_uv =
+							BQ25790_VREG_V_DEF_uV;
+
+		bat_info.precharge_current_ua = BQ25790_PRECHRG_I_DEF_uA;
+		bat_info.charge_term_current_ua = BQ25790_TERMCHRG_I_DEF_uA;
+		bq->init_data.ichg_max = BQ25790_ICHRG_I_MAX_uA;
+		bq->init_data.vreg_max = BQ25790_VREG_V_MAX_uV;
+	} else {
+		bq->init_data.ichg_max =
+				bat_info.constant_charge_current_max_ua;
+
+		bq->init_data.vreg_max =
+				bat_info.constant_charge_voltage_max_uv;
+	}
+
+	ret = bq25790_set_ichrg_curr(bq,
+				bat_info.constant_charge_current_max_ua);
+	if (ret)
+		goto err_out;
+
+	ret = bq25790_set_prechrg_curr(bq, bat_info.precharge_current_ua);
+	if (ret)
+		goto err_out;
+
+	ret = bq25790_set_chrg_volt(bq,
+				bat_info.constant_charge_voltage_max_uv);
+	if (ret)
+		goto err_out;
+
+	ret = bq25790_set_term_curr(bq, bat_info.charge_term_current_ua);
+	if (ret)
+		goto err_out;
+
+	ret = bq25790_set_input_volt_lim(bq, bq->init_data.vlim);
+	if (ret)
+		goto err_out;
+
+	ret = bq25790_set_input_curr_lim(bq, bq->init_data.ilim);
+	if (ret)
+		goto err_out;
+
+err_out:
+	return ret;
+}
+
+static int bq25790_parse_dt(struct bq25790_device *bq)
+{
+	int ret;
+
+	ret = device_property_read_u32(bq->dev, "ti,watchdog-timeout-ms",
+				       &bq->watchdog_timer);
+	if (ret)
+		bq->watchdog_timer = BQ25790_WATCHDOG_DIS;
+
+	if (bq->watchdog_timer > BQ25790_WATCHDOG_MAX ||
+	    bq->watchdog_timer < BQ25790_WATCHDOG_DIS)
+		return -EINVAL;
+
+	ret = device_property_read_u32(bq->dev,
+				       "input-voltage-limit-microvolt",
+				       &bq->init_data.vlim);
+	if (ret)
+		bq->init_data.vlim = BQ25790_VINDPM_DEF_uV;
+
+	if (bq->init_data.vlim > BQ25790_VINDPM_V_MAX_uV ||
+	    bq->init_data.vlim < BQ25790_VINDPM_V_MIN_uV)
+		return -EINVAL;
+
+	ret = device_property_read_u32(bq->dev,
+				       "input-current-limit-microamp",
+				       &bq->init_data.ilim);
+	if (ret)
+		bq->init_data.ilim = BQ25790_IINDPM_DEF_uA;
+
+	if (bq->init_data.ilim > BQ25790_IINDPM_I_MAX_uA ||
+	    bq->init_data.ilim < BQ25790_IINDPM_I_MIN_uA)
+		return -EINVAL;
+
+	return 0;
+}
+
+static int bq25790_probe(struct i2c_client *client,
+			 const struct i2c_device_id *id)
+{
+	struct device *dev = &client->dev;
+	struct bq25790_device *bq;
+	int ret;
+
+	bq = devm_kzalloc(dev, sizeof(*bq), GFP_KERNEL);
+	if (!bq)
+		return -ENOMEM;
+
+	bq->client = client;
+	bq->dev = dev;
+
+	mutex_init(&bq->lock);
+
+	strncpy(bq->model_name, id->name, I2C_NAME_SIZE);
+
+	bq->regmap = devm_regmap_init_i2c(client, &bq25790_regmap_config);
+	if (IS_ERR(bq->regmap)) {
+		dev_err(dev, "Failed to allocate register map\n");
+		return PTR_ERR(bq->regmap);
+	}
+
+	i2c_set_clientdata(client, bq);
+
+	ret = bq25790_parse_dt(bq);
+	if (ret) {
+		dev_err(dev, "Failed to read device tree properties%d\n", ret);
+		return ret;
+	}
+
+	/* OTG reporting */
+	bq->usb2_phy = devm_usb_get_phy(dev, USB_PHY_TYPE_USB2);
+	if (!IS_ERR_OR_NULL(bq->usb2_phy)) {
+		INIT_WORK(&bq->usb_work, bq25790_usb_work);
+		bq->usb_nb.notifier_call = bq25790_usb_notifier;
+		usb_register_notifier(bq->usb2_phy, &bq->usb_nb);
+	}
+
+	bq->usb3_phy = devm_usb_get_phy(dev, USB_PHY_TYPE_USB3);
+	if (!IS_ERR_OR_NULL(bq->usb3_phy)) {
+		INIT_WORK(&bq->usb_work, bq25790_usb_work);
+		bq->usb_nb.notifier_call = bq25790_usb_notifier;
+		usb_register_notifier(bq->usb3_phy, &bq->usb_nb);
+	}
+
+	if (client->irq) {
+		ret = devm_request_threaded_irq(dev, client->irq, NULL,
+						bq25790_irq_handler_thread,
+						IRQF_TRIGGER_FALLING |
+						IRQF_ONESHOT,
+						dev_name(&client->dev), bq);
+		if (ret)
+			goto error_out;
+	}
+
+	ret = bq25790_power_supply_init(bq, dev);
+	if (ret) {
+		dev_err(dev, "Failed to register power supply\n");
+		goto error_out;
+	}
+
+	ret = bq25790_hw_init(bq);
+	if (ret) {
+		dev_err(dev, "Cannot initialize the chip.\n");
+		goto error_out;
+	}
+
+	return ret;
+error_out:
+	if (!IS_ERR_OR_NULL(bq->usb2_phy))
+		usb_unregister_notifier(bq->usb2_phy, &bq->usb_nb);
+
+	if (!IS_ERR_OR_NULL(bq->usb3_phy))
+		usb_unregister_notifier(bq->usb3_phy, &bq->usb_nb);
+	return ret;
+}
+
+static const struct i2c_device_id bq25790_i2c_ids[] = {
+	{ BQ25790_NAME, 0 },
+	{ BQ25792_NAME, 1 },
+	{},
+};
+MODULE_DEVICE_TABLE(i2c, bq25790_i2c_ids);
+
+static const struct of_device_id bq25790_of_match[] = {
+	{ .compatible = "ti,bq25790", },
+	{ .compatible = "ti,bq25792", },
+	{ },
+};
+MODULE_DEVICE_TABLE(of, bq25790_of_match);
+
+static const struct acpi_device_id bq25790_acpi_match[] = {
+	{BQ25790_NAME, 0},
+	{BQ25792_NAME, 0},
+	{},
+};
+MODULE_DEVICE_TABLE(acpi, bq25790_acpi_match);
+
+static struct i2c_driver bq25790_driver = {
+	.driver = {
+		.name = "bq25790-charger",
+		.of_match_table = bq25790_of_match,
+		.acpi_match_table = ACPI_PTR(bq25790_acpi_match),
+	},
+	.probe = bq25790_probe,
+	.id_table = bq25790_i2c_ids,
+};
+module_i2c_driver(bq25790_driver);
+
+MODULE_AUTHOR("Dan Murphy <dmurphy@ti.com>");
+MODULE_AUTHOR("Ricardo Rivera-Matos <r-rivera-matos@ti.com>");
+MODULE_DESCRIPTION("bq25790 charger driver");
+MODULE_LICENSE("GPL v2");
diff --git a/drivers/power/supply/bq25790_charger.h b/drivers/power/supply/bq25790_charger.h
new file mode 100644
index 000000000000..77234e116a6b
--- /dev/null
+++ b/drivers/power/supply/bq25790_charger.h
@@ -0,0 +1,150 @@ 
+/* SPDX-License-Identifier: GPL-2.0-only */
+// BQ25790 Charger Driver
+// Copyright (C) 2020 Texas Instruments Incorporated - http://www.ti.com/
+
+#ifndef _BQ25790_CHARGER_H
+#define _BQ25790_CHARGER_H
+
+#define BQ25790_MANUFACTURER	"Texas Instruments"
+#define BQ25790_NAME		"bq25790"
+#define BQ25792_NAME		"bq25792"
+
+#define BQ25790_MIN_SYS_V	0x00
+#define BQ25790_CHRG_V_LIM_MSB	0x01
+#define BQ25790_CHRG_V_LIM_LSB	0x02
+#define BQ25790_CHRG_I_LIM_MSB	0x03
+#define BQ25790_CHRG_I_LIM_LSB	0x04
+#define BQ25790_INPUT_V_LIM	0x05
+#define BQ25790_INPUT_I_LIM_MSB	0x06
+#define BQ25790_INPUT_I_LIM_LSB	0x07
+#define BQ25790_PRECHRG_CTRL	0x08
+#define BQ25790_TERM_CTRL	0x09
+#define BQ25790_RECHRG_CTRL	0x0a
+#define BQ25790_VOTG_REG	0x0b
+#define BQ25790_IOTG_REG	0x0d
+#define BQ25790_TIMER_CTRL	0x0e
+#define BQ25790_CHRG_CTRL_0	0x0f
+#define BQ25790_CHRG_CTRL_1	0x10
+#define BQ25790_CHRG_CTRL_2	0x11
+#define BQ25790_CHRG_CTRL_3	0x12
+#define BQ25790_CHRG_CTRL_4	0x13
+#define BQ25790_CHRG_CTRL_5	0x14
+#define BQ25790_MPPT_CTRL	0x15
+#define BQ25790_TEMP_CTRL	0x16
+#define BQ25790_NTC_CTRL_0	0x17
+#define BQ25790_NTC_CTRL_1	0x18
+#define BQ25790_ICO_I_LIM	0x19
+#define BQ25790_CHRG_STAT_0	0x1b
+#define BQ25790_CHRG_STAT_1	0x1c
+#define BQ25790_CHRG_STAT_2	0x1d
+#define BQ25790_CHRG_STAT_3	0x1e
+#define BQ25790_CHRG_STAT_4	0x1f
+#define BQ25790_FAULT_STAT_0	0x20
+#define BQ25790_FAULT_STAT_1	0x21
+#define BQ25790_CHRG_FLAG_0	0x22
+#define BQ25790_CHRG_FLAG_1	0x23
+#define BQ25790_CHRG_FLAG_2	0x24
+#define BQ25790_CHRG_FLAG_3	0x25
+#define BQ25790_FAULT_FLAG_0	0x26
+#define BQ25790_FAULT_FLAG_1	0x27
+#define BQ25790_CHRG_MSK_0	0x28
+#define BQ25790_CHRG_MSK_1	0x29
+#define BQ25790_CHRG_MSK_2	0x2a
+#define BQ25790_CHRG_MSK_3	0x2b
+#define BQ25790_FAULT_MSK_0	0x2c
+#define BQ25790_FAULT_MSK_1	0x2d
+#define BQ25790_ADC_CTRL	0x2e
+#define BQ25790_FN_DISABE_0	0x2f
+#define BQ25790_FN_DISABE_1	0x30
+#define BQ25790_ADC_IBUS	0x31
+#define BQ25790_ADC_IBAT_MSB	0x33
+#define BQ25790_ADC_IBAT_LSB	0x34
+#define BQ25790_ADC_VBUS_MSB	0x35
+#define BQ25790_ADC_VBUS_LSB	0x36
+#define BQ25790_ADC_VAC1	0x37
+#define BQ25790_ADC_VAC2	0x39
+#define BQ25790_ADC_VBAT_MSB	0x3b
+#define BQ25790_ADC_VBAT_LSB	0x3c
+#define BQ25790_ADC_VSYS_MSB	0x3d
+#define BQ25790_ADC_VSYS_LSB	0x3e
+#define BQ25790_ADC_TS		0x3f
+#define BQ25790_ADC_TDIE	0x41
+#define BQ25790_ADC_DP		0x43
+#define BQ25790_ADC_DM		0x45
+#define BQ25790_DPDM_DRV	0x47
+#define BQ25790_PART_INFO	0x48
+
+#define BQ25790_CHRG_EN		BIT(5)
+#define BQ25790_ADC_EN		BIT(7)
+
+/* Charger Status 1 */
+#define BQ25790_CHG_STAT_MSK	GENMASK(7, 5)
+#define BQ25790_NOT_CHRGING	0
+#define BQ25790_TRICKLE_CHRG	BIT(5)
+#define BQ25790_PRECHRG		BIT(6)
+#define BQ25790_FAST_CHRG	(BIT(5) | BIT(6))
+#define BQ25790_TAPER_CHRG	BIT(7)
+#define BQ25790_TOP_OFF_CHRG	(BIT(6) | BIT(7))
+#define BQ25790_TERM_CHRG	(BIT(5) | BIT(6) | BIT(7))
+#define BQ25790_VBUS_PRESENT	BIT(0)
+
+#define BQ25790_VBUS_STAT_MSK	GENMASK(4, 1)
+#define BQ25790_USB_SDP		BIT(1)
+#define BQ25790_USB_CDP		BIT(2)
+#define BQ25790_USB_DCP		(BIT(1) | BIT(2))
+#define BQ25790_HVDCP		BIT(3)
+#define BQ25790_UNKNOWN_3A	(BIT(3) | BIT(1))
+#define BQ25790_NON_STANDARD	(BIT(3) | BIT(2))
+#define BQ25790_OTG_MODE	(BIT(3) | BIT(2) | BIT(1))
+#define BQ25790_UNQUAL_ADAPT	BIT(4)
+#define BQ25790_DIRECT_PWR	(BIT(4) | BIT(2) | BIT(1))
+
+/* Charger Status 4 */
+#define BQ25790_TEMP_HOT	BIT(0)
+#define BQ25790_TEMP_WARM	BIT(1)
+#define BQ25790_TEMP_COOL	BIT(2)
+#define BQ25790_TEMP_COLD	BIT(3)
+#define BQ25790_TEMP_MASK	GENMASK(3, 0)
+
+#define BQ25790_OTG_OVP		BIT(5)
+#define BQ25790_VSYS_OVP	BIT(6)
+#define BQ25790_VSYS_STAT	BIT(4)
+
+#define BQ25790_PRECHRG_CUR_MASK		GENMASK(5, 0)
+#define BQ25790_PRECHRG_CURRENT_STEP_uA		40000
+#define BQ25790_PRECHRG_I_MIN_uA		40000
+#define BQ25790_PRECHRG_I_MAX_uA		2000000
+#define BQ25790_PRECHRG_I_DEF_uA		120000
+#define BQ25790_TERMCHRG_CUR_MASK		GENMASK(4, 0)
+#define BQ25790_TERMCHRG_CURRENT_STEP_uA	40000
+#define BQ25790_TERMCHRG_I_MIN_uA		40000
+#define BQ25790_TERMCHRG_I_MAX_uA		1000000
+#define BQ25790_TERMCHRG_I_DEF_uA		200000
+#define BQ25790_ICHRG_CURRENT_STEP_uA		10000
+#define BQ25790_ICHRG_I_MIN_uA			50000
+#define BQ25790_ICHRG_I_MAX_uA			5000000
+#define BQ25790_ICHRG_I_DEF_uA			1000000
+
+#define BQ25790_VREG_V_MAX_uV	18800000
+#define BQ25790_VREG_V_MIN_uV	3000000
+#define BQ25790_VREG_V_DEF_uV	3600000
+#define BQ25790_VREG_V_STEP_uV	10000
+
+#define BQ25790_IINDPM_I_MIN_uA	100000
+#define BQ25790_IINDPM_I_MAX_uA	3300000
+#define BQ25790_IINDPM_STEP_uA	10000
+#define BQ25790_IINDPM_DEF_uA	1000000
+
+#define BQ25790_VINDPM_V_MIN_uV 3600000
+#define BQ25790_VINDPM_V_MAX_uV 22000000
+#define BQ25790_VINDPM_STEP_uV	100000
+#define BQ25790_VINDPM_DEF_uV	3600000
+
+#define BQ25790_ADC_VOLT_STEP_uV	1000
+#define BQ25790_ADC_CURR_STEP_uA	1000
+
+#define BQ25790_WATCHDOG_MASK	GENMASK(2, 0)
+#define BQ25790_WATCHDOG_DIS	0
+#define BQ25790_WATCHDOG_MAX	160000
+
+#endif /* _BQ25790_CHARGER_H */