drivers/console: xtensa_sim_console: implement arch_printk_char_out()

andyross · andyross · commit 8e7d434f64d6 · 2023-10-28T05:27:58.000-07:00
This is an older driver and didn't support the weak
arch_printk_char_out() hook, which is a link-time symbol that allows
logging to work from the first instruction.  Some drivers can't do
that because they need an initialization step, but this one works
great.

Signed-off-by: Andy Ross &lt;andyross@google.com&gt;
diff --git a/drivers/console/xtensa_sim_console.c b/drivers/console/xtensa_sim_console.c
@@ -13,7 +13,7 @@
  * @param c Character to output
  * @return The character passed as input.
  */
-static int console_out(int c)
+int arch_printk_char_out(int c)
 {
 	char buf[16];
 
@@ -54,8 +54,8 @@ extern void __printk_hook_install(int (*fn)(int));
  */
 static void xt_sim_console_hook_install(void)
 {
-	__stdout_hook_install(console_out);
-	__printk_hook_install(console_out);
+	__stdout_hook_install(arch_printk_char_out);
+	__printk_hook_install(arch_printk_char_out);
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@`
`13`	`13`	`* @param c Character to output`
`14`	`14`	`* @return The character passed as input.`
`15`	`15`	`*/`
`16`		`-static int console_out(int c)`
	`16`	`+int arch_printk_char_out(int c)`
`17`	`17`	`{`
`18`	`18`	`char buf[16];`
`19`	`19`
`@@ -54,8 +54,8 @@ extern void __printk_hook_install(int (*fn)(int));`
`54`	`54`	`*/`
`55`	`55`	`static void xt_sim_console_hook_install(void)`
`56`	`56`	`{`
`57`		`- __stdout_hook_install(console_out);`
`58`		`- __printk_hook_install(console_out);`
	`57`	`+ __stdout_hook_install(arch_printk_char_out);`
	`58`	`+ __printk_hook_install(arch_printk_char_out);`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`/**`