Furquan Shaikh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43281 )
Change subject: soc/amd/picasso: Drop the inclusion of entry16.ld and reset16.ld
......................................................................
soc/amd/picasso: Drop the inclusion of entry16.ld and reset16.ld
This change drops the inclusion of entry16.ld and reset16.ld instead
adds the content of those files directly in memlayout.ld in
picasso. This is done to allow the work for top-aligning bootblock to
happen independent of Picasso layout. Once that is complete, Picasso
layout can be re-evaluated to see if it can make use of the common
bootblock linker file includes.
TEST=Verified that coreboot.rom generated using --timeless is the same
with and without this change for trembyle.
Change-Id: Ib1218b24a06d0f69b856fb21458a6183fd21fcbc
Signed-off-by: Furquan Shaikh <furquan(a)google.com>
---
M src/soc/amd/picasso/memlayout.ld
1 file changed, 16 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/43281/1
diff --git a/src/soc/amd/picasso/memlayout.ld b/src/soc/amd/picasso/memlayout.ld
index 6f43ba1..b7f9167 100644
--- a/src/soc/amd/picasso/memlayout.ld
+++ b/src/soc/amd/picasso/memlayout.ld
@@ -95,7 +95,20 @@
}
#if ENV_BOOTBLOCK
-/* Bootblock specific scripts which provide more SECTION directives. */
-#include <cpu/x86/16bit/entry16.ld>
-#include <cpu/x86/16bit/reset16.ld>
+
+gdtptr16_offset = gdtptr16 & 0xffff;
+nullidt_offset = nullidt & 0xffff;
+
+SECTIONS {
+ /* Trigger an error if I have an unuseable start address */
+ _TOO_LOW = CONFIG_X86_RESET_VECTOR - 0xfff0;
+ _bogus = ASSERT(_start16bit >= _TOO_LOW, "_start16bit too low. Please report.");
+
+ . = CONFIG_X86_RESET_VECTOR;
+ .reset . : {
+ *(.reset);
+ . = 15;
+ BYTE(0x00);
+ }
+}
#endif /* ENV_BOOTBLOCK */
--
To view, visit https://review.coreboot.org/c/coreboot/+/43281
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib1218b24a06d0f69b856fb21458a6183fd21fcbc
Gerrit-Change-Number: 43281
Gerrit-PatchSet: 1
Gerrit-Owner: Furquan Shaikh <furquan(a)google.com>
Gerrit-MessageType: newchange
Hello Shelley Chen, build bot (Jenkins), Furquan Shaikh, Tim Wawrzynczak, Nick Vaccaro,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/46620
to look at the new patch set (#4).
Change subject: mb/google/poppy/v/atlas: Reset bluetooth during boot
......................................................................
mb/google/poppy/v/atlas: Reset bluetooth during boot
Bluetooth is part of the Thunder Peak 2 module. This is a USB device
that is not connected to any system level reset signals. Reset is done
via a GPIO. When the SoC reboots, we need to reset Bluetooth so the
device is in a known good state. BT_DISABLE_L needs to be asserted for
a fairly long time - longer than the gap between early GPIO and main
GPIO initialization (about 170ms). So, the recently added late GPIO
init step is used to provide the necessary delay before taking the BT
chip out of reset.
BRANCH=atlas
BUG=b:168757348
TEST=Verified that we successfully reset the bluetooth chip. The quick
way to check if the chip was reset is to look for dmesg
entries from btusb_setup_intel_new indicating that firmware
was downloaded right after rebooting.
Change-Id: I4c489963f7a353e8fc5f55b6cba7aaee0b347a37
Signed-off-by: Caveh Jalali <caveh(a)chromium.org>
---
M src/mainboard/google/poppy/variants/atlas/gpio.c
1 file changed, 23 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/20/46620/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/46620
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4c489963f7a353e8fc5f55b6cba7aaee0b347a37
Gerrit-Change-Number: 46620
Gerrit-PatchSet: 4
Gerrit-Owner: Caveh Jalali <caveh(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset
Caveh Jalali has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46619 )
Change subject: mb/google/poppy: Add late_gpio support
......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46619/1//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/46619/1//COMMIT_MSG@18
PS1, Line 18: TEST=tested with patch stack to verify that we successfully reset the
> I assume you'll want to cherry-pick this into firmware-poppy-10431. […]
atlas has its own firmware branch.
--
To view, visit https://review.coreboot.org/c/coreboot/+/46619
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I847f90275740ca825e4c93e6b8d5650143228476
Gerrit-Change-Number: 46619
Gerrit-PatchSet: 3
Gerrit-Owner: Caveh Jalali <caveh(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Tue, 17 Nov 2020 02:04:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-MessageType: comment
Hello Shelley Chen, build bot (Jenkins), Furquan Shaikh, Tim Wawrzynczak, Nick Vaccaro,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/46619
to look at the new patch set (#3).
Change subject: mb/google/poppy: Add late_gpio support
......................................................................
mb/google/poppy: Add late_gpio support
This adds support for configuring GPIOs late in the boot process. When
GPIOs are used for reset sequencing, long delays may be necessary. We
can achieve these long delays by adding a step to control GPIOs just
before we hand off to a payload. On an m3 atlas, this provides about a
320ms delay between the romstage GPIO initialization and the late
stage GPIO initialization. For reference, the delay between romstage
GPIO initialization and the main GPIO initialization is about 170ms.
BRANCH=atlas
BUG=b:168757348
TEST=tested with patch stack to verify that we successfully reset the
bluetooth chip.
Change-Id: I847f90275740ca825e4c93e6b8d5650143228476
Signed-off-by: Caveh Jalali <caveh(a)chromium.org>
---
M src/mainboard/google/poppy/variants/baseboard/gpio.c
M src/mainboard/google/poppy/variants/baseboard/include/baseboard/variants.h
2 files changed, 19 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/19/46619/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/46619
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I847f90275740ca825e4c93e6b8d5650143228476
Gerrit-Change-Number: 46619
Gerrit-PatchSet: 3
Gerrit-Owner: Caveh Jalali <caveh(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset
Caveh Jalali has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47654 )
Change subject: mb/google/volteer: Use SoC header for TCSS port control
......................................................................
Patch Set 1:
This change is ready for review.
--
To view, visit https://review.coreboot.org/c/coreboot/+/47654
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9ded34ad2316764ce42851b3d9b631ed5a30860e
Gerrit-Change-Number: 47654
Gerrit-PatchSet: 1
Gerrit-Owner: Caveh Jalali <caveh(a)chromium.org>
Gerrit-Reviewer: Brandon Breitenstein <brandon.breitenstein(a)intel.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Comment-Date: Tue, 17 Nov 2020 02:01:29 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment