[U-Boot] [PATCH 6/9] efi_loader: Add runtime services

Leif Lindholm leif.lindholm at linaro.org
Fri Jan 15 15:22:27 CET 2016


On Fri, Jan 15, 2016 at 03:15:37PM +0100, Alexander Graf wrote:
> On 15.01.16 14:52, Leif Lindholm wrote:
> >>> "The ResetSystem() function does not return."
> >>
> >> Hrm, I think returning EFI_UNSUPPORTED is still better than while(1) {
> >> }. With the return an OS at least has the chance to fix things up itself.
> > 
> > I'm not saying it isn't better, I'm saying it's not compliant - there
> > is no valid return value. I would prefer simply having the pointer set
> > to NULL.
> 
> Is a NULL function pointer valid here?

More than returning is.

/
    Leif


More information about the U-Boot mailing list