On 08/04/2025 2:23 pm, Marek Marczykowski-Górecki wrote:
> Consider also "Display controller" an IGD, not only "VGA compatible
> controller" in few more places.
>
> Fixes: 4191619e0893 ("libxl: extend IGD check")
> Signed-off-by: Marek Marczykowski-Górecki <marma...@invisiblethingslab.com>
> ---
> Do you prefer this to be split into two patches (libxl, hvmloader)?
Probably not for something this trivial.
>
> tools/firmware/hvmloader/pci.c | 1 +
> tools/libs/light/libxl_pci.c | 3 ++-
> 2 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/tools/firmware/hvmloader/pci.c b/tools/firmware/hvmloader/pci.c
> index c3c61ca060a6..1ee97a5b4b20 100644
> --- a/tools/firmware/hvmloader/pci.c
> +++ b/tools/firmware/hvmloader/pci.c
> @@ -173,6 +173,7 @@ void pci_setup(void)
> switch ( class )
> {
> case 0x0300:
> + case 0x0380:
Now we've got multiple, we should have /* VGA */ and /* Display */
comments like libxl has.
Can fix on commit.
Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>
