Re: [PATCH 2/2] seabios: remove PCI drivers from bios.bin

[Volker Rümelin]

Am 21.09.23 um 14:10 schrieb Paolo Bonzini:
> bios.bin is now used only by ISA PC, so PCI drivers are not necessary.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  pc-bios/bios.bin         | Bin 131072 -> 131072 bytes
>  roms/config.seabios-128k |  30 ++++++++++++++++++++++--------
>  2 files changed, 22 insertions(+), 8 deletions(-)
> diff --git a/roms/config.seabios-128k b/roms/config.seabios-128k
> index d18c802c46e..06f4ba35bbe 100644
> --- a/roms/config.seabios-128k
> +++ b/roms/config.seabios-128k
> @@ -1,21 +1,35 @@
> -# for qemu machine types 1.7 + older
> -# need to turn off features (xhci,uas) to make it fit into 128k
> +# SeaBIOS Configuration for -M isapc
> +
> +#
> +# General Features
> +#
>  CONFIG_QEMU=y
>  CONFIG_ROM_SIZE=128
>  CONFIG_ATA_DMA=n
>  CONFIG_BOOTSPLASH=n
>  CONFIG_XEN=n
> -CONFIG_USB_OHCI=n
> -CONFIG_USB_XHCI=n
> -CONFIG_USB_UAS=n
> +CONFIG_ATA_PIO32=n
> +CONFIG_AHCI=n
>  CONFIG_SDCARD=n
>  CONFIG_TCGBIOS=n
> -CONFIG_MPT_SCSI=n
> -CONFIG_ESP_SCSI=n
> -CONFIG_MEGASAS=n
> +CONFIG_VIRTIO_BLK=n
> +CONFIG_VIRTIO_SCSI=n
>  CONFIG_PVSCSI=n
> +CONFIG_ESP_SCSI=n
> +CONFIG_LSI_SCSI=n
> +CONFIG_MEGASAS=n
> +CONFIG_MPT_SCSI=n
>  CONFIG_NVME=n
>  CONFIG_USE_SMM=n
>  CONFIG_VGAHOOKS=n
>  CONFIG_HOST_BIOS_GEOMETRY=n
> +CONFIG_PS2PORT=n

Hi Paolo,

CONFIG_PS2PORT shouldn't be disabled. Otherwise FreeDOS and other
programs don't have a working keyboard.

With best regards,
Volker

> +CONFIG_USB=n
> +CONFIG_PMTIMER=n
> +CONFIG_PCIBIOS=n
> +CONFIG_DISABLE_A20=n
> +CONFIG_WRITABLE_UPPERMEMORY=n
> +CONFIG_ACPI=n
>  CONFIG_ACPI_PARSE=n
> +CONFIG_DEBUG_SERIAL=n
> +CONFIG_DEBUG_SERIAL_MMIO=n