Commit 9d807761 authored by Tsuchiya Yuto's avatar Tsuchiya Yuto Committed by Mauro Carvalho Chehab

media: atomisp: Remove remaining instance of call to trace_printk

(patch based on intel-aero kernel patch:
 https://github.com/intel-aero/meta-intel-aero-base/commit/26fc9fe5030b63bc9dcf0b5f32981948911ca272)

Here is the original commit message from the aforementioned patch:

	From 26fc9fe5030b63bc9dcf0b5f32981948911ca272 Mon Sep 17 00:00:00 2001
	From: Lucas De Marchi <lucas.demarchi@intel.com>
	Date: Fri, 7 Jul 2017 14:23:53 -0700
	Subject: [PATCH] linux-yocto: Remove remaining instance of call to
	 trace_printk

	It's not sufficient to leave trace_printk() out of "normal call chains" since
	the way trace infrastructure works is that it will allocate the trace_printk
	buffers if the symbol is there (by using a separate section for the function
	and checking if __start_* and __stop_* symbols are different.

	Therefore, even if the default value for the param tells the module to use
	printk(), just the fact that it can be changed to trace_printk() means the
	initialization code will be called.

The trace_printk() was replaced by pr_info() on commit 3d81099c
("media: atomisp: Replace trace_printk by pr_info") for the upstreamed
atomisp, too. However, as the aforementioned commit message says, there
is still a remaining instance. This causes the "trace_printk() being
used" kernel warning message to still appear on the first driver load.

Based on the aforementioned patch, this patch removes the call to
ftrace_vprintk(). This removes that kernel warning.

In addition to this, this patch also removes the following now unused
things:

    - now empty atomisp_css2_dbg_ftrace_print()
    - trace_printk option from dbg_func kernel parameter

Link: https://lore.kernel.org/r/20211017161958.44351-15-kitakar@gmail.comSigned-off-by: default avatarTsuchiya Yuto <kitakar@gmail.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@kernel.org>
parent edd076fe
...@@ -158,13 +158,6 @@ static void atomisp_css2_hw_load(hrt_address addr, void *to, uint32_t n) ...@@ -158,13 +158,6 @@ static void atomisp_css2_hw_load(hrt_address addr, void *to, uint32_t n)
spin_unlock_irqrestore(&mmio_lock, flags); spin_unlock_irqrestore(&mmio_lock, flags);
} }
static int __printf(1, 0) atomisp_css2_dbg_ftrace_print(const char *fmt,
va_list args)
{
ftrace_vprintk(fmt, args);
return 0;
}
static int __printf(1, 0) atomisp_vprintk(const char *fmt, va_list args) static int __printf(1, 0) atomisp_vprintk(const char *fmt, va_list args)
{ {
vprintk(fmt, args); vprintk(fmt, args);
...@@ -801,9 +794,6 @@ static inline int __set_css_print_env(struct atomisp_device *isp, int opt) ...@@ -801,9 +794,6 @@ static inline int __set_css_print_env(struct atomisp_device *isp, int opt)
if (opt == 0) if (opt == 0)
isp->css_env.isp_css_env.print_env.debug_print = NULL; isp->css_env.isp_css_env.print_env.debug_print = NULL;
else if (opt == 1) else if (opt == 1)
isp->css_env.isp_css_env.print_env.debug_print =
atomisp_css2_dbg_ftrace_print;
else if (opt == 2)
isp->css_env.isp_css_env.print_env.debug_print = atomisp_vprintk; isp->css_env.isp_css_env.print_env.debug_print = atomisp_vprintk;
else else
ret = -EINVAL; ret = -EINVAL;
......
...@@ -64,10 +64,10 @@ module_param(dbg_level, int, 0644); ...@@ -64,10 +64,10 @@ module_param(dbg_level, int, 0644);
MODULE_PARM_DESC(dbg_level, "debug message level (default:0)"); MODULE_PARM_DESC(dbg_level, "debug message level (default:0)");
/* log function switch */ /* log function switch */
int dbg_func = 2; int dbg_func = 1;
module_param(dbg_func, int, 0644); module_param(dbg_func, int, 0644);
MODULE_PARM_DESC(dbg_func, MODULE_PARM_DESC(dbg_func,
"log function switch non/trace_printk/printk (default:printk)"); "log function switch non/printk (default:printk)");
int mipicsi_flag; int mipicsi_flag;
module_param(mipicsi_flag, int, 0644); module_param(mipicsi_flag, int, 0644);
......
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