Attention is currently required from: Maximilian Brune, Philipp Hug.
ron minnich has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81031?usp=email )
Change subject: SiFive Unmatched: add support for spi1 x4 mode
......................................................................
Patch Set 1:
(1 comment)
File src/mainboard/sifive/hifive-unleashed/media.c:
https://review.coreboot.org/c/coreboot/+/81031/comment/6cb90999_c43243c1 :
PS1, Line 55: die("Wrong configuration of MSEL\n");
> did you want to fix the unleashed as well? or was that edit intended on the unmatched?
what was I thinking. Fixed. Thanks.
--
To view, visit https://review.coreboot.org/c/coreboot/+/81031?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ida7f195eb6e4fc85018ceb83cf317595127c4af5
Gerrit-Change-Number: 81031
Gerrit-PatchSet: 1
Gerrit-Owner: ron minnich <rminnich(a)gmail.com>
Gerrit-Reviewer: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Reviewer: Philipp Hug <philipp(a)hug.cx>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Philipp Hug <philipp(a)hug.cx>
Gerrit-Attention: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Comment-Date: Mon, 04 Mar 2024 18:46:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Philipp Hug <philipp(a)hug.cx>
Gerrit-MessageType: comment
Attention is currently required from: Maximilian Brune, ron minnich.
Hello Maximilian Brune, Philipp Hug, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/81031?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified+1 by build bot (Jenkins)
Change subject: SiFive Unmatched: add support for spi1 x4 mode
......................................................................
SiFive Unmatched: add support for spi1 x4 mode
Tested on an unmatched, both SPI1 x1 and x4
work now.
Change-Id: Ida7f195eb6e4fc85018ceb83cf317595127c4af5
Signed-off-by: Ronald G Minnich <rminnich(a)gmail.com>
---
M src/mainboard/sifive/hifive-unmatched/cbfs_spi.c
1 file changed, 25 insertions(+), 11 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/31/81031/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/81031?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ida7f195eb6e4fc85018ceb83cf317595127c4af5
Gerrit-Change-Number: 81031
Gerrit-PatchSet: 2
Gerrit-Owner: ron minnich <rminnich(a)gmail.com>
Gerrit-Reviewer: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Reviewer: Philipp Hug <philipp(a)hug.cx>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Attention: ron minnich <rminnich(a)gmail.com>
Gerrit-MessageType: newpatchset
Felix Singer has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/81073?usp=email )
Change subject: foo
......................................................................
Abandoned
--
To view, visit https://review.coreboot.org/c/coreboot/+/81073?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I945b380934e5a9f993cc9adbd7e1764fc12d5212
Gerrit-Change-Number: 81073
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-MessageType: abandon
Attention is currently required from: Nicholas Chin.
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81073?usp=email )
Change subject: foo
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> For testing, I presume?
Yeah, we wanted to try if one can push to the master branch.
--
To view, visit https://review.coreboot.org/c/coreboot/+/81073?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I945b380934e5a9f993cc9adbd7e1764fc12d5212
Gerrit-Change-Number: 81073
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Mon, 04 Mar 2024 18:27:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Felix Singer.
Nicholas Chin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81073?usp=email )
Change subject: foo
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
For testing, I presume?
--
To view, visit https://review.coreboot.org/c/coreboot/+/81073?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I945b380934e5a9f993cc9adbd7e1764fc12d5212
Gerrit-Change-Number: 81073
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Attention: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Comment-Date: Mon, 04 Mar 2024 18:23:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Ashish Kumar Mishra, Ashish Kumar Mishra, Subrata Banik.
Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80780?usp=email )
Change subject: mb/google/brox: Enable Wake on WLAN for SKU1
......................................................................
Patch Set 1:
(1 comment)
File src/mainboard/google/brox/variants/brox/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/80780/comment/b4371ae4_7111050a :
PS1, Line 273: chip drivers/wifi/generic
: register "wake" = "GPE0_DW0_03"
: register "add_acpi_dma_property" = "true"
: device pci 00.0 on
: probe WIFI_BT WIFI_BT_PCIE
: end
: end
> There's no wake knob in rtd3. Hence added this.
Just to clarify, this means that we need to load two drivers in the kernel for the wifi chip? The generic one to take care of the wake functionality and rtd3 one to take care of resetting the device?
--
To view, visit https://review.coreboot.org/c/coreboot/+/80780?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I04c35da2c9ac57cafdf7f7a35d83ab2e7a05fe4a
Gerrit-Change-Number: 80780
Gerrit-PatchSet: 1
Gerrit-Owner: Ashish Kumar Mishra <ashish.k.mishra(a)intel.com>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Ashish Kumar Mishra <ashish.k.mishra(a)intel.corp-partner.google.com>
Gerrit-Attention: Ashish Kumar Mishra <ashish.k.mishra(a)intel.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Ashish Kumar Mishra <ashish.k.mishra(a)intel.corp-partner.google.com>
Gerrit-Comment-Date: Mon, 04 Mar 2024 18:20:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Shelley Chen <shchen(a)google.com>
Comment-In-Reply-To: Ashish Kumar Mishra <ashish.k.mishra(a)intel.corp-partner.google.com>
Gerrit-MessageType: comment
Attention is currently required from: Maximilian Brune, ron minnich.
Philipp Hug has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81031?usp=email )
Change subject: SiFive Unmatched: add support for spi1 x4 mode
......................................................................
Patch Set 1:
(1 comment)
File src/mainboard/sifive/hifive-unleashed/media.c:
https://review.coreboot.org/c/coreboot/+/81031/comment/fc8cfb38_e6c324d5 :
PS1, Line 55: die("Wrong configuration of MSEL\n");
did you want to fix the unleashed as well? or was that edit intended on the unmatched?
--
To view, visit https://review.coreboot.org/c/coreboot/+/81031?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ida7f195eb6e4fc85018ceb83cf317595127c4af5
Gerrit-Change-Number: 81031
Gerrit-PatchSet: 1
Gerrit-Owner: ron minnich <rminnich(a)gmail.com>
Gerrit-Reviewer: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Reviewer: Philipp Hug <philipp(a)hug.cx>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Attention: ron minnich <rminnich(a)gmail.com>
Gerrit-Comment-Date: Mon, 04 Mar 2024 18:19:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment