svn commit: r336514 - head/sys/dev/vt/hw/ofwfb
Justin Hibbits
jhibbits at freebsd.org
Thu Jul 19 23:49:11 UTC 2018
You're right. I'm an idiot. I compiled but didn't try booting.
Fixing it after I fix the build.
- Justin
On Thu, Jul 19, 2018 at 3:37 PM Nathan Whitehorn <nwhitehorn at freebsd.org> wrote:
>
> Isn't stdout-path supposed to contain a string rather than an ihandle?
> This code assumes it is an ihandle (or xref phandle), which I think is
> wrong.
> -Nathan
>
> On 07/19/18 13:22, Justin Hibbits wrote:
> > Author: jhibbits
> > Date: Thu Jul 19 20:22:46 2018
> > New Revision: 336514
> > URL: https://svnweb.freebsd.org/changeset/base/336514
> >
> > Log:
> > ofwfb: Check for /chosen/stdout-path in addition to /chosen/stdout
> >
> > Some platforms use /chosen/stdout-path as the property containing the path to
> > the stdout node, not /chosen/stdout.
> >
> > Modified:
> > head/sys/dev/vt/hw/ofwfb/ofwfb.c
> >
> > Modified: head/sys/dev/vt/hw/ofwfb/ofwfb.c
> > ==============================================================================
> > --- head/sys/dev/vt/hw/ofwfb/ofwfb.c Thu Jul 19 20:20:43 2018 (r336513)
> > +++ head/sys/dev/vt/hw/ofwfb/ofwfb.c Thu Jul 19 20:22:46 2018 (r336514)
> > @@ -102,6 +102,10 @@ ofwfb_probe(struct vt_device *vd)
> > if (OF_getprop(chosen, "stdout", &stdout, sizeof(stdout)) ==
> > sizeof(stdout))
> > node = OF_instance_to_package(stdout);
> > + if (node == -1)
> > + if (OF_getprop(chosen, "stdout-path", &stdout, sizeof(stdout)) ==
> > + sizeof(stdout))
> > + node = OF_instance_to_package(stdout);
> > if (node == -1) {
> > /*
> > * The "/chosen/stdout" does not exist try
> >
>
More information about the svn-src-all
mailing list