On Wed, 29 May 2024 07:15:28 +0200 Philippe Mathieu-Daudé <phi...@linaro.org> wrote:
> 'smbios_encode_uuid' is always true, remove it, > simplifying smbios_encode_uuid(). > > Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> > Reviewed-by: Zhao Liu <zhao1....@intel.com> I'd squash this into previous commit, but won't insist. Reviewed-by: Igor Mammedov <imamm...@redhat.com> > --- > hw/smbios/smbios.c | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-) > > diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c > index 8261eb716f..3b7703489d 100644 > --- a/hw/smbios/smbios.c > +++ b/hw/smbios/smbios.c > @@ -30,7 +30,6 @@ > #include "hw/pci/pci_device.h" > #include "smbios_build.h" > > -static const bool smbios_uuid_encoded = true; > /* > * SMBIOS tables provided by user with '-smbios file=<foo>' option > */ > @@ -600,11 +599,9 @@ static void smbios_build_type_0_table(void) > static void smbios_encode_uuid(struct smbios_uuid *uuid, QemuUUID *in) > { > memcpy(uuid, in, 16); > - if (smbios_uuid_encoded) { > - uuid->time_low = bswap32(uuid->time_low); > - uuid->time_mid = bswap16(uuid->time_mid); > - uuid->time_hi_and_version = bswap16(uuid->time_hi_and_version); > - } > + uuid->time_low = bswap32(uuid->time_low); > + uuid->time_mid = bswap16(uuid->time_mid); > + uuid->time_hi_and_version = bswap16(uuid->time_hi_and_version); > } > > static void smbios_build_type_1_table(void)