On 6/6/25 12:40, Philippe Mathieu-Daudé wrote:
On 6/6/25 11:24, Zhenzhong Duan wrote:
SH_SERIAL is declared with OBJECT_DECLARE_SIMPLE_TYPE but defined with
OBJECT_DEFINE_TYPE, SHSerialStateClass is also a dummy class which
missed its parent.
Change to use OBJECT_DEFINE_SIMPLE_TYPE and remove SHSerialStateClass.
Closes: https://lists.gnu.org/archive/html/qemu-devel/2025-06/
msg00586.html
Suggested-by: David Hildenbrand <da...@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.d...@intel.com>
---
hw/char/sh_serial.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/hw/char/sh_serial.c b/hw/char/sh_serial.c
index 6abd80386f..8ccc2234ba 100644
--- a/hw/char/sh_serial.c
+++ b/hw/char/sh_serial.c
@@ -78,9 +78,7 @@ struct SHSerialState {
qemu_irq bri;
};
-typedef struct {} SHSerialStateClass;
-
-OBJECT_DEFINE_TYPE(SHSerialState, sh_serial, SH_SERIAL, SYS_BUS_DEVICE)
+OBJECT_DEFINE_SIMPLE_TYPE(SHSerialState, sh_serial, SH_SERIAL,
SYS_BUS_DEVICE)
I'm surprised this is the first time we use OBJECT_DEFINE_SIMPLE_TYPE.
Actually I already posted a patch cleaning that...:
https://lore.kernel.org/qemu-devel/20250124175053.74461-1-phi...@linaro.org/
Also I now notice I never replied to Peter's questions. I'll and
queue the series, along with yours.
Regards,
Phil.
