fsi: master-gpio: Add more tracepoints

This adds a few more tracepoints that have proven useful when
debugging issues with the FSI bus.

This also makes echo_delay() use clock_zeros() instead of
open-code it in order to share the tracepoint.
Signed-off-by: default avatarBenjamin Herrenschmidt <benh@kernel.crashing.org>
Reviewed-by: default avatarJoel Stanley <joel@jms.id.au>
parent 75854c14
...@@ -130,10 +130,17 @@ static void set_sda_output(struct fsi_master_gpio *master, int value) ...@@ -130,10 +130,17 @@ static void set_sda_output(struct fsi_master_gpio *master, int value)
static void clock_zeros(struct fsi_master_gpio *master, int count) static void clock_zeros(struct fsi_master_gpio *master, int count)
{ {
trace_fsi_master_gpio_clock_zeros(master, count);
set_sda_output(master, 1); set_sda_output(master, 1);
clock_toggle(master, count); clock_toggle(master, count);
} }
static void echo_delay(struct fsi_master_gpio *master)
{
clock_zeros(master, master->t_echo_delay);
}
static void serial_in(struct fsi_master_gpio *master, struct fsi_gpio_msg *msg, static void serial_in(struct fsi_master_gpio *master, struct fsi_gpio_msg *msg,
uint8_t num_bits) uint8_t num_bits)
{ {
...@@ -279,16 +286,19 @@ static void build_ar_command(struct fsi_master_gpio *master, ...@@ -279,16 +286,19 @@ static void build_ar_command(struct fsi_master_gpio *master,
addr_bits = 2; addr_bits = 2;
opcode_bits = 2; opcode_bits = 2;
opcode = FSI_GPIO_CMD_SAME_AR; opcode = FSI_GPIO_CMD_SAME_AR;
trace_fsi_master_gpio_cmd_same_addr(master);
} else if (check_relative_address(master, id, addr, &rel_addr)) { } else if (check_relative_address(master, id, addr, &rel_addr)) {
/* 8 bits plus sign */ /* 8 bits plus sign */
addr_bits = 9; addr_bits = 9;
addr = rel_addr; addr = rel_addr;
opcode = FSI_GPIO_CMD_REL_AR; opcode = FSI_GPIO_CMD_REL_AR;
trace_fsi_master_gpio_cmd_rel_addr(master, rel_addr);
} else { } else {
addr_bits = 21; addr_bits = 21;
opcode = FSI_GPIO_CMD_ABS_AR; opcode = FSI_GPIO_CMD_ABS_AR;
trace_fsi_master_gpio_cmd_abs_addr(master, addr);
} }
/* /*
...@@ -337,12 +347,6 @@ static void build_epoll_command(struct fsi_gpio_msg *cmd, uint8_t slave_id) ...@@ -337,12 +347,6 @@ static void build_epoll_command(struct fsi_gpio_msg *cmd, uint8_t slave_id)
msg_push_crc(cmd); msg_push_crc(cmd);
} }
static void echo_delay(struct fsi_master_gpio *master)
{
set_sda_output(master, 1);
clock_toggle(master, master->t_echo_delay);
}
static void build_term_command(struct fsi_gpio_msg *cmd, uint8_t slave_id) static void build_term_command(struct fsi_gpio_msg *cmd, uint8_t slave_id)
{ {
cmd->bits = 0; cmd->bits = 0;
......
...@@ -50,6 +50,22 @@ TRACE_EVENT(fsi_master_gpio_out, ...@@ -50,6 +50,22 @@ TRACE_EVENT(fsi_master_gpio_out,
) )
); );
TRACE_EVENT(fsi_master_gpio_clock_zeros,
TP_PROTO(const struct fsi_master_gpio *master, int clocks),
TP_ARGS(master, clocks),
TP_STRUCT__entry(
__field(int, master_idx)
__field(int, clocks)
),
TP_fast_assign(
__entry->master_idx = master->master.idx;
__entry->clocks = clocks;
),
TP_printk("fsi-gpio%d clock %d zeros",
__entry->master_idx, __entry->clocks
)
);
TRACE_EVENT(fsi_master_gpio_break, TRACE_EVENT(fsi_master_gpio_break,
TP_PROTO(const struct fsi_master_gpio *master), TP_PROTO(const struct fsi_master_gpio *master),
TP_ARGS(master), TP_ARGS(master),
...@@ -107,6 +123,49 @@ TRACE_EVENT(fsi_master_gpio_poll_response_busy, ...@@ -107,6 +123,49 @@ TRACE_EVENT(fsi_master_gpio_poll_response_busy,
__entry->master_idx, __entry->busy) __entry->master_idx, __entry->busy)
); );
TRACE_EVENT(fsi_master_gpio_cmd_abs_addr,
TP_PROTO(const struct fsi_master_gpio *master, u32 addr),
TP_ARGS(master, addr),
TP_STRUCT__entry(
__field(int, master_idx)
__field(u32, addr)
),
TP_fast_assign(
__entry->master_idx = master->master.idx;
__entry->addr = addr;
),
TP_printk("fsi-gpio%d: Sending ABS_ADR %06x",
__entry->master_idx, __entry->addr)
);
TRACE_EVENT(fsi_master_gpio_cmd_rel_addr,
TP_PROTO(const struct fsi_master_gpio *master, u32 rel_addr),
TP_ARGS(master, rel_addr),
TP_STRUCT__entry(
__field(int, master_idx)
__field(u32, rel_addr)
),
TP_fast_assign(
__entry->master_idx = master->master.idx;
__entry->rel_addr = rel_addr;
),
TP_printk("fsi-gpio%d: Sending REL_ADR %03x",
__entry->master_idx, __entry->rel_addr)
);
TRACE_EVENT(fsi_master_gpio_cmd_same_addr,
TP_PROTO(const struct fsi_master_gpio *master),
TP_ARGS(master),
TP_STRUCT__entry(
__field(int, master_idx)
),
TP_fast_assign(
__entry->master_idx = master->master.idx;
),
TP_printk("fsi-gpio%d: Sending SAME_ADR",
__entry->master_idx)
);
#endif /* _TRACE_FSI_MASTER_GPIO_H */ #endif /* _TRACE_FSI_MASTER_GPIO_H */
#include <trace/define_trace.h> #include <trace/define_trace.h>
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