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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot