On Fri, Sep 13, 2019 at 2:06 AM Ard Biesheuvel <ard.biesheu...@linaro.org> wrote: > > On Fri, 30 Aug 2019 at 03:06, Dan Williams <dan.j.willi...@intel.com> wrote: > > > > In preparation for adding another EFI_MEMMAP dependent call that needs > > to occur before e820__memblock_setup() fixup the existing efi calls to > > check for EFI_MEMMAP internally. This ends up being cleaner than the > > alternative of checking EFI_MEMMAP multiple times in setup_arch(). > > > > Cc: <x...@kernel.org> > > Cc: Ingo Molnar <mi...@redhat.com> > > Cc: "H. Peter Anvin" <h...@zytor.com> > > Cc: Andy Lutomirski <l...@kernel.org> > > Cc: Thomas Gleixner <t...@linutronix.de> > > Cc: Peter Zijlstra <pet...@infradead.org> > > Reviewed-by: Dave Hansen <dave.han...@linux.intel.com> > > Signed-off-by: Dan Williams <dan.j.willi...@intel.com> > > I'd prefer it if the spurious whitespace changes could be dropped, but > otherwise, this looks fine to me, so I am not going to obsess about > it.
Fair point, I'll drop those when I resubmit after -rc1. > Reviewed-by: Ard Biesheuvel <ard.biesheu...@linaro.org> Thanks!