Hi Julius,
So this patch is without the interface table support. Only loading the qclib blob. Will add the support for the interface table in later patches. Was planning to add all the relevant structures at that time. Hence trimmed this patch to be minimal like this. Not ok ?

Well... I'm just generally not a fan of landing premature hacks. It's just taking time away from working on the actual fix. You might as well leave this floating until you can clean it up.

Note that this patch also has the same problem in the Makefile as the other one I pointed out anyway. We shouldn't submit those hacks which work around valid build errors, we should rather spend our energy on getting the licensing issues worked out for good.

View Change

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I534af71163d034ea04420dda6a94ce31b08c8a07
Gerrit-Change-Number: 29967
Gerrit-PatchSet: 14
Gerrit-Owner: nsekar@codeaurora.org
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Reviewer: nsekar@codeaurora.org
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-CC: Sricharan Ramabadhran <srichara@qualcomm.corp-partner.google.com>
Gerrit-Comment-Date: Fri, 08 Mar 2019 22:36:07 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment