[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 07/58] opentitan: Rename memmap enum constants
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v2 07/58] opentitan: Rename memmap enum constants |
Date: |
Tue, 25 Aug 2020 12:09:46 +0100 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Wed, Aug 19, 2020 at 08:11:45PM -0400, Eduardo Habkost wrote:
> Some of the enum constant names conflict with the QOM type check
> macros (IBEX_PLIC, IBEX_UART). This needs to be addressed to
> allow us to transform the QOM type check macros into functions
> generated by OBJECT_DECLARE_TYPE().
>
> Rename all the constants to IBEX_DEV_*, to avoid conflicts.
>
> Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Changes v1 -> v2:
> * Added more details to commit message
>
> ---
> Cc: Alistair Francis <Alistair.Francis@wdc.com>
> Cc: Palmer Dabbelt <palmer@dabbelt.com>
> Cc: Sagar Karandikar <sagark@eecs.berkeley.edu>
> Cc: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
> Cc: qemu-riscv@nongnu.org
> Cc: qemu-devel@nongnu.org
> ---
> include/hw/riscv/opentitan.h | 38 ++++++++--------
> hw/riscv/opentitan.c | 84 ++++++++++++++++++------------------
> 2 files changed, 61 insertions(+), 61 deletions(-)
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|