Patch set 1:Code-Review -2
1 comment:
Patch Set #1, Line 164: return 1;
Why exactly is this branch considered to be an error? This most likely breaks DualBIOS support.
Thank you for the review Angel.
Nikolai, thanks for raising this with me. Angel and you are correct, this is a specific hack on cros side that needs unwinding. The downstream commit is 5e79c9f6954127319376a8b166e501ccfb1331cd .
Nikolai, just open a bug and assign me it as this overlaps patches I have on cros side that overlaps with this while ripping out the ec alias mechanism into a separate path. I believe we can abandon this patch not to wast upstreams limited time and we can deal with it in the bug.
To view, visit change 46439. To unsubscribe, or for help writing mail filters, visit settings.