[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 10/10] hw/mips/jazz: specify correct endian for dp8393x de
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 10/10] hw/mips/jazz: specify correct endian for dp8393x device |
Date: |
Fri, 25 Jun 2021 10:51:17 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 |
On 6/25/21 8:54 AM, Mark Cave-Ayland wrote:
> The MIPS magnum machines are available in both big endian (mips64) and little
> endian (mips64el) configurations. Ensure that the dp893x big_endian property
> is set accordingly using logic similar to that used for the MIPS malta
> machines.
>
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> ---
> hw/mips/jazz.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/hw/mips/jazz.c b/hw/mips/jazz.c
> index 89ca8bb910..ee1789183e 100644
> --- a/hw/mips/jazz.c
> +++ b/hw/mips/jazz.c
> @@ -126,7 +126,7 @@ static void mips_jazz_init(MachineState *machine,
> {
> MemoryRegion *address_space = get_system_memory();
> char *filename;
> - int bios_size, n;
> + int bios_size, n, big_endian;
Why not use a boolean directly?
> Clock *cpuclk;
> MIPSCPU *cpu;
> MIPSCPUClass *mcc;
> @@ -158,6 +158,12 @@ static void mips_jazz_init(MachineState *machine,
> [JAZZ_PICA61] = {33333333, 4},
> };
>
> +#ifdef TARGET_WORDS_BIGENDIAN
> + big_endian = 1;
> +#else
> + big_endian = 0;
> +#endif
> +
> if (machine->ram_size > 256 * MiB) {
> error_report("RAM size more than 256Mb is not supported");
> exit(EXIT_FAILURE);
> @@ -290,6 +296,7 @@ static void mips_jazz_init(MachineState *machine,
> dev = qdev_new("dp8393x");
> qdev_set_nic_properties(dev, nd);
> qdev_prop_set_uint8(dev, "it_shift", 2);
> + qdev_prop_set_bit(dev, "big_endian", big_endian > 0);
> object_property_set_link(OBJECT(dev), "dma_mr",
> OBJECT(rc4030_dma_mr), &error_abort);
> sysbus = SYS_BUS_DEVICE(dev);
>
- Re: [PATCH v2 02/10] dp8393x: convert to trace-events, (continued)
- [PATCH v2 04/10] hw/m68k/q800: move PROM and checksum calculation from dp8393x device to board, Mark Cave-Ayland, 2021/06/25
- [PATCH v2 06/10] qemu/bitops.h: add bitrev8 implementation, Mark Cave-Ayland, 2021/06/25
- [PATCH v2 01/10] dp8393x: checkpatch fixes, Mark Cave-Ayland, 2021/06/25
- [PATCH v2 05/10] dp8393x: remove onboard PROM containing MAC address and checksum, Mark Cave-Ayland, 2021/06/25
- [PATCH v2 09/10] dp8393x: fix CAM descriptor entry index, Mark Cave-Ayland, 2021/06/25
- [PATCH v2 07/10] hw/m68k/q800: fix PROM checksum and MAC address storage, Mark Cave-Ayland, 2021/06/25
- [PATCH v2 08/10] dp8393x: don't force 32-bit register access, Mark Cave-Ayland, 2021/06/25
- [PATCH v2 10/10] hw/mips/jazz: specify correct endian for dp8393x device, Mark Cave-Ayland, 2021/06/25
- Re: [PATCH v2 10/10] hw/mips/jazz: specify correct endian for dp8393x device,
Philippe Mathieu-Daudé <=
- Re: [PATCH v2 00/10] dp8393x: fixes for MacOS toolbox ROM, Finn Thain, 2021/06/26