[U-Boot] [PATCH v3] x86: Drop FSP error defines and use EFI instead

Simon Glass sjg at chromium.org
Fri Aug 14 18:49:47 CEST 2015


On 12 August 2015 at 19:54, Bin Meng <bmeng.cn at gmail.com> wrote:
> On Thu, Aug 13, 2015 at 9:33 AM, Simon Glass <sjg at chromium.org> wrote:
>> Now that we have an efi.h header we can use that for FSP error defines.
>> Drop the FSP ones.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> ---
>>
>> Changes in v3:
>> - Drop #include of efi.h since it is not needed
>>
>> Changes in v2:
>> - Drop checking against EFI_STATUS since this reduces code readability
>>
>>  arch/x86/include/asm/fsp/fsp_types.h | 11 -----------
>>  arch/x86/lib/fsp/fsp_common.c        |  2 +-
>>  2 files changed, 1 insertion(+), 12 deletions(-)
>>
>> diff --git a/arch/x86/include/asm/fsp/fsp_types.h b/arch/x86/include/asm/fsp/fsp_types.h
>> index f32d827..4fe69f2 100644
>> --- a/arch/x86/include/asm/fsp/fsp_types.h
>> +++ b/arch/x86/include/asm/fsp/fsp_types.h
>> @@ -68,15 +68,4 @@ struct efi_guid {
>>  #define SIGNATURE_64(A, B, C, D, E, F, G, H)   \
>>         (SIGNATURE_32(A, B, C, D) | ((u64)(SIGNATURE_32(E, F, G, H)) << 32))
>>
>> -/*
>> - * Define FSP API return status code.
>> - * Compatiable with EFI_STATUS defined in PI Spec.
>> - */
>> -#define FSP_SUCCESS            0
>> -#define FSP_INVALID_PARAM      0x80000002
>> -#define FSP_UNSUPPORTED                0x80000003
>> -#define FSP_DEVICE_ERROR       0x80000007
>> -#define FSP_NOT_FOUND          0x8000000E
>> -#define FSP_ALREADY_STARTED    0x80000014
>> -
>>  #endif
>> diff --git a/arch/x86/lib/fsp/fsp_common.c b/arch/x86/lib/fsp/fsp_common.c
>> index 6f72c6d..d564cb9 100644
>> --- a/arch/x86/lib/fsp/fsp_common.c
>> +++ b/arch/x86/lib/fsp/fsp_common.c
>> @@ -46,7 +46,7 @@ void board_final_cleanup(void)
>>         /* call into FspNotify */
>>         debug("Calling into FSP (notify phase INIT_PHASE_BOOT): ");
>>         status = fsp_notify(NULL, INIT_PHASE_BOOT);
>> -       if (status != FSP_SUCCESS)
>> +       if (status)
>>                 debug("fail, error code %x\n", status);
>>         else
>>                 debug("OK\n");
>> --
>
> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>

Applied to u-boot-x86.


More information about the U-Boot mailing list