[PATCH 13/38] part: Allow setting the partition-table type

[Simon Glass]

Hi Heinrich,

On Fri, 31 Mar 2023 at 13:17, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
>
>
> Am 31. März 2023 01:49:05 MESZ schrieb Simon Glass <sjg at chromium.org>:
> >Hi Heinrich,
> >
> >On Fri, 31 Mar 2023 at 11:33, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
> >>
> >>
> >>
> >> Am 30. März 2023 23:32:02 MESZ schrieb Simon Glass <sjg at chromium.org>:
> >> >Some devices have multiple partition types available on the same media.
> >> >It is sometimes useful to see these to check that everything is working
> >> >correctly.
> >> >
> >> >Provide a way to manually set the partition-table type, avoiding the
> >> >auto-detection process.
> >>
> >> Do you have an example image where we get it wrong?
> >>
> >> Linux does not need that. What is different in our  table type  priorities to Linux?
> >>
> >> I am not yet convinced we need to set this manually.
> >
> >There is an example in the documentation I added. That is using the
> >Ubuntu 22.04 ISO. Can you give it a try?
>
> Linux would mount this as iso9660.
>
> U-Boot lacks a driver for this file system.
>
> There is no need to mount the file as CD-ROM drive. You could mount it as virtio drive instead or as USB. Then U-Boot can read the EFI partition with sector size 512.
>
> Computers nowadays are installed from USB. Most workstation cases don't foresee optical drives anymore.

Yes, the example I have shown is for virtio and you can see the
different filesystems present there.

Regards,
Simon


>
> Best regards
>
> Heinrich
>
> >
> >You can also try it with a CDROM drive, something like:
> >
> >qemu-system-x86_64  -drive format=raw,file=root.img -bios
> >/tmp/b/qemu-x86_64/u-boot.rom -cdrom ubuntu-22.04.2-desktop-amd64.iso
> >-hdb fat:rw:/home/sglass/cosarm/win/seabios -nographic -m 4096
> >
> >Regards,
> >Simon
> >
> >> >
> >> >Signed-off-by: Simon Glass <sjg at chromium.org>
> >> >---
> >> >
> >> > cmd/part.c             | 34 +++++++++++++++++++
> >> > disk/part.c            | 16 +++++++++
> >> > doc/usage/cmd/part.rst | 74 ++++++++++++++++++++++++++++++++++++++++++
> >> > include/part.h         |  9 +++++
> >> > 4 files changed, 133 insertions(+)
> >
> >[..]