Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/33997 )
Change subject: flashchips.h: merge definitions from Chromium fork
......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/33997/2/flashchips.h
File flashchips.h:
https://review.coreboot.org/#/c/33997/2/flashchips.h@470
PS2, Line 470: #define INTEL_HWSEQ 0xFFFE /* dummy ID for hardware sequencing */
Here in upstream we don't have intel hwseq support so better to leave this out for now and ship it w […]
There is some hwseq support here in upstream, but I don't know much about it.
--
To view, visit
https://review.coreboot.org/c/flashrom/+/33997
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ie280f351045bdcbc454d6f829fc071af820382b1
Gerrit-Change-Number: 33997
Gerrit-PatchSet: 2
Gerrit-Owner: Alan Green
avg@google.com
Gerrit-Reviewer: Edward O'Callaghan
quasisec@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Angel Pons
th3fanbus@gmail.com
Gerrit-Comment-Date: Wed, 03 Jul 2019 09:05:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Edward O'Callaghan
quasisec@chromium.org
Gerrit-MessageType: comment