Hi, Paolo
> > static DevicePropertyInfo *make_device_property_info(ObjectClass *klass,
> > const
> char *name,
> > - const char
> *default_type)
> > + const char
> *default_type,
> > + const char
> *description)
> > {
> > DevicePropertyInfo *info;
> > Property *prop;
> > @@ -465,7 +466,12 @@ static DevicePropertyInfo
> *make_device_property_info(ObjectClass *klass,
> >
> > info = g_malloc0(sizeof(*info));
> > info->name = g_strdup(prop->name);
> > - info->type = g_strdup(prop->info->legacy_name ?:
> prop->info->name);
> > + if (prop->info->description) {
> > + info->type = g_strdup_printf("%s (%s)",
> prop->info->name,
> > +
> prop->info->description);
> > + } else {
> > + info->type = g_strdup(prop->info->name);
> > + }
> > return info;
> > }
> > klass = object_class_get_parent(klass);
> > @@ -474,7 +480,11 @@ static DevicePropertyInfo
> *make_device_property_info(ObjectClass *klass,
> > /* Not a qdev property, use the default type */
> > info = g_malloc0(sizeof(*info));
> > info->name = g_strdup(name);
> > - info->type = g_strdup(default_type);
> > + if (description) {
> > + info->type = g_strdup_printf("%s (%s)", default_type, description);
>
> Please add a new "description" field to DevicePropertyInfo, and format
> it in qdev_device_help.
>
> You can send v3 of just this patch.
>
OK, will do it. Thanks! :)
Best regards,
-Gonglei
