Agree with Paul's comments. @ongqiang, can you move this to latter in the chain? We should add drivers first and then enable config, unless if something will break in compilation.

View Change

To view, visit change 31521. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I478e06686158dd77b075bcef8a41763ae26c79f9
Gerrit-Change-Number: 31521
Gerrit-PatchSet: 3
Gerrit-Owner: yongqiang niu <yongqiang.niu@mediatek.com>
Gerrit-Reviewer: Hung-Te Lin <hungte@chromium.org>
Gerrit-Reviewer: You-Cheng Syu <youcheng@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Reviewer: jitao shi <jitao.shi@mediatek.com>
Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-CC: Daisuke Nojiri <dnojiri@chromium.org>
Gerrit-CC: Nico Huber <nico.h@gmx.de>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Wed, 10 Apr 2019 07:26:53 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment