[coreboot-gerrit] Change in coreboot[master]: wifi: Add support for new revisions of SAR table entries

[Tim Wawrzynczak (Code Review)]

Attention is currently required from: Furquan Shaikh, Sugnan Prabhu S, Paul Menzel.
Tim Wawrzynczak 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 25: Code-Review+1

(3 comments)

File src/vendorcode/google/chromeos/sar.c:

https://review.coreboot.org/c/coreboot/+/56750/comment/220b3f9b_dd2636b1 
PS25, Line 59: (1 + sar->dsar_set_count)
comment why the `1 + ` is required?

https://review.coreboot.org/c/coreboot/+/56750/comment/2ba395c1_d52165bb 
PS25, Line 99: void *
a `uint8_t *` should be able to cast to anything, this can be:
`struct sar_header *`

https://review.coreboot.org/c/coreboot/+/56750/comment/afbe0e3e_26ead37e 
PS25, Line 220: !strncmp(sar_str, SAR_STR_PREFIX, SAR_STR_PREFIX_SIZE) != 0
extra `!` at the beginning
`if (strncmp(sar_str, SAR_STR_PREFIX, SAR_STR_PREFIX_SIZE) != 0)`
?