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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx>
Signed-off-by: James Morse <james.morse@xxxxxxx>
---
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@xxxxxxxxxxxxxxx
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.

--
Jeffrey Hugo
Qualcomm Datacenter Technologies as an affiliate of Qualcomm Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.
--
To unsubscribe from this list: send the line "unsubscribe linux-efi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux