Attention is currently required from: Giammarco.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/76765?usp=email )
Change subject: flashchips.c: Add support for BY25D40A ......................................................................
Patch Set 1:
(3 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/76765/comment/a0b97175_6491452d : PS1, Line 14: You need to add testing info into commit message, which operations you ran on the chip. From the chip definition, looks like you did probing. Keep the rest of commit message, it's all fine. Just add testing info.
It would be ideal if you share the logs from successful runs, can you upload logs to https://paste.flashrom.org/ ? and add the link in a comment?
https://review.coreboot.org/c/flashrom/+/76765/comment/6176ea4e_a6e4464e : PS1, Line 16: Signed-off-by: Giammarco Marzano stich86@gmail.com Did you add this line manually? Because when you ask git to do this, it adds <> around email. You need to have:
Signed-off-by: Giammarco Marzano stich86@gmail.com
You can either correct manually, or next time you amend your commit, ask git to sign-off `git commit --amend -s` If you ask git, it will add new line, and then you need to remove this current line.
Patchset:
PS1: Giammarco, thank you for the patch! I have just two comments.
Also thanks for adding link to pull request, from it I discovered that the links in auto-response are broken. I will fix them, but I am glad you found guidelines anyway ;)