4 comments:
File src/mainboard/google/drallion/Makefile.inc:
Patch Set #9, Line 32: CONFIG_EC_GOOGLE_WILCO
Nit: I think these could just be changed to '-y' I don't think the board would build without wilco ec being selected. If I'm wrong, please feel free to ignore this.
Or ignore it even if I'm right. It's just a nit.
File src/mainboard/google/drallion/dsdt.asl:
Patch Set #9, Line 62: #if CONFIG(EC_GOOGLE_WILCO)
Nit: You're selecting EC_GOOGLE_WILCO in the kconfig. You have calls to various wilco ec functions all over the mainboard. I doubt it would build without the EC_GOOGLE_WILCO enabled, so this seems like a useless #if.
File src/mainboard/google/drallion/ramstage.c:
Patch Set #9, Line 25: #if CONFIG(GENERATE_SMBIOS_TABLES)
Is this guard needed? Wouldn't they just not get called?
File src/mainboard/google/drallion/variants/drallion/devicetree.cb:
Patch Set #9, Line 210: "MISCCFG_GPSIDEDPCGEN | MISCCFG_GPRTCDLCGEN | MISCCFG_GSXSLCGEN | MISCCFG_GPDPCGEN | MISCCFG_GPDLCGEN"
Nit: Maybe wrap this line?
To view, visit change 34497. To unsubscribe, or for help writing mail filters, visit settings.