Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43902 )
Change subject: [NOTFORMERGE]3rdparty/blobs: Update submodule pointer ......................................................................
[NOTFORMERGE]3rdparty/blobs: Update submodule pointer
Use microcode for Intel family 6, model 0x37, stepping 8 from 3rdparty/intel-microcode as this a more recent version.
Change-Id: Id886c5d71cc8b8a4f2caa9bea847cf6cfba5660b Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M 3rdparty/blobs M src/soc/intel/baytrail/Makefile.inc 2 files changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/02/43902/1
diff --git a/3rdparty/blobs b/3rdparty/blobs index bbe5d99..2fbe86c 160000 --- a/3rdparty/blobs +++ b/3rdparty/blobs @@ -1 +1 @@ -Subproject commit bbe5d99780d2d085e92d9bae2c0f7b6787419d72 +Subproject commit 2fbe86cf5e298ba189b418196ec1c522d6dbc49a diff --git a/src/soc/intel/baytrail/Makefile.inc b/src/soc/intel/baytrail/Makefile.inc index 451f29e..25cd261 100644 --- a/src/soc/intel/baytrail/Makefile.inc +++ b/src/soc/intel/baytrail/Makefile.inc @@ -62,7 +62,7 @@ postcar-y += ../../../cpu/intel/car/non-evict/exit_car.S
cpu_microcode_bins += 3rdparty/blobs/soc/intel/baytrail/microcode.bin \ - 3rdparty/intel-microcode/intel-ucode/06-37-09 + 3rdparty/intel-microcode/intel-ucode/06-37-*
CPPFLAGS_common += -Isrc/soc/intel/baytrail/include
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43902 )
Change subject: [NOTFORMERGE]3rdparty/blobs: Update submodule pointer ......................................................................
Patch Set 1: Code-Review-2
-2 until blob repo is updated to get proper git hash.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43902 )
Change subject: [NOTFORMERGE]3rdparty/blobs: Update submodule pointer ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/43902/1/src/soc/intel/baytrail/Make... File src/soc/intel/baytrail/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/43902/1/src/soc/intel/baytrail/Make... PS1, Line 65: 06-37-09 FWIW this update must have failed since the microcode in 3rdparty/blobs is 0 terminated for some reason.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43902 )
Change subject: [NOTFORMERGE]3rdparty/blobs: Update submodule pointer ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
https://review.coreboot.org/c/coreboot/+/43902/1/src/soc/intel/baytrail/Make... File src/soc/intel/baytrail/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/43902/1/src/soc/intel/baytrail/Make... PS1, Line 65: 06-37-09
FWIW this update must have failed since the microcode in 3rdparty/blobs is 0 terminated for some rea […]
Weird.
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Angel Pons, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/43902
to look at the new patch set (#2).
Change subject: 3rdparty/blobs: Update submodule pointer ......................................................................
3rdparty/blobs: Update submodule pointer
Use microcode for Intel family 6, model 0x37, stepping 8 from 3rdparty/intel-microcode as this a more recent version.
This also fixes updating Intel family 6, model 0x37, stepping 9 microcode as the microcode in 3rdparty/blobs was terminated with zeros which confuses the microcode update code.
Change-Id: Id886c5d71cc8b8a4f2caa9bea847cf6cfba5660b Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M 3rdparty/blobs M src/soc/intel/baytrail/Makefile.inc 2 files changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/02/43902/2
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43902 )
Change subject: 3rdparty/blobs: Update submodule pointer ......................................................................
Patch Set 2: -Code-Review
Ready now.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43902 )
Change subject: 3rdparty/blobs: Update submodule pointer ......................................................................
Patch Set 2: Code-Review+1
Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/43902?usp=email )
Change subject: 3rdparty/blobs: Update submodule pointer ......................................................................
Abandoned
No longer needed. https://review.coreboot.org/76946
-
Angel Pons (Code Review) -
Arthur Heymans (Code Review)
-
Martin L Roth (Code Review)