diff options
| author | Marc-André Lureau <marcandre.lureau@redhat.com> | 2017-06-07 20:36:09 +0400 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2017-06-20 14:31:32 +0200 |
| commit | 3fb2111fc9bb9c73bae81c6696bcf333690147b0 (patch) | |
| tree | 42de6d088d7f095d601d17ceee95678937bc3dfa /hw/core | |
| parent | 76318657a803c542cdee232c8507dec8004e2478 (diff) | |
| download | qemu-3fb2111fc9bb9c73bae81c6696bcf333690147b0.zip qemu-3fb2111fc9bb9c73bae81c6696bcf333690147b0.tar.gz qemu-3fb2111fc9bb9c73bae81c6696bcf333690147b0.tar.bz2 | |
qdev: avoid type casts between signed and unsigned
Modify the unsigned type for various properties to use QNUM_U64, to
avoid type casts.
There are a few empty lines added to improve code reading/style.
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-Id: <20170607163635.17635-18-marcandre.lureau@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
[Change to set_default_value_enum() dropped]
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'hw/core')
| -rw-r--r-- | hw/core/qdev-properties.c | 21 |
1 files changed, 13 insertions, 8 deletions
diff --git a/hw/core/qdev-properties.c b/hw/core/qdev-properties.c index f3f682c..68cd653 100644 --- a/hw/core/qdev-properties.c +++ b/hw/core/qdev-properties.c @@ -128,7 +128,7 @@ static void prop_set_bit(Object *obj, Visitor *v, const char *name, static void set_default_value_bool(Object *obj, const Property *prop) { - object_property_set_bool(obj, prop->defval.i, prop->name, &error_abort); + object_property_set_bool(obj, prop->defval.u, prop->name, &error_abort); } PropertyInfo qdev_prop_bit = { @@ -264,11 +264,16 @@ static void set_default_value_int(Object *obj, const Property *prop) object_property_set_int(obj, prop->defval.i, prop->name, &error_abort); } +static void set_default_value_uint(Object *obj, const Property *prop) +{ + object_property_set_uint(obj, prop->defval.u, prop->name, &error_abort); +} + PropertyInfo qdev_prop_uint8 = { .name = "uint8", .get = get_uint8, .set = set_uint8, - .set_default_value = set_default_value_int, + .set_default_value = set_default_value_uint, }; /* --- 16bit integer --- */ @@ -302,7 +307,7 @@ PropertyInfo qdev_prop_uint16 = { .name = "uint16", .get = get_uint16, .set = set_uint16, - .set_default_value = set_default_value_int, + .set_default_value = set_default_value_uint, }; /* --- 32bit integer --- */ @@ -361,7 +366,7 @@ PropertyInfo qdev_prop_uint32 = { .name = "uint32", .get = get_uint32, .set = set_uint32, - .set_default_value = set_default_value_int, + .set_default_value = set_default_value_uint, }; PropertyInfo qdev_prop_int32 = { @@ -402,7 +407,7 @@ PropertyInfo qdev_prop_uint64 = { .name = "uint64", .get = get_uint64, .set = set_uint64, - .set_default_value = set_default_value_int, + .set_default_value = set_default_value_uint, }; /* --- string --- */ @@ -725,7 +730,7 @@ PropertyInfo qdev_prop_blocksize = { .description = "A power of two between 512 and 32768", .get = get_uint16, .set = set_blocksize, - .set_default_value = set_default_value_int, + .set_default_value = set_default_value_uint, }; /* --- pci host address --- */ @@ -948,7 +953,7 @@ PropertyInfo qdev_prop_arraylen = { .name = "uint32", .get = get_uint32, .set = set_prop_arraylen, - .set_default_value = set_default_value_int, + .set_default_value = set_default_value_uint, }; /* --- public helpers --- */ @@ -1185,5 +1190,5 @@ PropertyInfo qdev_prop_size = { .name = "size", .get = get_size, .set = set_size, - .set_default_value = set_default_value_int, + .set_default_value = set_default_value_uint, }; |