[coreboot-gerrit] Change in coreboot[master]: mb/google/octopus/variants/garg: fix LTE power sequence while DUT war...

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46022 )

Change subject: mb/google/octopus/variants/garg: fix LTE power sequence while DUT warmboot
......................................................................

Patch Set 1:

(2 comments)

Is the Makefile change, all that is needed? Please elaborate in the commit message.

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

https://review.coreboot.org/c/coreboot/+/46022/1//COMMIT_MSG@16 
PS1, Line 16:       - pull down GPIO_161 for 30ms then release it.
Please name the datasheet name and version, where the power sequence is described in.

https://review.coreboot.org/c/coreboot/+/46022/1//COMMIT_MSG@15 
PS1, Line 15:       - keep GPIO_67 and GPIO_117 high and
            :       - pull down GPIO_161 for 30ms then release it.
Indentation for lists is not needed. You might loosely follow [Markdown][1].

[1]: https://daringfireball.net/projects/markdown/syntax