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

[Karthik Ramasubramanian (Code Review)]

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

Change subject: mb/google/octopus/variants/bobba: fix LTE power sequence
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/37649/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/37649/1//COMMIT_MSG@10 
PS1, Line 10:
> Here the problem is function power_off_lte_module() can't called in smm stage, so add variant. […]

Can get_board_sku() be put into a separate file - eg. board_info.c or mainboard_misc.c. Then that file can be compiled in ramstage and smm stages.

That way we don't have to re-implement it.

Also you can mention the problem details you shared in the response in the commit summary.