[coreboot-gerrit] [XS] Change in coreboot[master]: soc/amd/common/block/acpi/ivrs: zero-initialize ivhd_f0 struct

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/76072?usp=email )

Change subject: soc/amd/common/block/acpi/ivrs: zero-initialize ivhd_f0 struct
......................................................................

soc/amd/common/block/acpi/ivrs: zero-initialize ivhd_f0 struct

Zero-initialize the ivhd_f0 struct right at the beginning of the
ivhd_describe_f0_device function to make sure that the whole struct is
in a defined state.

Signed-off-by: Felix Held 
Change-Id: Ia6750b58dacb9b9192ed21128eb6e3a4495b96d0
Reviewed-on: https://review.coreboot.org/c/coreboot/+/76072
Tested-by: build bot (Jenkins) 
Reviewed-by: Fred Reitberger 
Reviewed-by: Jason Glenesk 
---
M src/soc/amd/common/block/acpi/ivrs.c
1 file changed, 1 insertion(+), 5 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Jason Glenesk: Looks good to me, approved
  Fred Reitberger: Looks good to me, approved

diff --git a/src/soc/amd/common/block/acpi/ivrs.c b/src/soc/amd/common/block/acpi/ivrs.c
index 79b4ff6..8d127e9 100644
--- a/src/soc/amd/common/block/acpi/ivrs.c
+++ b/src/soc/amd/common/block/acpi/ivrs.c
@@ -52,6 +52,7 @@
    					uint16_t dev_id, uint8_t datasetting)
 {
    ivrs_ivhd_f0_entry_t *ivhd_f0 = (ivrs_ivhd_f0_entry_t *)current;
+	memset(ivhd_f0, 0, sizeof(*ivhd_f0));

ivhd_f0->type = IVHD_DEV_VARIABLE;
    ivhd_f0->dev_id = dev_id;
@@ -65,11 +66,6 @@
    ivhd_f0->hardware_id[6] = '4';
    ivhd_f0->hardware_id[7] = '0';

-	memset(ivhd_f0->compatible_id, 0, sizeof(ivhd_f0->compatible_id));
-
-	ivhd_f0->uuid_format = 0;
-	ivhd_f0->uuid_length = 0;
-
    current += sizeof(ivrs_ivhd_f0_entry_t);
    return current;
 }