From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sinan Kaya Subject: Re: [PATCH V2 2/2] efi/fb: Convert PCI bus address to resource if translated by the bridge Date: Mon, 25 Jun 2018 13:31:29 -0400 Message-ID: References: <1526653072-7153-1-git-send-email-okaya@codeaurora.org> <1526653072-7153-2-git-send-email-okaya@codeaurora.org> <20180619222921.GA90490@bhelgaas-glaptop.roam.corp.google.com> <2a805337-c0b5-e134-7695-5a543ecaa26a@codeaurora.org> <37289a27-eb99-6a73-4d32-4a75edd11dcd@codeaurora.org> <7777f7bfead902f2f5175d48907fccec@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Ard Biesheuvel Cc: Bjorn Helgaas , "open list:EFIFB FRAMEBUFFER DRIVER" , Bartlomiej Zolnierkiewicz , linux-arm-msm@vger.kernel.org, Timur Tabi , open list , "open list:FRAMEBUFFER LAYER" , Peter Jones , linux-arm-kernel List-Id: linux-arm-msm@vger.kernel.org On 6/25/2018 1:29 PM, Ard Biesheuvel wrote: >>> 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. >> > Excellent! Thanks for following up with the vendor ... > Sure, appreciate the help. -- 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.