[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device |
Date: |
Mon, 12 Jan 2015 13:57:42 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 11/01/2015 04:57, address@hidden wrote:
> +static const MemoryRegionOps rocker_mmio_ops = {
> + .read = rocker_mmio_read,
> + .write = rocker_mmio_write,
> + .endianness = DEVICE_LITTLE_ENDIAN,
> + .valid = {
> + .min_access_size = 4,
> + .max_access_size = 8,
> + },
> + .impl = {
> + .min_access_size = 4,
> + .max_access_size = 8,
> + },
> +};
I suggest that you only use 32-bit registers in the internal
implementation, where writing to the low part of a 64-bit register only
writes to a latch. You can then use .impl.max_access_size == 4 but keep
.valid.max_access_size == 8.
QEMU will then take care of passing 64-bit writes down as two 32-bit
writes, in increasing address.
Paolo
- Re: [Qemu-devel] [PATCH v3 3/9] rocker: add register programming guide, (continued)
- [Qemu-devel] [PATCH v3 1/9] net: add MAC address string printer, sfeldma, 2015/01/10
- [Qemu-devel] [PATCH v3 4/9] pci: add rocker device ID, sfeldma, 2015/01/10
- [Qemu-devel] [PATCH v3 8/9] rocker: add tests, sfeldma, 2015/01/10
- [Qemu-devel] [PATCH v3 9/9] MAINTAINERS: add rocker, sfeldma, 2015/01/10
- [Qemu-devel] [PATCH v3 7/9] qmp: add rocker device support, sfeldma, 2015/01/10
- [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device, sfeldma, 2015/01/10
- Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device, Jason Wang, 2015/01/16