12 comments:
Patch Set #1, Line 7: mediaek
mediatek
Ack
Where is the data sheet (or the name, revision), and is that driver written from scratch or taken fr […]
Ack
Can you please add a datasheet name/revision.
Ack
Patch Set #25, Line 16: mesaages
messages
Ack
File src/soc/mediatek/mt8183/dsi.c:
Is all this still gonna get deduplicated with MT8173 code?
yes and the work to refactor is in https://review.coreboot.org/c/coreboot/+/34562
Consider changing to ">=" for consistency
This needs MTK to confirm.
@jitao/yungnian?
Patch Set #25, Line 93: lanes);
Can be moved to the previous line
Done
Patch Set #25, Line 295: printk(BIOS_ERR, "HFP less than d-phy, FPS will under 60Hz\n");
Exceeds 80 columns
Done
Patch Set #25, Line 302: printk(BIOS_ERR, "HFP less than d-phy, FPS will under 60Hz\n");
Exceeds 80 columns
Done
File src/soc/mediatek/mt8183/include/soc/dsi.h:
Patch Set #24, Line 105: #define END_OF_TABLE 1
We should make this one 0, so that if someone accidentally just uses an empty struct initializer to […]
Done
Patch Set #24, Line 106: #define INIT_GENENIC_CMD 2
typo
Done
Patch Set #24, Line 112: u8 data[64];
Considering how many of these you throw in for some of those panels, we should really try to optimiz […]
Let us change that to 7 first.
To view, visit change 31591. To unsubscribe, or for help writing mail filters, visit settings.