November 2020 - flashrom-gerrit

Change in flashrom[master]: ft2232_spi.c: Pack WREN and op in one ftdi_write_data() call
by Simon Buhrow (Code Review) 12 Nov '20

12 Nov '20

Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/40477 ) Change subject: ft2232_spi.c: Pack WREN and op in one ftdi_write_data() call ...................................................................... Patch Set 16: (3 comments) Thanks. Much better than no feedback! I will work on this when I get time. May be anyone (Nico?) can give a comment on ft2232_spi.c#497 meanwhile. https://review.coreboot.org/c/flashrom/+/40477/16/ft2232_spi.c File ft2232_spi.c: https://review.coreboot.org/c/flashrom/+/40477/16/ft2232_spi.c@172 PS16, Line 172: .multicommand = default_spi_send_multicommand, > You just need to implement this and will get both commands passed at once. I got it. This looks much better than my suggested patch! https://review.coreboot.org/c/flashrom/+/40477/16/ft2232_spi.c@497 PS16, Line 497: bufsize = max(writecnt + 19, 261 + 19); > This assumes `i` is at most 10. But there is really no guarantee because […] The optimization done in this change (to 261+19) was not due to USB-Transfer itself, but due to FTDI-Response time after every ftdi_write_data-call (which in this case is the some number as the number of USB-transfers). So this is a significant optimization. But with the good idea of implementing spi_send_multicommand for ftdi I have to check what this does with bufsize of (261+19). What do you want to drop? Just the change from 260+9 to 261+19 or the whole max()-line? I would drop the whole line (see PS11). IMHO the easiest and best way would be so set bufsize (to chunksize) to 4096. Is there any benefit of saving ca. 3.8kByte of space on a PC to the cost of more complex code? As Angel Pons suggested, this could be done in an extra patch. https://review.coreboot.org/c/flashrom/+/40477/16/ft2232_spi.c@562 PS16, Line 562: if (writearr[0] == JEDEC_WREN) { > What if the chip isn't JEDEC? 0x06 (WREN) could mean something very different. […] You are right, I was assuming JEDEC_WREN in any case. Thank´s for that hint! -- To view, visit https://review.coreboot.org/c/flashrom/+/40477 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ie4a07499ec5ef0af23818593f45dc427285a9e8a Gerrit-Change-Number: 40477 Gerrit-PatchSet: 16 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Nico Huber <nico.h(a)gmx.de> Gerrit-CC: Patrick Georgi <pgeorgi(a)google.com> Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Comment-Date: Thu, 12 Nov 2020 13:45:34 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h(a)gmx.de> Gerrit-MessageType: comment

1 0

Change in flashrom[master]: Disable ENE_LPC and MEC1308 on non-x86 arch
by Victor Ding (Code Review) 12 Nov '20

12 Nov '20

Victor Ding has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/47394 ) Change subject: Disable ENE_LPC and MEC1308 on non-x86 arch ...................................................................... Disable ENE_LPC and MEC1308 on non-x86 arch Both requires PCI port I/O and hence works only on x86. TEST=builds on Ubuntu for Raspberry Pi Signed-off-by: Victor Ding <victording(a)google.com> Change-Id: I69e1fbd87819b0b6370f31e9ee4c474500fb3759 --- M Makefile 1 file changed, 10 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/94/47394/1 diff --git a/Makefile b/Makefile index 225db9d..0498624 100644 --- a/Makefile +++ b/Makefile @@ -544,6 +544,16 @@ else override CONFIG_SATAMV = no endif +ifeq ($(CONFIG_ENE_LPC), yes) +UNSUPPORTED_FEATURES += CONFIG_ENE_LPC=yes +else +override CONFIG_ENE_LPC = no +endif +ifeq ($(CONFIG_MEC1308), yes) +UNSUPPORTED_FEATURES += CONFIG_MEC1308=yes +else +override CONFIG_MEC1308 = no +endif endif # Disable all drivers needing raw access (memory, PCI, port I/O) on -- To view, visit https://review.coreboot.org/c/flashrom/+/47394 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I69e1fbd87819b0b6370f31e9ee4c474500fb3759 Gerrit-Change-Number: 47394 Gerrit-PatchSet: 1 Gerrit-Owner: Victor Ding <victording(a)google.com> Gerrit-MessageType: newchange

3 3

Change in flashrom[master]: ft2232_spi.c: Pack WREN and op in one ftdi_write_data() call
by Nico Huber (Code Review) 10 Nov '20

