On 09/02/2012 10:45 AM, Wolfgang Denk wrote:
Dear Stephen Warren,
In message 1345158942-31512-1-git-send-email-swarren@wwwdotorg.org you wrote:
From: Stephen Warren swarren@nvidia.com
Each EFI partition table entry contains a UUID. Extend U-Boot's struct disk_partition to be able to store this information, and modify get_partition_info_efi() to fill it in.
The implementation of uuid_string() was stolen from the Linux kernel.
Signed-off-by: Stephen Warren swarren@nvidia.com
Can we please make this addition dependent on EFI support being enabled? Otherwise the increased memory footprint will hit all users, even if they never use EFI at all.
(I assume you're talking about the change to the partition info structure, since IIRC all the other code was already in either the EFI partition code already, or a new command under its own ifdef).
I'm also planning a patch that fills in this UUID field for MBR/DOS partition tables too. That'd require the ifdef to check 2 different config symbols, and perhaps more in the future. It seems a little messy to ifdef this just to save 4 bytes. Are you sure you want to do this?