Attention is currently required from: Thomas Heijligen, Peter Marheine.
Patch set 6:Code-Review +1
View Change
4 comments:
File Makefile:
Patch Set #3, Line 104: CONFIG_INTERNAL_MIPSLE \
I haven't looked it up yet
Done
Patch Set #3, Line 332: # Disable the internal programmer on unsupported architectures (everything but x86 and mipsel)
> (mark_unsupported) works fine, it's just hard to understand what it does. […]
Done, I think.
File Makefile:
Patch Set #5, Line 594: CONFIG_INTERNAL_X86 := yes
The new patchset should fix all this. […]
A bit more redundancy, but also more explicit, looks ok.
File Makefile:
Patch Set #6, Line 315: #ifneq ($(ARCH)-$(TARGET_OS), $(filter $(ARCH)-$(TARGET_OS), x86-$(TARGET_OS) $(ARCH)-Linux))
leftover?
To view, visit change 61300. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ia607ea60c3d7d15fe231fa412595992dadc535ad
Gerrit-Change-Number: 61300
Gerrit-PatchSet: 6
Gerrit-Owner: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-CC: Peter Marheine <pmarheine@chromium.org>
Gerrit-Attention: Thomas Heijligen <src@posteo.de>
Gerrit-Attention: Peter Marheine <pmarheine@chromium.org>
Gerrit-Comment-Date: Sat, 19 Feb 2022 16:07:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Nico Huber <nico.h@gmx.de>
Comment-In-Reply-To: Thomas Heijligen <src@posteo.de>
Comment-In-Reply-To: Peter Marheine <pmarheine@chromium.org>
Gerrit-MessageType: comment