Attention is currently required from: Nikolai Artemiev, Ssunk, Stefan Reinauer.
Anastasia Klimchuk has posted comments on this change by Ssunk. ( https://review.coreboot.org/c/flashrom/+/83243?usp=email )
Change subject: flashchips: Add Support for XMC XM25QU512C/XM25QU512D
......................................................................
Patch Set 4: Code-Review+2
(2 comments)
Patchset:
PS4:
Wait until the tag v1.4.0 done and after that the patch can be submitted. ETA 26 July.
https://mail.coreboot.org/hyperkitty/list/flashrom@flashrom.org/thread/UUDQ7...
I am leaving the comment unresolved, so that not to accidentally submit before the tag.
File flashchips.c:
https://review.coreboot.org/c/flashrom/+/83243/comment/967f7302_f4987e50?usp... :
PS2, Line 22495: FEATURE_4BA_ENTER_WREN
Since 256Mb or more requires 4 bytes for addressing, the enter/exit 4-byte address mode operation is […]
Thanks so much for the explanation! I understand now, all good.
--
To view, visit
https://review.coreboot.org/c/flashrom/+/83243?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I251c66b5d3b4fc94242c2c9d6b7c0f03c1bd7d0b
Gerrit-Change-Number: 83243
Gerrit-PatchSet: 4
Gerrit-Owner: Ssunk
ssunkkan@gmail.com
Gerrit-Reviewer: Anastasia Klimchuk
aklm@chromium.org
Gerrit-Reviewer: Nikolai Artemiev
nartemiev@google.com
Gerrit-Reviewer: Stefan Reinauer
stefan.reinauer@coreboot.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Stefan Reinauer
stefan.reinauer@coreboot.org
Gerrit-Attention: Ssunk
ssunkkan@gmail.com
Gerrit-Attention: Nikolai Artemiev
nartemiev@google.com
Gerrit-Comment-Date: Sat, 20 Jul 2024 08:29:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Ssunk
ssunkkan@gmail.com
Comment-In-Reply-To: Anastasia Klimchuk
aklm@chromium.org