Re: [PATCH V4 07/15] Move x86 specific video and disk probing code

["Jan Beulich" <JBeulich@suse.com>]

>>> On 10.09.14 at 02:51, <roy.franz@linaro.org> wrote:
> @@ -811,189 +785,7 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE *SystemTable)
>      cfg.addr = 0;
>  
>      dir_handle->Close(dir_handle);
> -
> -    if ( gop && !base_video )
> -    {
> -        for ( i = size = 0; i < gop->Mode->MaxMode; ++i )
> -        {
> -            unsigned int bpp = 0;
> -
> -            status = gop->QueryMode(gop, i, &info_size, &mode_info);
> -            if ( EFI_ERROR(status) )
> -                continue;
> -            switch ( mode_info->PixelFormat )
> -            {
> -            case PixelBitMask:
> -                bpp = hweight32(mode_info->PixelInformation.RedMask |
> -                                mode_info->PixelInformation.GreenMask |
> -                                mode_info->PixelInformation.BlueMask);
> -                break;
> -            case PixelRedGreenBlueReserved8BitPerColor:
> -            case PixelBlueGreenRedReserved8BitPerColor:
> -                bpp = 24;
> -                break;
> -            default:
> -                continue;
> -            }
> -            if ( cols == mode_info->HorizontalResolution &&
> -                 rows == mode_info->VerticalResolution &&
> -                 (!depth || bpp == depth) )
> -            {
> -                gop_mode = i;
> -                break;
> -            }
> -            if ( !cols && !rows &&
> -                 mode_info->HorizontalResolution *
> -                 mode_info->VerticalResolution > size )
> -            {
> -                size = mode_info->HorizontalResolution *
> -                       mode_info->VerticalResolution;
> -                gop_mode = i;
> -            }
> -        }
> -    }

This together with ...
 
> @@ -1075,77 +867,7 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE *SystemTable)
>      l3_bootmap[l3_table_offset(xen_phys_start + (8 << L2_PAGETABLE_SHIFT) - 1)] =
>          l3e_from_paddr((UINTN)l2_bootmap, __PAGE_HYPERVISOR);
>  
> -    if ( gop )
> -    {
> -        int bpp = 0;
> -
> -        /* Set graphics mode. */
> -        if ( gop_mode < gop->Mode->MaxMode && gop_mode != gop->Mode->Mode )
> -            gop->SetMode(gop, gop_mode);

... this isn't arch-specific again.

Jan