On 11/15/13 at 04:21pm, Toshi Kani wrote: > On Tue, 2013-11-05 at 16:20 +0800, dyoung at redhat.com wrote: > > Add two small functions: > > efi_merge_regions and efi_map_regions, efi_enter_virtual_mode > > calls them instead of embedding two long for loop. > > : > > > +void __init efi_enter_virtual_mode(void) > > +{ > > + efi_status_t status; > > + void *p, *new_memmap = NULL; > > *p is unused. The *p was removed in later patch, will remove it in this patch. Thanks for review. Dave