From patchwork Fri Oct 5 19:29:47 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [U-Boot, 2/9] disk: part_efi: re-order partition list printf, change case X-Patchwork-Submitter: Stephen Warren X-Patchwork-Id: 189570 X-Patchwork-Delegate: trini@ti.com Message-Id: <1349465394-3157-2-git-send-email-swarren@wwwdotorg.org> To: Tom Rini Cc: u-boot@lists.denx.de, Stephen Warren , Rob Herring Date: Fri, 5 Oct 2012 13:29:47 -0600 From: Stephen Warren List-Id: U-Boot discussion From: Stephen Warren The partition name is a long variable-length string. Move it last on the line to ensure consistent layout and that the entries align with the "header" line. Also, surround it in quotes, so if it's empty, it's obvious that something is still being printed. Also, change the case of the LBA numbers; lower-case looks nicer in my opinion, and will be more consistent with the UUID printing that is added later in this series. Signed-off-by: Stephen Warren --- disk/part_efi.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/disk/part_efi.c b/disk/part_efi.c index 008177e..b6b2bf5 100644 --- a/disk/part_efi.c +++ b/disk/part_efi.c @@ -136,16 +136,16 @@ void print_part_efi(block_dev_desc_t * dev_desc) debug("%s: gpt-entry at %p\n", __func__, gpt_pte); - printf("Part\tName\t\t\tStart LBA\tEnd LBA\n"); + printf("Part\tStart LBA\tEnd LBA\t\tName\n"); for (i = 0; i < le32_to_int(gpt_head->num_partition_entries); i++) { /* Stop at the first non valid PTE */ if (!is_pte_valid(&gpt_pte[i])) break; - printf("%3d\t%-18s\t0x%08llX\t0x%08llX\n", (i + 1), - print_efiname(&gpt_pte[i]), + printf("%3d\t0x%08llx\t0x%08llx\t\"%s\"\n", (i + 1), le64_to_int(gpt_pte[i].starting_lba), - le64_to_int(gpt_pte[i].ending_lba)); + le64_to_int(gpt_pte[i].ending_lba), + print_efiname(&gpt_pte[i])); } /* Remember to free pte */