Chen-Tsung Hsieh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46442 )
Change subject: mb/google/kukui: fix SKUID config for burnet/esche ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46442/1/src/mainboard/google/kukui/... File src/mainboard/google/kukui/boardid.c:
https://review.coreboot.org/c/coreboot/+/46442/1/src/mainboard/google/kukui/... PS1, Line 99: if (CONFIG(BOARD_GOOGLE_BURNET)) { : cached_sku_id = (0x10 | get_adc_index(SKU_ID_CHANNEL)); : return cached_sku_id; : } :
given Jacuzzi followers should not use need LCD module, I'd recommend changing this for all Jacuzzi: […]
The LCM_ID HW strap of kappa & juniper are actually 1. Their sku-ids are 0x10 (not 0x0) now. I'm worried about changing the rule of sku-id for MP devices.
I'd recommend that we still focus on debuging the floating HW strap value of LCM_ID. Then, we don't have to patch anything.