[coreboot-gerrit] Change in coreboot[master]: mb/google/octopus: Add custom SAR values for Foob360

[Karthik Ramasubramanian (Code Review)]

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39424 )

Change subject: mb/google/octopus: Add custom SAR values for Foob360
......................................................................

Patch Set 9:

(2 comments)

https://review.coreboot.org/c/coreboot/+/39424/9/src/mainboard/google/octopu... 
File src/mainboard/google/octopus/variants/baseboard/include/baseboard/variants.h:

https://review.coreboot.org/c/coreboot/+/39424/9/src/mainboard/google/octopu... 
PS9, Line 39: uint32_t get_board_sku(void);
No need to re-introduce this function.

https://review.coreboot.org/c/coreboot/+/39424/9/src/mainboard/google/octopu... 
File src/mainboard/google/octopus/variants/foob/variant.c:

https://review.coreboot.org/c/coreboot/+/39424/9/src/mainboard/google/octopu... 
PS9, Line 55: 	uint32_t sku_id = get_board_sku();
Please use google_chromeec_get_board_sku() like in line 34 instead of get_board_sku().