On Thu, Aug 05, 2021 at 05:47:39PM -0400, Eduardo Habkost wrote: > commit 58b350280e97 ("hw/arm/bcm2836: Restrict BCM283XInfo > declaration to C source") didn't just move the struct > BCM283XClass definition to bcm2836.c. It also introduced a > typedef (BCM283XClass) and two type checking macros > (BCM283X_CLASS, BCM283X_GET_CLASS). > > The typedef and macros duplicate what is already defined by the > OBJECT_DECLARE_TYPE declaration at bcm2836.h. Remove the > redundant declarations. > > Signed-off-by: Eduardo Habkost <ehabk...@redhat.com> > --- > Cc: Philippe Mathieu-Daudé <f4...@amsat.org> > Cc: Luc Michel <luc.mic...@greensocs.com> > Cc: Peter Maydell <peter.mayd...@linaro.org> > --- > hw/arm/bcm2836.c | 9 ++------- > 1 file changed, 2 insertions(+), 7 deletions(-)
Reviewed-by: Daniel P. Berrangé <berra...@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 :|