[S] Change in coreboot[main]: mb/google/corsola: Distinguish MT8186T's SKU ID from MT8186

[Yu-Ping Wu (Code Review)]

Attention is currently required from: Hung-Te Lin, Knox Chiou, Paul Menzel, Xinxiong Xu, Yang Wu, Yidi Lin.

Yu-Ping Wu has posted comments on this change by Yang Wu. ( https://review.coreboot.org/c/coreboot/+/84342?usp=email )

Change subject: mb/google/corsola: Distinguish MT8186T's SKU ID from MT8186
......................................................................

Patch Set 8:

(3 comments)

File src/mainboard/google/corsola/boardid.c:

https://review.coreboot.org/c/coreboot/+/84342/comment/291a24a0_7686c458?usp... :
PS6, Line 118: 	if (cached_sku_code == CROS_SKU_UNPROVISIONED ||
             : 	    cached_sku_code == CROS_SKU_UNKNOWN) {
             : 		printk(BIOS_WARNING, "SKU code from EC: 0x%x\n", cached_sku_code);
             : 
             : 		if (CONFIG(BOARD_GOOGLE_STARYU_COMMON)) {
             : 			if (get_cpu_id() == MTK_CPU_ID_MT8186T) {
             : 				/* Reserve last 4 bits to report PANEL_ID */
             : 				cached_sku_code = 0x7FFFFEF0UL | panel_id();
             : 			} else {
             : 				cached_sku_code = 0x7FFFFFF0UL | panel_id();
             : 			}
             : 		} else if (get_cpu_id() == MTK_CPU_ID_MT8186T &&
             : 			  !CONFIG(BOARD_GOOGLE_VOLTORB) &&
             : 			  !CONFIG(BOARD_GOOGLE_SQUIRTLE)) {
             : 			/*
             : 			* Distinguish MT8186T from MT8186 to select different device trees
             : 			* in the payload. Voltorb/Squirtle are exception here, because right now
             : 			* they use 0x7FFFFFFF in the DTS compatible string. We'll need to change
             : 			* that to 0x7FFFFEFF in kernel before the exception can be dropped.
             : 			*/
             : 			cached_sku_code = CROS_SKU_UNPROVISIONED_MT8186T;
             : 		}
             : 	}
> or […]

That also works. Given that we've decided to drop the exception for voltorb and squirtle here, I think PS8 looks clean enough.

File src/mainboard/google/corsola/boardid.c:

https://review.coreboot.org/c/coreboot/+/84342/comment/fde96ee4_42af43b9?usp... :
PS8, Line 119: cached_sku_code
Align with `cached_sku_code` above.

https://review.coreboot.org/c/coreboot/+/84342/comment/09ce8001_61a55284?usp... :
PS8, Line 125: 		if (CONFIG(BOARD_GOOGLE_STARYU_COMMON)) {
Keep the `/* Reserve last 4 bits to report PANEL_ID */` comment.