Commit f2a8aa05 authored by Adam Thomson's avatar Adam Thomson Committed by Greg Kroah-Hartman

typec: tcpm: Represent source supply through power_supply

This commit adds a power_supply class instance to represent a
PD source's voltage and current properties. This provides an
interface for reading these properties from user-space or other
drivers.

For PPS enabled Sources, this also provides write access to set
the current and voltage and allows for swapping between standard
PDO and PPS APDO.

As this represents a superset of the information provided in the
fusb302 driver, the power_supply instance in that code is removed
as part of this change, so reverting the commit titled
'typec: tcpm: Represent source supply through power_supply class'
Signed-off-by: default avatarAdam Thomson <Adam.Thomson.Opensource@diasemi.com>
Reviewed-by: default avatarGuenter Roeck <linux@roeck-us.net>
Reviewed-by: default avatarHeikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent cf450041
...@@ -49,6 +49,7 @@ config TYPEC_TCPM ...@@ -49,6 +49,7 @@ config TYPEC_TCPM
tristate "USB Type-C Port Controller Manager" tristate "USB Type-C Port Controller Manager"
depends on USB depends on USB
select USB_ROLE_SWITCH select USB_ROLE_SWITCH
select POWER_SUPPLY
help help
The Type-C Port Controller Manager provides a USB PD and USB Type-C The Type-C Port Controller Manager provides a USB PD and USB Type-C
state machine for use with Type-C Port Controllers. state machine for use with Type-C Port Controllers.
......
config TYPEC_FUSB302 config TYPEC_FUSB302
tristate "Fairchild FUSB302 Type-C chip driver" tristate "Fairchild FUSB302 Type-C chip driver"
depends on I2C && POWER_SUPPLY depends on I2C
help help
The Fairchild FUSB302 Type-C chip driver that works with The Fairchild FUSB302 Type-C chip driver that works with
Type-C Port Controller Manager to provide USB PD and USB Type-C Port Controller Manager to provide USB PD and USB
......
...@@ -18,7 +18,6 @@ ...@@ -18,7 +18,6 @@
#include <linux/of_device.h> #include <linux/of_device.h>
#include <linux/of_gpio.h> #include <linux/of_gpio.h>
#include <linux/pinctrl/consumer.h> #include <linux/pinctrl/consumer.h>
#include <linux/power_supply.h>
#include <linux/proc_fs.h> #include <linux/proc_fs.h>
#include <linux/regulator/consumer.h> #include <linux/regulator/consumer.h>
#include <linux/sched/clock.h> #include <linux/sched/clock.h>
...@@ -99,11 +98,6 @@ struct fusb302_chip { ...@@ -99,11 +98,6 @@ struct fusb302_chip {
/* lock for sharing chip states */ /* lock for sharing chip states */
struct mutex lock; struct mutex lock;
/* psy + psy status */
struct power_supply *psy;
u32 current_limit;
u32 supply_voltage;
/* chip status */ /* chip status */
enum toggling_mode toggling_mode; enum toggling_mode toggling_mode;
enum src_current_status src_current_status; enum src_current_status src_current_status;
...@@ -862,13 +856,11 @@ static int tcpm_set_vbus(struct tcpc_dev *dev, bool on, bool charge) ...@@ -862,13 +856,11 @@ static int tcpm_set_vbus(struct tcpc_dev *dev, bool on, bool charge)
chip->vbus_on = on; chip->vbus_on = on;
fusb302_log(chip, "vbus := %s", on ? "On" : "Off"); fusb302_log(chip, "vbus := %s", on ? "On" : "Off");
} }
if (chip->charge_on == charge) { if (chip->charge_on == charge)
fusb302_log(chip, "charge is already %s", fusb302_log(chip, "charge is already %s",
charge ? "On" : "Off"); charge ? "On" : "Off");
} else { else
chip->charge_on = charge; chip->charge_on = charge;
power_supply_changed(chip->psy);
}
done: done:
mutex_unlock(&chip->lock); mutex_unlock(&chip->lock);
...@@ -884,11 +876,6 @@ static int tcpm_set_current_limit(struct tcpc_dev *dev, u32 max_ma, u32 mv) ...@@ -884,11 +876,6 @@ static int tcpm_set_current_limit(struct tcpc_dev *dev, u32 max_ma, u32 mv)
fusb302_log(chip, "current limit: %d ma, %d mv (not implemented)", fusb302_log(chip, "current limit: %d ma, %d mv (not implemented)",
max_ma, mv); max_ma, mv);
chip->supply_voltage = mv;
chip->current_limit = max_ma;
power_supply_changed(chip->psy);
return 0; return 0;
} }
...@@ -1682,43 +1669,6 @@ static irqreturn_t fusb302_irq_intn(int irq, void *dev_id) ...@@ -1682,43 +1669,6 @@ static irqreturn_t fusb302_irq_intn(int irq, void *dev_id)
return IRQ_HANDLED; return IRQ_HANDLED;
} }
static int fusb302_psy_get_property(struct power_supply *psy,
enum power_supply_property psp,
union power_supply_propval *val)
{
struct fusb302_chip *chip = power_supply_get_drvdata(psy);
switch (psp) {
case POWER_SUPPLY_PROP_ONLINE:
val->intval = chip->charge_on;
break;
case POWER_SUPPLY_PROP_VOLTAGE_NOW:
val->intval = chip->supply_voltage * 1000; /* mV -> µV */
break;
case POWER_SUPPLY_PROP_CURRENT_MAX:
val->intval = chip->current_limit * 1000; /* mA -> µA */
break;
default:
return -ENODATA;
}
return 0;
}
static enum power_supply_property fusb302_psy_properties[] = {
POWER_SUPPLY_PROP_ONLINE,
POWER_SUPPLY_PROP_VOLTAGE_NOW,
POWER_SUPPLY_PROP_CURRENT_MAX,
};
static const struct power_supply_desc fusb302_psy_desc = {
.name = "fusb302-typec-source",
.type = POWER_SUPPLY_TYPE_USB_TYPE_C,
.properties = fusb302_psy_properties,
.num_properties = ARRAY_SIZE(fusb302_psy_properties),
.get_property = fusb302_psy_get_property,
};
static int init_gpio(struct fusb302_chip *chip) static int init_gpio(struct fusb302_chip *chip)
{ {
struct device_node *node; struct device_node *node;
...@@ -1781,7 +1731,6 @@ static int fusb302_probe(struct i2c_client *client, ...@@ -1781,7 +1731,6 @@ static int fusb302_probe(struct i2c_client *client,
struct fusb302_chip *chip; struct fusb302_chip *chip;
struct i2c_adapter *adapter; struct i2c_adapter *adapter;
struct device *dev = &client->dev; struct device *dev = &client->dev;
struct power_supply_config cfg = {};
const char *name; const char *name;
int ret = 0; int ret = 0;
u32 v; u32 v;
...@@ -1823,14 +1772,6 @@ static int fusb302_probe(struct i2c_client *client, ...@@ -1823,14 +1772,6 @@ static int fusb302_probe(struct i2c_client *client,
return -EPROBE_DEFER; return -EPROBE_DEFER;
} }
cfg.drv_data = chip;
chip->psy = devm_power_supply_register(dev, &fusb302_psy_desc, &cfg);
if (IS_ERR(chip->psy)) {
ret = PTR_ERR(chip->psy);
dev_err(chip->dev, "Error registering power-supply: %d\n", ret);
return ret;
}
ret = fusb302_debugfs_init(chip); ret = fusb302_debugfs_init(chip);
if (ret < 0) if (ret < 0)
return ret; return ret;
......
...@@ -12,6 +12,7 @@ ...@@ -12,6 +12,7 @@
#include <linux/kernel.h> #include <linux/kernel.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/mutex.h> #include <linux/mutex.h>
#include <linux/power_supply.h>
#include <linux/proc_fs.h> #include <linux/proc_fs.h>
#include <linux/sched/clock.h> #include <linux/sched/clock.h>
#include <linux/seq_file.h> #include <linux/seq_file.h>
...@@ -276,6 +277,11 @@ struct tcpm_port { ...@@ -276,6 +277,11 @@ struct tcpm_port {
u32 current_limit; u32 current_limit;
u32 supply_voltage; u32 supply_voltage;
/* Used to export TA voltage and current */
struct power_supply *psy;
struct power_supply_desc psy_desc;
enum power_supply_usb_type usb_type;
u32 bist_request; u32 bist_request;
/* PD state for Vendor Defined Messages */ /* PD state for Vendor Defined Messages */
...@@ -1895,6 +1901,7 @@ static int tcpm_pd_select_pdo(struct tcpm_port *port, int *sink_pdo, ...@@ -1895,6 +1901,7 @@ static int tcpm_pd_select_pdo(struct tcpm_port *port, int *sink_pdo,
int ret = -EINVAL; int ret = -EINVAL;
port->pps_data.supported = false; port->pps_data.supported = false;
port->usb_type = POWER_SUPPLY_USB_TYPE_PD;
/* /*
* Select the source PDO providing the most power which has a * Select the source PDO providing the most power which has a
...@@ -1915,8 +1922,11 @@ static int tcpm_pd_select_pdo(struct tcpm_port *port, int *sink_pdo, ...@@ -1915,8 +1922,11 @@ static int tcpm_pd_select_pdo(struct tcpm_port *port, int *sink_pdo,
min_src_mv = pdo_min_voltage(pdo); min_src_mv = pdo_min_voltage(pdo);
break; break;
case PDO_TYPE_APDO: case PDO_TYPE_APDO:
if (pdo_apdo_type(pdo) == APDO_TYPE_PPS) if (pdo_apdo_type(pdo) == APDO_TYPE_PPS) {
port->pps_data.supported = true; port->pps_data.supported = true;
port->usb_type =
POWER_SUPPLY_USB_TYPE_PD_PPS;
}
continue; continue;
default: default:
tcpm_log(port, "Invalid source PDO type, ignoring"); tcpm_log(port, "Invalid source PDO type, ignoring");
...@@ -2472,6 +2482,9 @@ static void tcpm_reset_port(struct tcpm_port *port) ...@@ -2472,6 +2482,9 @@ static void tcpm_reset_port(struct tcpm_port *port)
port->try_snk_count = 0; port->try_snk_count = 0;
port->supply_voltage = 0; port->supply_voltage = 0;
port->current_limit = 0; port->current_limit = 0;
port->usb_type = POWER_SUPPLY_USB_TYPE_C;
power_supply_changed(port->psy);
} }
static void tcpm_detach(struct tcpm_port *port) static void tcpm_detach(struct tcpm_port *port)
...@@ -2999,6 +3012,8 @@ static void run_state_machine(struct tcpm_port *port) ...@@ -2999,6 +3012,8 @@ static void run_state_machine(struct tcpm_port *port)
tcpm_check_send_discover(port); tcpm_check_send_discover(port);
tcpm_pps_complete(port, port->pps_status); tcpm_pps_complete(port, port->pps_status);
power_supply_changed(port->psy);
break; break;
/* Accessory states */ /* Accessory states */
...@@ -3849,7 +3864,7 @@ static int tcpm_try_role(const struct typec_capability *cap, int role) ...@@ -3849,7 +3864,7 @@ static int tcpm_try_role(const struct typec_capability *cap, int role)
return ret; return ret;
} }
static int __maybe_unused tcpm_pps_set_op_curr(struct tcpm_port *port, u16 op_curr) static int tcpm_pps_set_op_curr(struct tcpm_port *port, u16 op_curr)
{ {
unsigned int target_mw; unsigned int target_mw;
int ret; int ret;
...@@ -3901,7 +3916,7 @@ static int __maybe_unused tcpm_pps_set_op_curr(struct tcpm_port *port, u16 op_cu ...@@ -3901,7 +3916,7 @@ static int __maybe_unused tcpm_pps_set_op_curr(struct tcpm_port *port, u16 op_cu
return ret; return ret;
} }
static int __maybe_unused tcpm_pps_set_out_volt(struct tcpm_port *port, u16 out_volt) static int tcpm_pps_set_out_volt(struct tcpm_port *port, u16 out_volt)
{ {
unsigned int target_mw; unsigned int target_mw;
int ret; int ret;
...@@ -3954,7 +3969,7 @@ static int __maybe_unused tcpm_pps_set_out_volt(struct tcpm_port *port, u16 out_ ...@@ -3954,7 +3969,7 @@ static int __maybe_unused tcpm_pps_set_out_volt(struct tcpm_port *port, u16 out_
return ret; return ret;
} }
static int __maybe_unused tcpm_pps_activate(struct tcpm_port *port, bool activate) static int tcpm_pps_activate(struct tcpm_port *port, bool activate)
{ {
int ret = 0; int ret = 0;
...@@ -4159,6 +4174,230 @@ int tcpm_update_sink_capabilities(struct tcpm_port *port, const u32 *pdo, ...@@ -4159,6 +4174,230 @@ int tcpm_update_sink_capabilities(struct tcpm_port *port, const u32 *pdo,
} }
EXPORT_SYMBOL_GPL(tcpm_update_sink_capabilities); EXPORT_SYMBOL_GPL(tcpm_update_sink_capabilities);
/* Power Supply access to expose source power information */
enum tcpm_psy_online_states {
TCPM_PSY_OFFLINE = 0,
TCPM_PSY_FIXED_ONLINE,
TCPM_PSY_PROG_ONLINE,
};
static enum power_supply_property tcpm_psy_props[] = {
POWER_SUPPLY_PROP_USB_TYPE,
POWER_SUPPLY_PROP_ONLINE,
POWER_SUPPLY_PROP_VOLTAGE_MIN,
POWER_SUPPLY_PROP_VOLTAGE_MAX,
POWER_SUPPLY_PROP_VOLTAGE_NOW,
POWER_SUPPLY_PROP_CURRENT_MAX,
POWER_SUPPLY_PROP_CURRENT_NOW,
};
static int tcpm_psy_get_online(struct tcpm_port *port,
union power_supply_propval *val)
{
if (port->vbus_charge) {
if (port->pps_data.active)
val->intval = TCPM_PSY_PROG_ONLINE;
else
val->intval = TCPM_PSY_FIXED_ONLINE;
} else {
val->intval = TCPM_PSY_OFFLINE;
}
return 0;
}
static int tcpm_psy_get_voltage_min(struct tcpm_port *port,
union power_supply_propval *val)
{
if (port->pps_data.active)
val->intval = port->pps_data.min_volt * 1000;
else
val->intval = port->supply_voltage * 1000;
return 0;
}
static int tcpm_psy_get_voltage_max(struct tcpm_port *port,
union power_supply_propval *val)
{
if (port->pps_data.active)
val->intval = port->pps_data.max_volt * 1000;
else
val->intval = port->supply_voltage * 1000;
return 0;
}
static int tcpm_psy_get_voltage_now(struct tcpm_port *port,
union power_supply_propval *val)
{
val->intval = port->supply_voltage * 1000;
return 0;
}
static int tcpm_psy_get_current_max(struct tcpm_port *port,
union power_supply_propval *val)
{
if (port->pps_data.active)
val->intval = port->pps_data.max_curr * 1000;
else
val->intval = port->current_limit * 1000;
return 0;
}
static int tcpm_psy_get_current_now(struct tcpm_port *port,
union power_supply_propval *val)
{
val->intval = port->current_limit * 1000;
return 0;
}
static int tcpm_psy_get_prop(struct power_supply *psy,
enum power_supply_property psp,
union power_supply_propval *val)
{
struct tcpm_port *port = power_supply_get_drvdata(psy);
int ret = 0;
switch (psp) {
case POWER_SUPPLY_PROP_USB_TYPE:
val->intval = port->usb_type;
break;
case POWER_SUPPLY_PROP_ONLINE:
ret = tcpm_psy_get_online(port, val);
break;
case POWER_SUPPLY_PROP_VOLTAGE_MIN:
ret = tcpm_psy_get_voltage_min(port, val);
break;
case POWER_SUPPLY_PROP_VOLTAGE_MAX:
ret = tcpm_psy_get_voltage_max(port, val);
break;
case POWER_SUPPLY_PROP_VOLTAGE_NOW:
ret = tcpm_psy_get_voltage_now(port, val);
break;
case POWER_SUPPLY_PROP_CURRENT_MAX:
ret = tcpm_psy_get_current_max(port, val);
break;
case POWER_SUPPLY_PROP_CURRENT_NOW:
ret = tcpm_psy_get_current_now(port, val);
break;
default:
ret = -EINVAL;
break;
}
return ret;
}
static int tcpm_psy_set_online(struct tcpm_port *port,
const union power_supply_propval *val)
{
int ret;
switch (val->intval) {
case TCPM_PSY_FIXED_ONLINE:
ret = tcpm_pps_activate(port, false);
break;
case TCPM_PSY_PROG_ONLINE:
ret = tcpm_pps_activate(port, true);
break;
default:
ret = -EINVAL;
break;
}
return ret;
}
static int tcpm_psy_set_prop(struct power_supply *psy,
enum power_supply_property psp,
const union power_supply_propval *val)
{
struct tcpm_port *port = power_supply_get_drvdata(psy);
int ret;
switch (psp) {
case POWER_SUPPLY_PROP_ONLINE:
ret = tcpm_psy_set_online(port, val);
break;
case POWER_SUPPLY_PROP_VOLTAGE_NOW:
if (val->intval < port->pps_data.min_volt * 1000 ||
val->intval > port->pps_data.max_volt * 1000)
ret = -EINVAL;
else
ret = tcpm_pps_set_out_volt(port, val->intval / 1000);
break;
case POWER_SUPPLY_PROP_CURRENT_NOW:
if (val->intval > port->pps_data.max_curr * 1000)
ret = -EINVAL;
else
ret = tcpm_pps_set_op_curr(port, val->intval / 1000);
break;
default:
ret = -EINVAL;
break;
}
return ret;
}
static int tcpm_psy_prop_writeable(struct power_supply *psy,
enum power_supply_property psp)
{
switch (psp) {
case POWER_SUPPLY_PROP_ONLINE:
case POWER_SUPPLY_PROP_VOLTAGE_NOW:
case POWER_SUPPLY_PROP_CURRENT_NOW:
return 1;
default:
return 0;
}
}
static enum power_supply_usb_type tcpm_psy_usb_types[] = {
POWER_SUPPLY_USB_TYPE_C,
POWER_SUPPLY_USB_TYPE_PD,
POWER_SUPPLY_USB_TYPE_PD_PPS,
};
static const char *tcpm_psy_name_prefix = "tcpm-source-psy-";
static int devm_tcpm_psy_register(struct tcpm_port *port)
{
struct power_supply_config psy_cfg = {};
const char *port_dev_name = dev_name(port->dev);
size_t psy_name_len = strlen(tcpm_psy_name_prefix) +
strlen(port_dev_name) + 1;
char *psy_name;
psy_cfg.drv_data = port;
psy_name = devm_kzalloc(port->dev, psy_name_len, GFP_KERNEL);
if (!psy_name)
return -ENOMEM;
snprintf(psy_name, psy_name_len, "%s%s", tcpm_psy_name_prefix,
port_dev_name);
port->psy_desc.name = psy_name;
port->psy_desc.type = POWER_SUPPLY_TYPE_USB,
port->psy_desc.usb_types = tcpm_psy_usb_types;
port->psy_desc.num_usb_types = ARRAY_SIZE(tcpm_psy_usb_types);
port->psy_desc.properties = tcpm_psy_props,
port->psy_desc.num_properties = ARRAY_SIZE(tcpm_psy_props),
port->psy_desc.get_property = tcpm_psy_get_prop,
port->psy_desc.set_property = tcpm_psy_set_prop,
port->psy_desc.property_is_writeable = tcpm_psy_prop_writeable,
port->usb_type = POWER_SUPPLY_USB_TYPE_C;
port->psy = devm_power_supply_register(port->dev, &port->psy_desc,
&psy_cfg);
return PTR_ERR_OR_ZERO(port->psy);
}
struct tcpm_port *tcpm_register_port(struct device *dev, struct tcpc_dev *tcpc) struct tcpm_port *tcpm_register_port(struct device *dev, struct tcpc_dev *tcpc)
{ {
struct tcpm_port *port; struct tcpm_port *port;
...@@ -4234,6 +4473,10 @@ struct tcpm_port *tcpm_register_port(struct device *dev, struct tcpc_dev *tcpc) ...@@ -4234,6 +4473,10 @@ struct tcpm_port *tcpm_register_port(struct device *dev, struct tcpc_dev *tcpc)
goto out_destroy_wq; goto out_destroy_wq;
} }
err = devm_tcpm_psy_register(port);
if (err)
goto out_destroy_wq;
port->typec_port = typec_register_port(port->dev, &port->typec_caps); port->typec_port = typec_register_port(port->dev, &port->typec_caps);
if (IS_ERR(port->typec_port)) { if (IS_ERR(port->typec_port)) {
err = PTR_ERR(port->typec_port); err = PTR_ERR(port->typec_port);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment