[coreboot-gerrit] Change in coreboot[master]: mb/google/octopus/variants/garg: support LTE power sequence

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34188 )

Change subject: mb/google/octopus/variants/garg: support LTE power sequence
......................................................................

Patch Set 7:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34188/7/src/mainboard/google/octopu... 
File src/mainboard/google/octopus/variants/baseboard/include/baseboard/gpio.h:

https://review.coreboot.org/c/coreboot/+/34188/7/src/mainboard/google/octopu... 
PS7, Line 36: /* LTE reset GPIO */
            : #define GPIO_LTE_RESET	GPIO_161
            : 
Not required anymore.

https://review.coreboot.org/c/coreboot/+/34188/7/src/mainboard/google/octopu... 
File src/mainboard/google/octopus/variants/garg/variant.c:

https://review.coreboot.org/c/coreboot/+/34188/7/src/mainboard/google/octopu... 
PS7, Line 37: if (slp_typ != ACPI_S5)
            : 		return;
nit: If you do this before calling get_board_sku(), one transaction to the EC can be saved in non-S5 cases.