Re: [edk2-devel] [PATCH v2] MdeModulePkg/PciHostBridgeDxe: Add readback after final Cfg-Write

2023-11-13 Thread Laszlo Ersek
On 11/8/23 02:06, Michael D Kinney wrote: > Hi Jose, > >   > > 1. This logic needs to move into an AARCH64 specific directory/file.  > Other architectures handle this in other ways. > 2. There are many places throughout edk2 sources that perform PCI > config write operations.  You are

Re: [edk2-devel] [PATCH v2] MdeModulePkg/PciHostBridgeDxe: Add readback after final Cfg-Write

2023-11-07 Thread Michael D Kinney
Hi Jose, 1. This logic needs to move into an AARCH64 specific directory/file. Other architectures handle this in other ways. 2. There are many places throughout edk2 sources that perform PCI config write operations. You are only fixing it in a single location. You may want to look at

Re: [edk2-devel] [PATCH v2] MdeModulePkg/PciHostBridgeDxe: Add readback after final Cfg-Write

2023-11-07 Thread Joe L
++ Laszlo and Michael On Tue, Nov 7, 2023 at 10:54 AM Jose Lopez wrote: > ++ CC'd > > > On Mon, Nov 6, 2023 at 6:02 PM Joe Lopez wrote: > >> From: joelopez333 >> >> REF:https://edk2.groups.io/g/devel/topic/102310377#110456 >> >> Problem Report: >> >> On AARCH64, there is no

Re: [edk2-devel] [PATCH v2] MdeModulePkg/PciHostBridgeDxe: Add readback after final Cfg-Write

2023-11-07 Thread Joe L
++ CC'd On Mon, Nov 6, 2023 at 6:02 PM Joe Lopez wrote: > From: joelopez333 > > REF:https://edk2.groups.io/g/devel/topic/102310377#110456 > > Problem Report: > > On AARCH64, there is no ordering guarantee between configuration > space (ECAM) writes and memory space reads

[edk2-devel] [PATCH v2] MdeModulePkg/PciHostBridgeDxe: Add readback after final Cfg-Write

2023-11-06 Thread Joe L
From: joelopez333 REF:https://edk2.groups.io/g/devel/topic/102310377#110456 Problem Report: On AARCH64, there is no ordering guarantee between configuration space (ECAM) writes and memory space reads (MMIO). ARM AMBA CHI only guarantees ordering for reads and writes within