Commit a5235417 authored by Krishna chaitanya chundru's avatar Krishna chaitanya chundru Committed by Manivannan Sadhasivam

bus: mhi: host: Change the trace string for the userspace tools mapping

User space tools can't map strings if we use directly, as the string
address is internal to kernel.

So add trace point strings for the user space tools to map strings
properly.
Suggested-by: default avatarSteven Rostedt <rostedt@goodmis.org>
Signed-off-by: default avatarKrishna chaitanya chundru <quic_krichai@quicinc.com>
Reviewed-by: default avatarManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Link: https://lore.kernel.org/r/20240218-ftrace_string-v1-1-27da85c1f844@quicinc.comSigned-off-by: default avatarManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
parent 27711860
...@@ -1340,7 +1340,7 @@ static int mhi_update_channel_state(struct mhi_controller *mhi_cntrl, ...@@ -1340,7 +1340,7 @@ static int mhi_update_channel_state(struct mhi_controller *mhi_cntrl,
enum mhi_cmd_type cmd = MHI_CMD_NOP; enum mhi_cmd_type cmd = MHI_CMD_NOP;
int ret; int ret;
trace_mhi_channel_command_start(mhi_cntrl, mhi_chan, to_state, "Updating"); trace_mhi_channel_command_start(mhi_cntrl, mhi_chan, to_state, TPS("Updating"));
switch (to_state) { switch (to_state) {
case MHI_CH_STATE_TYPE_RESET: case MHI_CH_STATE_TYPE_RESET:
write_lock_irq(&mhi_chan->lock); write_lock_irq(&mhi_chan->lock);
...@@ -1407,7 +1407,7 @@ static int mhi_update_channel_state(struct mhi_controller *mhi_cntrl, ...@@ -1407,7 +1407,7 @@ static int mhi_update_channel_state(struct mhi_controller *mhi_cntrl,
write_unlock_irq(&mhi_chan->lock); write_unlock_irq(&mhi_chan->lock);
} }
trace_mhi_channel_command_end(mhi_cntrl, mhi_chan, to_state, "Updated"); trace_mhi_channel_command_end(mhi_cntrl, mhi_chan, to_state, TPS("Updated"));
exit_channel_update: exit_channel_update:
mhi_cntrl->runtime_put(mhi_cntrl); mhi_cntrl->runtime_put(mhi_cntrl);
mhi_device_put(mhi_cntrl->mhi_dev); mhi_device_put(mhi_cntrl->mhi_dev);
......
...@@ -84,6 +84,8 @@ DEV_ST_TRANSITION_LIST ...@@ -84,6 +84,8 @@ DEV_ST_TRANSITION_LIST
#define dev_st_trans(a, b) { DEV_ST_TRANSITION_##a, b }, #define dev_st_trans(a, b) { DEV_ST_TRANSITION_##a, b },
#define dev_st_trans_end(a, b) { DEV_ST_TRANSITION_##a, b } #define dev_st_trans_end(a, b) { DEV_ST_TRANSITION_##a, b }
#define TPS(x) tracepoint_string(x)
TRACE_EVENT(mhi_gen_tre, TRACE_EVENT(mhi_gen_tre,
TP_PROTO(struct mhi_controller *mhi_cntrl, struct mhi_chan *mhi_chan, TP_PROTO(struct mhi_controller *mhi_cntrl, struct mhi_chan *mhi_chan,
......
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