5 comments:
File src/mainboard/google/trogdor/Kconfig:
Patch Set #25, Line 21: select DRIVERS_TI_SN65DSI86BRIDGE
In the internal patch set submitted, i have updated the order. Not sure , it got missed here.
Well, I mean it's still not fixed. Please talk to Ravi or whoever is in charge of uploading these here and make sure they get ordered correctly the next time.
File src/mainboard/google/trogdor/mainboard.c:
Patch Set #32, Line 8: #include "board.h"
Please order the headers with "double quotes" below all the headers with <angled brackets> with a blank line in between. Also, please order headers alphabetically (i.e. most of these should come above <soc/qupv3_config.h>).
Note: This number likely needs an update for Pompom (see issuetracker.google.com/issues/161373813). Depending on whether we get the question of exactly how much resolved in time, we can either slip that into this patch or fix it as a follow-up.
File src/soc/qualcomm/sc7180/Makefile.inc:
Patch Set #25, Line 65: ramstage-$(CONFIG_MAINBOARD_DO_NATIVE_VGA_INIT) += display/dsi.c
Please put the Makefile lines for each file together with the patch that is adding that file.
Done
File src/soc/qualcomm/sc7180/display/mdss.c:
Patch Set #25, Line 156: 0x00000003
Where do all these magic numbers come from? This at least needs some comments.
*ping* (still unresolved)
To view, visit change 39615. To unsubscribe, or for help writing mail filters, visit settings.