diff options
author | Pridhiviraj Paidipeddi <ppaidipe@linux.vnet.ibm.com> | 2018-03-01 22:33:19 +0530 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2018-03-01 20:27:01 -0600 |
commit | 20f685a3627a2a522c465716377561a8fbcc608f (patch) | |
tree | a1980c0de43a39ea35db51267d602381f75ff922 /hw/fsp/fsp-console.c | |
parent | f23240f50653f3404b21e064f80e8c74b095c2ea (diff) | |
download | skiboot-20f685a3627a2a522c465716377561a8fbcc608f.zip skiboot-20f685a3627a2a522c465716377561a8fbcc608f.tar.gz skiboot-20f685a3627a2a522c465716377561a8fbcc608f.tar.bz2 |
console(lpc/fsp-console): Use only stdout-path property on P9 and above
dtc tool complaining about below warning as usage of linux,stdout-path
property under /chosen node is deprecated.
dts: Warning
(chosen_node_stdout_path): Use 'stdout-path' instead of 'linux,stdout-path'
So this patch fix this by using stdout-path property on all the systems
and keep linux,stdout-path only on P8 and before. This property refers to
a node which represents the device to be used for boot console output.
Verified boot on both P8 and P9 systems with new and older kernels.
And also verified dtc warnings got fixed in both P8 and P9.
Signed-off-by: Pridhiviraj Paidipeddi <ppaidipe@linux.vnet.ibm.com>
[stewart: simplify logic]
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hw/fsp/fsp-console.c')
-rw-r--r-- | hw/fsp/fsp-console.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/hw/fsp/fsp-console.c b/hw/fsp/fsp-console.c index 8d845d8..fc092ce 100644 --- a/hw/fsp/fsp-console.c +++ b/hw/fsp/fsp-console.c @@ -1,4 +1,4 @@ -/* Copyright 2013-2014 IBM Corp. +/* Copyright 2013-2018 IBM Corp. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -1072,14 +1072,22 @@ void fsp_console_select_stdout(void) */ } } + dt_check_del_prop(dt_chosen, "linux,stdout-path"); + dt_check_del_prop(dt_chosen, "stdout-path"); if (fsp_serials[1].open && use_serial) { - dt_add_property_string(dt_chosen, "linux,stdout-path", + if (proc_gen < proc_gen_p9) + dt_add_property_string(dt_chosen, "linux,stdout-path", + "/ibm,opal/consoles/serial@1"); + dt_add_property_string(dt_chosen, "stdout-path", "/ibm,opal/consoles/serial@1"); prlog(PR_NOTICE, "FSPCON: default console set to serial A\n"); } else { - dt_add_property_string(dt_chosen, "linux,stdout-path", + if (proc_gen < proc_gen_p9) + dt_add_property_string(dt_chosen, "linux,stdout-path", + "/ibm,opal/consoles/serial@0"); + dt_add_property_string(dt_chosen, "stdout-path", "/ibm,opal/consoles/serial@0"); prlog(PR_NOTICE, "FSPCON: default console set to SOL/DVS\n"); } |