Commit 3478c4c1 authored by Guido Gunther's avatar Guido Gunther Committed by Martin Kepplinger
Browse files

usb: typec: tps6598x: Add trace event for power status register



Together with the PD status register this is vital for debugging power
negotiations at runtime.
Signed-off-by: Guido Gunther's avatarGuido Günther <agx@sigxcpu.org>
parent f121cde9
......@@ -50,11 +50,6 @@ enum {
TPS_PORTINFO_SOURCE,
};
/* TPS_REG_POWER_STATUS bits */
#define TPS_POWER_STATUS_CONNECTION BIT(0)
#define TPS_POWER_STATUS_SOURCESINK BIT(1)
#define TPS_POWER_STATUS_PWROPMODE(p) (((p) & GENMASK(3, 2)) >> 2)
/* TPS_REG_RX_IDENTITY_SOP */
struct tps6598x_rx_identity_reg {
u8 status;
......@@ -433,6 +428,15 @@ static irqreturn_t tps6598x_interrupt(int irq, void *data)
}
trace_tps6598x_status(status);
if ((event1 | event2) & TPS_REG_INT_POWER_STATUS_UPDATE) {
ret = tps6598x_read16(tps, TPS_REG_POWER_STATUS, &pwr_status);
if (ret < 0) {
dev_err(tps->dev, "failed to read power status: %d\n", ret);
goto err_clear_ints;
}
trace_tps6598x_power_status(pwr_status);
}
/* Handle plug insert or removal */
if ((event1 | event2) & TPS_REG_INT_PLUG_EVENT) {
if (status & TPS_STATUS_PLUG_PRESENT) {
......
......@@ -124,4 +124,23 @@
#define TPS_REG_INT_HARD_RESET BIT(1)
#define TPS_REG_INT_PD_SOFT_RESET BIT(0)
/* TPS_REG_POWER_STATUS bits */
#define TPS_POWER_STATUS_CONNECTION(x) FIELD_GET(BIT(0), (x))
#define TPS_POWER_STATUS_SOURCESINK(x) FIELD_GET(BIT(1), (x))
#define TPS_POWER_STATUS_BC12_DET(x) FIELD_GET(BIT(2), (x))
#define TPS_POWER_STATUS_TYPEC_CURRENT_MASK GENMASK(3, 2)
#define TPS_POWER_STATUS_PWROPMODE(p) FIELD_GET(TPS_POWER_STATUS_TYPEC_CURRENT_MASK, (p))
#define TPS_POWER_STATUS_BC12_STATUS_MASK GENMASK(6, 5)
#define TPS_POWER_STATUS_BC12_STATUS(p) FIELD_GET(TPS_POWER_STATUS_BC12_STATUS_MASK, (p))
#define TPS_POWER_STATUS_TYPEC_CURRENT_USB 0
#define TPS_POWER_STATUS_TYPEC_CURRENT_1A5 1
#define TPS_POWER_STATUS_TYPEC_CURRENT_3A0 2
#define TPS_POWER_STATUS_TYPEC_CURRENT_PD 3
#define TPS_POWER_STATUS_BC12_STATUS_SDP 0
#define TPS_POWER_STATUS_BC12_STATUS_CDP 2
#define TPS_POWER_STATUS_BC12_STATUS_DCP 3
#endif /* __TPS6598X_H__ */
......@@ -134,6 +134,24 @@
{ TPS_STATUS_HIGH_VOLAGE_WARNING, "HIGH_VOLAGE_WARNING" }, \
{ TPS_STATUS_HIGH_LOW_VOLTAGE_WARNING, "HIGH_LOW_VOLTAGE_WARNING" })
#define show_power_status_source_sink(power_status) \
__print_symbolic(TPS_POWER_STATUS_SOURCESINK(power_status), \
{ 1, "sink" }, \
{ 0, "source" })
#define show_power_status_typec_status(power_status) \
__print_symbolic(TPS_POWER_STATUS_PWROPMODE(power_status), \
{ TPS_POWER_STATUS_TYPEC_CURRENT_PD, "pd" }, \
{ TPS_POWER_STATUS_TYPEC_CURRENT_3A0, "3.0A" }, \
{ TPS_POWER_STATUS_TYPEC_CURRENT_1A5, "1.5A" }, \
{ TPS_POWER_STATUS_TYPEC_CURRENT_USB, "usb" })
#define show_power_status_bc12_status(power_status) \
__print_symbolic(TPS_POWER_STATUS_BC12_STATUS(power_status), \
{ TPS_POWER_STATUS_BC12_STATUS_DCP, "dcp" }, \
{ TPS_POWER_STATUS_BC12_STATUS_CDP, "cdp" }, \
{ TPS_POWER_STATUS_BC12_STATUS_SDP, "sdp" })
TRACE_EVENT(tps6598x_irq,
TP_PROTO(u64 event1,
u64 event2),
......@@ -181,6 +199,26 @@ TRACE_EVENT(tps6598x_status,
)
);
TRACE_EVENT(tps6598x_power_status,
TP_PROTO(u16 power_status),
TP_ARGS(power_status),
TP_STRUCT__entry(
__field(u16, power_status)
),
TP_fast_assign(
__entry->power_status = power_status;
),
TP_printk("conn: %d, pwr-role: %s, typec: %s, bc: %s",
!!TPS_POWER_STATUS_CONNECTION(__entry->power_status),
show_power_status_source_sink(__entry->power_status),
show_power_status_typec_status(__entry->power_status),
show_power_status_bc12_status(__entry->power_status)
)
);
#endif /* _TPS6598X_TRACE_H_ */
/* This part must be outside protection */
......
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