Rev
3 comments:
File src/mainboard/google/kukui/display.c:
static struct edid *get_edid(struct board_display_intf *intf)
{
struct panel_info *info = intf->cur_panel_info;
if (info)
return info->edid;
return NULL;
}
I think this function is not needed since it's only called one time, and the intf is provided by developer so simply assert() would be sufficient. Please move it to the caller.
Patch Set #19, Line 84: get_panel_name
this is also not needed...
Patch Set #19, Line 100: update_panel_info
In fact we don't need to call intf->get_panel_id.
For Kukui family, the panel ID is decided by ADC (see boardid.c).
We may add a new get_panel_id() calling ADC if needed,
or simply do get_sku_id() >> 4.
To view, visit change 32511. To unsubscribe, or for help writing mail filters, visit settings.