[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH 1/4] uninorth: add impl min_access_size and max_ac
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH 1/4] uninorth: add impl min_access_size and max_access_size to unin_ops |
Date: |
Mon, 4 Jun 2018 10:39:25 +1000 |
User-agent: |
Mutt/1.9.5 (2018-04-13) |
On Sun, May 06, 2018 at 03:20:02PM +0100, Mark Cave-Ayland wrote:
> >From testing all my local images the uninorth registers are only ever
> read or written with 32-bit accesses.
If that's the case, shouldn't you be setting the range of valid
accesses, rather than the range of implemented accesses?
>
> Signed-off-by: Mark Cave-Ayland <address@hidden>
> ---
> hw/pci-host/uninorth.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/hw/pci-host/uninorth.c b/hw/pci-host/uninorth.c
> index ba76b84dbc..a658f9230a 100644
> --- a/hw/pci-host/uninorth.c
> +++ b/hw/pci-host/uninorth.c
> @@ -548,6 +548,10 @@ static const MemoryRegionOps unin_ops = {
> .read = unin_read,
> .write = unin_write,
> .endianness = DEVICE_BIG_ENDIAN,
> + .impl = {
> + .min_access_size = 4,
> + .max_access_size = 4,
> + },
> };
>
> static void unin_init(Object *obj)
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-ppc] [PATCH 1/4] uninorth: add impl min_access_size and max_access_size to unin_ops,
David Gibson <=