10 Nov '20

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/40477 ) Change subject: ft2232_spi.c: Pack WREN and op in one ftdi_write_data() call ...................................................................... Patch Set 16: (5 comments) > hm... anyone having time to review/merge this? I´d like to get this done... Sorry to let you know after you went through many cycles already, but I think this needs a re-write :-/ There is already everything in place in flashrom to avoid such a layering violation, namely the `.multicommand` member of struct spi_master. The biggest problem with your current implementation is that you assume that there is always a JEDEC flash chip, but that's not generally true. So, the correct way to do this is to implement `.multicommand` instead of `.command`. I would roughly do it as follows: Loop Put write part of command into buffer. If not last command and read part is empty, Continue with next loop iteration. Execute buffer. Execute read part if any. While not last command. https://review.coreboot.org/c/flashrom/+/40477/16//COMMIT_MSG Commit Message: https://review.coreboot.org/c/flashrom/+/40477/16//COMMIT_MSG@25 PS16, Line 25: in a similar way. The infrastructure is already in place. You just have to implement `.multicommand` instead of `.command` for the programmer. https://review.coreboot.org/c/flashrom/+/40477/16/ft2232_spi.c File ft2232_spi.c: https://review.coreboot.org/c/flashrom/+/40477/16/ft2232_spi.c@172 PS16, Line 172: .multicommand = default_spi_send_multicommand, You just need to implement this and will get both commands passed at once. https://review.coreboot.org/c/flashrom/+/40477/16/ft2232_spi.c@497 PS16, Line 497: bufsize = max(writecnt + 19, 261 + 19); This assumes `i` is at most 10. But there is really no guarantee because we could have been called with WREN multiple times (by accident). Easy to fix though, replace the assumed 10 in `writecnt + 19` with `i`: `i + writecnt + 9`. Btw. this `max(..., 261 + 19)` optimization seems insignificant in the presence of USB transfers (which probably have 100 times more overhead than realloc()). I would probably drop it. https://review.coreboot.org/c/flashrom/+/40477/16/ft2232_spi.c@562 PS16, Line 562: if (writearr[0] == JEDEC_WREN) { What if the chip isn't JEDEC? 0x06 (WREN) could mean something very different. This is pretty much a blocker. Especially because the proper infrastructure (.multicommand) is already in place. https://review.coreboot.org/c/flashrom/+/40477/16/ft2232_spi.c@568 PS16, Line 568: } else { To "quote" checkpatch from memory: `else` is not needed after a return. -- To view, visit https://review.coreboot.org/c/flashrom/+/40477 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ie4a07499ec5ef0af23818593f45dc427285a9e8a Gerrit-Change-Number: 40477 Gerrit-PatchSet: 16 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Nico Huber <nico.h(a)gmx.de> Gerrit-CC: Patrick Georgi <pgeorgi(a)google.com> Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Comment-Date: Tue, 10 Nov 2020 11:51:10 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: ft2232_spi.c: Pack WREN and op in one ftdi_write_data() call
by Simon Buhrow (Code Review) 10 Nov '20

10 Nov '20

Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/40477 ) Change subject: ft2232_spi.c: Pack WREN and op in one ftdi_write_data() call ...................................................................... Patch Set 16: hm... anyone having time to review/merge this? I´d like to get this done... -- To view, visit https://review.coreboot.org/c/flashrom/+/40477 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ie4a07499ec5ef0af23818593f45dc427285a9e8a Gerrit-Change-Number: 40477 Gerrit-PatchSet: 16 Gerrit-Owner: Simon Buhrow Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Patrick Georgi <pgeorgi(a)google.com> Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Comment-Date: Tue, 10 Nov 2020 07:34:36 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: test patch
by Eizan Miyamoto (Code Review) 10 Nov '20

10 Nov '20

Eizan Miyamoto has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/47379 ) Change subject: test patch ...................................................................... test patch Change-Id: I171ecd540524e4f11859b8b95c2e57cc6941bba0 --- M README 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/79/47379/1 diff --git a/README b/README index 4e7bd4f..9dd92f9 100644 --- a/README +++ b/README @@ -2,6 +2,7 @@ flashrom README ------------------------------------------------------------------------------- + flashrom is a utility for detecting, reading, writing, verifying and erasing flash chips. It is often used to flash BIOS/EFI/coreboot/firmware images in-system using a supported mainboard, but it also supports flashing of network -- To view, visit https://review.coreboot.org/c/flashrom/+/47379 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I171ecd540524e4f11859b8b95c2e57cc6941bba0 Gerrit-Change-Number: 47379 Gerrit-PatchSet: 1 Gerrit-Owner: Eizan Miyamoto <eizan(a)chromium.org> Gerrit-MessageType: newchange

