From 627c8443a3ddaded2c1cf4ac0811164eb1943815 Mon Sep 17 00:00:00 2001 From: Sugnan Prabhu S Date: Mon, 6 Sep 2021 15:05:56 +0530 Subject: [PATCH] wifi: Update deny list entry size to uint16_t As per the connectivity document deny list entry size should be uint16 559910_Intel_Connectivity_Platforms_BIOS_Guidelines_Rev6_4.pdf Fixes: cc50770cd0("wifi: Add support for wifi time average SAR config") Change-Id: I045c21350cf4c2266df108eede6350d090322ba0 Signed-off-by: Sugnan Prabhu S Reviewed-on: https://review.coreboot.org/c/coreboot/+/57407 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Tim Wawrzynczak --- src/drivers/wifi/generic/acpi.c | 6 +++--- src/include/sar.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/drivers/wifi/generic/acpi.c b/src/drivers/wifi/generic/acpi.c index e829185d6e..11fc0e084f 100644 --- a/src/drivers/wifi/generic/acpi.c +++ b/src/drivers/wifi/generic/acpi.c @@ -450,10 +450,10 @@ static void sar_emit_wtas(struct avg_profile *wtas) acpigen_write_dword(wtas->revision); acpigen_write_package(package_size); acpigen_write_dword(DOMAIN_TYPE_WIFI); - acpigen_write_dword(wtas->tas_selection); - acpigen_write_dword(wtas->tas_list_size); + acpigen_write_byte(wtas->tas_selection); + acpigen_write_byte(wtas->tas_list_size); for (i = 0; i < MAX_DENYLIST_ENTRY; i++) - acpigen_write_byte(wtas->deny_list_entry[i]); + acpigen_write_word(wtas->deny_list_entry[i]); acpigen_write_package_end(); acpigen_write_package_end(); diff --git a/src/include/sar.h b/src/include/sar.h index 5a4f7ae58a..beccb377cd 100644 --- a/src/include/sar.h +++ b/src/include/sar.h @@ -44,7 +44,7 @@ struct avg_profile { uint8_t revision; uint8_t tas_selection; uint8_t tas_list_size; - uint8_t deny_list_entry[MAX_DENYLIST_ENTRY]; + uint16_t deny_list_entry[MAX_DENYLIST_ENTRY]; } __packed; struct dsm_profile {