mbox series

[0/2] hw/pci-host/gt64120: Fix regression on big-endian targets

Message ID 20230118094023.49517-1-philmd@linaro.org
Headers show
Series hw/pci-host/gt64120: Fix regression on big-endian targets | expand

Message

Philippe Mathieu-Daudé Jan. 18, 2023, 9:40 a.m. UTC 
Klauss reported a regression on big-endian targets, introduced
by commit 145e2198d749 ("hw/mips/gt64xxx_pci: Endian-swap using
PCI_HOST_BRIDGE MemoryRegionOps"). Fix it and add the Klauss'
reproducer as Avocado test.

Philippe Mathieu-Daudé (2):
  hw/pci-host/gt64120: Fix PCI I/O config register endianness
  tests/avocado: Add test accessing NVMe on big-endian MIPS target

 hw/pci-host/gt64120.c               | 25 +++++-----------
 tests/avocado/boot_linux_console.py | 44 +++++++++++++++++++++++++++++
 2 files changed, 51 insertions(+), 18 deletions(-)

Comments

Philippe Mathieu-Daudé Jan. 18, 2023, 9:43 a.m. UTC | #1 
On 18/1/23 10:40, Philippe Mathieu-Daudé wrote:
> Klauss reported a regression on big-endian targets, introduced
> by commit 145e2198d749 ("hw/mips/gt64xxx_pci: Endian-swap using
> PCI_HOST_BRIDGE MemoryRegionOps"). Fix it and add the Klauss'
> reproducer as Avocado test.

Sorry Klaus for adding an extra 's' in your name :\
Klaus Jensen Jan. 18, 2023, 9:52 a.m. UTC | #2 
On Jan 18 10:43, Philippe Mathieu-Daudé wrote:
> On 18/1/23 10:40, Philippe Mathieu-Daudé wrote:
> > Klauss reported a regression on big-endian targets, introduced
> > by commit 145e2198d749 ("hw/mips/gt64xxx_pci: Endian-swap using
> > PCI_HOST_BRIDGE MemoryRegionOps"). Fix it and add the Klauss'
> > reproducer as Avocado test.
> 
> Sorry Klaus for adding an extra 's' in your name :\

Don't worry about it ;) Thanks for the quick fix!