[coreboot-gerrit] [S] Change in coreboot[master]: mb/google/corsola: Add detachable Starmie as variant

[Yidi Lin (Code Review)]

Attention is currently required from: Hung-Te Lin, Ruihai Zhou, Yu-Ping Wu.

Yidi Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74048 )

Change subject: mb/google/corsola: Add detachable Starmie as variant
......................................................................

Patch Set 10:

(2 comments)

Patchset:

PS10:
> sorry, I don't know why I accidentally added too many reviewers

This is expected.
The gerrit will add Hung-Te, Yu-Ping and me to the CLs related to MTK platform from now on.

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

https://review.coreboot.org/c/coreboot/+/74048/comment/b9a64dd7_42dc9ec2 
PS10, Line 143: bridge->power_on
It seems that all bridge callbacks are required. Just check them before proceeding to the display initialization.
```
if (!bridge || !bridge->power_on || !bridge->get_edid || !bridge->post_power_on)
  return -1;
```
And remove the assertions in the below.

(We may need to refactor this file after adding the MIPI support.)