[coreboot-gerrit] [S] Change in coreboot[main]: soc/mediatek/mt8196: Save HW protect temperature to SRAM

[Yidi Lin (Code Review)]

Attention is currently required from: Hung-Te Lin, Jarried Lin, Paul Menzel, Yu-Ping Wu.

Yidi Lin has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/86551?usp=email )

Change subject: soc/mediatek/mt8196: Save HW protect temperature to SRAM
......................................................................

Patch Set 3:

(1 comment)

File src/soc/mediatek/mt8196/thermal.c:

https://review.coreboot.org/c/coreboot/+/86551/comment/18c435cb_7453b162?usp... :
PS2, Line 606: 	if (tc_num == LVTS_AP_CONTROLLER0)
             : 		tc_index = 0;
             : 	else if (tc_num == LVTS_AP_CONTROLLER1)
             : 		tc_index = 1;
             : 
             : 	if (tc_index != 0xff) {
             : 		thermal_write_reboot_msr_sram(tc_index, raw_high);
             : 		if (tc_index == 0)
             : 			thermal_write_reboot_temp_sram(tc->reboot_temperature);
             : 	}
> Then, can we add `int reboot_temp_sram_offset` (whose value equals to `tc_index * 4` on valid contro […]

So, we can add two members to the struct. One is `reboot_temp_sram_offset` another is for `reboot_msr_sram_offset` (feel free to make the naming shorter). The driver saves the value to the memory when the offset is valid.