1 1

Change in flashrom[master]: programmer.h: Reorder MEC1308 and ENE_LPC enumerators
by Edward O'Callaghan (Code Review) 09 Nov '20

09 Nov '20

Edward O'Callaghan has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/47353 ) Change subject: programmer.h: Reorder MEC1308 and ENE_LPC enumerators ...................................................................... programmer.h: Reorder MEC1308 and ENE_LPC enumerators BUG=b:172668501 TEST=None BRANCH=None Original-Change-Id: I2e9d3df6a6fd6d5d8e3a5a13ee56f5997b10ea52 Original-Signed-off-by: Sam McNally <sammc(a)chromium.org> Change-Id: I8e7a57b7c30e2dd2306d6fe7268eee8bb9d0c8a5 Signed-off-by: Edward O'Callaghan <quasisec(a)google.com> --- M programmer.h 1 file changed, 6 insertions(+), 6 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/53/47353/1 diff --git a/programmer.h b/programmer.h index 9a7892d..1bd0d37 100644 --- a/programmer.h +++ b/programmer.h @@ -31,6 +31,9 @@ #if CONFIG_DUMMY == 1 PROGRAMMER_DUMMY, #endif +#if CONFIG_MEC1308 == 1 + PROGRAMMER_MEC1308, +#endif #if CONFIG_NIC3COM == 1 PROGRAMMER_NIC3COM, #endif @@ -61,9 +64,6 @@ #if CONFIG_ATAPROMISE == 1 PROGRAMMER_ATAPROMISE, #endif -#if CONFIG_ENE_LPC == 1 - PROGRAMMER_ENE_LPC, -#endif #if CONFIG_IT8212 == 1 PROGRAMMER_IT8212, #endif @@ -82,6 +82,9 @@ #if CONFIG_DEVELOPERBOX_SPI == 1 PROGRAMMER_DEVELOPERBOX_SPI, #endif +#if CONFIG_ENE_LPC == 1 + PROGRAMMER_ENE_LPC, +#endif #if CONFIG_RAYER_SPI == 1 PROGRAMMER_RAYER_SPI, #endif @@ -112,9 +115,6 @@ #if CONFIG_USBBLASTER_SPI == 1 PROGRAMMER_USBBLASTER_SPI, #endif -#if CONFIG_MEC1308 == 1 - PROGRAMMER_MEC1308, -#endif #if CONFIG_MSTARDDC_SPI == 1 PROGRAMMER_MSTARDDC_SPI, #endif -- To view, visit https://review.coreboot.org/c/flashrom/+/47353 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I8e7a57b7c30e2dd2306d6fe7268eee8bb9d0c8a5 Gerrit-Change-Number: 47353 Gerrit-PatchSet: 1 Gerrit-Owner: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-MessageType: newchange

3 5

Change in flashrom[master]: test_build.sh: Move build test procedure to repository
by Patrick Georgi (Code Review) 03 Nov '20

03 Nov '20

Patrick Georgi has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/46894 ) Change subject: test_build.sh: Move build test procedure to repository ...................................................................... test_build.sh: Move build test procedure to repository Instead of hard coding the test procedure on qa.coreboot.org, allow running a script in the repo instead. The server is already adapted to do that, so once there's a test_build.sh file in the toplevel directory, it's run in place of the default operation. The content of this change mirrors the default operation exactly so should serve as a good starting point. The script is executed in an encapsulate[0] context with the workspace, /tmp and $HOME/.ccache writable, everything else read-only and network disabled. It should return 0 on success, anything else on failure, as is normal for UNIX processes. [0] https://review.coreboot.org/cgit/encapsulate.git Change-Id: I37a8e925d1b283c3b8f87cb3d0f1ed8920f2cf95 Signed-off-by: Patrick Georgi <pgeorgi(a)google.com> --- A test_build.sh 1 file changed, 8 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/94/46894/1 diff --git a/test_build.sh b/test_build.sh new file mode 100755 index 0000000..0e7f37a --- /dev/null +++ b/test_build.sh @@ -0,0 +1,8 @@ +#!/bin/sh +set -e + +make CONFIG_EVERYTHING=yes WARNERROR=yes + +meson -p out +(cd out && ninja) +(cd out && ninja test) -- To view, visit https://review.coreboot.org/c/flashrom/+/46894 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I37a8e925d1b283c3b8f87cb3d0f1ed8920f2cf95 Gerrit-Change-Number: 46894 Gerrit-PatchSet: 1 Gerrit-Owner: Patrick Georgi <pgeorgi(a)google.com> Gerrit-MessageType: newchange

