Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak, Paul Menzel.
Sugnan Prabhu S has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56750 )
Change subject: wifi: Add support for new revisions of SAR table entries
......................................................................
Patch Set 26:
(3 comments)
File src/vendorcode/google/chromeos/sar.c:
https://review.coreboot.org/c/coreboot/+/56750/comment/46538a95_d2b2b00c
PS25, Line 59: (1 + sar->dsar_set_count)
comment why the `1 + ` is required?
https://review.coreboot.org/c/coreboot/+/56750/25/src/include/sar.h#30
sar_table will have data for both WRDS and EWRD. So it will be 1 + sar->dsar_set_count.
https://review.coreboot.org/c/coreboot/+/56750/comment/99959d2c_4f5bc5a6
PS25, Line 99: void *
a `uint8_t *` should be able to cast to anything, this can be: […]
Done
https://review.coreboot.org/c/coreboot/+/56750/comment/aed7bf64_5d84ff51
PS25, Line 220: !strncmp(sar_str, SAR_STR_PREFIX, SAR_STR_PREFIX_SIZE) != 0
extra `!` at the beginning […]
Done
Typo :( updated thanks
--
To view, visit
https://review.coreboot.org/c/coreboot/+/56750
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I08c3f321938eba04e8bcff4d87cb215422715bb2
Gerrit-Change-Number: 56750
Gerrit-PatchSet: 26
Gerrit-Owner: Sugnan Prabhu S
sugnan.prabhu.s@intel.com
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Dtrain Hsu
dtrain_hsu@compal.corp-partner.google.com
Gerrit-CC: Jamie Chen
jamie.chen@intel.com
Gerrit-CC: Janiex Tu
janiex.tu@intel.corp-partner.google.com
Gerrit-CC: Matt Chen
matt.chen@intel.corp-partner.google.com
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-CC: Rizwan Qureshi
rizwan.qureshi@intel.com
Gerrit-CC: Simon Yang
simon1.yang@intel.com
Gerrit-Attention: Furquan Shaikh
furquan@google.com
Gerrit-Attention: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Attention: Paul Menzel
paulepanter@mailbox.org
Gerrit-Comment-Date: Thu, 02 Sep 2021 00:39:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-MessageType: comment