From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bin Meng <bmeng.cn@gmail.com>
Date: Tue, 30 Aug 2016 09:29:03 +0800
Subject: [U-Boot] [PATCH v2 2/3] x86: efi: Fix EFI 64-bit payload build
	warnings
In-Reply-To: <CAPnjgZ3f2RLoKLd4jpA-KVNfPo9yWSh2_RHXvGfSMFgbjJ26BA@mail.gmail.com>
References: <1472114839-6792-1-git-send-email-bmeng.cn@gmail.com>
 <1472114839-6792-2-git-send-email-bmeng.cn@gmail.com>
 <CAPnjgZ3f2RLoKLd4jpA-KVNfPo9yWSh2_RHXvGfSMFgbjJ26BA@mail.gmail.com>
Message-ID: <CAEUhbmWmL=HHVTVNMhF4LjUJoF=qPrKoHRyrC8GYhu7g_PtQsg@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Tue, Aug 30, 2016 at 3:38 AM, Simon Glass <sjg@chromium.org> wrote:
> On 25 August 2016 at 02:47, Bin Meng <bmeng.cn@gmail.com> wrote:
>> There are lots of warnings when building EFI 64-bit payload.
>>
>> include/asm-generic/bitops/__fls.h:17:2:
>>   warning: left shift count >= width of type
>>         if (!(word & (~0ul << 32))) {
>>                         ^
>>
>> In fact, U-Boot itself as EFI payload is running in 32-bit mode.
>> So BITS_PER_LONG needs to still be 32, but EFI status codes are
>> 64-bit when booting from 64-bit EFI. Introduce EFI_BITS_PER_LONG
>> to bridge those status codes with U-Boot's BITS_PER_LONG.
>>
>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>>
>> ---
>>
>> Changes in v2:
>> - Rework the patch to fix 64-bit payload boot issue
>>
>>  arch/x86/include/asm/types.h |  4 ----
>>  include/efi.h                | 32 ++++++++++++++++++++------------
>>  2 files changed, 20 insertions(+), 16 deletions(-)
>
> Reviewed-by: Simon Glass <sjg@chromium.org>

applied to u-boot-x86, thanks!