Re: [patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-17 Thread Dave Young
On 11/15/13 at 04:21pm, Toshi Kani wrote: > On Tue, 2013-11-05 at 16:20 +0800, dyo...@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

Re: [patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-17 Thread Dave Young
On 11/15/13 at 04:21pm, Toshi Kani wrote: On Tue, 2013-11-05 at 16:20 +0800, dyo...@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)

Re: [patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-15 Thread Toshi Kani
On Tue, 2013-11-05 at 16:20 +0800, dyo...@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; > +

Re: [patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-15 Thread Toshi Kani
On Tue, 2013-11-05 at 16:20 +0800, dyo...@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,

Re: [patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-13 Thread Dave Young
On 11/13/13 at 03:50pm, Matt Fleming wrote: > On Tue, 05 Nov, at 04:20:10PM, dyo...@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. > > > > v1->v2: > > refresh; coding style

Re: [patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-13 Thread Matt Fleming
On Tue, 05 Nov, at 04:20:10PM, dyo...@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. > > v1->v2: > refresh; coding style fixes. > > Signed-off-by: Dave Young > --- >

Re: [patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-13 Thread Matt Fleming
On Tue, 05 Nov, at 04:20:10PM, dyo...@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. v1-v2: refresh; coding style fixes. Signed-off-by: Dave Young dyo...@redhat.com ---

Re: [patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-13 Thread Dave Young
On 11/13/13 at 03:50pm, Matt Fleming wrote: On Tue, 05 Nov, at 04:20:10PM, dyo...@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. v1-v2: refresh; coding style fixes.

[patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-05 Thread dyoung
Add two small functions: efi_merge_regions and efi_map_regions, efi_enter_virtual_mode calls them instead of embedding two long for loop. v1->v2: refresh; coding style fixes. Signed-off-by: Dave Young --- arch/x86/platform/efi/efi.c | 107 +--- 1 file

[patch 3/7 v2] Cleanup efi_enter_virtual_mode function

2013-11-05 Thread dyoung
Add two small functions: efi_merge_regions and efi_map_regions, efi_enter_virtual_mode calls them instead of embedding two long for loop. v1-v2: refresh; coding style fixes. Signed-off-by: Dave Young dyo...@redhat.com --- arch/x86/platform/efi/efi.c | 107