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

[Yidi Lin (Code Review)]

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

Yidi Lin 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 6:

(5 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84342/comment/68c61247_1392e0e3?usp... :
PS6, Line 9: default
unprovisioned

https://review.coreboot.org/c/coreboot/+/84342/comment/e83153a9_491c18b8?usp... :
PS6, Line 10: dts
DTS

https://review.coreboot.org/c/coreboot/+/84342/comment/d1876f5d_27c58dc3?usp... :
PS6, Line 10: load
use

https://review.coreboot.org/c/coreboot/+/84342/comment/bc6af773_fad616a5?usp... :
PS6, Line 11: 0x7fffffff
unprovisioned

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

https://review.coreboot.org/c/coreboot/+/84342/comment/b8926bf5_43341976?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;
             : 		}
             : 	}
What about
```
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);
        cached_sku_code = CROS_SKU_UNPROVISIONED;
    if (get_cpu_id() == MTK_CPU_ID_MT8186T &&
        !CONFIG(BOARD_GOOGLE_VOLTORB) && !CONFIG(BOARD_GOOGLE_SQUIRTLE)
        cached_sku_code = CROS_SKU_UNPROVISIONED_MT8186T;

if (CONFIG(BOARD_GOOGLE_STARYU_COMMON)) {
        cached_sku_code &= ~0xF;
        cached_sku_code |= panel_id();
    }
}
```