[coreboot-gerrit] Change in coreboot[master]: drivers/wifi/generic: fix package_size to align with WLAN driver

[Matt Chen (Code Review)]

Attention is currently required from: Sugnan Prabhu S, Tim Wawrzynczak, Paul Menzel, Subrata Banik, EricR Lai.
Matt Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58951 )

Change subject: drivers/wifi/generic: fix package_size to align with WLAN driver
......................................................................

Patch Set 6:

(1 comment)

Patchset:

PS6:
> Primus doesn't have this flag. But other two have this. I can check on Brya as well.

Hi Eric,

Can you apply below for Brya, see if it is printing 3 or 1 ?

diff --git a/src/drivers/wifi/generic/acpi.c b/src/drivers/wifi/generic/acpi.c
index 11fc0e084f..743a313144 100644
--- a/src/drivers/wifi/generic/acpi.c
+++ b/src/drivers/wifi/generic/acpi.c
@@ -273,6 +273,7 @@ static void sar_emit_ewrd(const struct sar_profile *sar)
         * Emit 'Domain Type' + 'Dynamic SAR Enable' + 'Extended SAR sets count'
         * + number of bytes for Set#2 & 3 & 4
         */
+       printk(BIOS_INFO, "sar->dsar_set_count = %d\n", sar->dsar_set_count);
        package_size = 1 + 1 + 1 + table_size * sar->dsar_set_count;
        acpigen_write_package(package_size);
        acpigen_write_dword(DOMAIN_TYPE_WIFI);