Skip to content
This repository has been archived by the owner on Sep 18, 2020. It is now read-only.

Commit

Permalink
disk_util: use FAT32 on ESP
Browse files Browse the repository at this point in the history
mkfs.vfat was defaulting to FAT16 based on the size of the partition.
The UEFI spec (2.7 errata A, section 13.3) implies that only FAT32 is
necessarily supported on the ESP, and we've received a report of
hardware that doesn't recognize FAT16.
  • Loading branch information
bgilbert committed Nov 13, 2017
1 parent 3f195c7 commit 7f058d6
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions build_library/disk_util
Original file line number Diff line number Diff line change
Expand Up @@ -423,6 +423,9 @@ def FormatFat(part, device):
cmd = ['mkfs.vfat']
if 'fs_label' in part:
cmd += ['-n', part['fs_label']]
if part['type'] == 'efi':
# ESP is FAT32 irrespective of size
cmd += ['-F', '32']
Sudo(cmd + [device, vfat_blocks], stdout_null=True)


Expand Down

0 comments on commit 7f058d6

Please sign in to comment.