From 47f570fcfd5a5a8c58913fee8fa2db2f99591ae9 Mon Sep 17 00:00:00 2001 From: "Vineeth Pillai (Google)" Date: Mon, 23 Mar 2026 12:00:32 -0400 Subject: [PATCH] spi: Use trace_call__##name() at guarded tracepoint call sites MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Replace trace_foo() with the new trace_call__foo() at sites already guarded by trace_foo_enabled(), avoiding a redundant static_branch_unlikely() re-evaluation inside the tracepoint. trace_call__foo() calls the tracepoint callbacks directly without utilizing the static branch again. Cc: Michael Hennerich Cc: Nuno Sá Cc: David Lechner Link: https://patch.msgid.link/20260323160052.17528-14-vineeth@bitbyteword.org Suggested-by: Steven Rostedt Suggested-by: Peter Zijlstra Signed-off-by: Vineeth Pillai (Google) Assisted-by: Claude:claude-sonnet-4-6 Acked-by: Mark Brown Signed-off-by: Steven Rostedt (Google) --- drivers/spi/spi-axi-spi-engine.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/spi/spi-axi-spi-engine.c b/drivers/spi/spi-axi-spi-engine.c index 48ee5bf48173..02bbc5d0cfc5 100644 --- a/drivers/spi/spi-axi-spi-engine.c +++ b/drivers/spi/spi-axi-spi-engine.c @@ -953,7 +953,7 @@ static int spi_engine_transfer_one_message(struct spi_controller *host, struct spi_transfer *xfer; list_for_each_entry(xfer, &msg->transfers, transfer_list) - trace_spi_transfer_start(msg, xfer); + trace_call__spi_transfer_start(msg, xfer); } spin_lock_irqsave(&spi_engine->lock, flags); @@ -987,7 +987,7 @@ static int spi_engine_transfer_one_message(struct spi_controller *host, struct spi_transfer *xfer; list_for_each_entry(xfer, &msg->transfers, transfer_list) - trace_spi_transfer_stop(msg, xfer); + trace_call__spi_transfer_stop(msg, xfer); } out: