Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/em100/+/36836 )
Change subject: Make firmware update hardware version aware
......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/em100/+/36836/1/firmware.c
File firmware.c:
https://review.coreboot.org/c/em100/+/36836/1/firmware.c@243
PS1, Line 243: em100->hwversion == 4
Even then this is saying "this is an EM100Pro (non-G2) file if and only if there's also an EM100Pro […]
It says "This is NOT an EM100Pro file", not "This is an EM100Pro file.
At this point you will not get to this code if hwversion is anything but 4. The reason for this additional check is that on G2 (when the G2 code is added), the check and message will not make sense (and there is a second check for G2 specifically)
--
To view, visit
https://review.coreboot.org/c/em100/+/36836
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: em100
Gerrit-Branch: master
Gerrit-Change-Id: Ia284adca4cb4d341a8d229374b6ea0b5d2a5d5c0
Gerrit-Change-Number: 36836
Gerrit-PatchSet: 1
Gerrit-Owner: Stefan Reinauer
stefan.reinauer@coreboot.org
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Stefan Reinauer
stefan.reinauer@coreboot.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Thu, 14 Nov 2019 22:14:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Stefan Reinauer
stefan.reinauer@coreboot.org
Comment-In-Reply-To: Patrick Georgi
pgeorgi@google.com
Gerrit-MessageType: comment