coreboot 4.8.1: add upstreamed coreboot sutil patch and librem_15v3(Q4.0 compatible) board to CircleCI config to test build.

Author: tlaurion

.circleci/config.yml

@@ -88,8 +88,12 @@ jobs:
 
       - run:
           name: Download and neuter xx30 ME (keep generated GBE and extracted IFD in tree)
-          # me_cleaner.py present under heads xx30 blobs dir comes from https://github.com/corna/me_cleaner/blob/43612a630c79f3bc6f2653bfe90dfe0b7b137e08/me_cleaner.py
+          # Below when we download me_cleaner it's not really that important to validate it's
+          # integrity. This is because ./blobs/xx30/download_clean_me.sh does a hash check on
+          # it's result anyway.
           command: |
+            wget -O ./blobs/xx30/me_cleaner.py https://raw.githubusercontent.com/corna/me_cleaner/master/me_cleaner.py
+            chmod +x ./blobs/xx30/me_cleaner.py
             ./blobs/xx30/download_clean_me.sh -m $(readlink -f ./blobs/xx30/me_cleaner.py)
 
       - persist_to_workspace:
@@ -163,6 +167,13 @@ workflows:
 # version. The last board in the sequence is the dependency
 # for the parallel boards built at the end, and also save_cache.
 
+      # Coreboot 4.8.1
+      - build_and_persist:
+          name: librem_15v3
+          target: librem_15v3
+          requires:
+            - prep_env
+
       # Coreboot 4.11
       - build_and_persist:
           name: kgpe-d16_workstation
@@ -266,12 +277,6 @@ workflows:
           requires:
             - x230-hotp-maximized
 
-      - build:
-          name: qemu-coreboot
-          target: qemu-coreboot
-          requires:
-            - librem_mini
-
       - build:
           name: qemu-coreboot-fbwhiptail
           target: qemu-coreboot-fbwhiptail

patches/coreboot-4.8.1/0073-sutil-remove_dup_symbols.patch

@@ -0,0 +1,33 @@
+From f02ed614d2a031aeb73777b06dc0934bb9520e50 Mon Sep 17 00:00:00 2001
+From: Patrick Georgi <patrick@georgi.software>
+Date: Fri, 22 Oct 2021 07:46:14 +0200
+Subject: [PATCH] util/sconfig: Avoid duplicate symbols in link step
+
+Recent linkers became more picky about unifying identical symbols, so
+avoid struct *device head to appear several times by marking it
+"extern".
+
+Follows a change in commit 79e8412665567c, but without carrying all the
+other refactorings done there.
+
+Change-Id: Icede0936cc99f1c183c09812bffc3cc9a3993efa
+Signed-off-by: Patrick Georgi <patrick@georgi.software>
+Reviewed-on: https://review.coreboot.org/c/coreboot/+/58534
+Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
+Reviewed-by: Martin Roth <martinroth@google.com>
+---
+
+diff --git a/util/sconfig/sconfig.h b/util/sconfig/sconfig.h
+index 119d7b5..3ca13b3 100644
+--- a/util/sconfig/sconfig.h
++++ b/util/sconfig/sconfig.h
+@@ -77,7 +77,7 @@
+ 	struct reg *reg;
+ };
+ 
+-struct device *head;
++extern struct device *head;
+ 
+ struct header;
+ struct header {
+