Re: [PATCH] efi/libstub: arm*: double cast memory_map to hide compiler warning

[Ard Biesheuvel <ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]

On 3 January 2017 at 21:11, Jeffrey Hugo <jhugo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> wrote:
> On 12/20/2016 7:07 AM, Matt Fleming wrote:
>>
>> On Tue, 20 Dec, at 01:52:07PM, James Morse wrote:
>>>
>>> "efi/libstub: arm*: Pass latest memory map to the kernel" added a new
>>> call to update fdt in-place. This caused a new warning on 32bit ARM:
>>>
>>> ../drivers/firmware/efi/libstub/fdt.c: In function
>>> 'allocate_new_fdt_and_exit_boot':
>>> ../drivers/firmware/efi/libstub/fdt.c:303:9: warning: cast from pointer
>>> to integer of different size [-Wpointer-to-int-cast]
>>>          (u64)memory_map, (u32)map_size);
>>>          ^
>>>
>>> Cast to an integer of native pointer size first.
>>>
>>> Reported-by: kbuild test robot <fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
>>> Signed-off-by: James Morse <james.morse-5wv7dgnIgG8@public.gmane.org>
>>> ---
>>> I guess this too needs to go to stable unless you intend to squash it?
>>>
>>>  drivers/firmware/efi/libstub/fdt.c | 3 ++-
>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>>
>> Thanks James, I've folded this into your previous patch.
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-efi" in
>> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
>
> Thanks James.  I'm happy to see this got cleaned up and sorted out while I
> was on vacation.  I see Ard submitted a pull request already, but for what
> its worth, the latest version on the EFI urgent branch tested ok on the
> platforms I have available.
>

Thanks for confirming

Cheers,
Ard.