On 9/12/19 6:51 AM, AKASHI Takahiro wrote:
> Currently, a whole disk without any partitions is not associated
> with EFI_SIMPLE_FILE_SYSTEM_PROTOCOL. So even if it houses FAT
> file system, there is a chance that we may not be able to access
> it, particularly, when accesses are to be attempted after searching
> that protocol against a device handle.
>
> With this patch, EFI_SIMPLE_FILE_SYSTEM_PROTOCOL is installed
> to such a disk if part_get_info() shows there is not partition
> table installed on it.
>
> Signed-off-by: AKASHI Takahiro <takahiro.aka...@linaro.org>
> ---
> lib/efi_loader/efi_disk.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/lib/efi_loader/efi_disk.c b/lib/efi_loader/efi_disk.c
> index d72f455901f2..d36f22cedc52 100644
> --- a/lib/efi_loader/efi_disk.c
> +++ b/lib/efi_loader/efi_disk.c
> @@ -253,6 +253,7 @@ static efi_status_t efi_disk_add_dev(
> struct efi_disk_obj **disk)
> {
> struct efi_disk_obj *diskobj;
> + disk_partition_t info;
> efi_status_t ret;
>
> /* Don't add empty devices */
> @@ -284,7 +285,9 @@ static efi_status_t efi_disk_add_dev(
> diskobj->dp);
> if (ret != EFI_SUCCESS)
> return ret;
> - if (part >= 1 && efi_fs_exists(desc, part)) {
> + /* partitions or whole disk without partitions */
> + if ((part >= 1 || part_get_info(desc, part, &info)) &&
part_get_info() returns -1 for part = 0 on a DOS partioned disk. So this
check does not work.
Best regards
Heinrich
> + efi_fs_exists(desc, part)) {
> diskobj->volume = efi_simple_file_system(desc, part,
> diskobj->dp);
> ret = efi_add_protocol(&diskobj->header,
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
