svn commit: r336514 - head/sys/dev/vt/hw/ofwfb
Emmanuel Vadot
manu at bidouilliste.com
Thu Jul 19 20:48:18 UTC 2018
On Thu, 19 Jul 2018 15:38:49 -0500
Kyle Evans <kevans at freebsd.org> wrote:
> Seems like some bad decisions have been made with regards to naming...
> e.g. https://lkml.org/lkml/2018/6/25/784
See this thread too :
http://lists.infradead.org/pipermail/linux-arm-kernel/2018-June/584613.html
They did bad things.
It seems that the IMX linux guys don't really know what they are
talking about.
stdout-path is supposed to contain a string.
> On Thu, Jul 19, 2018 at 3:26 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
> >>
> >
> >
--
Emmanuel Vadot <manu at bidouilliste.com> <manu at freebsd.org>
More information about the svn-src-head
mailing list