[XS] Change in coreboot[main]: mb/google/brox: Select USE_UNIFIED_AP_FIRMWARE_FOR_UFS_AND_NON_UFS - coreboot-gerrit

14 Mar 2024


      Attention is currently required from: Ashish Kumar Mishra, Karthik Ramasubramanian.
Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81224?usp=email )
Change subject: mb/google/brox: Select USE_UNIFIED_AP_FIRMWARE_FOR_UFS_AND_NON_UFS
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
...
No change required.
So I tried applying this patch and was still getting failures on suspend_stress_test.  I was never able to successfully getting through a call to `suspend_stress_test -c 10`.  Is Intel seeing any suspend failures with this fix?  Although it does seem like it's an EC interrupt that is causing the premature wake.
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/81224?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: Iabd0b3a83aa386e09310b671632368807a4018d4
Gerrit-Change-Number: 81224
Gerrit-PatchSet: 2
Gerrit-Owner: Ashish Kumar Mishra ashish.k.mishra@intel.com
Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com
Gerrit-Reviewer: Shelley Chen shchen@google.com
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Attention: Ashish Kumar Mishra ashish.k.mishra@intel.com
Gerrit-Attention: Karthik Ramasubramanian kramasub@google.com
Gerrit-Comment-Date: Thu, 14 Mar 2024 18:57:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Shelley Chen shchen@google.com
Comment-In-Reply-To: Ashish Kumar Mishra ashish.k.mishra@intel.com
Gerrit-MessageType: comment