Name of user not set #1002476 has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/35144 )
Change subject: util/crossgcc/patches: facilitate successful build of ipxe ......................................................................
util/crossgcc/patches: facilitate successful build of ipxe
New changes in the latest binutils 2.32 leads to assembler errors causes ipxe build failure. IPXE uses the divide test which requires /dev/null as input as well as the output file name.
This patch facilitates the /dev/null as an exception to the current changes in binutils package while building crossgcc for coreboot leads to successful build of ipxe and further tests to pass based on /dev/null and applies automatically during the crossgcc rebuild.
Also, this can be reverted once binutils/ipxe provides update release in this respect.
Change-Id: I111cfae98286baef38bb95b9f30f0382b047ea03 Signed-off-by: Himanshu Sahdev himanshusah@hcl.com --- M 3rdparty/blobs M 3rdparty/fsp M 3rdparty/libgfxinit M 3rdparty/libhwbase M 3rdparty/opensbi M 3rdparty/vboot M util/crossgcc/patches/binutils-2.32_as-ipxe.patch M util/nvidia/cbootimage 8 files changed, 8 insertions(+), 8 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/35144/1
diff --git a/3rdparty/blobs b/3rdparty/blobs index 62aa0e0..ca6cfcd 160000 --- a/3rdparty/blobs +++ b/3rdparty/blobs @@ -1 +1 @@ -Subproject commit 62aa0e0c54295bbb7b1a3e5e73f960bafdb59d04 +Subproject commit ca6cfcdbe1cdeb38c2622ee2e5236cc4657e3377 diff --git a/3rdparty/fsp b/3rdparty/fsp index 5996417..162719b 160000 --- a/3rdparty/fsp +++ b/3rdparty/fsp @@ -1 +1 @@ -Subproject commit 59964173e18950debcc6b8856c5c928935ce0b4f +Subproject commit 162719b6cb5efc6c50dcb23dfdb378358573e2b6 diff --git a/3rdparty/libgfxinit b/3rdparty/libgfxinit index a815704..f70edda 160000 --- a/3rdparty/libgfxinit +++ b/3rdparty/libgfxinit @@ -1 +1 @@ -Subproject commit a815704c84b4823f5b723404a37efed9d6c85d66 +Subproject commit f70eddafbc2c6045a14e2f8bbb3273ee738fbaf7 diff --git a/3rdparty/libhwbase b/3rdparty/libhwbase index bd0ed91..637f2a4 160000 --- a/3rdparty/libhwbase +++ b/3rdparty/libhwbase @@ -1 +1 @@ -Subproject commit bd0ed91cb985a697033edd9fd62d322aa017e791 +Subproject commit 637f2a4f21ead8ccc45d5256834eb27ce72088db diff --git a/3rdparty/opensbi b/3rdparty/opensbi index ce228ee..804b997 160000 --- a/3rdparty/opensbi +++ b/3rdparty/opensbi @@ -1 +1 @@ -Subproject commit ce228ee0919deb9957192d723eecc8aaae2697c6 +Subproject commit 804b997ed415001097803e4b537fd63d043874b9 diff --git a/3rdparty/vboot b/3rdparty/vboot index e6700f4..304aa42 160000 --- a/3rdparty/vboot +++ b/3rdparty/vboot @@ -1 +1 @@ -Subproject commit e6700f4c70fe72850ae4f3f5df19c9281ebcefc8 +Subproject commit 304aa429c1a04cda3ab2ce37b9e31af84405bfca diff --git a/util/crossgcc/patches/binutils-2.32_as-ipxe.patch b/util/crossgcc/patches/binutils-2.32_as-ipxe.patch index df95630..4d2c00a 100644 --- a/util/crossgcc/patches/binutils-2.32_as-ipxe.patch +++ b/util/crossgcc/patches/binutils-2.32_as-ipxe.patch @@ -1,5 +1,5 @@ From 6984bd861cc595e56d26ea033851d9174e855129 Mon Sep 17 00:00:00 2001 -From: CunningLearner sahdev.himan@gmail.com +From: Himanshu Sahdev aka CunningLearner sahdev.himan@gmail.com Date: Mon, 26 Aug 2019 16:57:13 +0530 Subject: [PATCH] as: facilitate tests based on /dev/null
diff --git a/util/nvidia/cbootimage b/util/nvidia/cbootimage index 65a6d94..64045f9 160000 --- a/util/nvidia/cbootimage +++ b/util/nvidia/cbootimage @@ -1 +1 @@ -Subproject commit 65a6d94dd5f442578551e0a81ecbe5235e673fd4 +Subproject commit 64045f993c2cd8989838aeaad3d22107d96d5596
Hello Julius Werner,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/35144
to look at the new patch set (#2).
Change subject: util/crossgcc/patches: facilitate successful build of ipxe ......................................................................
util/crossgcc/patches: facilitate successful build of ipxe
New changes in the latest binutils 2.32 leads to assembler errors causes ipxe build failure. IPXE uses the divide test which requires /dev/null as input as well as the output file name.
This patch facilitates the /dev/null as an exception to the current changes in binutils package while building crossgcc for coreboot leads to successful build of ipxe and further tests to pass based on /dev/null and applies automatically during the crossgcc rebuild.
Also, this can be reverted once binutils/ipxe provides update release in this respect.
Change-Id: I111cfae98286baef38bb95b9f30f0382b047ea03 Signed-off-by: Himanshu Sahdev himanshusah@hcl.com --- D 3rdparty/blobs D 3rdparty/fsp D 3rdparty/libgfxinit D 3rdparty/libhwbase D 3rdparty/opensbi D 3rdparty/vboot D util/crossgcc/patches/binutils-2.32_as-ipxe.patch D util/nvidia/cbootimage 8 files changed, 0 insertions(+), 43 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/35144/2
Hello Julius Werner, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/35144
to look at the new patch set (#3).
Change subject: Not for review ......................................................................
Not for review
Change-Id: I111cfae98286baef38bb95b9f30f0382b047ea03 Signed-off-by: Himanshu Sahdev himanshusah@hcl.com --- D 3rdparty/blobs D 3rdparty/fsp D 3rdparty/libgfxinit D 3rdparty/libhwbase D 3rdparty/opensbi D 3rdparty/vboot D util/crossgcc/patches/binutils-2.32_as-ipxe.patch D util/nvidia/cbootimage 8 files changed, 0 insertions(+), 43 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/35144/3
Name of user not set #1002476 has removed Julius Werner from this change. ( https://review.coreboot.org/c/coreboot/+/35144 )
Change subject: Not for review ......................................................................
Removed reviewer Julius Werner.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/35144
to look at the new patch set (#4).
Change subject: Not for review ......................................................................
Not for review
Change-Id: I111cfae98286baef38bb95b9f30f0382b047ea03 Signed-off-by: Himanshu Sahdev himanshusah@hcl.com --- D 3rdparty/blobs D 3rdparty/fsp D 3rdparty/libgfxinit D 3rdparty/libhwbase D 3rdparty/opensbi D 3rdparty/vboot M util/crossgcc/patches/binutils-2.32_as-ipxe.patch D util/nvidia/cbootimage 8 files changed, 1 insertion(+), 8 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/35144/4
Hello Julius Werner, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/35144
to look at the new patch set (#5).
Change subject: Not for review ......................................................................
Not for review
Change-Id: I111cfae98286baef38bb95b9f30f0382b047ea03 Signed-off-by: Himanshu Sahdev himanshusah@hcl.com --- D 3rdparty/blobs D 3rdparty/fsp D 3rdparty/libgfxinit D 3rdparty/libhwbase D 3rdparty/opensbi D 3rdparty/vboot M util/crossgcc/patches/binutils-2.32_as-ipxe.patch D util/nvidia/cbootimage 8 files changed, 0 insertions(+), 42 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/35144/5
Name of user not set #1002476 has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/35144 )
Change subject: Not for review ......................................................................
Abandoned
-
Name of user not set (Code Review)