On Wed, 18 May 2016 17:15:10 +0200
Zbigniew Bodek <z...@semihalf.com> wrote:
> 2016-05-18 17:05 GMT+02:00 Andrew Turner <and...@freebsd.org>:
...
> > #ifdef INTRNG
> > xref = OF_xref_from_node(ofw_bus_get_node(dev));
> > - if (intr_pic_register(dev, xref) != 0) {
> > + if (intr_pic_register(dev, xref) == NULL) {
> > device_printf(dev, "could not register PIC\n");
> > goto error;
> > }
> > @@ -172,7 +172,7 @@ error:
> > /* Failure so free resources */
> > gic_v3_detach(dev);
> >
> > - return (err);
> > + return (ENXIO);
> >
>
>
> Few line above we have:
> err = gic_v3_attach(dev);
> if (err != 0)
> goto error;
>
> So now we would not return error code from gic_v3_attach() but always
> ENXIO...
The error value doesn't matter, as long as it's non-zero. This also
fixes the return value in the intrng case if either of
intr_pic_register or intr_pic_claim_root fail. You might get a little
information from the return value being printed, however it would be
more useful to have verbose logging to print an error message.
In all of these failure cases there isn't much we can do as there is no
root interrupt controller, the boot will fail later on when we enable
interrupts.
Andrew
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
