From: Jonathan Cormier <jcormier@criticallink.com>
To: Nishanth Menon <nm@ti.com>, Tero Kristo <kristo@kernel.org>,
Santosh Shilimkar <ssantosh@kernel.org>,
Steven Rostedt <rostedt@goodmis.org>,
Masami Hiramatsu <mhiramat@kernel.org>,
Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
Nishanth Menon <nm@ti.com>,
Vignesh Raghavendra <vigneshr@ti.com>
Cc: linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-trace-kernel@vger.kernel.org,
Jonathan Cormier <jcormier@criticallink.com>
Subject: [PATCH 2/2] firmware: ti_sci: trace: Decode message types
Date: Wed, 20 Aug 2025 14:10:45 -0400 [thread overview]
Message-ID: <20250820-linux_master_ti_sci_trace-v1-2-2a69c0abf55e@criticallink.com> (raw)
In-Reply-To: <20250820-linux_master_ti_sci_trace-v1-0-2a69c0abf55e@criticallink.com>
Make debugging easier by decoding the ti_sci message types
Signed-off-by: Jonathan Cormier <jcormier@criticallink.com>
---
drivers/firmware/ti_sci_trace.h | 69 ++++++++++++++++++++++++++++++++++++++---
1 file changed, 64 insertions(+), 5 deletions(-)
diff --git a/drivers/firmware/ti_sci_trace.h b/drivers/firmware/ti_sci_trace.h
index 7f99c41ea08aec6cf64d601576cad29b14c6ad5b..88a159e13907d7d7806b312a2ad3103c106d1972 100644
--- a/drivers/firmware/ti_sci_trace.h
+++ b/drivers/firmware/ti_sci_trace.h
@@ -7,6 +7,63 @@
#include <linux/tracepoint.h>
+#define show_ti_sci_msg_type(x) \
+ __print_symbolic(x, \
+ { TI_SCI_MSG_ENABLE_WDT, "ENABLE_WDT" }, \
+ { TI_SCI_MSG_WAKE_RESET, "WAKE_RESET" }, \
+ { TI_SCI_MSG_VERSION, "VERSION" }, \
+ { TI_SCI_MSG_WAKE_REASON, "WAKE_REASON" }, \
+ { TI_SCI_MSG_GOODBYE, "GOODBYE" }, \
+ { TI_SCI_MSG_SYS_RESET, "SYS_RESET" }, \
+ { TI_SCI_MSG_QUERY_FW_CAPS, "QUERY_FW_CAPS" }, \
+ { TI_SCI_MSG_SET_DEVICE_STATE, "SET_DEVICE_STATE" }, \
+ { TI_SCI_MSG_GET_DEVICE_STATE, "GET_DEVICE_STATE" }, \
+ { TI_SCI_MSG_SET_DEVICE_RESETS, "SET_DEVICE_RESETS" }, \
+ { TI_SCI_MSG_SET_CLOCK_STATE, "SET_CLOCK_STATE" }, \
+ { TI_SCI_MSG_GET_CLOCK_STATE, "GET_CLOCK_STATE" }, \
+ { TI_SCI_MSG_SET_CLOCK_PARENT, "SET_CLOCK_PARENT" }, \
+ { TI_SCI_MSG_GET_CLOCK_PARENT, "GET_CLOCK_PARENT" }, \
+ { TI_SCI_MSG_GET_NUM_CLOCK_PARENTS, "GET_NUM_CLOCK_PARENTS" }, \
+ { TI_SCI_MSG_SET_CLOCK_FREQ, "SET_CLOCK_FREQ" }, \
+ { TI_SCI_MSG_QUERY_CLOCK_FREQ, "QUERY_CLOCK_FREQ" }, \
+ { TI_SCI_MSG_GET_CLOCK_FREQ, "GET_CLOCK_FREQ" }, \
+ { TI_SCI_MSG_PREPARE_SLEEP, "PREPARE_SLEEP" }, \
+ { TI_SCI_MSG_LPM_WAKE_REASON, "LPM_WAKE_REASON" }, \
+ { TI_SCI_MSG_SET_IO_ISOLATION, "SET_IO_ISOLATION" }, \
+ { TI_SCI_MSG_LPM_SET_DEVICE_CONSTRAINT, "LPM_SET_DEVICE_CONSTRAINT" }, \
+ { TI_SCI_MSG_LPM_SET_LATENCY_CONSTRAINT, "LPM_SET_LATENCY_CONSTRAINT" }, \
+ { TI_SCI_MSG_GET_RESOURCE_RANGE, "GET_RESOURCE_RANGE" }, \
+ { TI_SCI_MSG_SET_IRQ, "SET_IRQ" }, \
+ { TI_SCI_MSG_FREE_IRQ, "FREE_IRQ" }, \
+ { TI_SCI_MSG_RM_RING_ALLOCATE, "RM_RING_ALLOCATE" }, \
+ { TI_SCI_MSG_RM_RING_FREE, "RM_RING_FREE" }, \
+ { TI_SCI_MSG_RM_RING_RECONFIG, "RM_RING_RECONFIG" }, \
+ { TI_SCI_MSG_RM_RING_RESET, "RM_RING_RESET" }, \
+ { TI_SCI_MSG_RM_RING_CFG, "RM_RING_CFG" }, \
+ { TI_SCI_MSG_RM_PSIL_PAIR, "RM_PSIL_PAIR" }, \
+ { TI_SCI_MSG_RM_PSIL_UNPAIR, "RM_PSIL_UNPAIR" }, \
+ { TI_SCI_MSG_RM_UDMAP_TX_ALLOC, "RM_UDMAP_TX_ALLOC" }, \
+ { TI_SCI_MSG_RM_UDMAP_TX_FREE, "RM_UDMAP_TX_FREE" }, \
+ { TI_SCI_MSG_RM_UDMAP_RX_ALLOC, "RM_UDMAP_RX_ALLOC" }, \
+ { TI_SCI_MSG_RM_UDMAP_RX_FREE, "RM_UDMAP_RX_FREE" }, \
+ { TI_SCI_MSG_RM_UDMAP_FLOW_CFG, "RM_UDMAP_FLOW_CFG" }, \
+ { TI_SCI_MSG_RM_UDMAP_OPT_FLOW_CFG, "RM_UDMAP_OPT_FLOW_CFG" }, \
+ { TISCI_MSG_RM_UDMAP_TX_CH_CFG, "RM_UDMAP_TX_CH_CFG" }, \
+ { TISCI_MSG_RM_UDMAP_TX_CH_GET_CFG, "RM_UDMAP_TX_CH_GET_CFG" }, \
+ { TISCI_MSG_RM_UDMAP_RX_CH_CFG, "RM_UDMAP_RX_CH_CFG" }, \
+ { TISCI_MSG_RM_UDMAP_RX_CH_GET_CFG, "RM_UDMAP_RX_CH_GET_CFG" }, \
+ { TISCI_MSG_RM_UDMAP_FLOW_CFG, "RM_UDMAP_FLOW_CFG" }, \
+ { TISCI_MSG_RM_UDMAP_FLOW_SIZE_THRESH_CFG, "RM_UDMAP_FLOW_SIZE_THRESH_CFG" }, \
+ { TISCI_MSG_RM_UDMAP_FLOW_GET_CFG, "RM_UDMAP_FLOW_GET_CFG" }, \
+ { TISCI_MSG_RM_UDMAP_FLOW_SIZE_THRESH_GET_CFG, "RM_UDMAP_FLOW_SIZE_THRESH_GET_CFG" }, \
+ { TI_SCI_MSG_PROC_REQUEST, "PROC_REQUEST" }, \
+ { TI_SCI_MSG_PROC_RELEASE, "PROC_RELEASE" }, \
+ { TI_SCI_MSG_PROC_HANDOVER, "PROC_HANDOVER" }, \
+ { TI_SCI_MSG_SET_CONFIG, "SET_CONFIG" }, \
+ { TI_SCI_MSG_SET_CTRL, "SET_CTRL" }, \
+ { TI_SCI_MSG_GET_STATUS, "GET_STATUS" } \
+ )
+
DECLARE_EVENT_CLASS(ti_sci_hdr_event_class,
TP_PROTO(struct ti_sci_msg_hdr *hdr, int status),
@@ -28,8 +85,9 @@ DECLARE_EVENT_CLASS(ti_sci_hdr_event_class,
__entry->status = status;
),
- TP_printk("type=%04X host=%02X seq=%02X flags=%08X status=%d",
- __entry->type, __entry->host, __entry->seq, __entry->flags, __entry->status)
+ TP_printk("type=%s host=%02X seq=%02X flags=%08X status=%d",
+ show_ti_sci_msg_type(__entry->type), __entry->host,
+ __entry->seq, __entry->flags, __entry->status)
);
DEFINE_EVENT(ti_sci_hdr_event_class,
@@ -73,9 +131,10 @@ TRACE_EVENT(ti_sci_msg_dump,
memcpy(__get_dynamic_array(cmd), xfer->xfer_buf, __entry->len);
),
- TP_printk("type=%04X host=%02X seq=%02X flags=%08X data=%s",
- __entry->type, __entry->host, __entry->seq, __entry->flags,
- __print_hex_str(__get_dynamic_array(cmd), __entry->len))
+ TP_printk("type=%s host=%02X seq=%02X flags=%08X data=%s",
+ show_ti_sci_msg_type(__entry->type), __entry->host,
+ __entry->seq, __entry->flags,
+ __print_hex(__get_dynamic_array(cmd), __entry->len))
);
#endif /* _TRACE_TI_SCI_H */
--
2.43.0
next prev parent reply other threads:[~2025-08-20 18:11 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-20 18:10 [PATCH 0/2] PATCH: firmware: ti_sci: Add trace events to TI SCI Jonathan Cormier
2025-08-20 18:10 ` [PATCH 1/2] firmware: ti_sci: Add trace events Jonathan Cormier
2025-08-20 19:59 ` Andrew Davis
2025-08-20 18:10 ` Jonathan Cormier [this message]
2025-08-20 20:04 ` [PATCH 0/2] PATCH: firmware: ti_sci: Add trace events to TI SCI Andrew Davis
2025-08-22 16:05 ` Jon Cormier
2025-08-22 17:39 ` Andrew Davis
2025-08-22 18:30 ` Jon Cormier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250820-linux_master_ti_sci_trace-v1-2-2a69c0abf55e@criticallink.com \
--to=jcormier@criticallink.com \
--cc=kristo@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-trace-kernel@vger.kernel.org \
--cc=mathieu.desnoyers@efficios.com \
--cc=mhiramat@kernel.org \
--cc=nm@ti.com \
--cc=rostedt@goodmis.org \
--cc=ssantosh@kernel.org \
--cc=vigneshr@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).