Change in flashrom[master]: ft2232_spi.c: change the chunksize to 280

List overview All Threads
Download

newer

older

Change in flashrom[master]: os.h:...

Change in flashrom[master]:...

Simon Buhrow (Code Review)

8 Jul 2020 8 Jul '20

7:21 a.m.

Simon Buhrow has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

ft2232_spi.c: change the chunksize to 280

Signed-off-by: Simon Buhrow simon.buhrow@posteo.de Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a --- M ft2232_spi.c 1 file changed, 12 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/82/43282/1

diff --git a/ft2232_spi.c b/ft2232_spi.c index 0799af4..0a8d631 100644 --- a/ft2232_spi.c +++ b/ft2232_spi.c @@ -398,7 +398,18 @@ msg_perr("Unable to set latency timer (%s).\n", ftdi_get_error_string(ftdic)); }

- if (ftdi_write_data_set_chunksize(ftdic, 270)) { + if (ftdi_write_data_set_chunksize(ftdic, 280)) { + /* + * 280 bytes = + * + 9 B (CMD) + * + 1 B (WREN) + * + 9 B (CMD) + * + 1 B (op) + * + 4 B (addr) + * + 256 B (page data) + * + * With op: PageProgram or Erase; CMD: FTDI-Chip commands + */ msg_perr("Unable to set chunk size (%s).\n", ftdi_get_error_string(ftdic)); }

Attachments:

attachment.htm (text/html — 3.1 KB)

Show replies by date

Simon Buhrow (Code Review)

8 Jul 8 Jul

7:26 a.m.

Hello build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/flashrom/+/43282

to look at the new patch set (#2).

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

ft2232_spi.c: change the chunksize to 280

This is necessary for change 40477 (time saving patch).

/* * 280 bytes = * + 9 B (CMD) * + 1 B (WREN) * + 9 B (CMD) * + 1 B (op) * + 4 B (addr) * + 256 B (page data) * * With op: PageProgram or Erase; CMD: FTDI-Chip commands */

Signed-off-by: Simon Buhrow simon.buhrow@posteo.de Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a --- M ft2232_spi.c 1 file changed, 12 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/82/43282/2

Simon Buhrow (Code Review)

8:04 a.m.

Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

Patch Set 2:

The value of 280 is the minimum size needed to realize less ftdi_write_data() calls in patch 40477.

Nevertheless a value of 4096 would be reasonable as well. In fact I can´t see any real need to set chunksize smaller than the default value (here: default equals to max) of the ftdi chunksize which equals to the ftdi buffer size (fix hardware buffer!).

A value of 4096 should result in the same behavior. However 280 is like having (/showing to have) some more control and knowledge about what happens.

I do not prefer any of the options. As there was possibly a reason to set chunksize smaller than the default value I chose 280.

Simon Buhrow (Code Review)

8:06 a.m.

Hello build bot (Jenkins), Angel Pons,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/flashrom/+/43282

to look at the new patch set (#3).

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

ft2232_spi.c: change the chunksize to 280

This is necessary for change 40477 (time saving patch).

280 bytes = + 9 B (CMD) + 1 B (WREN) + 9 B (CMD) + 1 B (op) + 4 B (addr) + 256 B (page data)

With op: PageProgram or Erase; CMD: FTDI-Chip commands

Signed-off-by: Simon Buhrow simon.buhrow@posteo.de Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a --- M ft2232_spi.c 1 file changed, 12 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/82/43282/3

Angel Pons (Code Review)

9:39 a.m.

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

Patch Set 3: Code-Review+2

(2 comments)

...

Patch Set 2:

The value of 280 is the minimum size needed to realize less ftdi_write_data() calls in patch 40477.

Nevertheless a value of 4096 would be reasonable as well. In fact I can´t see any real need to set chunksize smaller than the default value (here: default equals to max) of the ftdi chunksize which equals to the ftdi buffer size (fix hardware buffer!).

A value of 4096 should result in the same behavior. However 280 is like having (/showing to have) some more control and knowledge about what happens.

I do not prefer any of the options. As there was possibly a reason to set chunksize smaller than the default value I chose 280.

I'm not sure if there's any particular reason to reduce the default bufsize.

https://review.coreboot.org/c/flashrom/+/43282/3//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/flashrom/+/43282/3//COMMIT_MSG@9 PS3, Line 9: change 40477 CB:40477

https://review.coreboot.org/c/flashrom/+/43282/3//COMMIT_MSG@20 PS3, Line 20: two blank lines, remove one?

Simon Buhrow (Code Review)

11:56 a.m.

Hello build bot (Jenkins), Angel Pons,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/flashrom/+/43282

to look at the new patch set (#4).

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

ft2232_spi.c: change the chunksize to 280

This is necessary for change CB:40477 (time saving patch).

280 bytes = + 9 B (CMD) + 1 B (WREN) + 9 B (CMD) + 1 B (op) + 4 B (addr) + 256 B (page data)

With op: PageProgram or Erase; CMD: FTDI-Chip commands

Signed-off-by: Simon Buhrow simon.buhrow@posteo.de Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a --- M ft2232_spi.c 1 file changed, 12 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/82/43282/4

Nico Huber (Code Review)

10 Nov 10 Nov

11:58 a.m.

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

Patch Set 4:

Too bad these things aren't well documented. Maybe the original FTDI library has some hint why one can set the size at all?

If we keep the chunksize setting, I would make this part of the write command. e.g. keep a global variable with the last set chunksize and then try to increase it everytime we need something higher. Or the other way around, #define a maximum and then check that this maximum isn't exceeded in the com- mand processing.

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 4 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Tue, 10 Nov 2020 11:58:48 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment

Simon Buhrow (Code Review)

12 Nov 12 Nov

12:37 p.m.

Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

Patch Set 4:

I checked the libftdi again and could not find any hint why this should be used. Furhtermore I checked D2XX_Programmer's_Guide from FTDI and could not find any such function (https://www.ftdichip.com/Support/Documents/ProgramGuides/D2XX_Programmer%27s...).

So may be the easiest and cleanest way is to set it to 4096 or to not set it at all (and use the libfdti default which is again 4096)?

Is there any argument to stay with setting the chunksize lower than 4096? (again as mentioned before: I chose 280 as there might has been any reason to set it lower than 4096. But if no one has an idea why, I would opt to set it to 4096).

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 4 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Thu, 12 Nov 2020 12:37:51 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment

Edward O'Callaghan (Code Review)

20 Apr 20 Apr

2:29 a.m.

Attention is currently required from: Simon Buhrow. Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

Patch Set 4:

(1 comment)

File ft2232_spi.c:

https://review.coreboot.org/c/flashrom/+/43282/comment/339d3e23_787f01a5 PS4, Line 401: 280) should this be a define up at the top of the file with the comment above as the rational for the value?

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 4 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Edward O'Callaghan quasisec@chromium.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@mailbox.org Gerrit-Attention: Simon Buhrow Gerrit-Comment-Date: Tue, 20 Apr 2021 02:29:02 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

Angel Pons (Code Review)

8:28 a.m.

Attention is currently required from: Simon Buhrow, Edward O'Callaghan. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

Patch Set 4: Code-Review+1

(2 comments)

Commit Message:

https://review.coreboot.org/c/flashrom/+/43282/comment/cfa1a450_5be3f683 PS4, Line 9: This is necessary for change CB:40477 (time saving patch). Does CB:40477 require this change? If so, please reorder the commits. You can reorder the last two commits using `git rebase`: https://stackoverflow.com/a/49452944

File ft2232_spi.c:

https://review.coreboot.org/c/flashrom/+/43282/comment/5c38a5fe_ba13c8cc PS4, Line 401: 280)

...

should this be a define up at the top of the file with the comment above as the rational for the val […]

Good idea. The define could even be spelled out:

#define FTDI_CHUNK_SIZE (9 + 1 + 9 + 1 + 4 + 256)

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 4 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Edward O'Callaghan quasisec@chromium.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@mailbox.org Gerrit-Attention: Simon Buhrow Gerrit-Attention: Edward O'Callaghan quasisec@chromium.org Gerrit-Comment-Date: Tue, 20 Apr 2021 08:28:33 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: Edward O'Callaghan quasisec@chromium.org Gerrit-MessageType: comment

Simon Buhrow (Code Review)

23 Apr 23 Apr

11:38 a.m.

Attention is currently required from: Edward O'Callaghan, Angel Pons. Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

Patch Set 4:

(1 comment)

File ft2232_spi.c:

https://review.coreboot.org/c/flashrom/+/43282/comment/cac5a1f3_ec0f387f PS4, Line 401: 280)

...

Good idea. The define could even be spelled out: […]

Yepp, good idea!

However this could become obsolet. see patch 40477

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 4 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Edward O'Callaghan quasisec@chromium.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@mailbox.org Gerrit-Attention: Edward O'Callaghan quasisec@chromium.org Gerrit-Attention: Angel Pons th3fanbus@gmail.com Gerrit-Comment-Date: Fri, 23 Apr 2021 11:38:34 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Edward O'Callaghan quasisec@chromium.org Comment-In-Reply-To: Angel Pons th3fanbus@gmail.com Gerrit-MessageType: comment

Simon Buhrow (Code Review)

29 Apr 29 Apr

11:35 a.m.

Attention is currently required from: Edward O'Callaghan, Angel Pons. Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 280 ......................................................................

Patch Set 4:

(1 comment)

Patchset:

PS4: Sorry, I was wrong: It is not getting obsolete but important because of https://review.coreboot.org/c/flashrom/+/40477

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 4 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Edward O'Callaghan quasisec@chromium.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@mailbox.org Gerrit-Attention: Edward O'Callaghan quasisec@chromium.org Gerrit-Attention: Angel Pons th3fanbus@gmail.com Gerrit-Comment-Date: Thu, 29 Apr 2021 11:35:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

Simon Buhrow (Code Review)

11:36 a.m.

New subject: Change in flashrom[master]: ft2232_spi.c: change the chunksize to 4096

Attention is currently required from: Edward O'Callaghan, Angel Pons. Hello build bot (Jenkins), Angel Pons,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/flashrom/+/43282

to look at the new patch set (#5).

Change subject: ft2232_spi.c: change the chunksize to 4096 ......................................................................

ft2232_spi.c: change the chunksize to 4096

4096 Bytes is the hardware buffer size of the FT2232 chip.

This is necessary to get full benefit of time saving patch https://review.coreboot.org/c/flashrom/+/40477

Signed-off-by: Simon Buhrow simon.buhrow@posteo.de Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a --- M ft2232_spi.c 1 file changed, 12 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/82/43282/5

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 5 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Edward O'Callaghan quasisec@chromium.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@mailbox.org Gerrit-Attention: Edward O'Callaghan quasisec@chromium.org Gerrit-Attention: Angel Pons th3fanbus@gmail.com Gerrit-MessageType: newpatchset

Simon Buhrow (Code Review)

12:08 p.m.

New subject: Change in flashrom[master]: ft2232_spi.c: change the chunksize to 4096

Attention is currently required from: Edward O'Callaghan, Angel Pons. Hello build bot (Jenkins), Angel Pons,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/flashrom/+/43282

to look at the new patch set (#6).

Change subject: ft2232_spi.c: change the chunksize to 4096 ......................................................................

ft2232_spi.c: change the chunksize to 4096

4096 Bytes is the hardware buffer size of the FT2232 chip.

This is necessary to get full benefit of time saving patch https://review.coreboot.org/c/flashrom/+/40477

Signed-off-by: Simon Buhrow simon.buhrow@posteo.de Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a --- M ft2232_spi.c 1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/82/43282/6

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 6 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Edward O'Callaghan quasisec@chromium.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@mailbox.org Gerrit-Attention: Edward O'Callaghan quasisec@chromium.org Gerrit-Attention: Angel Pons th3fanbus@gmail.com Gerrit-MessageType: newpatchset

Nico Huber (Code Review)

2 May 2 May

5:30 p.m.

New subject: Change in flashrom[master]: ft2232_spi.c: change the chunksize to 4096

Attention is currently required from: Simon Buhrow, Edward O'Callaghan, Angel Pons. Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 4096 ......................................................................

Patch Set 6:

(1 comment)

Patchset:

PS6: Turns out this is not about the FTDI hardware but how much data libftdi hands over to libusb at once. As 4096 is the default, I guess we can just remove it.

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 6 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Edward O'Callaghan quasisec@chromium.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@mailbox.org Gerrit-Attention: Simon Buhrow Gerrit-Attention: Edward O'Callaghan quasisec@chromium.org Gerrit-Attention: Angel Pons th3fanbus@gmail.com Gerrit-Comment-Date: Sun, 02 May 2021 17:30:40 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

Simon Buhrow (Code Review)

3 May 3 May

8:12 a.m.

New subject: Change in flashrom[master]: ft2232_spi.c: change the chunksize to 4096

Attention is currently required from: Nico Huber, Edward O'Callaghan, Angel Pons. Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 4096 ......................................................................

Patch Set 6:

(1 comment)

Patchset:

PS6:

...

Turns out this is not about the FTDI hardware but how much data […]

Yes, you are right.

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 6 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Edward O'Callaghan quasisec@chromium.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@mailbox.org Gerrit-Attention: Nico Huber nico.h@gmx.de Gerrit-Attention: Edward O'Callaghan quasisec@chromium.org Gerrit-Attention: Angel Pons th3fanbus@gmail.com Gerrit-Comment-Date: Mon, 03 May 2021 08:12:32 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber nico.h@gmx.de Gerrit-MessageType: comment

Simon Buhrow (Code Review)

27 Sep 27 Sep

8:30 a.m.

New subject: Change in flashrom[master]: ft2232_spi.c: change the chunksize to 4096

Attention is currently required from: Nico Huber, Edward O'Callaghan, Angel Pons. Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 4096 ......................................................................

Patch Set 6:

(1 comment)

Patchset:

PS6: Became obsolete s. https://review.coreboot.org/c/flashrom/+/55683

-- To view, visit https://review.coreboot.org/c/flashrom/+/43282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5e096937834fc8fc8623c8c59ade529de081b72a Gerrit-Change-Number: 43282 Gerrit-PatchSet: 6 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Edward O'Callaghan quasisec@chromium.org Gerrit-CC: Nico Huber nico.h@gmx.de Gerrit-CC: Paul Menzel paulepanter@mailbox.org Gerrit-Attention: Nico Huber nico.h@gmx.de Gerrit-Attention: Edward O'Callaghan quasisec@chromium.org Gerrit-Attention: Angel Pons th3fanbus@gmail.com Gerrit-Comment-Date: Mon, 27 Sep 2021 08:30:37 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

Simon Buhrow (Code Review)

8:31 a.m.

New subject: Change in flashrom[master]: ft2232_spi.c: change the chunksize to 4096

Simon Buhrow has abandoned this change. ( https://review.coreboot.org/c/flashrom/+/43282 )

Change subject: ft2232_spi.c: change the chunksize to 4096 ......................................................................

Abandoned

Became obsolete s. https://review.coreboot.org/c/flashrom/+/55683

1181

days inactive

1627

days old

flashrom-gerrit@flashrom.org

17 comments

4 participants

tags (0)

participants (4)

Angel Pons (Code Review)
Edward O'Callaghan (Code Review)
Nico Huber (Code Review)
Simon Buhrow (Code Review)