m...@console-pimps.org writes:

> On Wed, 12 Feb, at 09:15:03AM, Toshi Kani wrote:
>> 
>> Hi Matt,
>> 
>> Yes, I agree that the table size should be 0x38.  However, ACPI spec
>> states that bit0 of status indicates if the boot image graphic is valid.
>> This bit is set to 0 (invalid) on the system.  Can you check this bit
>> and return when invalid? 
>
> Madper, could you try out this patch?
>
Thanks Matt. This patch fix the panic. And ioremap warning disappeared
after apply your patch.
> ---
>
> diff --git a/arch/x86/platform/efi/efi-bgrt.c 
> b/arch/x86/platform/efi/efi-bgrt.c
> index 4df9591eadad..f15103dff4b4 100644
> --- a/arch/x86/platform/efi/efi-bgrt.c
> +++ b/arch/x86/platform/efi/efi-bgrt.c
> @@ -42,7 +42,7 @@ void __init efi_bgrt_init(void)
>  
>       if (bgrt_tab->header.length < sizeof(*bgrt_tab))
>               return;
> -     if (bgrt_tab->version != 1)
> +     if (bgrt_tab->version != 1 || bgrt_tab->status != 1)
>               return;
>       if (bgrt_tab->image_type != 0 || !bgrt_tab->image_address)
>               return;

-- 
Madper
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to