Re: [PATCH v4] PCI/VGA: Make the vga_is_firmware_default() less arch-dependent

[suijingfeng]

Hi,


On 2023/8/18 06:08, Bjorn Helgaas wrote:
> > Please note that before apply this patch, vgaarb can not select the
> > right boot vga due to weird logic introduced with the commit
> > 57fc7323a8e7c ("LoongArch: Add PCI controller support")
> If we need this reference to 57fc7323a8e7c, we need more specifics
> about what the "weird logic" is.  pci_fixup_vgadev() is the only
> obvious VGA connection, so I suppose it's related to that.

Sorry, re-reading your reviews,
I realized that I may miss the point,  here, the sentence is not well phased,
Correct expression should be:


Please note that before apply this patch, there don't have a reasonable, system level solution on loongarch.
VGAARB select a wrong device as primary GPU due to the pci_fixup_vgadev() function
introduced with the commit 57fc7323a8e7c ("LoongArch: Add PCI controller support").