Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/47812 )
Change subject: Doc/releases/checklist.md: Fix up URLs
......................................................................
Doc/releases/checklist.md: Fix up URLs
Use angle brackets so that they appear as links, and update a link to a
Gerrit change to use the current format.
Change-Id: I41f82986429dcfd1cbc5b5c088a0c47bd24a57c4
Signed-off-by: Angel Pons <th3fanbus(a)gmail.com>
---
M Documentation/releases/checklist.md
1 file changed, 5 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/47812/1
diff --git a/Documentation/releases/checklist.md b/Documentation/releases/checklist.md
index 1cfb110..d777f0e 100644
--- a/Documentation/releases/checklist.md
+++ b/Documentation/releases/checklist.md
@@ -198,16 +198,16 @@
````
People can now see the release tarballs on the website at
-https://www.coreboot.org/releases/
+<https://www.coreboot.org/releases/>
-The downloads page is the official place to download the releases from, and it needs to be updated with links to the new release tarballs and .sig files. It can be found at https://review.coreboot.org/cgit/homepage.git/tree/downloads.html
+The downloads page is the official place to download the releases from, and it needs to be updated with links to the new release tarballs and .sig files. It can be found at <https://review.coreboot.org/cgit/homepage.git/tree/downloads.html>
-Here is an example commit to change it: https://review.coreboot.org/#/c/19515/
+Here is an example commit to change it: <https://review.coreboot.org/c/homepage/+/19515>
## Upload crossgcc sources
Sometimes the source files for older revisions of
crossgcc disappear. To deal with that we maintain a mirror at
-https://www.coreboot.org/releases/crossgcc-sources/ where we host the
+<https://www.coreboot.org/releases/crossgcc-sources/> where we host the
sources used by the crossgcc scripts that are part of coreboot releases.
Run
@@ -221,7 +221,7 @@
directory on the server.
## After the release is complete
-Post the release notes on https://blogs.coreboot.org
+Post the release notes on <https://blogs.coreboot.org>
## Making a branch
At times we will need to create a branch, generally for patch fixes.
--
To view, visit https://review.coreboot.org/c/coreboot/+/47812
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I41f82986429dcfd1cbc5b5c088a0c47bd24a57c4
Gerrit-Change-Number: 47812
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-MessageType: newchange
Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/47811 )
Change subject: Doc/releases/checklist.md: Add reminder to unpack relnotes
......................................................................
Doc/releases/checklist.md: Add reminder to unpack relnotes
Explicitly add this easy-to-forget step. Also add a missing period.
Change-Id: Iaf13155fcc8a70f3565fb2404cef886524fa5161
Signed-off-by: Angel Pons <th3fanbus(a)gmail.com>
---
M Documentation/releases/checklist.md
1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/11/47811/1
diff --git a/Documentation/releases/checklist.md b/Documentation/releases/checklist.md
index a80fd85..1cfb110 100644
--- a/Documentation/releases/checklist.md
+++ b/Documentation/releases/checklist.md
@@ -75,7 +75,8 @@
- [ ] Test the release from the actual release tarballs.
- [ ] Push signed Tag to repo.
- [ ] Announce that the release tag is done on IRC.
-- [ ] Upload release files to web server
+- [ ] Upload release files to web server.
+- [ ] Also extract the release notes and place them on the web server.
- [ ] Upload crossgcc sources to web server.
- [ ] Update download page to point to files, push to repo.
- [ ] Write and publish blog post with release notes.
--
To view, visit https://review.coreboot.org/c/coreboot/+/47811
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iaf13155fcc8a70f3565fb2404cef886524fa5161
Gerrit-Change-Number: 47811
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-MessageType: newchange
Frans Hendriks has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/47760 )
Change subject: src/drivers/intel/fsp1_1/cache_as_ram.S: Clear _bss area only
......................................................................
src/drivers/intel/fsp1_1/cache_as_ram.S: Clear _bss area only
Whole car region is cleared, while only small part needs to be done.
Clear .bss area only
Tested on Facebook FBG1701
Change-Id: I021c2f7d3531c553015fde98d155915f897b434d
Signed-off-by: Frans Hendriks <fhendriks(a)eltan.com>
---
M src/drivers/intel/fsp1_1/cache_as_ram.S
1 file changed, 8 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/47760/1
diff --git a/src/drivers/intel/fsp1_1/cache_as_ram.S b/src/drivers/intel/fsp1_1/cache_as_ram.S
index 31c3580..e8d5091 100644
--- a/src/drivers/intel/fsp1_1/cache_as_ram.S
+++ b/src/drivers/intel/fsp1_1/cache_as_ram.S
@@ -144,15 +144,16 @@
* mm0: low 32-bits of TSC value
* mm1: high 32-bits of TSC value
*/
-
- /* coreboot assumes stack/heap region will be zero */
+ movl %edx, temp_memory_end
+ movl %ecx, temp_memory_start
+
+ /* clear .bss section */
cld
- movl %ecx, %edi
- neg %ecx
- /* Clear up to Temp Ram top. */
- add %edx, %ecx
+ xor %eax, %eax
+ movl $(_ebss), %ecx
+ movl $(_bss), %edi
+ sub %edi, %ecx
shrl $2, %ecx
- xorl %eax, %eax
rep stosl
/* Need to align stack to 16 bytes at call instruction. Account for
--
To view, visit https://review.coreboot.org/c/coreboot/+/47760
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I021c2f7d3531c553015fde98d155915f897b434d
Gerrit-Change-Number: 47760
Gerrit-PatchSet: 1
Gerrit-Owner: Frans Hendriks <fhendriks(a)eltan.com>
Gerrit-MessageType: newchange
Furquan Shaikh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/47832 )
Change subject: vc/amd/pi/00670F00: Add raw AGESA binary only to COREBOOT CBFS
......................................................................
vc/amd/pi/00670F00: Add raw AGESA binary only to COREBOOT CBFS
If AGESA is added as a raw binary (and not a stage), then cbfstool
does not perform relocation. In this case, it should be added only to
COREBOOT (i.e. default) CBFS since the binary needs to be present only
in one specific location that is present in the default CBFS.
Change-Id: I7a7edc217663f9d1d36b05308bbd35f56a28b9b1
Signed-off-by: Furquan Shaikh <furquan(a)google.com>
---
M src/vendorcode/amd/pi/00670F00/Makefile.inc
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/32/47832/1
diff --git a/src/vendorcode/amd/pi/00670F00/Makefile.inc b/src/vendorcode/amd/pi/00670F00/Makefile.inc
index 46af655..71898a3 100644
--- a/src/vendorcode/amd/pi/00670F00/Makefile.inc
+++ b/src/vendorcode/amd/pi/00670F00/Makefile.inc
@@ -170,6 +170,7 @@
else
$(CONFIG_AGESA_CBFS_NAME)-type := raw
$(CONFIG_AGESA_CBFS_NAME)-position := $(CONFIG_AGESA_BINARY_PI_LOCATION)
+regions-for-file-$(CONFIG_AGESA_CBFS_NAME) = COREBOOT
endif # CONFIG_AGESA_BINARY_PI_AS_STAGE
endif # AGESA_BINARYPI_INPUT_FILE == ""
--
To view, visit https://review.coreboot.org/c/coreboot/+/47832
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7a7edc217663f9d1d36b05308bbd35f56a28b9b1
Gerrit-Change-Number: 47832
Gerrit-PatchSet: 1
Gerrit-Owner: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-MessageType: newchange