Attention is currently required from: Nikolai Artemiev, Stefan Reinauer, Victor Lim.
Anastasia Klimchuk has posted comments on this change by Victor Lim. ( https://review.coreboot.org/c/flashrom/+/83012?usp=email )
Change subject: flashchips: Add support for chip model GD25LQ128E/GD25LB128E/GD25LR128E/GD25LQ128D/GD25LQ128C ......................................................................
Patch Set 1:
(3 comments)
Patchset:
PS1: Victor thank you, the patch looks good!
I only have comments about commit message. You can modify the commit message in Gerrit, there is an Edit button below commit message box.
Also just to check, you have tested the models you are adding, is that right? Do you have the link to datasheet for GD25LR128E ? Thank you!
Commit Message:
https://review.coreboot.org/c/flashrom/+/83012/comment/7e6ffec5_8d7577c3?usp... : PS1, Line 7: GD25LQ128E/GD25LB128E/GD25LR128E/GD25LQ128D/GD25LQ128C You can replace this list with GD25LB128E/GD25LR128E , because you add two models, and the others were here before. So commit title will be
flashchips: Add support for chip model GD25LB128E/GD25LR128E
https://review.coreboot.org/c/flashrom/+/83012/comment/253fbd98_60bf60da?usp... : PS1, Line 9: Adding GD25LQ128E/GD25LB128E/GD25LR128E/GD25LQ128D/GD25LQ128C to flashchip.c : : These part # sharing the same ID, form, fit, function, produced with different process node: 1.8V 128Mbit You need to wrap the commit message to be 72 chars max width. (links to datasheets are exception, leave them long)
Keep the same text, but break the lines so that each line no longer than 72 chars.