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

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

Philippe Mathieu-Daudé posted 2 patches 1 year, 3 months ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230118094023.49517-1-philmd@linaro.org

Maintainers: "Philippe Mathieu-Daudé" <philmd@linaro.org>, Aurelien Jarno <aurelien@aurel32.net>, Cleber Rosa <crosa@redhat.com>, Wainer dos Santos Moschetta <wainersm@redhat.com>, Beraldo Leal <bleal@redhat.com>

There is a newer version of this series

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

Expand all Fold all

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

Posted by Philippe Mathieu-Daudé 1 year, 3 months ago

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(-)

-- 
2.38.1

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

Posted by Philippe Mathieu-Daudé 1 year, 3 months ago

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 :\

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

Posted by Klaus Jensen 1 year, 3 months ago

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!