Re: [PATCH V2 2/2] efi/fb: Convert PCI bus address to resource if translated by the bridge

From: Sinan Kaya
Date: Mon Jun 25 2018 - 13:28:23 EST


On 6/25/2018 11:52 AM, okaya@xxxxxxxxxxxxxx wrote:
>> Note that this is not the only MMIO translation related change made by
>> Heyi Guo to the generic PCI host bridge and bus drivers, but given
>> that those did not support MMIO translation at all, I take it your
>> affected platforms will already have their own changes to accommodate
>> this.
>
> Platform has been doing mmio translation for quite a while. Because all accesses go through pci io protocol, the rest of the UEFI never needed to be aware of bar address or do direct access.
>
> This is the first time I hear of direct access. Maybe, GOP is a special case.
>
> I started copying your response to the bios vendor.
>
> They are probably missing that patch. I will pass it along.

I was informed that they fixed the issue in BIOS by presenting CPU physical
address instead of PCI bus address in FrameBufferBase of the AST GOP UEFI driver.

We can drop this patch now.

--
Sinan Kaya
Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.