3 5

Change in flashrom[master]: chipset_enable.c: Add all the lakes support
by Edward O'Callaghan (Code Review) 02 Nov '20

02 Nov '20

Edward O'Callaghan has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/47092 ) Change subject: chipset_enable.c: Add all the lakes support ...................................................................... chipset_enable.c: Add all the lakes support Pulled out of ChromiumOS tree. - pid=0x02a4 -> "Cometlake" - pid=0x34a4 -> "Icelake" - pid=0x4da4 -> "Jasperlake" - pid=0x51a4 -> "Alderlake-P" - pid=0x7aa4 -> "Alderlake-S" - pid=0x9d24 -> "Skylake" - pid=0x9da4 -> "Cannonlake" - pid=0xa0a4 -> "Tigerlake" BUG=b:XXX BRANCH=none TEST=none Change-Id: I403b11bbfc698e593fedd272921ade2bf8b3c2ca Signed-off-by: Edward O'Callaghan <quasisec(a)google.com> --- M chipset_enable.c 1 file changed, 10 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/92/47092/1 diff --git a/chipset_enable.c b/chipset_enable.c index 01d0291..3312483 100644 --- a/chipset_enable.c +++ b/chipset_enable.c @@ -1815,6 +1815,7 @@ {0x1166, 0x0200, B_P, OK, "Broadcom", "OSB4", enable_flash_osb4}, {0x1166, 0x0205, B_PFL, OK, "Broadcom", "HT-1000", enable_flash_ht1000}, {0x17f3, 0x6030, B_PFL, OK, "RDC", "R8610/R3210", enable_flash_rdc_r8610}, + {0x8086, 0x02a4, B_FS, OK, "Intel", "Cometlake", enable_flash_pch100}, {0x8086, 0x0c60, B_FS, NT, "Intel", "S12x0", enable_flash_s12x0}, {0x8086, 0x0f1c, B_FS, OK, "Intel", "Bay Trail", enable_flash_silvermont}, {0x8086, 0x0f1d, B_FS, NT, "Intel", "Bay Trail", enable_flash_silvermont}, @@ -1897,6 +1898,7 @@ {0x8086, 0x2918, B_FS, DEP, "Intel", "ICH9", enable_flash_ich9}, {0x8086, 0x2919, B_FS, DEP, "Intel", "ICH9M", enable_flash_ich9}, {0x8086, 0x31f0, B_FS, OK, "Intel", "Geminilake", enable_flash_apl}, + {0x8086, 0x34a4, B_FS, OK, "Intel", "Icelake", enable_flash_pch100}, {0x8086, 0x3a10, B_FS, NT, "Intel", "ICH10R Eng. Sample", enable_flash_ich10}, {0x8086, 0x3a14, B_FS, DEP, "Intel", "ICH10DO", enable_flash_ich10}, {0x8086, 0x3a16, B_FS, DEP, "Intel", "ICH10R", enable_flash_ich10}, @@ -1920,11 +1922,14 @@ {0x8086, 0x3b14, B_FS, DEP, "Intel", "3420", enable_flash_pch5}, {0x8086, 0x3b16, B_FS, NT, "Intel", "3450", enable_flash_pch5}, {0x8086, 0x3b1e, B_FS, NT, "Intel", "B55", enable_flash_pch5}, + {0x8086, 0x4da4, B_FS, OK, "Intel", "Jasperlake", enable_flash_pch100}, {0x8086, 0x5031, B_FS, OK, "Intel", "EP80579", enable_flash_ich7}, + {0x8086, 0x51a4, B_FS, OK, "Intel", "Alderlake-P", enable_flash_pch100}, {0x8086, 0x5af0, B_FS, OK, "Intel", "Apollolake", enable_flash_apl}, {0x8086, 0x7000, B_P, OK, "Intel", "PIIX3", enable_flash_piix4}, {0x8086, 0x7110, B_P, OK, "Intel", "PIIX4/4E/4M", enable_flash_piix4}, {0x8086, 0x7198, B_P, OK, "Intel", "440MX", enable_flash_piix4}, + {0x8086, 0x7aa4, B_FS, OK, "Intel", "Alderlake-S", enable_flash_pch100}, {0x8086, 0x8119, B_FL, OK, "Intel", "SCH Poulsbo", enable_flash_poulsbo}, {0x8086, 0x8186, B_FS, OK, "Intel", "Atom E6xx(T) (Tunnel Creek)", enable_flash_tunnelcreek}, {0x8086, 0x8c40, B_FS, NT, "Intel", "Lynx Point", enable_flash_pch8}, @@ -2008,6 +2013,7 @@ {0x8086, 0x9cc7, B_FS, NT, "Intel", "Broadwell Y Premium", enable_flash_pch9_lp}, {0x8086, 0x9cc9, B_FS, NT, "Intel", "Broadwell Y Base", enable_flash_pch9_lp}, {0x8086, 0x9ccb, B_FS, NT, "Intel", "Broadwell H", enable_flash_pch9}, + {0x8086, 0x9d24, B_FS, OK, "Intel", "Skylake", enable_flash_pch100}, {0x8086, 0x9d41, B_S, NT, "Intel", "Skylake / Kaby Lake Sample", enable_flash_pch100}, {0x8086, 0x9d43, B_S, NT, "Intel", "Skylake U Base", enable_flash_pch100}, {0x8086, 0x9d46, B_S, NT, "Intel", "Skylake Y Premium", enable_flash_pch100}, @@ -2020,7 +2026,9 @@ {0x8086, 0x9d56, B_S, NT, "Intel", "Kaby Lake Y Premium", enable_flash_pch100}, {0x8086, 0x9d58, B_S, NT, "Intel", "Kaby Lake U Premium", enable_flash_pch100}, {0x8086, 0x9d84, B_S, DEP, "Intel", "Cannon Lake U Premium", enable_flash_pch300}, + {0x8086, 0x9da4, B_FS, OK, "Intel", "Cannonlake", enable_flash_pch100}, {0x8086, 0x0284, B_S, DEP, "Intel", "Comet Lake U Premium", enable_flash_pch400}, + {0x8086, 0xa0a4, B_FS, OK, "Intel", "Tigerlake", enable_flash_pch100}, {0x8086, 0xa141, B_S, NT, "Intel", "Sunrise Point Desktop Sample", enable_flash_pch100}, {0x8086, 0xa142, B_S, NT, "Intel", "Sunrise Point Unknown Sample", enable_flash_pch100}, {0x8086, 0xa143, B_S, NT, "Intel", "H110", enable_flash_pch100}, @@ -2055,6 +2063,7 @@ {0x8086, 0xa1cb, B_S, NT, "Intel", "C621A Series Chipset (QS/PRQ)", enable_flash_c620}, {0x8086, 0xa1cc, B_S, NT, "Intel", "C627A Series Chipset (QS/PRQ)", enable_flash_c620}, {0x8086, 0xa1cd, B_S, NT, "Intel", "C629A Series Chipset (QS/PRQ)", enable_flash_c620}, + {0x8086, 0xa224, B_FS, OK, "Intel", "Lewisburg", enable_flash_pch100}, {0x8086, 0xa240, B_S, NT, "Intel", "C620 Series Chipset Supersku", enable_flash_c620}, {0x8086, 0xa241, B_S, NT, "Intel", "C620 Series Chipset Supersku", enable_flash_c620}, {0x8086, 0xa242, B_S, NT, "Intel", "C624 Series Chipset Supersku", enable_flash_c620}, @@ -2085,6 +2094,7 @@ {0x8086, 0xa30d, B_S, NT, "Intel", "HM370", enable_flash_pch300}, {0x8086, 0xa30e, B_S, DEP, "Intel", "CM246", enable_flash_pch300}, {0x8086, 0x3482, B_S, DEP, "Intel", "Ice Lake U Premium", enable_flash_pch300}, + {0x8086, 0xa0a4, B_FS, OK, "Intel", "Tigerlake", enable_flash_pch100}, #endif {0}, }; -- To view, visit https://review.coreboot.org/c/flashrom/+/47092 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I403b11bbfc698e593fedd272921ade2bf8b3c2ca Gerrit-Change-Number: 47092 Gerrit-PatchSet: 1 Gerrit-Owner: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-MessageType: newchange

