Change in coreboot[master]: Documentation/Intel: s5 charging applet design document

[Karthik Ramasubramanian (Code Review)]

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

Change subject: Documentation/Intel: s5 charging applet design document
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/c/coreboot/+/37762/4/Documentation/soc/intel/s5_... 
File Documentation/soc/intel/s5_charging_applet/s5_charging_applet.md:

https://review.coreboot.org/c/coreboot/+/37762/4/Documentation/soc/intel/s5_... 
PS4, Line 36: Show critical|
            : |  (Battery      |	   or          |  Battery     |
            : |   critical)    |	Lid Open       |  symbol
Will it turn off automatically after a timeout? If so, can it be configurable to balance the power as well as user-experience.

How does coreboot know if battery is critical?

Chrome Devices do display critical battery and shutdown automatically - at the OS level and not at the coreboot stage.

https://review.coreboot.org/c/coreboot/+/37762/4/Documentation/soc/intel/s5_... 
File Documentation/soc/intel/s5_charging_applet/s5_charging_applet_design_flowchart.png:

PS4: 
Memory Init happens in ROM stage and Silicon Init happens in RAM Stage.