[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 08/16] hw/ssi/xilinx_spi: Make device endianness configurable
From: |
Philippe Mathieu-Daudé |
Subject: |
[PATCH v2 08/16] hw/ssi/xilinx_spi: Make device endianness configurable |
Date: |
Thu, 7 Nov 2024 01:22:14 +0000 |
Replace the DEVICE_NATIVE_ENDIAN MemoryRegionOps by a pair
of DEVICE_LITTLE_ENDIAN / DEVICE_BIG_ENDIAN.
Add the "little-endian" property to select the device
endianness, defaulting to little endian.
Set the proper endianness on the single machine using the
device.
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/arm/xlnx-zynqmp.c | 4 ++++
hw/ssi/xilinx_spi.c | 29 +++++++++++++++++++++--------
2 files changed, 25 insertions(+), 8 deletions(-)
diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c
index ab2d50e31b..e735dbdf82 100644
--- a/hw/arm/xlnx-zynqmp.c
+++ b/hw/arm/xlnx-zynqmp.c
@@ -714,6 +714,10 @@ static void xlnx_zynqmp_realize(DeviceState *dev, Error
**errp)
for (i = 0; i < XLNX_ZYNQMP_NUM_SPIS; i++) {
gchar *bus_name;
+ if (!object_property_set_bool(OBJECT(&s->spi[i])), "little-endian",
+ true, errp)) {
+ return;
+ }
if (!sysbus_realize(SYS_BUS_DEVICE(&s->spi[i]), errp)) {
return;
}
diff --git a/hw/ssi/xilinx_spi.c b/hw/ssi/xilinx_spi.c
index 7f1e1808c5..2a0c9bca05 100644
--- a/hw/ssi/xilinx_spi.c
+++ b/hw/ssi/xilinx_spi.c
@@ -83,6 +83,7 @@ OBJECT_DECLARE_SIMPLE_TYPE(XilinxSPI, XILINX_SPI)
struct XilinxSPI {
SysBusDevice parent_obj;
+ bool little_endian_model;
MemoryRegion mmio;
qemu_irq irq;
@@ -313,13 +314,23 @@ done:
xlx_spi_update_irq(s);
}
-static const MemoryRegionOps spi_ops = {
- .read = spi_read,
- .write = spi_write,
- .endianness = DEVICE_NATIVE_ENDIAN,
- .valid = {
- .min_access_size = 4,
- .max_access_size = 4
+static const MemoryRegionOps spi_ops[2] = {
+ {
+ .read = spi_read,
+ .write = spi_write,
+ .endianness = DEVICE_BIG_ENDIAN,
+ .valid = {
+ .min_access_size = 4,
+ .max_access_size = 4,
+ },
+ }, {
+ .read = spi_read,
+ .write = spi_write,
+ .endianness = DEVICE_LITTLE_ENDIAN,
+ .valid = {
+ .min_access_size = 4,
+ .max_access_size = 4,
+ },
}
};
@@ -339,7 +350,8 @@ static void xilinx_spi_realize(DeviceState *dev, Error
**errp)
sysbus_init_irq(sbd, &s->cs_lines[i]);
}
- memory_region_init_io(&s->mmio, OBJECT(s), &spi_ops, s,
+ memory_region_init_io(&s->mmio, OBJECT(s),
+ &spi_ops[s->little_endian_model], s,
"xilinx-spi", R_MAX * 4);
sysbus_init_mmio(sbd, &s->mmio);
@@ -362,6 +374,7 @@ static const VMStateDescription vmstate_xilinx_spi = {
};
static Property xilinx_spi_properties[] = {
+ DEFINE_PROP_BOOL("little-endian", XilinxSPI, little_endian_model, true),
DEFINE_PROP_UINT8("num-ss-bits", XilinxSPI, num_cs, 1),
DEFINE_PROP_END_OF_LIST(),
};
--
2.45.2
- [PATCH v2 01/16] hw/microblaze: Restrict MemoryRegionOps are implemented as 32-bit, (continued)
- [PATCH v2 01/16] hw/microblaze: Restrict MemoryRegionOps are implemented as 32-bit, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 02/16] hw/microblaze: Propagate CPU endianness to microblaze_load_kernel(), Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 03/16] hw/intc/xilinx_intc: Make device endianness configurable, Philippe Mathieu-Daudé, 2024/11/06
- [RFC PATCH v2 04/16] hw/net/xilinx_ethlite: Simplify by having configurable endianness, Philippe Mathieu-Daudé, 2024/11/06
- [RFC PATCH v2 05/16] hw/timer/xilinx_timer: Allow down to 8-bit memory access, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 06/16] hw/timer/xilinx_timer: Make device endianness configurable, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 07/16] hw/char/xilinx_uartlite: Make device endianness configurable, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 08/16] hw/ssi/xilinx_spi: Make device endianness configurable,
Philippe Mathieu-Daudé <=
- [PATCH v2 09/16] hw/ssi/xilinx_spips: Make device endianness configurable, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 10/16] target/microblaze: Explode MO_TExx -> MO_TE | MO_xx, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 11/16] target/microblaze: Set MO_TE once in do_load() / do_store(), Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 12/16] target/microblaze: Introduce mo_endian() helper, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 13/16] target/microblaze: Consider endianness while translating code, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 14/16] hw/microblaze: Support various endianness for s3adsp1800 machines, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 15/16] tests/functional: Explicit endianness of microblaze assets, Philippe Mathieu-Daudé, 2024/11/06
- [PATCH v2 16/16] tests/functional: Add microblaze cross-endianness tests, Philippe Mathieu-Daudé, 2024/11/06