Attention is currently required from: Angel Pons, Anastasia Klimchuk.
Alex Mikhalev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/70140 )
Change subject: flashchips: Add support for IS25WP016
......................................................................
Patch Set 4:
(2 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/70140/comment/25e3ea33_be5ee7ea
PS2, Line 9: Tested using `linux_spi` programmer on Raspberry Pi CM4.
> Please test reading, writing and erasing.
Done
https://review.coreboot.org/c/flashrom/+/70140/comment/06dfc242_1ccb091d
PS2, Line 11: Datasheet: https://www.issi.com/WW/pdf/25LP-WP016D.pdf
> Vendors often remove datasheets from their websites. I suggest using a caching service, e.g. […]
Done
--
To view, visit https://review.coreboot.org/c/flashrom/+/70140
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I2b8caea229ffda72f1b04183c31715faccb64ad5
Gerrit-Change-Number: 70140
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Mikhalev <alex(a)corvus-robotics.com>
Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Comment-Date: Fri, 19 May 2023 19:49:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-MessageType: comment
Attention is currently required from: Angel Pons, Anastasia Klimchuk.
Alex Mikhalev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/70140 )
Change subject: flashchips: Add support for IS25WP016
......................................................................
Patch Set 4:
(1 comment)
Patchset:
PS2:
> Hello Alex, sorry for delay: are you still here? (hopefully :) ) […]
I addressed the comments and fixed the merge conflicts
--
To view, visit https://review.coreboot.org/c/flashrom/+/70140
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I2b8caea229ffda72f1b04183c31715faccb64ad5
Gerrit-Change-Number: 70140
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Mikhalev <alex(a)corvus-robotics.com>
Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Comment-Date: Fri, 19 May 2023 19:48:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-MessageType: comment
Attention is currently required from: Angel Pons, Alex Mikhalev.
Hello build bot (Jenkins), Angel Pons, Anastasia Klimchuk,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/70140
to look at the new patch set (#4).
Change subject: flashchips: Add support for IS25WP016
......................................................................
flashchips: Add support for IS25WP016
Tested reading, writing and erasing using `linux_spi` programmer on Raspberry
Pi CM4.
Datasheet: https://web.archive.org/web/20221129211027/https://www.issi.com/WW/pdf/25LP…
Change-Id: I2b8caea229ffda72f1b04183c31715faccb64ad5
Signed-off-by: Alex Mikhalev <alex(a)corvus-robotics.com>
---
M flashchips.c
M include/flashchips.h
2 files changed, 58 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/40/70140/4
--
To view, visit https://review.coreboot.org/c/flashrom/+/70140
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I2b8caea229ffda72f1b04183c31715faccb64ad5
Gerrit-Change-Number: 70140
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Mikhalev <alex(a)corvus-robotics.com>
Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Alex Mikhalev <alex(a)corvus-robotics.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Simon Buhrow, Thomas Heijligen, Aarya, Anastasia Klimchuk.
Hello build bot (Jenkins), Simon Buhrow, Thomas Heijligen, Edward O'Callaghan, Aarya,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/67535
to look at the new patch set (#7).
Change subject: [WIP] tests: Unit tests for erase function selection algo
......................................................................
[WIP] tests: Unit tests for erase function selection algo
Few things left to do , so I marked WIP.
But I am working to complete this test and merge.
Change-Id: I8f3fdefb76e71f6f8dc295d9dead5f38642aace7
Signed-off-by: Anastasia Klimchuk <aklm(a)chromium.org>
---
A tests/erase_func_algo.c
M tests/meson.build
M tests/tests.c
M tests/tests.h
4 files changed, 566 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/35/67535/7
--
To view, visit https://review.coreboot.org/c/flashrom/+/67535
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I8f3fdefb76e71f6f8dc295d9dead5f38642aace7
Gerrit-Change-Number: 67535
Gerrit-PatchSet: 7
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Aarya <aarya.chaumal(a)gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org>
Gerrit-Reviewer: Simon Buhrow
Gerrit-Reviewer: Thomas Heijligen <src(a)posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-CC: Nikolai Artemiev <nartemiev(a)google.com>
Gerrit-Attention: Simon Buhrow
Gerrit-Attention: Thomas Heijligen <src(a)posteo.de>
Gerrit-Attention: Aarya <aarya.chaumal(a)gmail.com>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Simon Buhrow, Thomas Heijligen, Aarya, Anastasia Klimchuk.
Hello build bot (Jenkins), Simon Buhrow, Thomas Heijligen, Edward O'Callaghan, Aarya,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/67535
to look at the new patch set (#6).
Change subject: [WIP] tests: Unit tests for erase function selection algo
......................................................................
[WIP] tests: Unit tests for erase function selection algo
Few things left to do , so I marked WIP.
But I am working to complete this test and merge.
Change-Id: I8f3fdefb76e71f6f8dc295d9dead5f38642aace7
Signed-off-by: Anastasia Klimchuk <aklm(a)chromium.org>
---
A tests/erase_func_algo.c
M tests/meson.build
M tests/tests.c
M tests/tests.h
4 files changed, 560 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/35/67535/6
--
To view, visit https://review.coreboot.org/c/flashrom/+/67535
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I8f3fdefb76e71f6f8dc295d9dead5f38642aace7
Gerrit-Change-Number: 67535
Gerrit-PatchSet: 6
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Aarya <aarya.chaumal(a)gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org>
Gerrit-Reviewer: Simon Buhrow
Gerrit-Reviewer: Thomas Heijligen <src(a)posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-CC: Nikolai Artemiev <nartemiev(a)google.com>
Gerrit-Attention: Simon Buhrow
Gerrit-Attention: Thomas Heijligen <src(a)posteo.de>
Gerrit-Attention: Aarya <aarya.chaumal(a)gmail.com>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Martin L Roth, Anastasia Klimchuk.
Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/74954 )
Change subject: flashrom MAINTAINERS: Add Martin Roth as reviewer for AMD SPI
......................................................................
Patch Set 1:
(1 comment)
File MAINTAINERS:
https://review.coreboot.org/c/flashrom/+/74954/comment/8124f4db_f04665c1
PS1, Line 133: gaumless(a)tutanota.com
> Just to double-check, this is not the email you signed-off the patch with? is this WAI?
Yeah, I've got a few addresses right now. The tutanota account is my personal account that I'm slowly switching to.
--
To view, visit https://review.coreboot.org/c/flashrom/+/74954
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I27a07be1549ef070ad72b8e657d72170c7e85620
Gerrit-Change-Number: 74954
Gerrit-PatchSet: 1
Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin Roth <martin.roth(a)amd.corp-partner.google.com>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Comment-Date: Fri, 19 May 2023 08:22:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-MessageType: comment