Nico Huber posted comments on this change.
View Change
Patch set 1:
(1 comment)
File nicintel_eeprom.c:
Patch Set #1, Line 106: return (device_id & 0xfff0) == 0x1530;
Are you sure about this? 0x157b and 0x157c seem valid in addition to 0x153x
As this was supposed to be a fixup!, I tried to anticipate what
the author intended. And, as the comment suggests, this is not
supposed to be future proof.
Anyway, `device_id >= 0x1530` should work as well (but doesn't
align with current nicintel_spi.c).
To view, visit change 21934. To unsubscribe, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: staging
Gerrit-MessageType: comment
Gerrit-Change-Id: I5d3f71e4e0f77cc8793e7f395baf69e1fad930a3
Gerrit-Change-Number: 21934
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Stefan Tauner <stefan.tauner@gmx.at>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Fri, 20 Oct 2017 15:19:12 +0000
Gerrit-HasComments: Yes