[PATCH 21/23] hw/rtc/ds1338: Prefer DEFINE_TYPES() macro

Bernhard Beschow posted 23 patches 4 weeks, 1 day ago
There is a newer version of this series
[PATCH 21/23] hw/rtc/ds1338: Prefer DEFINE_TYPES() macro
Posted by Bernhard Beschow 4 weeks, 1 day ago
---
 hw/rtc/ds1338.c | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/hw/rtc/ds1338.c b/hw/rtc/ds1338.c
index a5fe221418..6de13caf99 100644
--- a/hw/rtc/ds1338.c
+++ b/hw/rtc/ds1338.c
@@ -14,7 +14,6 @@
 #include "hw/i2c/i2c.h"
 #include "migration/vmstate.h"
 #include "qemu/bcd.h"
-#include "qemu/module.h"
 #include "qom/object.h"
 #include "sysemu/rtc.h"
 
@@ -227,16 +226,13 @@ static void ds1338_class_init(ObjectClass *klass, void *data)
     dc->vmsd = &vmstate_ds1338;
 }
 
-static const TypeInfo ds1338_info = {
-    .name          = TYPE_DS1338,
-    .parent        = TYPE_I2C_SLAVE,
-    .instance_size = sizeof(DS1338State),
-    .class_init    = ds1338_class_init,
+static const TypeInfo types[] = {
+    {
+        .name          = TYPE_DS1338,
+        .parent        = TYPE_I2C_SLAVE,
+        .instance_size = sizeof(DS1338State),
+        .class_init    = ds1338_class_init,
+    },
 };
 
-static void ds1338_register_types(void)
-{
-    type_register_static(&ds1338_info);
-}
-
-type_init(ds1338_register_types)
+DEFINE_TYPES(types)
-- 
2.46.1
Re: [PATCH 21/23] hw/rtc/ds1338: Prefer DEFINE_TYPES() macro
Posted by Cédric Le Goater 3 weeks, 6 days ago
Sob is missing.


Thanks,

C.

On 9/23/24 11:30, Bernhard Beschow wrote:
> ---
>   hw/rtc/ds1338.c | 20 ++++++++------------
>   1 file changed, 8 insertions(+), 12 deletions(-)
> 
> diff --git a/hw/rtc/ds1338.c b/hw/rtc/ds1338.c
> index a5fe221418..6de13caf99 100644
> --- a/hw/rtc/ds1338.c
> +++ b/hw/rtc/ds1338.c
> @@ -14,7 +14,6 @@
>   #include "hw/i2c/i2c.h"
>   #include "migration/vmstate.h"
>   #include "qemu/bcd.h"
> -#include "qemu/module.h"
>   #include "qom/object.h"
>   #include "sysemu/rtc.h"
>   
> @@ -227,16 +226,13 @@ static void ds1338_class_init(ObjectClass *klass, void *data)
>       dc->vmsd = &vmstate_ds1338;
>   }
>   
> -static const TypeInfo ds1338_info = {
> -    .name          = TYPE_DS1338,
> -    .parent        = TYPE_I2C_SLAVE,
> -    .instance_size = sizeof(DS1338State),
> -    .class_init    = ds1338_class_init,
> +static const TypeInfo types[] = {
> +    {
> +        .name          = TYPE_DS1338,
> +        .parent        = TYPE_I2C_SLAVE,
> +        .instance_size = sizeof(DS1338State),
> +        .class_init    = ds1338_class_init,
> +    },
>   };
>   
> -static void ds1338_register_types(void)
> -{
> -    type_register_static(&ds1338_info);
> -}
> -
> -type_init(ds1338_register_types)
> +DEFINE_TYPES(types)