[PATCH] staging: media: atomisp: remove functions doing ftrace like logging

Adarsh Das posted 1 patch 5 days, 20 hours ago
.../media/atomisp/pci/ia_css_isp_configs.c    | 26 -------------------
1 file changed, 26 deletions(-)
[PATCH] staging: media: atomisp: remove functions doing ftrace like logging
Posted by Adarsh Das 5 days, 20 hours ago
Remove ia_css_debug_dtrace() calls which only log function entry.
ftrace should be used instead.

Signed-off-by: Adarsh Das <adarshdas950@gmail.com>
---
 .../media/atomisp/pci/ia_css_isp_configs.c    | 26 -------------------
 1 file changed, 26 deletions(-)

diff --git a/drivers/staging/media/atomisp/pci/ia_css_isp_configs.c b/drivers/staging/media/atomisp/pci/ia_css_isp_configs.c
index 38c9c62366d6..1dce9fa926ab 100644
--- a/drivers/staging/media/atomisp/pci/ia_css_isp_configs.c
+++ b/drivers/staging/media/atomisp/pci/ia_css_isp_configs.c
@@ -18,8 +18,6 @@ int ia_css_configure_iterator(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -41,8 +39,6 @@ int ia_css_configure_copy_output(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -66,8 +62,6 @@ int ia_css_configure_crop(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -89,8 +83,6 @@ int ia_css_configure_fpn(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -111,8 +103,6 @@ int ia_css_configure_dvs(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -133,8 +123,6 @@ int ia_css_configure_qplane(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -156,8 +144,6 @@ int ia_css_configure_output0(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -179,8 +165,6 @@ int ia_css_configure_output1(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -202,8 +186,6 @@ int ia_css_configure_output(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -225,8 +207,6 @@ int ia_css_configure_raw(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -248,8 +228,6 @@ int ia_css_configure_tnr(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -271,8 +249,6 @@ int ia_css_configure_ref(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
@@ -294,8 +270,6 @@ int ia_css_configure_vf(const struct ia_css_binary *binary,
 	unsigned int offset = 0;
 	unsigned int size   = 0;
 
-	ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "%s:\n", __func__);
-
 	if (!binary->info->mem_offsets.offsets.config)
 		return 0;
 
-- 
2.52.0
Re: [PATCH] staging: media: atomisp: remove functions doing ftrace like logging
Posted by Andy Shevchenko 4 days, 2 hours ago
On Sun, Feb 1, 2026 at 4:24 PM Adarsh Das <adarshdas950@gmail.com> wrote:
>
> Remove ia_css_debug_dtrace() calls which only log function entry.
> ftrace should be used instead.

LGTM,
Reviewed-by: Andy Shevchenko <andy@kernel.org>

-- 
With Best Regards,
Andy Shevchenko