On Thu, Aug 20, 2015 at 04:07:06PM +0100, Matt Fleming wrote: > On Tue, 11 Aug, at 02:16:27PM, Lee, Chun-Yi wrote: > > Moved the function of transferring EFI status to kernel error for > > later used by EFI stub. > > Might I suggest: > > "Move the function for converting EFI status to kernel error values > from drivers/firmware/efi/ to include/linux/efi.h for use by the x86 > EFI stub in an upcoming patch." > > ? > Thanks for your suggestion, I will follow it. > > Reviewed-by: Jiri Kosina <jkosina@xxxxxxxx> > > Tested-by: Jiri Kosina <jkosina@xxxxxxxx> > > Signed-off-by: Lee, Chun-Yi <jlee@xxxxxxxx> > > --- > > drivers/firmware/efi/vars.c | 33 --------------------------------- > > include/linux/efi.h | 33 +++++++++++++++++++++++++++++++++ > > 2 files changed, 33 insertions(+), 33 deletions(-) > > The patch content is fine, > > Reviewed-by: Matt Fleming <matt.fleming@xxxxxxxxx> > Thanks > -- > Matt Fleming, Intel Open Source Technology Center Joey Lee -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html