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

Leif Lindholm leif.lindholm at linaro.org
Fri Jan 15 14:52:14 CET 2016


On Fri, Jan 15, 2016 at 01:26:42AM +0100, Alexander Graf wrote:
> On 26.12.15 19:33, Leif Lindholm wrote:
> >> +	.reset_system = (void *)&efi_unimplemented,
> > 
> > "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.

/
    Leif


More information about the U-Boot mailing list