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

[Xinxiong Xu (Code Review)]

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

Xinxiong Xu 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:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/84342/comment/a6575902_7a4179d8?usp... :
PS6, Line 131: BOARD_GOOGLE_SQUIRTLE
> Yes if the schedule can be met, otherwise there will be 2 exceptions. […]

If we don't want Squirtle to be an exception, can we apply the same modification to Voltorb? We can add 0x7FFFFEFF to the config.star file and support both 0x7FFFFEFF and 0x7FFFFFFF in the same DTS configuration, ensuring consistency. 
Since we need to add 0x7FFFFEFF to Squirtle's config.star, and also add 0x7FFFFEFF to the DTS that supports 0x7FFFFFFF, this shouldn't cause any confusion. 
In that case, there will be no need to add exceptions on the coreboot side.
What do you think?