summaryrefslogtreecommitdiffstats
path: root/src/mainboard/emulation/qemu-i440fx
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2021-06-28 15:36:23 +0200
committerPatrick Georgi <pgeorgi@google.com>2021-07-01 07:35:02 +0000
commit35b99c64e62fd84866dd269a3641b66646f9557f (patch)
tree8a95363becb766da16e47fadbe87ebca9a5fe06e /src/mainboard/emulation/qemu-i440fx
parentc6dab80b1cd8fb35115e9876b79abefb6c4f2d91 (diff)
downloadcoreboot-35b99c64e62fd84866dd269a3641b66646f9557f.tar.gz
coreboot-35b99c64e62fd84866dd269a3641b66646f9557f.tar.bz2
coreboot-35b99c64e62fd84866dd269a3641b66646f9557f.zip
SMBIOS: Avoid `sizeof` on struct type
Where applicable, use the size of the associated variable. Change-Id: Ibbac2a82893232a6f87182a6a965b84a599d633e Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/55904 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-by: Michał Żygowski <michal.zygowski@3mdeb.com>
Diffstat (limited to 'src/mainboard/emulation/qemu-i440fx')
-rw-r--r--src/mainboard/emulation/qemu-i440fx/northbridge.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/mainboard/emulation/qemu-i440fx/northbridge.c b/src/mainboard/emulation/qemu-i440fx/northbridge.c
index 8579b8f8869e..01d036a6a0fa 100644
--- a/src/mainboard/emulation/qemu-i440fx/northbridge.c
+++ b/src/mainboard/emulation/qemu-i440fx/northbridge.c
@@ -166,9 +166,9 @@ static void cpu_pci_domain_read_resources(struct device *dev)
static int qemu_get_smbios_data16(int handle, unsigned long *current)
{
struct smbios_type16 *t = (struct smbios_type16 *)*current;
- int len = sizeof(struct smbios_type16);
+ int len = sizeof(*t);
- memset(t, 0, sizeof(struct smbios_type16));
+ memset(t, 0, sizeof(*t));
t->type = SMBIOS_PHYS_MEMORY_ARRAY;
t->handle = handle;
t->length = len - 2;
@@ -185,11 +185,11 @@ static int qemu_get_smbios_data17(int handle, int parent_handle, unsigned long *
struct smbios_type17 *t = (struct smbios_type17 *)*current;
int len;
- memset(t, 0, sizeof(struct smbios_type17));
+ memset(t, 0, sizeof(*t));
t->type = SMBIOS_MEMORY_DEVICE;
t->handle = handle;
t->phys_memory_array_handle = parent_handle;
- t->length = sizeof(struct smbios_type17) - 2;
+ t->length = sizeof(*t) - 2;
t->size = qemu_get_memory_size() / 1024;
t->data_width = 64;
t->total_width = 64;