[Skiboot] [PATCH] console(lpc/fsp-console): Use just stdout-path.
stewart at linux.vnet.ibm.com
Thu Mar 1 18:06:17 AEDT 2018
Pridhiviraj Paidipeddi <ppaidipe at linux.vnet.ibm.com> writes:
> 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 will fix this by using just stdout-path property for specifying
> to the node which represents the device to be used for boot console output.
> Signed-off-by: Pridhiviraj Paidipeddi <ppaidipe at linux.vnet.ibm.com>
> core/console.c | 4 ++--
> core/init.c | 2 +-
> hw/fsp/fsp-console.c | 6 +++---
> hw/lpc-uart.c | 4 ++--
> 4 files changed, 8 insertions(+), 8 deletions(-)
As kind of discussed on internal slack, the kernel commit that will pick
up the 'stdout-path' property is this one:
Author: Grant Likely <grant.likely at linaro.org>
Date: Thu Mar 27 17:11:23 2014 -0700
of: Add support for ePAPR "stdout-path" property
ePAPR 1.1 defines the "stdout-path" property for specifying the console
device, but Linux currently only handles the older "linux,stdout-path"
property. This patch adds parsing for the new property name.
Signed-off-by: Grant Likely <grant.likely at linaro.org>
which came in in v3.14-rc5-29-g676e1b2fcd9d - which is after when it's
generally accepted that things should work on p8.
So, if we do this with a "if power9" (as in, p9 cpu or greater) then we
should be okay.
I'm not sure what the effect is of putting both in on p8? That may be
okay too (I'm not sure what the test looks for).
OPAL Architect, IBM.
More information about the Skiboot