Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/27277 )
Change subject: soc/amd/common: Fix AmdLateRunApTask()
......................................................................
Patch Set 5:
(1 comment)
https://review.coreboot.org/#/c/27277/5/src/soc/amd/common/block/pi/agesawr…
File src/soc/amd/common/block/pi/agesawrapper.c:
https://review.coreboot.org/#/c/27277/5/src/soc/amd/common/block/pi/agesawr…
PS5, Line 356: amd_late_run_ap_task
> Just for compatibility with the remaining functions within agesawrapper, even though it's not called […]
That prefix agesawrapper_ was stupid in the first place and followups shall remove the rest. So they shall be symmetrical at the end of the day.
--
To view, visit https://review.coreboot.org/c/coreboot/+/27277
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1dad64b955b53bd19363737665235f95aa3d451e
Gerrit-Change-Number: 27277
Gerrit-PatchSet: 5
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 20 Feb 2019 04:00:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-MessageType: comment
Kyösti Mälkki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31369
Change subject: sifive/hifive-unleashed: Drop unneeded console_tx_flush()
......................................................................
sifive/hifive-unleashed: Drop unneeded console_tx_flush()
Every printk() call already does console_tx_flush()
so there should not be anything in transmit buffer
when we return from console_init().
Change-Id: Iff2927c02d2c8031907620a056782bb014f20162
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
---
M src/mainboard/sifive/hifive-unleashed/romstage.c
1 file changed, 0 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/31369/1
diff --git a/src/mainboard/sifive/hifive-unleashed/romstage.c b/src/mainboard/sifive/hifive-unleashed/romstage.c
index 8277141..0611a6d 100644
--- a/src/mainboard/sifive/hifive-unleashed/romstage.c
+++ b/src/mainboard/sifive/hifive-unleashed/romstage.c
@@ -27,12 +27,6 @@
/* TODO: Follow Section 6.3 (FSBL) of the FU540 manual */
- /*
- * Flush console before changing clock/UART divisor to prevent garbage
- * being printed.
- */
- console_tx_flush();
-
clock_init();
// re-initialize UART
--
To view, visit https://review.coreboot.org/c/coreboot/+/31369
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iff2927c02d2c8031907620a056782bb014f20162
Gerrit-Change-Number: 31369
Gerrit-PatchSet: 1
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-MessageType: newchange
Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/27277 )
Change subject: soc/amd/common: Fix AmdLateRunApTask()
......................................................................
Patch Set 5:
(2 comments)
https://review.coreboot.org/#/c/27277/5/src/soc/amd/common/block/pi/agesawr…
File src/soc/amd/common/block/pi/agesawrapper.c:
https://review.coreboot.org/#/c/27277/5/src/soc/amd/common/block/pi/agesawr…
PS5, Line 356: amd_late_run_ap_task
Just for compatibility with the remaining functions within agesawrapper, even though it's not called by AmdAgesaCallout), I believe this function should be called agesawrapper_amd_late_run_ap_task.
https://review.coreboot.org/#/c/27277/5/src/soc/amd/common/block/pi/def_cal…
File src/soc/amd/common/block/pi/def_callouts.c:
https://review.coreboot.org/#/c/27277/5/src/soc/amd/common/block/pi/def_cal…
PS5, Line 208: amd_late_run_ap_task
agesawrapper_amd_late_run_ap_task?
--
To view, visit https://review.coreboot.org/c/coreboot/+/27277
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1dad64b955b53bd19363737665235f95aa3d451e
Gerrit-Change-Number: 27277
Gerrit-PatchSet: 5
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 20 Feb 2019 00:23:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Felix Singer has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/30870 )
Change subject: Test
......................................................................
Abandoned
--
To view, visit https://review.coreboot.org/c/coreboot/+/30870
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie3fd19b493443df797cea21afbeeeae285b6eac9
Gerrit-Change-Number: 30870
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Singer <migy(a)darmstadt.ccc.de>
Gerrit-MessageType: abandon
Patrick Rudolph has uploaded a new patch set (#15) to the change originally created by Patrick Rudolph. ( https://review.coreboot.org/c/coreboot/+/29667 )
Change subject: mb/emulation/qemu-q35: Add x86_64 support
......................................................................
mb/emulation/qemu-q35: Add x86_64 support
* Enable optional x86_64 romstage, postcar and ramstage
* Add Kconfig for x86_64 compilation
* Add documentation for x86 qemu mainboards
* Increase CAR stack as x86_64 uses more than 0x4000 bytes
* Always setup MTRRs even though qemu doesn't care about them
Working:
* Boots to Linux
* Boots to SeaBios
* Drops to protected mode at end of ramstage
* Enumerates PCI devices
* Relocateable ramstage
Signed-off-by: Patrick Rudolph <siro(a)das-labor.org>
Change-Id: If2f02a95b2f91ab51043d4e81054354f4a6eb5d5
---
M Documentation/arch/x86/index.md
A Documentation/mainboard/emulation/qemu-i440fx.md
A Documentation/mainboard/emulation/qemu-q35.md
M Documentation/mainboard/index.md
M src/arch/x86/Kconfig
M src/cpu/qemu-x86/Kconfig
M src/cpu/qemu-x86/Makefile.inc
M src/mainboard/emulation/qemu-i440fx/Kconfig
M src/mainboard/emulation/qemu-i440fx/romstage.c
M src/mainboard/emulation/qemu-q35/Kconfig
M src/mainboard/emulation/qemu-q35/romstage.c
11 files changed, 211 insertions(+), 20 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/29667/15
--
To view, visit https://review.coreboot.org/c/coreboot/+/29667
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If2f02a95b2f91ab51043d4e81054354f4a6eb5d5
Gerrit-Change-Number: 29667
Gerrit-PatchSet: 15
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: ron minnich <rminnich(a)gmail.com>
Gerrit-CC: Nico Huber <nico.h(a)gmx.de>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset
Patrick Rudolph has uploaded a new patch set (#14) to the change originally created by Patrick Rudolph. ( https://review.coreboot.org/c/coreboot/+/29667 )
Change subject: mb/emulation/qemu-q35: x86_64 support
......................................................................
mb/emulation/qemu-q35: x86_64 support
* Enable optional x86_64 romstage, postcar and ramstage
* Add documentation for x86 qemu mainboards
* Increase CAR stack as x86_64 uses more than 0x4000 bytes
* Always setup MTRRs even though qemu doesn't care about them
Working:
* Boots to Linux
* Drops to protected mode at end of ramstage
* Enumerates PCI devices
* Relocateable ramstage
* postcar stage
Signed-off-by: Patrick Rudolph <siro(a)das-labor.org>
Change-Id: If2f02a95b2f91ab51043d4e81054354f4a6eb5d5
---
M Documentation/arch/x86/index.md
A Documentation/mainboard/emulation/qemu-i440fx.md
A Documentation/mainboard/emulation/qemu-q35.md
M Documentation/mainboard/index.md
M src/arch/x86/Kconfig
M src/cpu/qemu-x86/Kconfig
M src/cpu/qemu-x86/Makefile.inc
M src/mainboard/emulation/qemu-i440fx/Kconfig
M src/mainboard/emulation/qemu-i440fx/romstage.c
M src/mainboard/emulation/qemu-q35/Kconfig
M src/mainboard/emulation/qemu-q35/romstage.c
11 files changed, 211 insertions(+), 20 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/29667/14
--
To view, visit https://review.coreboot.org/c/coreboot/+/29667
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If2f02a95b2f91ab51043d4e81054354f4a6eb5d5
Gerrit-Change-Number: 29667
Gerrit-PatchSet: 14
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: ron minnich <rminnich(a)gmail.com>
Gerrit-CC: Nico Huber <nico.h(a)gmx.de>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset