2019-05-27 09:55:21 +03:00
|
|
|
// SPDX-License-Identifier: GPL-2.0-only
|
2015-05-04 19:46:07 +03:00
|
|
|
/*
|
|
|
|
* axp288_charger.c - X-power AXP288 PMIC Charger driver
|
|
|
|
*
|
2017-12-26 15:58:59 +03:00
|
|
|
* Copyright (C) 2016-2017 Hans de Goede <hdegoede@redhat.com>
|
2015-05-04 19:46:07 +03:00
|
|
|
* Copyright (C) 2014 Intel Corporation
|
|
|
|
* Author: Ramakrishna Pallala <ramakrishna.pallala@intel.com>
|
|
|
|
*/
|
|
|
|
|
2017-04-19 15:02:12 +03:00
|
|
|
#include <linux/acpi.h>
|
2018-11-26 18:27:55 +03:00
|
|
|
#include <linux/bitops.h>
|
2015-05-04 19:46:07 +03:00
|
|
|
#include <linux/module.h>
|
|
|
|
#include <linux/device.h>
|
|
|
|
#include <linux/regmap.h>
|
|
|
|
#include <linux/workqueue.h>
|
|
|
|
#include <linux/delay.h>
|
|
|
|
#include <linux/platform_device.h>
|
|
|
|
#include <linux/usb/otg.h>
|
|
|
|
#include <linux/notifier.h>
|
|
|
|
#include <linux/power_supply.h>
|
|
|
|
#include <linux/property.h>
|
|
|
|
#include <linux/mfd/axp20x.h>
|
|
|
|
#include <linux/extcon.h>
|
power: supply: axp288_charger: Add special handling for HP Pavilion x2 10
Some HP Pavilion x2 10 models use an AXP288 for charging and fuel-gauge.
We use a native power_supply / PMIC driver in this case, because on most
models with an AXP288 the ACPI AC / Battery code is either completely
missing or relies on custom / proprietary ACPI OpRegions which Linux
does not implement.
The native drivers mostly work fine, but there are 2 problems:
1. These model uses a Type-C connector for charging which the AXP288 does
not support. As long as a Type-A charger (which uses the USB data pins for
charger type detection) is used everything is fine. But if a Type-C
charger is used (such as the charger shipped with the device) then the
charger is not recognized.
So we end up slowly discharging the device even though a charger is
connected, because we are limiting the current from the charger to 500mA.
To make things worse this happens with the device's official charger.
Looking at the ACPI tables HP has "solved" the problem of the AXP288 not
being able to recognize Type-C chargers by simply always programming the
input-current-limit at 3000mA and relying on a Vhold setting of 4.7V
(normally 4.4V) to limit the current intake if the charger cannot handle
this.
2. If no charger is connected when the machine boots then it boots with the
vbus-path disabled. On other devices this is done when a 5V boost converter
is active to avoid the PMIC trying to charge from the 5V boost output.
This is done when an OTG host cable is inserted and the ID pin on the
micro-B receptacle is pulled low, the ID pin has an ACPI event handler
associated with it which re-enables the vbus-path when the ID pin is pulled
high when the OTG cable is removed. The Type-C connector has no ID pin,
there is no ID pin handler and there appears to be no 5V boost converter,
so we end up not charging because the vbus-path is disabled, until we
unplug the charger which automatically clears the vbus-path disable bit and
then on the second plug-in of the adapter we start charging.
The HP Pavilion x2 10 models with an AXP288 do have mostly working ACPI
AC / Battery code which does not rely on custom / proprietary ACPI
OpRegions. So one possible solution would be to blacklist the AXP288
native power_supply drivers and add the HP Pavilion x2 10 with AXP288
DMI ids to the list of devices which should use the ACPI AC / Battery
code even though they have an AXP288 PMIC. This would require changes to
4 files: drivers/acpi/ac.c, drivers/power/supply/axp288_charger.c,
drivers/acpi/battery.c and drivers/power/supply/axp288_fuel_gauge.c.
Beside needing adding the same DMI matches to 4 different files, this
approach also triggers problem 2. from above, but then when suspended,
during suspend the machine will not wakeup because the vbus path is
disabled by the AML code when not charging, so the Vbus low-to-high
IRQ is not triggered, the CPU never wakes up and the device does not
charge even though the user likely things it is charging, esp. since
the charge status LED is directly coupled to an adapter being plugged
in and does not reflect actual charging.
This could be worked by enabling vbus-path explicitly from say the
axp288_charger driver's suspend handler.
So neither situation is ideal, in both cased we need to explicitly enable
the vbus-path to work around different variants of problem 2 above, this
requires a quirk in the axp288_charger code.
If we go the route of using the ACPI AC / Battery drivers then we need
modifications to 3 other drivers; and we need to partially disable the
axp288_charger code, while at the same time keeping it around to enable
vbus-path on suspend.
OTOH we can copy the hardcoding of 3A input-current-limit (we never touch
Vhold, so that would stay at 4.7V) to the axp288_charger code, which needs
changes regardless, then we concentrate all special handling of this
interesting device model in the axp288_charger code. That is what this
commit does.
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1791098
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
2020-02-23 18:32:08 +03:00
|
|
|
#include <linux/dmi.h>
|
2015-05-04 19:46:07 +03:00
|
|
|
|
2018-11-26 18:27:55 +03:00
|
|
|
#define PS_STAT_VBUS_TRIGGER BIT(0)
|
|
|
|
#define PS_STAT_BAT_CHRG_DIR BIT(2)
|
|
|
|
#define PS_STAT_VBAT_ABOVE_VHOLD BIT(3)
|
|
|
|
#define PS_STAT_VBUS_VALID BIT(4)
|
|
|
|
#define PS_STAT_VBUS_PRESENT BIT(5)
|
2015-05-04 19:46:07 +03:00
|
|
|
|
2018-11-26 18:27:55 +03:00
|
|
|
#define CHRG_STAT_BAT_SAFE_MODE BIT(3)
|
|
|
|
#define CHRG_STAT_BAT_VALID BIT(4)
|
|
|
|
#define CHRG_STAT_BAT_PRESENT BIT(5)
|
|
|
|
#define CHRG_STAT_CHARGING BIT(6)
|
|
|
|
#define CHRG_STAT_PMIC_OTP BIT(7)
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
#define VBUS_ISPOUT_CUR_LIM_MASK 0x03
|
|
|
|
#define VBUS_ISPOUT_CUR_LIM_BIT_POS 0
|
|
|
|
#define VBUS_ISPOUT_CUR_LIM_900MA 0x0 /* 900mA */
|
|
|
|
#define VBUS_ISPOUT_CUR_LIM_1500MA 0x1 /* 1500mA */
|
|
|
|
#define VBUS_ISPOUT_CUR_LIM_2000MA 0x2 /* 2000mA */
|
|
|
|
#define VBUS_ISPOUT_CUR_NO_LIM 0x3 /* 2500mA */
|
2022-02-08 15:51:47 +03:00
|
|
|
#define VBUS_ISPOUT_VHOLD_SET_MASK 0x38
|
2015-05-04 19:46:07 +03:00
|
|
|
#define VBUS_ISPOUT_VHOLD_SET_BIT_POS 0x3
|
|
|
|
#define VBUS_ISPOUT_VHOLD_SET_OFFSET 4000 /* 4000mV */
|
|
|
|
#define VBUS_ISPOUT_VHOLD_SET_LSB_RES 100 /* 100mV */
|
2022-02-08 15:51:47 +03:00
|
|
|
#define VBUS_ISPOUT_VHOLD_SET_4400MV 0x4 /* 4400mV */
|
2018-11-26 18:27:55 +03:00
|
|
|
#define VBUS_ISPOUT_VBUS_PATH_DIS BIT(7)
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
#define CHRG_CCCV_CC_MASK 0xf /* 4 bits */
|
|
|
|
#define CHRG_CCCV_CC_BIT_POS 0
|
|
|
|
#define CHRG_CCCV_CC_OFFSET 200 /* 200mA */
|
|
|
|
#define CHRG_CCCV_CC_LSB_RES 200 /* 200mA */
|
2018-11-26 18:27:55 +03:00
|
|
|
#define CHRG_CCCV_ITERM_20P BIT(4) /* 20% of CC */
|
2015-05-04 19:46:07 +03:00
|
|
|
#define CHRG_CCCV_CV_MASK 0x60 /* 2 bits */
|
|
|
|
#define CHRG_CCCV_CV_BIT_POS 5
|
|
|
|
#define CHRG_CCCV_CV_4100MV 0x0 /* 4.10V */
|
|
|
|
#define CHRG_CCCV_CV_4150MV 0x1 /* 4.15V */
|
|
|
|
#define CHRG_CCCV_CV_4200MV 0x2 /* 4.20V */
|
|
|
|
#define CHRG_CCCV_CV_4350MV 0x3 /* 4.35V */
|
2018-11-26 18:27:55 +03:00
|
|
|
#define CHRG_CCCV_CHG_EN BIT(7)
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
#define CNTL2_CC_TIMEOUT_MASK 0x3 /* 2 bits */
|
|
|
|
#define CNTL2_CC_TIMEOUT_OFFSET 6 /* 6 Hrs */
|
|
|
|
#define CNTL2_CC_TIMEOUT_LSB_RES 2 /* 2 Hrs */
|
|
|
|
#define CNTL2_CC_TIMEOUT_12HRS 0x3 /* 12 Hrs */
|
2018-11-26 18:27:55 +03:00
|
|
|
#define CNTL2_CHGLED_TYPEB BIT(4)
|
|
|
|
#define CNTL2_CHG_OUT_TURNON BIT(5)
|
2015-05-04 19:46:07 +03:00
|
|
|
#define CNTL2_PC_TIMEOUT_MASK 0xC0
|
|
|
|
#define CNTL2_PC_TIMEOUT_OFFSET 40 /* 40 mins */
|
|
|
|
#define CNTL2_PC_TIMEOUT_LSB_RES 10 /* 10 mins */
|
|
|
|
#define CNTL2_PC_TIMEOUT_70MINS 0x3
|
|
|
|
|
2018-11-26 18:27:55 +03:00
|
|
|
#define CHRG_ILIM_TEMP_LOOP_EN BIT(3)
|
2015-05-04 19:46:07 +03:00
|
|
|
#define CHRG_VBUS_ILIM_MASK 0xf0
|
|
|
|
#define CHRG_VBUS_ILIM_BIT_POS 4
|
|
|
|
#define CHRG_VBUS_ILIM_100MA 0x0 /* 100mA */
|
|
|
|
#define CHRG_VBUS_ILIM_500MA 0x1 /* 500mA */
|
|
|
|
#define CHRG_VBUS_ILIM_900MA 0x2 /* 900mA */
|
|
|
|
#define CHRG_VBUS_ILIM_1500MA 0x3 /* 1500mA */
|
|
|
|
#define CHRG_VBUS_ILIM_2000MA 0x4 /* 2000mA */
|
|
|
|
#define CHRG_VBUS_ILIM_2500MA 0x5 /* 2500mA */
|
|
|
|
#define CHRG_VBUS_ILIM_3000MA 0x6 /* 3000mA */
|
2018-04-18 15:07:56 +03:00
|
|
|
#define CHRG_VBUS_ILIM_3500MA 0x7 /* 3500mA */
|
|
|
|
#define CHRG_VBUS_ILIM_4000MA 0x8 /* 4000mA */
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
#define CHRG_VLTFC_0C 0xA5 /* 0 DegC */
|
|
|
|
#define CHRG_VHTFC_45C 0x1F /* 45 DegC */
|
|
|
|
|
2018-11-26 18:27:55 +03:00
|
|
|
#define FG_CNTL_OCV_ADJ_EN BIT(3)
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
#define CV_4100MV 4100 /* 4100mV */
|
|
|
|
#define CV_4150MV 4150 /* 4150mV */
|
|
|
|
#define CV_4200MV 4200 /* 4200mV */
|
|
|
|
#define CV_4350MV 4350 /* 4350mV */
|
|
|
|
|
|
|
|
#define AXP288_EXTCON_DEV_NAME "axp288_extcon"
|
2017-04-19 15:02:12 +03:00
|
|
|
#define USB_HOST_EXTCON_HID "INT3496"
|
|
|
|
#define USB_HOST_EXTCON_NAME "INT3496:00"
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
enum {
|
|
|
|
VBUS_OV_IRQ = 0,
|
|
|
|
CHARGE_DONE_IRQ,
|
|
|
|
CHARGE_CHARGING_IRQ,
|
|
|
|
BAT_SAFE_QUIT_IRQ,
|
|
|
|
BAT_SAFE_ENTER_IRQ,
|
|
|
|
QCBTU_IRQ,
|
|
|
|
CBTU_IRQ,
|
|
|
|
QCBTO_IRQ,
|
|
|
|
CBTO_IRQ,
|
|
|
|
CHRG_INTR_END,
|
|
|
|
};
|
|
|
|
|
|
|
|
struct axp288_chrg_info {
|
|
|
|
struct platform_device *pdev;
|
|
|
|
struct regmap *regmap;
|
|
|
|
struct regmap_irq_chip_data *regmap_irqc;
|
|
|
|
int irq[CHRG_INTR_END];
|
|
|
|
struct power_supply *psy_usb;
|
|
|
|
|
|
|
|
/* OTG/Host mode */
|
|
|
|
struct {
|
|
|
|
struct work_struct work;
|
2016-03-21 04:36:26 +03:00
|
|
|
struct extcon_dev *cable;
|
2015-05-04 19:46:07 +03:00
|
|
|
struct notifier_block id_nb;
|
|
|
|
bool id_short;
|
|
|
|
} otg;
|
|
|
|
|
|
|
|
/* SDP/CDP/DCP USB charging cable notifications */
|
|
|
|
struct {
|
|
|
|
struct extcon_dev *edev;
|
2017-12-26 15:59:08 +03:00
|
|
|
struct notifier_block nb;
|
2015-05-04 19:46:07 +03:00
|
|
|
struct work_struct work;
|
|
|
|
} cable;
|
|
|
|
|
|
|
|
int cc;
|
|
|
|
int cv;
|
|
|
|
int max_cc;
|
|
|
|
int max_cv;
|
|
|
|
};
|
|
|
|
|
|
|
|
static inline int axp288_charger_set_cc(struct axp288_chrg_info *info, int cc)
|
|
|
|
{
|
|
|
|
u8 reg_val;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
if (cc < CHRG_CCCV_CC_OFFSET)
|
|
|
|
cc = CHRG_CCCV_CC_OFFSET;
|
|
|
|
else if (cc > info->max_cc)
|
|
|
|
cc = info->max_cc;
|
|
|
|
|
|
|
|
reg_val = (cc - CHRG_CCCV_CC_OFFSET) / CHRG_CCCV_CC_LSB_RES;
|
|
|
|
cc = (reg_val * CHRG_CCCV_CC_LSB_RES) + CHRG_CCCV_CC_OFFSET;
|
|
|
|
reg_val = reg_val << CHRG_CCCV_CC_BIT_POS;
|
|
|
|
|
|
|
|
ret = regmap_update_bits(info->regmap,
|
|
|
|
AXP20X_CHRG_CTRL1,
|
|
|
|
CHRG_CCCV_CC_MASK, reg_val);
|
|
|
|
if (ret >= 0)
|
|
|
|
info->cc = cc;
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static inline int axp288_charger_set_cv(struct axp288_chrg_info *info, int cv)
|
|
|
|
{
|
|
|
|
u8 reg_val;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
if (cv <= CV_4100MV) {
|
|
|
|
reg_val = CHRG_CCCV_CV_4100MV;
|
|
|
|
cv = CV_4100MV;
|
|
|
|
} else if (cv <= CV_4150MV) {
|
|
|
|
reg_val = CHRG_CCCV_CV_4150MV;
|
|
|
|
cv = CV_4150MV;
|
|
|
|
} else if (cv <= CV_4200MV) {
|
|
|
|
reg_val = CHRG_CCCV_CV_4200MV;
|
|
|
|
cv = CV_4200MV;
|
|
|
|
} else {
|
|
|
|
reg_val = CHRG_CCCV_CV_4350MV;
|
|
|
|
cv = CV_4350MV;
|
|
|
|
}
|
|
|
|
|
|
|
|
reg_val = reg_val << CHRG_CCCV_CV_BIT_POS;
|
|
|
|
|
|
|
|
ret = regmap_update_bits(info->regmap,
|
|
|
|
AXP20X_CHRG_CTRL1,
|
|
|
|
CHRG_CCCV_CV_MASK, reg_val);
|
|
|
|
|
|
|
|
if (ret >= 0)
|
|
|
|
info->cv = cv;
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2017-12-26 15:59:05 +03:00
|
|
|
static int axp288_charger_get_vbus_inlmt(struct axp288_chrg_info *info)
|
|
|
|
{
|
|
|
|
unsigned int val;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
ret = regmap_read(info->regmap, AXP20X_CHRG_BAK_CTRL, &val);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
val >>= CHRG_VBUS_ILIM_BIT_POS;
|
|
|
|
switch (val) {
|
|
|
|
case CHRG_VBUS_ILIM_100MA:
|
|
|
|
return 100000;
|
|
|
|
case CHRG_VBUS_ILIM_500MA:
|
|
|
|
return 500000;
|
|
|
|
case CHRG_VBUS_ILIM_900MA:
|
|
|
|
return 900000;
|
|
|
|
case CHRG_VBUS_ILIM_1500MA:
|
|
|
|
return 1500000;
|
|
|
|
case CHRG_VBUS_ILIM_2000MA:
|
|
|
|
return 2000000;
|
|
|
|
case CHRG_VBUS_ILIM_2500MA:
|
|
|
|
return 2500000;
|
|
|
|
case CHRG_VBUS_ILIM_3000MA:
|
|
|
|
return 3000000;
|
2018-04-18 15:07:56 +03:00
|
|
|
case CHRG_VBUS_ILIM_3500MA:
|
|
|
|
return 3500000;
|
2017-12-26 15:59:05 +03:00
|
|
|
default:
|
2018-04-18 15:07:56 +03:00
|
|
|
/* All b1xxx values map to 4000 mA */
|
|
|
|
return 4000000;
|
2017-12-26 15:59:05 +03:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2015-05-04 19:46:07 +03:00
|
|
|
static inline int axp288_charger_set_vbus_inlmt(struct axp288_chrg_info *info,
|
|
|
|
int inlmt)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
u8 reg_val;
|
|
|
|
|
2018-04-18 15:07:56 +03:00
|
|
|
if (inlmt >= 4000000)
|
|
|
|
reg_val = CHRG_VBUS_ILIM_4000MA << CHRG_VBUS_ILIM_BIT_POS;
|
|
|
|
else if (inlmt >= 3500000)
|
|
|
|
reg_val = CHRG_VBUS_ILIM_3500MA << CHRG_VBUS_ILIM_BIT_POS;
|
|
|
|
else if (inlmt >= 3000000)
|
2017-12-26 15:59:06 +03:00
|
|
|
reg_val = CHRG_VBUS_ILIM_3000MA << CHRG_VBUS_ILIM_BIT_POS;
|
|
|
|
else if (inlmt >= 2500000)
|
|
|
|
reg_val = CHRG_VBUS_ILIM_2500MA << CHRG_VBUS_ILIM_BIT_POS;
|
|
|
|
else if (inlmt >= 2000000)
|
|
|
|
reg_val = CHRG_VBUS_ILIM_2000MA << CHRG_VBUS_ILIM_BIT_POS;
|
|
|
|
else if (inlmt >= 1500000)
|
|
|
|
reg_val = CHRG_VBUS_ILIM_1500MA << CHRG_VBUS_ILIM_BIT_POS;
|
|
|
|
else if (inlmt >= 900000)
|
|
|
|
reg_val = CHRG_VBUS_ILIM_900MA << CHRG_VBUS_ILIM_BIT_POS;
|
|
|
|
else if (inlmt >= 500000)
|
|
|
|
reg_val = CHRG_VBUS_ILIM_500MA << CHRG_VBUS_ILIM_BIT_POS;
|
|
|
|
else
|
|
|
|
reg_val = CHRG_VBUS_ILIM_100MA << CHRG_VBUS_ILIM_BIT_POS;
|
2017-12-26 15:59:03 +03:00
|
|
|
|
|
|
|
ret = regmap_update_bits(info->regmap, AXP20X_CHRG_BAK_CTRL,
|
|
|
|
CHRG_VBUS_ILIM_MASK, reg_val);
|
2017-12-26 15:59:05 +03:00
|
|
|
if (ret < 0)
|
2015-05-04 19:46:07 +03:00
|
|
|
dev_err(&info->pdev->dev, "charger BAK control %d\n", ret);
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int axp288_charger_vbus_path_select(struct axp288_chrg_info *info,
|
|
|
|
bool enable)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
if (enable)
|
|
|
|
ret = regmap_update_bits(info->regmap, AXP20X_VBUS_IPSOUT_MGMT,
|
|
|
|
VBUS_ISPOUT_VBUS_PATH_DIS, 0);
|
|
|
|
else
|
|
|
|
ret = regmap_update_bits(info->regmap, AXP20X_VBUS_IPSOUT_MGMT,
|
|
|
|
VBUS_ISPOUT_VBUS_PATH_DIS, VBUS_ISPOUT_VBUS_PATH_DIS);
|
|
|
|
|
|
|
|
if (ret < 0)
|
|
|
|
dev_err(&info->pdev->dev, "axp288 vbus path select %d\n", ret);
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int axp288_charger_enable_charger(struct axp288_chrg_info *info,
|
|
|
|
bool enable)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
if (enable)
|
|
|
|
ret = regmap_update_bits(info->regmap, AXP20X_CHRG_CTRL1,
|
|
|
|
CHRG_CCCV_CHG_EN, CHRG_CCCV_CHG_EN);
|
|
|
|
else
|
|
|
|
ret = regmap_update_bits(info->regmap, AXP20X_CHRG_CTRL1,
|
|
|
|
CHRG_CCCV_CHG_EN, 0);
|
|
|
|
if (ret < 0)
|
|
|
|
dev_err(&info->pdev->dev, "axp288 enable charger %d\n", ret);
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int axp288_charger_is_present(struct axp288_chrg_info *info)
|
|
|
|
{
|
|
|
|
int ret, present = 0;
|
|
|
|
unsigned int val;
|
|
|
|
|
|
|
|
ret = regmap_read(info->regmap, AXP20X_PWR_INPUT_STATUS, &val);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (val & PS_STAT_VBUS_PRESENT)
|
|
|
|
present = 1;
|
|
|
|
return present;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int axp288_charger_is_online(struct axp288_chrg_info *info)
|
|
|
|
{
|
|
|
|
int ret, online = 0;
|
|
|
|
unsigned int val;
|
|
|
|
|
|
|
|
ret = regmap_read(info->regmap, AXP20X_PWR_INPUT_STATUS, &val);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (val & PS_STAT_VBUS_VALID)
|
|
|
|
online = 1;
|
|
|
|
return online;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int axp288_get_charger_health(struct axp288_chrg_info *info)
|
|
|
|
{
|
|
|
|
int ret, pwr_stat, chrg_stat;
|
|
|
|
int health = POWER_SUPPLY_HEALTH_UNKNOWN;
|
|
|
|
unsigned int val;
|
|
|
|
|
|
|
|
ret = regmap_read(info->regmap, AXP20X_PWR_INPUT_STATUS, &val);
|
|
|
|
if ((ret < 0) || !(val & PS_STAT_VBUS_PRESENT))
|
|
|
|
goto health_read_fail;
|
|
|
|
else
|
|
|
|
pwr_stat = val;
|
|
|
|
|
|
|
|
ret = regmap_read(info->regmap, AXP20X_PWR_OP_MODE, &val);
|
|
|
|
if (ret < 0)
|
|
|
|
goto health_read_fail;
|
|
|
|
else
|
|
|
|
chrg_stat = val;
|
|
|
|
|
|
|
|
if (!(pwr_stat & PS_STAT_VBUS_VALID))
|
|
|
|
health = POWER_SUPPLY_HEALTH_DEAD;
|
|
|
|
else if (chrg_stat & CHRG_STAT_PMIC_OTP)
|
|
|
|
health = POWER_SUPPLY_HEALTH_OVERHEAT;
|
|
|
|
else if (chrg_stat & CHRG_STAT_BAT_SAFE_MODE)
|
|
|
|
health = POWER_SUPPLY_HEALTH_SAFETY_TIMER_EXPIRE;
|
|
|
|
else
|
|
|
|
health = POWER_SUPPLY_HEALTH_GOOD;
|
|
|
|
|
|
|
|
health_read_fail:
|
|
|
|
return health;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int axp288_charger_usb_set_property(struct power_supply *psy,
|
|
|
|
enum power_supply_property psp,
|
|
|
|
const union power_supply_propval *val)
|
|
|
|
{
|
|
|
|
struct axp288_chrg_info *info = power_supply_get_drvdata(psy);
|
|
|
|
int ret = 0;
|
|
|
|
int scaled_val;
|
|
|
|
|
|
|
|
switch (psp) {
|
|
|
|
case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
|
|
|
|
scaled_val = min(val->intval, info->max_cc);
|
|
|
|
scaled_val = DIV_ROUND_CLOSEST(scaled_val, 1000);
|
|
|
|
ret = axp288_charger_set_cc(info, scaled_val);
|
|
|
|
if (ret < 0)
|
|
|
|
dev_warn(&info->pdev->dev, "set charge current failed\n");
|
|
|
|
break;
|
|
|
|
case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
|
|
|
|
scaled_val = min(val->intval, info->max_cv);
|
|
|
|
scaled_val = DIV_ROUND_CLOSEST(scaled_val, 1000);
|
|
|
|
ret = axp288_charger_set_cv(info, scaled_val);
|
|
|
|
if (ret < 0)
|
|
|
|
dev_warn(&info->pdev->dev, "set charge voltage failed\n");
|
|
|
|
break;
|
2017-12-26 15:59:07 +03:00
|
|
|
case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
|
|
|
|
ret = axp288_charger_set_vbus_inlmt(info, val->intval);
|
|
|
|
if (ret < 0)
|
|
|
|
dev_warn(&info->pdev->dev, "set input current limit failed\n");
|
|
|
|
break;
|
2015-05-04 19:46:07 +03:00
|
|
|
default:
|
|
|
|
ret = -EINVAL;
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int axp288_charger_usb_get_property(struct power_supply *psy,
|
|
|
|
enum power_supply_property psp,
|
|
|
|
union power_supply_propval *val)
|
|
|
|
{
|
|
|
|
struct axp288_chrg_info *info = power_supply_get_drvdata(psy);
|
2017-12-26 15:59:04 +03:00
|
|
|
int ret;
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
switch (psp) {
|
|
|
|
case POWER_SUPPLY_PROP_PRESENT:
|
|
|
|
/* Check for OTG case first */
|
|
|
|
if (info->otg.id_short) {
|
|
|
|
val->intval = 0;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
ret = axp288_charger_is_present(info);
|
|
|
|
if (ret < 0)
|
2017-12-26 15:59:04 +03:00
|
|
|
return ret;
|
2016-12-21 17:36:53 +03:00
|
|
|
val->intval = ret;
|
2015-05-04 19:46:07 +03:00
|
|
|
break;
|
|
|
|
case POWER_SUPPLY_PROP_ONLINE:
|
|
|
|
/* Check for OTG case first */
|
|
|
|
if (info->otg.id_short) {
|
|
|
|
val->intval = 0;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
ret = axp288_charger_is_online(info);
|
|
|
|
if (ret < 0)
|
2017-12-26 15:59:04 +03:00
|
|
|
return ret;
|
2016-12-21 17:36:53 +03:00
|
|
|
val->intval = ret;
|
2015-05-04 19:46:07 +03:00
|
|
|
break;
|
|
|
|
case POWER_SUPPLY_PROP_HEALTH:
|
|
|
|
val->intval = axp288_get_charger_health(info);
|
|
|
|
break;
|
|
|
|
case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
|
|
|
|
val->intval = info->cc * 1000;
|
|
|
|
break;
|
|
|
|
case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX:
|
|
|
|
val->intval = info->max_cc * 1000;
|
|
|
|
break;
|
|
|
|
case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
|
|
|
|
val->intval = info->cv * 1000;
|
|
|
|
break;
|
|
|
|
case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX:
|
|
|
|
val->intval = info->max_cv * 1000;
|
|
|
|
break;
|
2017-12-26 15:59:07 +03:00
|
|
|
case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
|
2017-12-26 15:59:05 +03:00
|
|
|
ret = axp288_charger_get_vbus_inlmt(info);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
val->intval = ret;
|
2015-05-04 19:46:07 +03:00
|
|
|
break;
|
|
|
|
default:
|
2017-12-26 15:59:04 +03:00
|
|
|
return -EINVAL;
|
2015-05-04 19:46:07 +03:00
|
|
|
}
|
|
|
|
|
2017-12-26 15:59:04 +03:00
|
|
|
return 0;
|
2015-05-04 19:46:07 +03:00
|
|
|
}
|
|
|
|
|
|
|
|
static int axp288_charger_property_is_writeable(struct power_supply *psy,
|
|
|
|
enum power_supply_property psp)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
switch (psp) {
|
|
|
|
case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
|
|
|
|
case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
|
2017-12-26 15:59:07 +03:00
|
|
|
case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
|
2015-05-04 19:46:07 +03:00
|
|
|
ret = 1;
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
ret = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static enum power_supply_property axp288_usb_props[] = {
|
|
|
|
POWER_SUPPLY_PROP_PRESENT,
|
|
|
|
POWER_SUPPLY_PROP_ONLINE,
|
|
|
|
POWER_SUPPLY_PROP_TYPE,
|
|
|
|
POWER_SUPPLY_PROP_HEALTH,
|
|
|
|
POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT,
|
|
|
|
POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX,
|
|
|
|
POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE,
|
|
|
|
POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX,
|
2017-12-26 15:59:07 +03:00
|
|
|
POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT,
|
2015-05-04 19:46:07 +03:00
|
|
|
};
|
|
|
|
|
|
|
|
static const struct power_supply_desc axp288_charger_desc = {
|
|
|
|
.name = "axp288_charger",
|
|
|
|
.type = POWER_SUPPLY_TYPE_USB,
|
|
|
|
.properties = axp288_usb_props,
|
|
|
|
.num_properties = ARRAY_SIZE(axp288_usb_props),
|
|
|
|
.get_property = axp288_charger_usb_get_property,
|
|
|
|
.set_property = axp288_charger_usb_set_property,
|
|
|
|
.property_is_writeable = axp288_charger_property_is_writeable,
|
|
|
|
};
|
|
|
|
|
|
|
|
static irqreturn_t axp288_charger_irq_thread_handler(int irq, void *dev)
|
|
|
|
{
|
|
|
|
struct axp288_chrg_info *info = dev;
|
|
|
|
int i;
|
|
|
|
|
|
|
|
for (i = 0; i < CHRG_INTR_END; i++) {
|
|
|
|
if (info->irq[i] == irq)
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (i >= CHRG_INTR_END) {
|
|
|
|
dev_warn(&info->pdev->dev, "spurious interrupt!!\n");
|
|
|
|
return IRQ_NONE;
|
|
|
|
}
|
|
|
|
|
|
|
|
switch (i) {
|
|
|
|
case VBUS_OV_IRQ:
|
|
|
|
dev_dbg(&info->pdev->dev, "VBUS Over Voltage INTR\n");
|
|
|
|
break;
|
|
|
|
case CHARGE_DONE_IRQ:
|
|
|
|
dev_dbg(&info->pdev->dev, "Charging Done INTR\n");
|
|
|
|
break;
|
|
|
|
case CHARGE_CHARGING_IRQ:
|
|
|
|
dev_dbg(&info->pdev->dev, "Start Charging IRQ\n");
|
|
|
|
break;
|
|
|
|
case BAT_SAFE_QUIT_IRQ:
|
|
|
|
dev_dbg(&info->pdev->dev,
|
|
|
|
"Quit Safe Mode(restart timer) Charging IRQ\n");
|
|
|
|
break;
|
|
|
|
case BAT_SAFE_ENTER_IRQ:
|
|
|
|
dev_dbg(&info->pdev->dev,
|
|
|
|
"Enter Safe Mode(timer expire) Charging IRQ\n");
|
|
|
|
break;
|
|
|
|
case QCBTU_IRQ:
|
|
|
|
dev_dbg(&info->pdev->dev,
|
|
|
|
"Quit Battery Under Temperature(CHRG) INTR\n");
|
|
|
|
break;
|
|
|
|
case CBTU_IRQ:
|
|
|
|
dev_dbg(&info->pdev->dev,
|
|
|
|
"Hit Battery Under Temperature(CHRG) INTR\n");
|
|
|
|
break;
|
|
|
|
case QCBTO_IRQ:
|
|
|
|
dev_dbg(&info->pdev->dev,
|
|
|
|
"Quit Battery Over Temperature(CHRG) INTR\n");
|
|
|
|
break;
|
|
|
|
case CBTO_IRQ:
|
|
|
|
dev_dbg(&info->pdev->dev,
|
|
|
|
"Hit Battery Over Temperature(CHRG) INTR\n");
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
dev_warn(&info->pdev->dev, "Spurious Interrupt!!!\n");
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
|
|
|
|
power_supply_changed(info->psy_usb);
|
|
|
|
out:
|
|
|
|
return IRQ_HANDLED;
|
|
|
|
}
|
|
|
|
|
power: supply: axp288_charger: Add special handling for HP Pavilion x2 10
Some HP Pavilion x2 10 models use an AXP288 for charging and fuel-gauge.
We use a native power_supply / PMIC driver in this case, because on most
models with an AXP288 the ACPI AC / Battery code is either completely
missing or relies on custom / proprietary ACPI OpRegions which Linux
does not implement.
The native drivers mostly work fine, but there are 2 problems:
1. These model uses a Type-C connector for charging which the AXP288 does
not support. As long as a Type-A charger (which uses the USB data pins for
charger type detection) is used everything is fine. But if a Type-C
charger is used (such as the charger shipped with the device) then the
charger is not recognized.
So we end up slowly discharging the device even though a charger is
connected, because we are limiting the current from the charger to 500mA.
To make things worse this happens with the device's official charger.
Looking at the ACPI tables HP has "solved" the problem of the AXP288 not
being able to recognize Type-C chargers by simply always programming the
input-current-limit at 3000mA and relying on a Vhold setting of 4.7V
(normally 4.4V) to limit the current intake if the charger cannot handle
this.
2. If no charger is connected when the machine boots then it boots with the
vbus-path disabled. On other devices this is done when a 5V boost converter
is active to avoid the PMIC trying to charge from the 5V boost output.
This is done when an OTG host cable is inserted and the ID pin on the
micro-B receptacle is pulled low, the ID pin has an ACPI event handler
associated with it which re-enables the vbus-path when the ID pin is pulled
high when the OTG cable is removed. The Type-C connector has no ID pin,
there is no ID pin handler and there appears to be no 5V boost converter,
so we end up not charging because the vbus-path is disabled, until we
unplug the charger which automatically clears the vbus-path disable bit and
then on the second plug-in of the adapter we start charging.
The HP Pavilion x2 10 models with an AXP288 do have mostly working ACPI
AC / Battery code which does not rely on custom / proprietary ACPI
OpRegions. So one possible solution would be to blacklist the AXP288
native power_supply drivers and add the HP Pavilion x2 10 with AXP288
DMI ids to the list of devices which should use the ACPI AC / Battery
code even though they have an AXP288 PMIC. This would require changes to
4 files: drivers/acpi/ac.c, drivers/power/supply/axp288_charger.c,
drivers/acpi/battery.c and drivers/power/supply/axp288_fuel_gauge.c.
Beside needing adding the same DMI matches to 4 different files, this
approach also triggers problem 2. from above, but then when suspended,
during suspend the machine will not wakeup because the vbus path is
disabled by the AML code when not charging, so the Vbus low-to-high
IRQ is not triggered, the CPU never wakes up and the device does not
charge even though the user likely things it is charging, esp. since
the charge status LED is directly coupled to an adapter being plugged
in and does not reflect actual charging.
This could be worked by enabling vbus-path explicitly from say the
axp288_charger driver's suspend handler.
So neither situation is ideal, in both cased we need to explicitly enable
the vbus-path to work around different variants of problem 2 above, this
requires a quirk in the axp288_charger code.
If we go the route of using the ACPI AC / Battery drivers then we need
modifications to 3 other drivers; and we need to partially disable the
axp288_charger code, while at the same time keeping it around to enable
vbus-path on suspend.
OTOH we can copy the hardcoding of 3A input-current-limit (we never touch
Vhold, so that would stay at 4.7V) to the axp288_charger code, which needs
changes regardless, then we concentrate all special handling of this
interesting device model in the axp288_charger code. That is what this
commit does.
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1791098
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
2020-02-23 18:32:08 +03:00
|
|
|
/*
|
|
|
|
* The HP Pavilion x2 10 series comes in a number of variants:
|
2020-11-18 15:13:12 +03:00
|
|
|
* Bay Trail SoC + AXP288 PMIC, Micro-USB, DMI_BOARD_NAME: "8021"
|
|
|
|
* Bay Trail SoC + AXP288 PMIC, Type-C, DMI_BOARD_NAME: "815D"
|
|
|
|
* Cherry Trail SoC + AXP288 PMIC, Type-C, DMI_BOARD_NAME: "813E"
|
|
|
|
* Cherry Trail SoC + TI PMIC, Type-C, DMI_BOARD_NAME: "827C" or "82F4"
|
power: supply: axp288_charger: Add special handling for HP Pavilion x2 10
Some HP Pavilion x2 10 models use an AXP288 for charging and fuel-gauge.
We use a native power_supply / PMIC driver in this case, because on most
models with an AXP288 the ACPI AC / Battery code is either completely
missing or relies on custom / proprietary ACPI OpRegions which Linux
does not implement.
The native drivers mostly work fine, but there are 2 problems:
1. These model uses a Type-C connector for charging which the AXP288 does
not support. As long as a Type-A charger (which uses the USB data pins for
charger type detection) is used everything is fine. But if a Type-C
charger is used (such as the charger shipped with the device) then the
charger is not recognized.
So we end up slowly discharging the device even though a charger is
connected, because we are limiting the current from the charger to 500mA.
To make things worse this happens with the device's official charger.
Looking at the ACPI tables HP has "solved" the problem of the AXP288 not
being able to recognize Type-C chargers by simply always programming the
input-current-limit at 3000mA and relying on a Vhold setting of 4.7V
(normally 4.4V) to limit the current intake if the charger cannot handle
this.
2. If no charger is connected when the machine boots then it boots with the
vbus-path disabled. On other devices this is done when a 5V boost converter
is active to avoid the PMIC trying to charge from the 5V boost output.
This is done when an OTG host cable is inserted and the ID pin on the
micro-B receptacle is pulled low, the ID pin has an ACPI event handler
associated with it which re-enables the vbus-path when the ID pin is pulled
high when the OTG cable is removed. The Type-C connector has no ID pin,
there is no ID pin handler and there appears to be no 5V boost converter,
so we end up not charging because the vbus-path is disabled, until we
unplug the charger which automatically clears the vbus-path disable bit and
then on the second plug-in of the adapter we start charging.
The HP Pavilion x2 10 models with an AXP288 do have mostly working ACPI
AC / Battery code which does not rely on custom / proprietary ACPI
OpRegions. So one possible solution would be to blacklist the AXP288
native power_supply drivers and add the HP Pavilion x2 10 with AXP288
DMI ids to the list of devices which should use the ACPI AC / Battery
code even though they have an AXP288 PMIC. This would require changes to
4 files: drivers/acpi/ac.c, drivers/power/supply/axp288_charger.c,
drivers/acpi/battery.c and drivers/power/supply/axp288_fuel_gauge.c.
Beside needing adding the same DMI matches to 4 different files, this
approach also triggers problem 2. from above, but then when suspended,
during suspend the machine will not wakeup because the vbus path is
disabled by the AML code when not charging, so the Vbus low-to-high
IRQ is not triggered, the CPU never wakes up and the device does not
charge even though the user likely things it is charging, esp. since
the charge status LED is directly coupled to an adapter being plugged
in and does not reflect actual charging.
This could be worked by enabling vbus-path explicitly from say the
axp288_charger driver's suspend handler.
So neither situation is ideal, in both cased we need to explicitly enable
the vbus-path to work around different variants of problem 2 above, this
requires a quirk in the axp288_charger code.
If we go the route of using the ACPI AC / Battery drivers then we need
modifications to 3 other drivers; and we need to partially disable the
axp288_charger code, while at the same time keeping it around to enable
vbus-path on suspend.
OTOH we can copy the hardcoding of 3A input-current-limit (we never touch
Vhold, so that would stay at 4.7V) to the axp288_charger code, which needs
changes regardless, then we concentrate all special handling of this
interesting device model in the axp288_charger code. That is what this
commit does.
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1791098
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
2020-02-23 18:32:08 +03:00
|
|
|
*
|
2020-11-18 15:13:12 +03:00
|
|
|
* The variants with the AXP288 + Type-C connector are all kinds of special:
|
power: supply: axp288_charger: Add special handling for HP Pavilion x2 10
Some HP Pavilion x2 10 models use an AXP288 for charging and fuel-gauge.
We use a native power_supply / PMIC driver in this case, because on most
models with an AXP288 the ACPI AC / Battery code is either completely
missing or relies on custom / proprietary ACPI OpRegions which Linux
does not implement.
The native drivers mostly work fine, but there are 2 problems:
1. These model uses a Type-C connector for charging which the AXP288 does
not support. As long as a Type-A charger (which uses the USB data pins for
charger type detection) is used everything is fine. But if a Type-C
charger is used (such as the charger shipped with the device) then the
charger is not recognized.
So we end up slowly discharging the device even though a charger is
connected, because we are limiting the current from the charger to 500mA.
To make things worse this happens with the device's official charger.
Looking at the ACPI tables HP has "solved" the problem of the AXP288 not
being able to recognize Type-C chargers by simply always programming the
input-current-limit at 3000mA and relying on a Vhold setting of 4.7V
(normally 4.4V) to limit the current intake if the charger cannot handle
this.
2. If no charger is connected when the machine boots then it boots with the
vbus-path disabled. On other devices this is done when a 5V boost converter
is active to avoid the PMIC trying to charge from the 5V boost output.
This is done when an OTG host cable is inserted and the ID pin on the
micro-B receptacle is pulled low, the ID pin has an ACPI event handler
associated with it which re-enables the vbus-path when the ID pin is pulled
high when the OTG cable is removed. The Type-C connector has no ID pin,
there is no ID pin handler and there appears to be no 5V boost converter,
so we end up not charging because the vbus-path is disabled, until we
unplug the charger which automatically clears the vbus-path disable bit and
then on the second plug-in of the adapter we start charging.
The HP Pavilion x2 10 models with an AXP288 do have mostly working ACPI
AC / Battery code which does not rely on custom / proprietary ACPI
OpRegions. So one possible solution would be to blacklist the AXP288
native power_supply drivers and add the HP Pavilion x2 10 with AXP288
DMI ids to the list of devices which should use the ACPI AC / Battery
code even though they have an AXP288 PMIC. This would require changes to
4 files: drivers/acpi/ac.c, drivers/power/supply/axp288_charger.c,
drivers/acpi/battery.c and drivers/power/supply/axp288_fuel_gauge.c.
Beside needing adding the same DMI matches to 4 different files, this
approach also triggers problem 2. from above, but then when suspended,
during suspend the machine will not wakeup because the vbus path is
disabled by the AML code when not charging, so the Vbus low-to-high
IRQ is not triggered, the CPU never wakes up and the device does not
charge even though the user likely things it is charging, esp. since
the charge status LED is directly coupled to an adapter being plugged
in and does not reflect actual charging.
This could be worked by enabling vbus-path explicitly from say the
axp288_charger driver's suspend handler.
So neither situation is ideal, in both cased we need to explicitly enable
the vbus-path to work around different variants of problem 2 above, this
requires a quirk in the axp288_charger code.
If we go the route of using the ACPI AC / Battery drivers then we need
modifications to 3 other drivers; and we need to partially disable the
axp288_charger code, while at the same time keeping it around to enable
vbus-path on suspend.
OTOH we can copy the hardcoding of 3A input-current-limit (we never touch
Vhold, so that would stay at 4.7V) to the axp288_charger code, which needs
changes regardless, then we concentrate all special handling of this
interesting device model in the axp288_charger code. That is what this
commit does.
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1791098
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
2020-02-23 18:32:08 +03:00
|
|
|
*
|
2020-11-18 15:13:12 +03:00
|
|
|
* 1. They use a Type-C connector which the AXP288 does not support, so when
|
|
|
|
* using a Type-C charger it is not recognized. Unlike most AXP288 devices,
|
power: supply: axp288_charger: Add special handling for HP Pavilion x2 10
Some HP Pavilion x2 10 models use an AXP288 for charging and fuel-gauge.
We use a native power_supply / PMIC driver in this case, because on most
models with an AXP288 the ACPI AC / Battery code is either completely
missing or relies on custom / proprietary ACPI OpRegions which Linux
does not implement.
The native drivers mostly work fine, but there are 2 problems:
1. These model uses a Type-C connector for charging which the AXP288 does
not support. As long as a Type-A charger (which uses the USB data pins for
charger type detection) is used everything is fine. But if a Type-C
charger is used (such as the charger shipped with the device) then the
charger is not recognized.
So we end up slowly discharging the device even though a charger is
connected, because we are limiting the current from the charger to 500mA.
To make things worse this happens with the device's official charger.
Looking at the ACPI tables HP has "solved" the problem of the AXP288 not
being able to recognize Type-C chargers by simply always programming the
input-current-limit at 3000mA and relying on a Vhold setting of 4.7V
(normally 4.4V) to limit the current intake if the charger cannot handle
this.
2. If no charger is connected when the machine boots then it boots with the
vbus-path disabled. On other devices this is done when a 5V boost converter
is active to avoid the PMIC trying to charge from the 5V boost output.
This is done when an OTG host cable is inserted and the ID pin on the
micro-B receptacle is pulled low, the ID pin has an ACPI event handler
associated with it which re-enables the vbus-path when the ID pin is pulled
high when the OTG cable is removed. The Type-C connector has no ID pin,
there is no ID pin handler and there appears to be no 5V boost converter,
so we end up not charging because the vbus-path is disabled, until we
unplug the charger which automatically clears the vbus-path disable bit and
then on the second plug-in of the adapter we start charging.
The HP Pavilion x2 10 models with an AXP288 do have mostly working ACPI
AC / Battery code which does not rely on custom / proprietary ACPI
OpRegions. So one possible solution would be to blacklist the AXP288
native power_supply drivers and add the HP Pavilion x2 10 with AXP288
DMI ids to the list of devices which should use the ACPI AC / Battery
code even though they have an AXP288 PMIC. This would require changes to
4 files: drivers/acpi/ac.c, drivers/power/supply/axp288_charger.c,
drivers/acpi/battery.c and drivers/power/supply/axp288_fuel_gauge.c.
Beside needing adding the same DMI matches to 4 different files, this
approach also triggers problem 2. from above, but then when suspended,
during suspend the machine will not wakeup because the vbus path is
disabled by the AML code when not charging, so the Vbus low-to-high
IRQ is not triggered, the CPU never wakes up and the device does not
charge even though the user likely things it is charging, esp. since
the charge status LED is directly coupled to an adapter being plugged
in and does not reflect actual charging.
This could be worked by enabling vbus-path explicitly from say the
axp288_charger driver's suspend handler.
So neither situation is ideal, in both cased we need to explicitly enable
the vbus-path to work around different variants of problem 2 above, this
requires a quirk in the axp288_charger code.
If we go the route of using the ACPI AC / Battery drivers then we need
modifications to 3 other drivers; and we need to partially disable the
axp288_charger code, while at the same time keeping it around to enable
vbus-path on suspend.
OTOH we can copy the hardcoding of 3A input-current-limit (we never touch
Vhold, so that would stay at 4.7V) to the axp288_charger code, which needs
changes regardless, then we concentrate all special handling of this
interesting device model in the axp288_charger code. That is what this
commit does.
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1791098
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
2020-02-23 18:32:08 +03:00
|
|
|
* this model actually has mostly working ACPI AC / Battery code, the ACPI code
|
|
|
|
* "solves" this by simply setting the input_current_limit to 3A.
|
|
|
|
* There are still some issues with the ACPI code, so we use this native driver,
|
|
|
|
* and to solve the charging not working (500mA is not enough) issue we hardcode
|
|
|
|
* the 3A input_current_limit like the ACPI code does.
|
|
|
|
*
|
|
|
|
* 2. If no charger is connected the machine boots with the vbus-path disabled.
|
|
|
|
* Normally this is done when a 5V boost converter is active to avoid the PMIC
|
|
|
|
* trying to charge from the 5V boost converter's output. This is done when
|
|
|
|
* an OTG host cable is inserted and the ID pin on the micro-B receptacle is
|
|
|
|
* pulled low and the ID pin has an ACPI event handler associated with it
|
|
|
|
* which re-enables the vbus-path when the ID pin is pulled high when the
|
|
|
|
* OTG host cable is removed. The Type-C connector has no ID pin, there is
|
|
|
|
* no ID pin handler and there appears to be no 5V boost converter, so we
|
|
|
|
* end up not charging because the vbus-path is disabled, until we unplug
|
|
|
|
* the charger which automatically clears the vbus-path disable bit and then
|
|
|
|
* on the second plug-in of the adapter we start charging. To solve the not
|
|
|
|
* charging on first charger plugin we unconditionally enable the vbus-path at
|
|
|
|
* probe on this model, which is safe since there is no 5V boost converter.
|
|
|
|
*/
|
|
|
|
static const struct dmi_system_id axp288_hp_x2_dmi_ids[] = {
|
|
|
|
{
|
|
|
|
.matches = {
|
2020-11-18 15:13:12 +03:00
|
|
|
DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
|
|
|
|
DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "HP Pavilion x2 Detachable"),
|
|
|
|
DMI_EXACT_MATCH(DMI_BOARD_NAME, "815D"),
|
|
|
|
},
|
|
|
|
},
|
|
|
|
{
|
|
|
|
.matches = {
|
|
|
|
DMI_EXACT_MATCH(DMI_SYS_VENDOR, "HP"),
|
|
|
|
DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "HP Pavilion x2 Detachable"),
|
|
|
|
DMI_EXACT_MATCH(DMI_BOARD_NAME, "813E"),
|
power: supply: axp288_charger: Add special handling for HP Pavilion x2 10
Some HP Pavilion x2 10 models use an AXP288 for charging and fuel-gauge.
We use a native power_supply / PMIC driver in this case, because on most
models with an AXP288 the ACPI AC / Battery code is either completely
missing or relies on custom / proprietary ACPI OpRegions which Linux
does not implement.
The native drivers mostly work fine, but there are 2 problems:
1. These model uses a Type-C connector for charging which the AXP288 does
not support. As long as a Type-A charger (which uses the USB data pins for
charger type detection) is used everything is fine. But if a Type-C
charger is used (such as the charger shipped with the device) then the
charger is not recognized.
So we end up slowly discharging the device even though a charger is
connected, because we are limiting the current from the charger to 500mA.
To make things worse this happens with the device's official charger.
Looking at the ACPI tables HP has "solved" the problem of the AXP288 not
being able to recognize Type-C chargers by simply always programming the
input-current-limit at 3000mA and relying on a Vhold setting of 4.7V
(normally 4.4V) to limit the current intake if the charger cannot handle
this.
2. If no charger is connected when the machine boots then it boots with the
vbus-path disabled. On other devices this is done when a 5V boost converter
is active to avoid the PMIC trying to charge from the 5V boost output.
This is done when an OTG host cable is inserted and the ID pin on the
micro-B receptacle is pulled low, the ID pin has an ACPI event handler
associated with it which re-enables the vbus-path when the ID pin is pulled
high when the OTG cable is removed. The Type-C connector has no ID pin,
there is no ID pin handler and there appears to be no 5V boost converter,
so we end up not charging because the vbus-path is disabled, until we
unplug the charger which automatically clears the vbus-path disable bit and
then on the second plug-in of the adapter we start charging.
The HP Pavilion x2 10 models with an AXP288 do have mostly working ACPI
AC / Battery code which does not rely on custom / proprietary ACPI
OpRegions. So one possible solution would be to blacklist the AXP288
native power_supply drivers and add the HP Pavilion x2 10 with AXP288
DMI ids to the list of devices which should use the ACPI AC / Battery
code even though they have an AXP288 PMIC. This would require changes to
4 files: drivers/acpi/ac.c, drivers/power/supply/axp288_charger.c,
drivers/acpi/battery.c and drivers/power/supply/axp288_fuel_gauge.c.
Beside needing adding the same DMI matches to 4 different files, this
approach also triggers problem 2. from above, but then when suspended,
during suspend the machine will not wakeup because the vbus path is
disabled by the AML code when not charging, so the Vbus low-to-high
IRQ is not triggered, the CPU never wakes up and the device does not
charge even though the user likely things it is charging, esp. since
the charge status LED is directly coupled to an adapter being plugged
in and does not reflect actual charging.
This could be worked by enabling vbus-path explicitly from say the
axp288_charger driver's suspend handler.
So neither situation is ideal, in both cased we need to explicitly enable
the vbus-path to work around different variants of problem 2 above, this
requires a quirk in the axp288_charger code.
If we go the route of using the ACPI AC / Battery drivers then we need
modifications to 3 other drivers; and we need to partially disable the
axp288_charger code, while at the same time keeping it around to enable
vbus-path on suspend.
OTOH we can copy the hardcoding of 3A input-current-limit (we never touch
Vhold, so that would stay at 4.7V) to the axp288_charger code, which needs
changes regardless, then we concentrate all special handling of this
interesting device model in the axp288_charger code. That is what this
commit does.
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1791098
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
2020-02-23 18:32:08 +03:00
|
|
|
},
|
|
|
|
},
|
|
|
|
{} /* Terminating entry */
|
|
|
|
};
|
|
|
|
|
2015-05-04 19:46:07 +03:00
|
|
|
static void axp288_charger_extcon_evt_worker(struct work_struct *work)
|
|
|
|
{
|
|
|
|
struct axp288_chrg_info *info =
|
|
|
|
container_of(work, struct axp288_chrg_info, cable.work);
|
|
|
|
int ret, current_limit;
|
|
|
|
struct extcon_dev *edev = info->cable.edev;
|
2017-12-26 15:58:59 +03:00
|
|
|
unsigned int val;
|
|
|
|
|
|
|
|
ret = regmap_read(info->regmap, AXP20X_PWR_INPUT_STATUS, &val);
|
|
|
|
if (ret < 0) {
|
|
|
|
dev_err(&info->pdev->dev, "Error reading status (%d)\n", ret);
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Offline? Disable charging and bail */
|
|
|
|
if (!(val & PS_STAT_VBUS_VALID)) {
|
|
|
|
dev_dbg(&info->pdev->dev, "USB charger disconnected\n");
|
|
|
|
axp288_charger_enable_charger(info, false);
|
|
|
|
power_supply_changed(info->psy_usb);
|
|
|
|
return;
|
|
|
|
}
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
/* Determine cable/charger type */
|
power: supply: axp288_charger: Add special handling for HP Pavilion x2 10
Some HP Pavilion x2 10 models use an AXP288 for charging and fuel-gauge.
We use a native power_supply / PMIC driver in this case, because on most
models with an AXP288 the ACPI AC / Battery code is either completely
missing or relies on custom / proprietary ACPI OpRegions which Linux
does not implement.
The native drivers mostly work fine, but there are 2 problems:
1. These model uses a Type-C connector for charging which the AXP288 does
not support. As long as a Type-A charger (which uses the USB data pins for
charger type detection) is used everything is fine. But if a Type-C
charger is used (such as the charger shipped with the device) then the
charger is not recognized.
So we end up slowly discharging the device even though a charger is
connected, because we are limiting the current from the charger to 500mA.
To make things worse this happens with the device's official charger.
Looking at the ACPI tables HP has "solved" the problem of the AXP288 not
being able to recognize Type-C chargers by simply always programming the
input-current-limit at 3000mA and relying on a Vhold setting of 4.7V
(normally 4.4V) to limit the current intake if the charger cannot handle
this.
2. If no charger is connected when the machine boots then it boots with the
vbus-path disabled. On other devices this is done when a 5V boost converter
is active to avoid the PMIC trying to charge from the 5V boost output.
This is done when an OTG host cable is inserted and the ID pin on the
micro-B receptacle is pulled low, the ID pin has an ACPI event handler
associated with it which re-enables the vbus-path when the ID pin is pulled
high when the OTG cable is removed. The Type-C connector has no ID pin,
there is no ID pin handler and there appears to be no 5V boost converter,
so we end up not charging because the vbus-path is disabled, until we
unplug the charger which automatically clears the vbus-path disable bit and
then on the second plug-in of the adapter we start charging.
The HP Pavilion x2 10 models with an AXP288 do have mostly working ACPI
AC / Battery code which does not rely on custom / proprietary ACPI
OpRegions. So one possible solution would be to blacklist the AXP288
native power_supply drivers and add the HP Pavilion x2 10 with AXP288
DMI ids to the list of devices which should use the ACPI AC / Battery
code even though they have an AXP288 PMIC. This would require changes to
4 files: drivers/acpi/ac.c, drivers/power/supply/axp288_charger.c,
drivers/acpi/battery.c and drivers/power/supply/axp288_fuel_gauge.c.
Beside needing adding the same DMI matches to 4 different files, this
approach also triggers problem 2. from above, but then when suspended,
during suspend the machine will not wakeup because the vbus path is
disabled by the AML code when not charging, so the Vbus low-to-high
IRQ is not triggered, the CPU never wakes up and the device does not
charge even though the user likely things it is charging, esp. since
the charge status LED is directly coupled to an adapter being plugged
in and does not reflect actual charging.
This could be worked by enabling vbus-path explicitly from say the
axp288_charger driver's suspend handler.
So neither situation is ideal, in both cased we need to explicitly enable
the vbus-path to work around different variants of problem 2 above, this
requires a quirk in the axp288_charger code.
If we go the route of using the ACPI AC / Battery drivers then we need
modifications to 3 other drivers; and we need to partially disable the
axp288_charger code, while at the same time keeping it around to enable
vbus-path on suspend.
OTOH we can copy the hardcoding of 3A input-current-limit (we never touch
Vhold, so that would stay at 4.7V) to the axp288_charger code, which needs
changes regardless, then we concentrate all special handling of this
interesting device model in the axp288_charger code. That is what this
commit does.
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1791098
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
2020-02-23 18:32:08 +03:00
|
|
|
if (dmi_check_system(axp288_hp_x2_dmi_ids)) {
|
|
|
|
/* See comment above axp288_hp_x2_dmi_ids declaration */
|
|
|
|
dev_dbg(&info->pdev->dev, "HP X2 with Type-C, setting inlmt to 3A\n");
|
|
|
|
current_limit = 3000000;
|
|
|
|
} else if (extcon_get_state(edev, EXTCON_CHG_USB_SDP) > 0) {
|
2017-12-26 15:59:00 +03:00
|
|
|
dev_dbg(&info->pdev->dev, "USB SDP charger is connected\n");
|
2017-12-26 15:59:06 +03:00
|
|
|
current_limit = 500000;
|
2016-11-30 08:57:31 +03:00
|
|
|
} else if (extcon_get_state(edev, EXTCON_CHG_USB_CDP) > 0) {
|
2017-12-26 15:59:00 +03:00
|
|
|
dev_dbg(&info->pdev->dev, "USB CDP charger is connected\n");
|
2017-12-26 15:59:06 +03:00
|
|
|
current_limit = 1500000;
|
2016-11-30 08:57:31 +03:00
|
|
|
} else if (extcon_get_state(edev, EXTCON_CHG_USB_DCP) > 0) {
|
2017-12-26 15:59:00 +03:00
|
|
|
dev_dbg(&info->pdev->dev, "USB DCP charger is connected\n");
|
2017-12-26 15:59:06 +03:00
|
|
|
current_limit = 2000000;
|
2015-05-04 19:46:07 +03:00
|
|
|
} else {
|
2017-12-26 15:58:59 +03:00
|
|
|
/* Charger type detection still in progress, bail. */
|
2015-05-04 19:46:07 +03:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2017-12-26 15:58:59 +03:00
|
|
|
/* Set vbus current limit first, then enable charger */
|
|
|
|
ret = axp288_charger_set_vbus_inlmt(info, current_limit);
|
|
|
|
if (ret == 0)
|
|
|
|
axp288_charger_enable_charger(info, true);
|
|
|
|
else
|
|
|
|
dev_err(&info->pdev->dev,
|
|
|
|
"error setting current limit (%d)\n", ret);
|
2015-05-04 19:46:07 +03:00
|
|
|
|
2016-12-21 17:36:53 +03:00
|
|
|
power_supply_changed(info->psy_usb);
|
2015-05-04 19:46:07 +03:00
|
|
|
}
|
|
|
|
|
2017-12-26 15:59:08 +03:00
|
|
|
static int axp288_charger_handle_cable_evt(struct notifier_block *nb,
|
|
|
|
unsigned long event, void *param)
|
2016-12-21 20:28:23 +03:00
|
|
|
{
|
|
|
|
struct axp288_chrg_info *info =
|
2017-12-26 15:59:08 +03:00
|
|
|
container_of(nb, struct axp288_chrg_info, cable.nb);
|
2016-12-21 20:28:23 +03:00
|
|
|
schedule_work(&info->cable.work);
|
2015-05-04 19:46:07 +03:00
|
|
|
return NOTIFY_OK;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void axp288_charger_otg_evt_worker(struct work_struct *work)
|
|
|
|
{
|
|
|
|
struct axp288_chrg_info *info =
|
|
|
|
container_of(work, struct axp288_chrg_info, otg.work);
|
2016-12-21 17:36:54 +03:00
|
|
|
struct extcon_dev *edev = info->otg.cable;
|
|
|
|
int ret, usb_host = extcon_get_state(edev, EXTCON_USB_HOST);
|
|
|
|
|
|
|
|
dev_dbg(&info->pdev->dev, "external connector USB-Host is %s\n",
|
|
|
|
usb_host ? "attached" : "detached");
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Set usb_id_short flag to avoid running charger detection logic
|
|
|
|
* in case usb host.
|
|
|
|
*/
|
|
|
|
info->otg.id_short = usb_host;
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
/* Disable VBUS path before enabling the 5V boost */
|
|
|
|
ret = axp288_charger_vbus_path_select(info, !info->otg.id_short);
|
|
|
|
if (ret < 0)
|
|
|
|
dev_warn(&info->pdev->dev, "vbus path disable failed\n");
|
|
|
|
}
|
|
|
|
|
|
|
|
static int axp288_charger_handle_otg_evt(struct notifier_block *nb,
|
|
|
|
unsigned long event, void *param)
|
|
|
|
{
|
|
|
|
struct axp288_chrg_info *info =
|
|
|
|
container_of(nb, struct axp288_chrg_info, otg.id_nb);
|
|
|
|
|
|
|
|
schedule_work(&info->otg.work);
|
|
|
|
|
|
|
|
return NOTIFY_OK;
|
|
|
|
}
|
|
|
|
|
2016-12-21 17:36:46 +03:00
|
|
|
static int charger_init_hw_regs(struct axp288_chrg_info *info)
|
2015-05-04 19:46:07 +03:00
|
|
|
{
|
|
|
|
int ret, cc, cv;
|
|
|
|
unsigned int val;
|
|
|
|
|
|
|
|
/* Program temperature thresholds */
|
|
|
|
ret = regmap_write(info->regmap, AXP20X_V_LTF_CHRG, CHRG_VLTFC_0C);
|
2016-12-21 17:36:46 +03:00
|
|
|
if (ret < 0) {
|
|
|
|
dev_err(&info->pdev->dev, "register(%x) write error(%d)\n",
|
2015-05-04 19:46:07 +03:00
|
|
|
AXP20X_V_LTF_CHRG, ret);
|
2016-12-21 17:36:46 +03:00
|
|
|
return ret;
|
|
|
|
}
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
ret = regmap_write(info->regmap, AXP20X_V_HTF_CHRG, CHRG_VHTFC_45C);
|
2016-12-21 17:36:46 +03:00
|
|
|
if (ret < 0) {
|
|
|
|
dev_err(&info->pdev->dev, "register(%x) write error(%d)\n",
|
2015-05-04 19:46:07 +03:00
|
|
|
AXP20X_V_HTF_CHRG, ret);
|
2016-12-21 17:36:46 +03:00
|
|
|
return ret;
|
|
|
|
}
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
/* Do not turn-off charger o/p after charge cycle ends */
|
|
|
|
ret = regmap_update_bits(info->regmap,
|
|
|
|
AXP20X_CHRG_CTRL2,
|
2016-12-21 17:36:55 +03:00
|
|
|
CNTL2_CHG_OUT_TURNON, CNTL2_CHG_OUT_TURNON);
|
2016-12-21 17:36:46 +03:00
|
|
|
if (ret < 0) {
|
|
|
|
dev_err(&info->pdev->dev, "register(%x) write error(%d)\n",
|
2015-05-04 19:46:07 +03:00
|
|
|
AXP20X_CHRG_CTRL2, ret);
|
2016-12-21 17:36:46 +03:00
|
|
|
return ret;
|
|
|
|
}
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
/* Setup ending condition for charging to be 10% of I(chrg) */
|
|
|
|
ret = regmap_update_bits(info->regmap,
|
|
|
|
AXP20X_CHRG_CTRL1,
|
|
|
|
CHRG_CCCV_ITERM_20P, 0);
|
2016-12-21 17:36:46 +03:00
|
|
|
if (ret < 0) {
|
|
|
|
dev_err(&info->pdev->dev, "register(%x) write error(%d)\n",
|
2015-05-04 19:46:07 +03:00
|
|
|
AXP20X_CHRG_CTRL1, ret);
|
2016-12-21 17:36:46 +03:00
|
|
|
return ret;
|
|
|
|
}
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
/* Disable OCV-SOC curve calibration */
|
|
|
|
ret = regmap_update_bits(info->regmap,
|
|
|
|
AXP20X_CC_CTRL,
|
|
|
|
FG_CNTL_OCV_ADJ_EN, 0);
|
2016-12-21 17:36:46 +03:00
|
|
|
if (ret < 0) {
|
|
|
|
dev_err(&info->pdev->dev, "register(%x) write error(%d)\n",
|
2015-05-04 19:46:07 +03:00
|
|
|
AXP20X_CC_CTRL, ret);
|
2016-12-21 17:36:46 +03:00
|
|
|
return ret;
|
|
|
|
}
|
2015-05-04 19:46:07 +03:00
|
|
|
|
power: supply: axp288_charger: Add special handling for HP Pavilion x2 10
Some HP Pavilion x2 10 models use an AXP288 for charging and fuel-gauge.
We use a native power_supply / PMIC driver in this case, because on most
models with an AXP288 the ACPI AC / Battery code is either completely
missing or relies on custom / proprietary ACPI OpRegions which Linux
does not implement.
The native drivers mostly work fine, but there are 2 problems:
1. These model uses a Type-C connector for charging which the AXP288 does
not support. As long as a Type-A charger (which uses the USB data pins for
charger type detection) is used everything is fine. But if a Type-C
charger is used (such as the charger shipped with the device) then the
charger is not recognized.
So we end up slowly discharging the device even though a charger is
connected, because we are limiting the current from the charger to 500mA.
To make things worse this happens with the device's official charger.
Looking at the ACPI tables HP has "solved" the problem of the AXP288 not
being able to recognize Type-C chargers by simply always programming the
input-current-limit at 3000mA and relying on a Vhold setting of 4.7V
(normally 4.4V) to limit the current intake if the charger cannot handle
this.
2. If no charger is connected when the machine boots then it boots with the
vbus-path disabled. On other devices this is done when a 5V boost converter
is active to avoid the PMIC trying to charge from the 5V boost output.
This is done when an OTG host cable is inserted and the ID pin on the
micro-B receptacle is pulled low, the ID pin has an ACPI event handler
associated with it which re-enables the vbus-path when the ID pin is pulled
high when the OTG cable is removed. The Type-C connector has no ID pin,
there is no ID pin handler and there appears to be no 5V boost converter,
so we end up not charging because the vbus-path is disabled, until we
unplug the charger which automatically clears the vbus-path disable bit and
then on the second plug-in of the adapter we start charging.
The HP Pavilion x2 10 models with an AXP288 do have mostly working ACPI
AC / Battery code which does not rely on custom / proprietary ACPI
OpRegions. So one possible solution would be to blacklist the AXP288
native power_supply drivers and add the HP Pavilion x2 10 with AXP288
DMI ids to the list of devices which should use the ACPI AC / Battery
code even though they have an AXP288 PMIC. This would require changes to
4 files: drivers/acpi/ac.c, drivers/power/supply/axp288_charger.c,
drivers/acpi/battery.c and drivers/power/supply/axp288_fuel_gauge.c.
Beside needing adding the same DMI matches to 4 different files, this
approach also triggers problem 2. from above, but then when suspended,
during suspend the machine will not wakeup because the vbus path is
disabled by the AML code when not charging, so the Vbus low-to-high
IRQ is not triggered, the CPU never wakes up and the device does not
charge even though the user likely things it is charging, esp. since
the charge status LED is directly coupled to an adapter being plugged
in and does not reflect actual charging.
This could be worked by enabling vbus-path explicitly from say the
axp288_charger driver's suspend handler.
So neither situation is ideal, in both cased we need to explicitly enable
the vbus-path to work around different variants of problem 2 above, this
requires a quirk in the axp288_charger code.
If we go the route of using the ACPI AC / Battery drivers then we need
modifications to 3 other drivers; and we need to partially disable the
axp288_charger code, while at the same time keeping it around to enable
vbus-path on suspend.
OTOH we can copy the hardcoding of 3A input-current-limit (we never touch
Vhold, so that would stay at 4.7V) to the axp288_charger code, which needs
changes regardless, then we concentrate all special handling of this
interesting device model in the axp288_charger code. That is what this
commit does.
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1791098
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
2020-02-23 18:32:08 +03:00
|
|
|
if (dmi_check_system(axp288_hp_x2_dmi_ids)) {
|
|
|
|
/* See comment above axp288_hp_x2_dmi_ids declaration */
|
|
|
|
ret = axp288_charger_vbus_path_select(info, true);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
2022-02-08 15:51:47 +03:00
|
|
|
} else {
|
|
|
|
/* Set Vhold to the factory default / recommended 4.4V */
|
|
|
|
val = VBUS_ISPOUT_VHOLD_SET_4400MV << VBUS_ISPOUT_VHOLD_SET_BIT_POS;
|
|
|
|
ret = regmap_update_bits(info->regmap, AXP20X_VBUS_IPSOUT_MGMT,
|
|
|
|
VBUS_ISPOUT_VHOLD_SET_MASK, val);
|
|
|
|
if (ret < 0) {
|
|
|
|
dev_err(&info->pdev->dev, "register(%x) write error(%d)\n",
|
|
|
|
AXP20X_VBUS_IPSOUT_MGMT, ret);
|
|
|
|
return ret;
|
|
|
|
}
|
power: supply: axp288_charger: Add special handling for HP Pavilion x2 10
Some HP Pavilion x2 10 models use an AXP288 for charging and fuel-gauge.
We use a native power_supply / PMIC driver in this case, because on most
models with an AXP288 the ACPI AC / Battery code is either completely
missing or relies on custom / proprietary ACPI OpRegions which Linux
does not implement.
The native drivers mostly work fine, but there are 2 problems:
1. These model uses a Type-C connector for charging which the AXP288 does
not support. As long as a Type-A charger (which uses the USB data pins for
charger type detection) is used everything is fine. But if a Type-C
charger is used (such as the charger shipped with the device) then the
charger is not recognized.
So we end up slowly discharging the device even though a charger is
connected, because we are limiting the current from the charger to 500mA.
To make things worse this happens with the device's official charger.
Looking at the ACPI tables HP has "solved" the problem of the AXP288 not
being able to recognize Type-C chargers by simply always programming the
input-current-limit at 3000mA and relying on a Vhold setting of 4.7V
(normally 4.4V) to limit the current intake if the charger cannot handle
this.
2. If no charger is connected when the machine boots then it boots with the
vbus-path disabled. On other devices this is done when a 5V boost converter
is active to avoid the PMIC trying to charge from the 5V boost output.
This is done when an OTG host cable is inserted and the ID pin on the
micro-B receptacle is pulled low, the ID pin has an ACPI event handler
associated with it which re-enables the vbus-path when the ID pin is pulled
high when the OTG cable is removed. The Type-C connector has no ID pin,
there is no ID pin handler and there appears to be no 5V boost converter,
so we end up not charging because the vbus-path is disabled, until we
unplug the charger which automatically clears the vbus-path disable bit and
then on the second plug-in of the adapter we start charging.
The HP Pavilion x2 10 models with an AXP288 do have mostly working ACPI
AC / Battery code which does not rely on custom / proprietary ACPI
OpRegions. So one possible solution would be to blacklist the AXP288
native power_supply drivers and add the HP Pavilion x2 10 with AXP288
DMI ids to the list of devices which should use the ACPI AC / Battery
code even though they have an AXP288 PMIC. This would require changes to
4 files: drivers/acpi/ac.c, drivers/power/supply/axp288_charger.c,
drivers/acpi/battery.c and drivers/power/supply/axp288_fuel_gauge.c.
Beside needing adding the same DMI matches to 4 different files, this
approach also triggers problem 2. from above, but then when suspended,
during suspend the machine will not wakeup because the vbus path is
disabled by the AML code when not charging, so the Vbus low-to-high
IRQ is not triggered, the CPU never wakes up and the device does not
charge even though the user likely things it is charging, esp. since
the charge status LED is directly coupled to an adapter being plugged
in and does not reflect actual charging.
This could be worked by enabling vbus-path explicitly from say the
axp288_charger driver's suspend handler.
So neither situation is ideal, in both cased we need to explicitly enable
the vbus-path to work around different variants of problem 2 above, this
requires a quirk in the axp288_charger code.
If we go the route of using the ACPI AC / Battery drivers then we need
modifications to 3 other drivers; and we need to partially disable the
axp288_charger code, while at the same time keeping it around to enable
vbus-path on suspend.
OTOH we can copy the hardcoding of 3A input-current-limit (we never touch
Vhold, so that would stay at 4.7V) to the axp288_charger code, which needs
changes regardless, then we concentrate all special handling of this
interesting device model in the axp288_charger code. That is what this
commit does.
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1791098
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
2020-02-23 18:32:08 +03:00
|
|
|
}
|
|
|
|
|
2015-05-04 19:46:07 +03:00
|
|
|
/* Read current charge voltage and current limit */
|
|
|
|
ret = regmap_read(info->regmap, AXP20X_CHRG_CTRL1, &val);
|
|
|
|
if (ret < 0) {
|
2016-12-21 17:36:47 +03:00
|
|
|
dev_err(&info->pdev->dev, "register(%x) read error(%d)\n",
|
|
|
|
AXP20X_CHRG_CTRL1, ret);
|
|
|
|
return ret;
|
|
|
|
}
|
2015-05-04 19:46:07 +03:00
|
|
|
|
2016-12-21 17:36:47 +03:00
|
|
|
/* Determine charge voltage */
|
|
|
|
cv = (val & CHRG_CCCV_CV_MASK) >> CHRG_CCCV_CV_BIT_POS;
|
|
|
|
switch (cv) {
|
|
|
|
case CHRG_CCCV_CV_4100MV:
|
|
|
|
info->cv = CV_4100MV;
|
|
|
|
break;
|
|
|
|
case CHRG_CCCV_CV_4150MV:
|
|
|
|
info->cv = CV_4150MV;
|
|
|
|
break;
|
|
|
|
case CHRG_CCCV_CV_4200MV:
|
|
|
|
info->cv = CV_4200MV;
|
|
|
|
break;
|
|
|
|
case CHRG_CCCV_CV_4350MV:
|
|
|
|
info->cv = CV_4350MV;
|
|
|
|
break;
|
|
|
|
}
|
2015-05-04 19:46:07 +03:00
|
|
|
|
2016-12-21 17:36:47 +03:00
|
|
|
/* Determine charge current limit */
|
2018-05-23 16:33:21 +03:00
|
|
|
cc = (val & CHRG_CCCV_CC_MASK) >> CHRG_CCCV_CC_BIT_POS;
|
2016-12-21 17:36:47 +03:00
|
|
|
cc = (cc * CHRG_CCCV_CC_LSB_RES) + CHRG_CCCV_CC_OFFSET;
|
|
|
|
info->cc = cc;
|
2015-05-04 19:46:07 +03:00
|
|
|
|
2016-12-21 17:36:47 +03:00
|
|
|
/*
|
|
|
|
* Do not allow the user to configure higher settings then those
|
|
|
|
* set by the firmware
|
|
|
|
*/
|
|
|
|
info->max_cv = info->cv;
|
|
|
|
info->max_cc = info->cc;
|
2015-05-04 19:46:07 +03:00
|
|
|
|
2016-12-21 17:36:46 +03:00
|
|
|
return 0;
|
2015-05-04 19:46:07 +03:00
|
|
|
}
|
|
|
|
|
2017-12-26 15:59:09 +03:00
|
|
|
static void axp288_charger_cancel_work(void *data)
|
|
|
|
{
|
|
|
|
struct axp288_chrg_info *info = data;
|
|
|
|
|
|
|
|
cancel_work_sync(&info->otg.work);
|
|
|
|
cancel_work_sync(&info->cable.work);
|
|
|
|
}
|
|
|
|
|
2015-05-04 19:46:07 +03:00
|
|
|
static int axp288_charger_probe(struct platform_device *pdev)
|
|
|
|
{
|
|
|
|
int ret, i, pirq;
|
|
|
|
struct axp288_chrg_info *info;
|
2016-12-21 17:36:48 +03:00
|
|
|
struct device *dev = &pdev->dev;
|
2015-05-04 19:46:07 +03:00
|
|
|
struct axp20x_dev *axp20x = dev_get_drvdata(pdev->dev.parent);
|
|
|
|
struct power_supply_config charger_cfg = {};
|
2018-04-18 15:08:20 +03:00
|
|
|
unsigned int val;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* On some devices the fuelgauge and charger parts of the axp288 are
|
|
|
|
* not used, check that the fuelgauge is enabled (CC_CTRL != 0).
|
|
|
|
*/
|
|
|
|
ret = regmap_read(axp20x->regmap, AXP20X_CC_CTRL, &val);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
if (val == 0)
|
|
|
|
return -ENODEV;
|
|
|
|
|
2021-08-11 13:49:29 +03:00
|
|
|
info = devm_kzalloc(dev, sizeof(*info), GFP_KERNEL);
|
2015-05-04 19:46:07 +03:00
|
|
|
if (!info)
|
|
|
|
return -ENOMEM;
|
|
|
|
|
|
|
|
info->pdev = pdev;
|
|
|
|
info->regmap = axp20x->regmap;
|
|
|
|
info->regmap_irqc = axp20x->regmap_irqc;
|
|
|
|
|
|
|
|
info->cable.edev = extcon_get_extcon_dev(AXP288_EXTCON_DEV_NAME);
|
2021-12-17 09:28:46 +03:00
|
|
|
if (IS_ERR(info->cable.edev)) {
|
|
|
|
dev_err_probe(dev, PTR_ERR(info->cable.edev),
|
|
|
|
"extcon_get_extcon_dev(%s) failed\n",
|
|
|
|
AXP288_EXTCON_DEV_NAME);
|
|
|
|
return PTR_ERR(info->cable.edev);
|
2015-05-04 19:46:07 +03:00
|
|
|
}
|
|
|
|
|
2017-04-19 15:02:12 +03:00
|
|
|
if (acpi_dev_present(USB_HOST_EXTCON_HID, NULL, -1)) {
|
|
|
|
info->otg.cable = extcon_get_extcon_dev(USB_HOST_EXTCON_NAME);
|
2021-12-17 09:28:46 +03:00
|
|
|
if (IS_ERR(info->otg.cable)) {
|
|
|
|
dev_err_probe(dev, PTR_ERR(info->otg.cable),
|
|
|
|
"extcon_get_extcon_dev(%s) failed\n",
|
|
|
|
USB_HOST_EXTCON_NAME);
|
|
|
|
return PTR_ERR(info->otg.cable);
|
2017-04-19 15:02:12 +03:00
|
|
|
}
|
2021-08-11 13:49:29 +03:00
|
|
|
dev_info(dev, "Using " USB_HOST_EXTCON_HID " extcon for usb-id\n");
|
2016-12-21 17:36:51 +03:00
|
|
|
}
|
|
|
|
|
2015-05-04 19:46:07 +03:00
|
|
|
platform_set_drvdata(pdev, info);
|
|
|
|
|
2016-12-21 17:36:50 +03:00
|
|
|
ret = charger_init_hw_regs(info);
|
|
|
|
if (ret)
|
|
|
|
return ret;
|
|
|
|
|
2015-05-04 19:46:07 +03:00
|
|
|
/* Register with power supply class */
|
|
|
|
charger_cfg.drv_data = info;
|
2016-12-21 17:36:48 +03:00
|
|
|
info->psy_usb = devm_power_supply_register(dev, &axp288_charger_desc,
|
|
|
|
&charger_cfg);
|
2015-05-04 19:46:07 +03:00
|
|
|
if (IS_ERR(info->psy_usb)) {
|
|
|
|
ret = PTR_ERR(info->psy_usb);
|
2016-12-21 17:36:48 +03:00
|
|
|
dev_err(dev, "failed to register power supply: %d\n", ret);
|
|
|
|
return ret;
|
2015-05-04 19:46:07 +03:00
|
|
|
}
|
|
|
|
|
2017-12-26 15:59:09 +03:00
|
|
|
/* Cancel our work on cleanup, register this before the notifiers */
|
|
|
|
ret = devm_add_action(dev, axp288_charger_cancel_work, info);
|
|
|
|
if (ret)
|
|
|
|
return ret;
|
|
|
|
|
2016-12-21 17:36:49 +03:00
|
|
|
/* Register for extcon notification */
|
|
|
|
INIT_WORK(&info->cable.work, axp288_charger_extcon_evt_worker);
|
2017-12-26 15:59:08 +03:00
|
|
|
info->cable.nb.notifier_call = axp288_charger_handle_cable_evt;
|
|
|
|
ret = devm_extcon_register_notifier_all(dev, info->cable.edev,
|
|
|
|
&info->cable.nb);
|
|
|
|
if (ret) {
|
|
|
|
dev_err(dev, "failed to register cable extcon notifier\n");
|
|
|
|
return ret;
|
2016-12-21 17:36:49 +03:00
|
|
|
}
|
2016-12-21 17:36:54 +03:00
|
|
|
schedule_work(&info->cable.work);
|
2016-12-21 17:36:49 +03:00
|
|
|
|
2015-05-04 19:46:07 +03:00
|
|
|
/* Register for OTG notification */
|
|
|
|
INIT_WORK(&info->otg.work, axp288_charger_otg_evt_worker);
|
|
|
|
info->otg.id_nb.notifier_call = axp288_charger_handle_otg_evt;
|
2017-04-19 15:02:12 +03:00
|
|
|
if (info->otg.cable) {
|
2021-08-11 13:49:29 +03:00
|
|
|
ret = devm_extcon_register_notifier(dev, info->otg.cable,
|
2016-11-30 08:57:31 +03:00
|
|
|
EXTCON_USB_HOST, &info->otg.id_nb);
|
2017-04-19 15:02:12 +03:00
|
|
|
if (ret) {
|
|
|
|
dev_err(dev, "failed to register EXTCON_USB_HOST notifier\n");
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
schedule_work(&info->otg.work);
|
2016-12-21 17:36:51 +03:00
|
|
|
}
|
2015-05-04 19:46:07 +03:00
|
|
|
|
|
|
|
/* Register charger interrupts */
|
|
|
|
for (i = 0; i < CHRG_INTR_END; i++) {
|
|
|
|
pirq = platform_get_irq(info->pdev, i);
|
2020-04-15 07:27:27 +03:00
|
|
|
if (pirq < 0)
|
2019-03-18 19:14:39 +03:00
|
|
|
return pirq;
|
2020-04-15 07:27:27 +03:00
|
|
|
|
2015-05-04 19:46:07 +03:00
|
|
|
info->irq[i] = regmap_irq_get_virq(info->regmap_irqc, pirq);
|
|
|
|
if (info->irq[i] < 0) {
|
|
|
|
dev_warn(&info->pdev->dev,
|
|
|
|
"failed to get virtual interrupt=%d\n", pirq);
|
2016-12-21 17:36:48 +03:00
|
|
|
return info->irq[i];
|
2015-05-04 19:46:07 +03:00
|
|
|
}
|
|
|
|
ret = devm_request_threaded_irq(&info->pdev->dev, info->irq[i],
|
|
|
|
NULL, axp288_charger_irq_thread_handler,
|
|
|
|
IRQF_ONESHOT, info->pdev->name, info);
|
|
|
|
if (ret) {
|
2021-08-11 13:49:29 +03:00
|
|
|
dev_err(dev, "failed to request interrupt=%d\n",
|
2015-05-04 19:46:07 +03:00
|
|
|
info->irq[i]);
|
2016-12-21 17:36:48 +03:00
|
|
|
return ret;
|
2015-05-04 19:46:07 +03:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2016-12-21 17:36:57 +03:00
|
|
|
static const struct platform_device_id axp288_charger_id_table[] = {
|
|
|
|
{ .name = "axp288_charger" },
|
|
|
|
{},
|
|
|
|
};
|
|
|
|
MODULE_DEVICE_TABLE(platform, axp288_charger_id_table);
|
|
|
|
|
2015-05-04 19:46:07 +03:00
|
|
|
static struct platform_driver axp288_charger_driver = {
|
|
|
|
.probe = axp288_charger_probe,
|
2016-12-21 17:36:57 +03:00
|
|
|
.id_table = axp288_charger_id_table,
|
2015-05-04 19:46:07 +03:00
|
|
|
.driver = {
|
|
|
|
.name = "axp288_charger",
|
|
|
|
},
|
|
|
|
};
|
|
|
|
|
|
|
|
module_platform_driver(axp288_charger_driver);
|
|
|
|
|
|
|
|
MODULE_AUTHOR("Ramakrishna Pallala <ramakrishna.pallala@intel.com>");
|
|
|
|
MODULE_DESCRIPTION("X-power AXP288 Charger Driver");
|
|
|
|
MODULE_LICENSE("GPL v2");
|