[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/267292e2_0ef66b24?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);
             : 	}
> I may prefer to remove controller2 and controller3, and change to: […]

Do you mean removing the controller 2 and 3 from`lvts_tscpu_g_tc` ?

Can you explain the reason ?