Hi Bin,
On 16 February 2016 at 03:09, Bin Meng bmeng.cn@gmail.com wrote:
Hi Simon,
On Mon, Feb 15, 2016 at 10:16 AM, Simon Glass sjg@chromium.org wrote:
Adjust the cast to avoid a warning when stdint.h is used.
Signed-off-by: Simon Glass sjg@chromium.org
disk/part_efi.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/disk/part_efi.c b/disk/part_efi.c index db5e7ed..7bd840f 100644 --- a/disk/part_efi.c +++ b/disk/part_efi.c @@ -658,11 +658,13 @@ int gpt_verify_partitions(struct blk_desc *dev_desc, gpt_part_size = le64_to_cpu(gpt_e[i].ending_lba) - le64_to_cpu(gpt_e[i].starting_lba) + 1; debug("size(LBA) - GPT: %8llu, ENV: %8llu ",
gpt_part_size, (u64) partitions[i].size);
(unsigned long long)gpt_part_size,(unsigned long long)partitions[i].size);Can you elaborate why this is needed?
When using stdint.h (CONFIG_USE_STDINT) I get this warning with sandbox. I don't know if it is my compiler or there is a better way to make stdint work...
if (le64_to_cpu(gpt_part_size) != partitions[i].size) { error("Partition %s size: %llu does not match %llu!\n",
efi_str, gpt_part_size, (u64) partitions[i].size);
efi_str, (unsigned long long)gpt_part_size,(unsigned long long)partitions[i].size); return -1; }@@ -678,12 +680,12 @@ int gpt_verify_partitions(struct blk_desc *dev_desc, /* Check if GPT and ENV start LBAs match */ debug("start LBA - GPT: %8llu, ENV: %8llu\n", le64_to_cpu(gpt_e[i].starting_lba),
(u64) partitions[i].start);
(unsigned long long)partitions[i].start); if (le64_to_cpu(gpt_e[i].starting_lba) != partitions[i].start) { error("Partition %s start: %llu does not match %llu!\n", efi_str, le64_to_cpu(gpt_e[i].starting_lba),
(u64) partitions[i].start);
(unsigned long long)partitions[i].start); return -1; } }--
Regards, Simon