Hello,
I am reworking the patch. Some comments below.
Mit freundlichen Grüßen / Best regards
Thomas Mittelstaedt
Cross-Domain Computing Solutions, Engineering BBM Vehicle Computer 3
(XC-CT/EBV3)
Robert Bosch GmbH | Postfach 10 02 61 | 31102 Hildesheim | GERMANY |
www.bosch.com
Tel. +49 5121 49-2965 | Mobil +49 160 90418883 | Telefax +49 711 811-5052965 |
thomas.mittelsta...@de.bosch.com
Sitz: Stuttgart, Registergericht: Amtsgericht Stuttgart, HRB 14000;
Aufsichtsratsvorsitzender: Prof. Dr. Stefan Asenkerschbaumer; Geschäftsführung:
Dr. Stefan Hartung,
Dr. Christian Fischer, Dr. Markus Forschner, Stefan Grosch, Dr. Markus Heyn,
Dr. Tanja Rückert
> -----Ursprüngliche Nachricht-----
> Von: Heinrich Schuchardt <xypron.g...@gmx.de>
> Gesendet: Freitag, 18. August 2023 10:34
> An: Mittelstaedt Thomas (XC-CT/EBV3) <thomas.mittelsta...@de.bosch.com>
> Cc: Simon Glass <s...@chromium.org>; Ilias Apalodimas
> <ilias.apalodi...@linaro.org>; u-boot@lists.denx.de
> Betreff: Re: [PATCH 1/1] Documentation extended with specific information for
> VirtualBox
>
> On 17.08.23 15:46, thomas.mittelsta...@de.bosch.com wrote:
> > From: Thomas Mittelstaedt <thomas.mittelsta...@de.bosch.com>
> >
> > Signed-off-by: Thomas Mittelstaedt <thomas.mittelsta...@de.bosch.com>
> > ---
> > doc/develop/uefi/u-boot_on_efi.rst | 76 ++++++++++++++++++++++++++++++
> > 1 file changed, 76 insertions(+)
> >
> > diff --git a/doc/develop/uefi/u-boot_on_efi.rst
> > b/doc/develop/uefi/u-boot_on_efi.rst
> > index acad6397e8..927e1dc266 100644
> > --- a/doc/develop/uefi/u-boot_on_efi.rst
> > +++ b/doc/develop/uefi/u-boot_on_efi.rst
> > @@ -254,6 +254,82 @@ This shows running with serial enabled (see
> `include/configs/efi-x86_app.h`)::
> >
> > => QEMU: Terminated
> >
> > +Run at VirtualBox (x86)
>
> %s/at/on/
>
OK. Will do.
> Below you assume a 64bit machine: %s/x86/x86_64/
OK. Will do.
>
> > +--------------------------
>
> The line length must match the heading.
>
> > +
> > +Enable EFI
> > +~~~~~~~~~~~~
>
> ditto
OK. Will do.
>
> > +At settings for virtual machine the flag at **System->Motherboard->Enable
> > EFI
> (special OSes only)** has to be enabled.
>
> Limit lines to 80 characters.
>
OK. Will do.
> > +
> > +Installation
> > +~~~~~~~~~~~~
> > +Install the needed system at a VDI disk and connect this to SATA
> > +controller (type AHCI) at settings
>
> Provide the preinstalled Linux system as a Virtual Disk Image (VDI) and
> assign it
> to a SATA controller (type AHCI) using the settings for the virtual machine at
> menu item **System->Storage->Controller:SATA**.
>
> > +for virtual machine at **System->Storage->Controller:SATA**.
> > +
> > +For the following description 3 partitions are assumed:
>
> three GPT partitions
>
OK. Will do.
> > +
> > +- Partition 1: formatted as **vfat**, used for U-Boot and and its
> > +environment filesenvironment, flag **boot** set
>
> formatted as FAT file-system and marked as EFI system partition (partition
> type
> 0xEF00) used for the U-Boot EFI binary.
>
> (If VirtualBox is UEFI compliant, it should recognize the ESP as the boot
> partition.)
>
> > +- Partition 2: formatted as **ext2/ext4**, used for Linux image and
> > +boot configuration, flag **boot** set
>
> ext2 or ext4
>
>
OK. Will do.
> > +- Partition 3: formatted as **ext4**, used for root file system
> > +
> > +Create an extlinux.conf or a boot script
> > +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > +
> > +Following files are assumed to be located at system for boot
> > configuration::
> > +
> > + Partition File Comment
> > + 1 EFI/BOOT/BOOTX64.efi # renamed U-Boot EFI image
>
> EFI/BOOT/BOOTX64.EFI
>
> Please, tell the reader which defconfig to use and which U-Boot file to copy.
>
OK. Will add some description.
> > + 2 Image # Linux image
>
> Please, remove the blank space at the end of line.
>
> > + 2 Initrd # Initramfs of Linux
> > +
> > +Boot script
> > +~~~~~~~~~~~
> > +
> > +The boot script **boot.scr** is assumed to be located at::
> > +
> > + Partition File Comment
> > + 2 boot.scr # Boot script, generated with mkimage from template
>
> Please, remove the blank space at the end of line.
>
OK. Will do.
> > +
> > +Content of **boot.scr**:
> > +
> > +.. code-block:: bash
> > +
> > + ext4load ${devtype} ${devnum}:${distro_bootpart} ${kernel_addr_r}
> > + ${prefix}Image setenv kernel_size ${filesize} ext4load ${devtype}
> > + ${devnum}:${distro_bootpart} ${ramdisk_addr_r} ${prefix}Initrd
> > + setenv initrd_size ${filesize} zboot ${kernel_addr_r}
> > + ${kernel_size} ${ramdisk_addr_r} ${initrd_size}
>
> Doesn't U-Boot need this file on the boot partition i.e. the ESP?
>
I'll change to a setup of 2 partitions, because I'm always separate board
specific partitions from generic boot partitions. At our Debian images we
always have a separation boot partition with boot scripts, exlinux.conf, Linux
images, device trees etc. But for this description I don't need it.
> Best regards
>
> Heinrich
>
> > +
> > +Extlinux configuration
> > +~~~~~~~~~~~~~~~~~~~~~~
> > +
> > +Alternatively a configuration **extlinux.conf** can be used.
> > **extlinux.conf** is
> assumed to be located at::
> > +
> > + Partition File Comment
> > + 2 extlinux/extlinux.conf # Extlinux boot configuration
> > +
> > +Content of **extlinux.conf**:
> > +
> > +.. code-block:: bash
> > +
> > + default l0
> > + menu title U-Boot menu
> > + prompt 0
> > + timeout 50
> > +
> > + label l0
> > + menu label Linux
> > + linux /Image
> > + initrd /Initrd
> > +
> > +
> > +Additionally something like (sda is assumed as disk device):
> > +
> > +.. code-block:: bash
> > +
> > + append root=/dev/sda3 console=tty0 console=ttyS0,115200n8 rootwait
> > +rw
> > +
> > +
> >
> > Future work
> > -----------