1 1

Change in flashrom[master]: cli_classic.c: Attempt to hook up fmap cros feature
by Edward O'Callaghan (Code Review) 01 Nov '20

01 Nov '20

Edward O'Callaghan has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/47066 ) Change subject: cli_classic.c: Attempt to hook up fmap cros feature ...................................................................... cli_classic.c: Attempt to hook up fmap cros feature Change-Id: Ieb8e96821832ed8a4ab6f1525a2b263925c76968 Signed-off-by: Edward O'Callaghan <quasisec(a)google.com> --- M cli_classic.c 1 file changed, 26 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/66/47066/1 diff --git a/cli_classic.c b/cli_classic.c index ae7f6ef..beb578c 100644 --- a/cli_classic.c +++ b/cli_classic.c @@ -154,11 +154,13 @@ int read_it = 0, write_it = 0, erase_it = 0, verify_it = 0; int dont_verify_it = 0, dont_verify_all = 0, list_supported = 0, operation_specified = 0; struct flashrom_layout *layout = NULL; + int set_ignore_fmap = 0; enum programmer prog = PROGRAMMER_INVALID; enum { OPTION_IFD = 0x0100, OPTION_FMAP, OPTION_FMAP_FILE, + OPTION_IGNORE_FMAP, OPTION_FLASH_CONTENTS, OPTION_FLASH_NAME, OPTION_FLASH_SIZE, @@ -187,6 +189,7 @@ {"ifd", 0, NULL, OPTION_IFD}, {"fmap", 0, NULL, OPTION_FMAP}, {"fmap-file", 1, NULL, OPTION_FMAP_FILE}, + {"ignore-fmap", 0, NULL, OPTION_IGNORE_FMAP}, {"image", 1, NULL, 'i'}, {"flash-contents", 1, NULL, OPTION_FLASH_CONTENTS}, {"flash-name", 0, NULL, OPTION_FLASH_NAME}, @@ -314,6 +317,9 @@ cli_classic_abort_usage("Error: --layout and --fmap both specified. Aborting.\n"); fmap = 1; break; + case OPTION_IGNORE_FMAP: + set_ignore_fmap = 1; + break; case 'i': tempstr = strdup(optarg); if (register_include_arg(&include_args, tempstr)) { @@ -641,6 +647,26 @@ goto out_shutdown; } +#if 0 + /* If the user doesn't specify any -i argument, then we can skip the + * fmap parsing to speed up. */ + if (get_num_include_args() == 0 && !extract_it) { + msg_gdbg("No -i argument is specified, set ignore_fmap.\n"); + set_ignore_fmap = 1; + } +#endif + + /* + * Add entries for regions specified in flashmap, unless the user + * explicitly requested not to look for fmap, or provided a layout + * file. + */ + if (!set_ignore_fmap && !layoutfile && + get_fmap_entries(filename, fill_flash) < 0) { + ret = 1; + goto out_shutdown; + } + if (set_wp_enable && set_wp_disable) { msg_ginfo("Error: --wp-enable and --wp-disable are mutually exclusive\n"); ret = 1; -- To view, visit https://review.coreboot.org/c/flashrom/+/47066 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ieb8e96821832ed8a4ab6f1525a2b263925c76968 Gerrit-Change-Number: 47066 Gerrit-PatchSet: 1 Gerrit-Owner: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: fmap.c: Port to upstream
by Edward O'Callaghan (Code Review) 01 Nov '20

01 Nov '20

Edward O'Callaghan has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/47065 ) Change subject: fmap.c: Port to upstream ...................................................................... fmap.c: Port to upstream Change-Id: I489761f531c3a44978f6450cdd0f1ff991f13d87 Signed-off-by: Edward O'Callaghan <quasisec(a)google.com> --- M fmap.c M layout.h 2 files changed, 2 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/65/47065/1 diff --git a/fmap.c b/fmap.c index 8155aa3..fb6e0d3 100644 --- a/fmap.c +++ b/fmap.c @@ -609,7 +609,7 @@ { struct flashctx *flash = handle; - return read_flash(flash, dest, offset, size); + return flash->chip->read(flash, dest, offset, size); } int get_fmap_entries(const char *filename, struct flashctx *flash) diff --git a/layout.h b/layout.h index 8e4eb13..b32d0ca 100644 --- a/layout.h +++ b/layout.h @@ -40,6 +40,7 @@ chipoff_t end; bool included; char *name; + char file[256]; /* file[0]=='\0' means not specified. */ }; struct flashrom_layout { -- To view, visit https://review.coreboot.org/c/flashrom/+/47065 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I489761f531c3a44978f6450cdd0f1ff991f13d87 Gerrit-Change-Number: 47065 Gerrit-PatchSet: 1 Gerrit-Owner: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-MessageType: newchange

1 0

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit November 2020