May 2022 - flashrom-gerrit - mail.coreboot.org

Change in flashrom[master]: libflashrom: Return progress state to the library user
by Daniel Campello (Code Review) 03 May '22

03 May '22

Attention is currently required from: Richard Hughes, Edward O'Callaghan, Angel Pons, Anastasia Klimchuk, Patrick Rudolph. Daniel Campello has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/49643 ) Change subject: libflashrom: Return progress state to the library user ...................................................................... Patch Set 11: (16 comments) Commit Message: https://review.coreboot.org/c/flashrom/+/49643/comment/10ec2b73_825cd270 PS11, Line 15: > It would be realy good to test it on at least one real programmer, not dummy. […] I am not completely familiar with the different programmers, but I think the programmer here is spi25 ??? File cli_output.c: https://review.coreboot.org/c/flashrom/+/49643/comment/51c4d99a_4b1b2e52 PS11, Line 87: > Space between function name and arguments list not needed. Done File flash.h: https://review.coreboot.org/c/flashrom/+/49643/comment/9ac059f2_ade1976b PS11, Line 481: set_progress > Looking into the usage of this function, maybe `update_progress` is better name? It is called repeat […] Done https://review.coreboot.org/c/flashrom/+/49643/comment/852a23df_50b31cc4 PS11, Line 481: void set_progress(struct flashctx *flash, enum flashrom_progress_stage stage, size_t current, size_t total); > Lets move it into the /* cli_output.c */ section, together with other output stuff. […] This is to support the API exposed via libflashrom and independent from the CLI. File libflashrom.h: https://review.coreboot.org/c/flashrom/+/49643/comment/6cb7a25b_0930f07b PS11, Line 106: flashrom_set_progress_callback > If set_progress becomes update_progress, this one becomes flashrom_update_progress_callback Done File libflashrom.c: https://review.coreboot.org/c/flashrom/+/49643/comment/574c5e32_13fe10da PS11, Line 102: chaned > chaned -> changed (typo) Done https://review.coreboot.org/c/flashrom/+/49643/comment/01c8e569_c5ccee22 PS11, Line 106: userdata > This is called progress_userdata not userdata? Done File spi.c: https://review.coreboot.org/c/flashrom/+/49643/comment/63a1f172_cec5592c PS11, Line 104: size_t progress_start = start; : size_t progress_total = len - start; > I think the variable naming can be improved, I spent a while trying to read this small piece. […] Done File spi25.c: https://review.coreboot.org/c/flashrom/+/49643/comment/a29094da_cb4f2f8e PS11, Line 677: size_t progress_start = start; : size_t progress_total = len - start; > This looks the same as previous, let do […] Done https://review.coreboot.org/c/flashrom/+/49643/comment/8f8f794e_91ac2cb5 PS11, Line 704: size_t progress_start = start; : size_t progress_total = len - start; > And one more here :) Done File tests/spi25.c: https://review.coreboot.org/c/flashrom/+/49643/comment/959fd74b_5467d951 PS11, Line 93: assert_true(0); : break; > This probably can be `fail()` , if it's the intention Done https://review.coreboot.org/c/flashrom/+/49643/comment/02d09a9d_d33e5c3a PS11, Line 106: > new line Done https://review.coreboot.org/c/flashrom/+/49643/comment/3f8e6cce_d86df9a3 PS11, Line 107: > new line Done https://review.coreboot.org/c/flashrom/+/49643/comment/86628ce5_824f7254 PS11, Line 110: > new line Done https://review.coreboot.org/c/flashrom/+/49643/comment/419bda1e_c048218a PS11, Line 110: > new line Done https://review.coreboot.org/c/flashrom/+/49643/comment/bc30e2b3_e66499b8 PS11, Line 110: > new line Done -- To view, visit https://review.coreboot.org/c/flashrom/+/49643 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I7197572bb7f19e3bdb2bde855d70a0f50fd3854c Gerrit-Change-Number: 49643 Gerrit-PatchSet: 11 Gerrit-Owner: Richard Hughes <richard(a)hughsie.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Daniel Campello <campello(a)chromium.org> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Peter Marheine <pmarheine(a)chromium.org> Gerrit-CC: Richard Hughes <hughsient(a)gmail.com> Gerrit-CC: Thomas Heijligen <src(a)posteo.de> Gerrit-Attention: Richard Hughes <richard(a)hughsie.com> Gerrit-Attention: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Comment-Date: Tue, 03 May 2022 19:55:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-MessageType: comment

1 0

Change in flashrom[master]: libflashrom: Return progress state to the library user
by Daniel Campello (Code Review) 03 May '22

03 May '22

Attention is currently required from: Richard Hughes, Edward O'Callaghan, Daniel Campello, Angel Pons, Patrick Rudolph. Daniel Campello has uploaded a new patch set (#12) to the change originally created by Richard Hughes. ( https://review.coreboot.org/c/flashrom/+/49643 ) Change subject: libflashrom: Return progress state to the library user ...................................................................... libflashrom: Return progress state to the library user Projects using libflashrom like fwupd expect the user to wait for the operation to complete. To avoid the user thinking the process has "hung" or "got stuck" report back the progress complete of the erase, write and read operations. Include a test for the dummy spi25 device. Change-Id: I7197572bb7f19e3bdb2bde855d70a0f50fd3854c Signed-off-by: Richard Hughes <richard(a)hughsie.com> --- M 82802ab.c M at45db.c M cli_classic.c M cli_output.c M dediprog.c M en29lv640b.c M include/flash.h M include/libflashrom.h M it87spi.c M jedec.c M libflashrom.c M linux_mtd.c M lspcon_i2c_spi.c M realtek_mst_i2c_spi.c M spi.c M spi25.c M sst28sf040.c M tests/spi25.c M tests/tests.c M tests/tests.h 20 files changed, 155 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/43/49643/12 -- To view, visit https://review.coreboot.org/c/flashrom/+/49643 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I7197572bb7f19e3bdb2bde855d70a0f50fd3854c Gerrit-Change-Number: 49643 Gerrit-PatchSet: 12 Gerrit-Owner: Richard Hughes <richard(a)hughsie.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Daniel Campello <campello(a)chromium.org> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Peter Marheine <pmarheine(a)chromium.org> Gerrit-CC: Richard Hughes <hughsient(a)gmail.com> Gerrit-CC: Thomas Heijligen <src(a)posteo.de> Gerrit-Attention: Richard Hughes <richard(a)hughsie.com> Gerrit-Attention: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Attention: Daniel Campello <campello(a)chromium.org> Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org> Gerrit-MessageType: newpatchset

1 0

Change in flashrom[master]: libflashrom: Move documentation to header
by Angel Pons (Code Review) 03 May '22

03 May '22

Attention is currently required from: Evan Benn. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63903 ) Change subject: libflashrom: Move documentation to header ...................................................................... Patch Set 7: Code-Review+1 (3 comments) Commit Message: https://review.coreboot.org/c/flashrom/+/63903/comment/8dfb69aa_04b16cc8 PS7, Line 10: the word `allows` went missing Patchset: PS4: > not sure if I lost R+1 by accidently setting WIP or just by uploading a new patch, PTAL. Code-review scores are lost after a new patchset is uploaded File include/libflashrom.h: https://review.coreboot.org/c/flashrom/+/63903/comment/c3925392_5fe815e9 PS6, Line 372: another > I am happy to make these edits, I'm just checking that you know I am moving these lines from one pla […] Ah, I see. Since it's hard to see the movement in the diff, I just went through the added lines to see if there were any weird things. -- To view, visit https://review.coreboot.org/c/flashrom/+/63903 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I856b74d5bfea13722539be15496755a95e701eea Gerrit-Change-Number: 63903 Gerrit-PatchSet: 7 Gerrit-Owner: Evan Benn <evanbenn(a)google.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Evan Benn <evanbenn(a)google.com> Gerrit-Comment-Date: Tue, 03 May 2022 16:51:44 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com> Comment-In-Reply-To: Evan Benn <evanbenn(a)google.com> Gerrit-MessageType: comment

1 0

Change in flashrom[master]: meson: add option to disable tests
by Felix Singer (Code Review) 03 May '22

03 May '22

Attention is currently required from: Thomas Heijligen, Edward O'Callaghan, Anastasia Klimchuk. Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63832 ) Change subject: meson: add option to disable tests ...................................................................... Patch Set 6: (3 comments) Commit Message: https://review.coreboot.org/c/flashrom/+/63832/comment/9a2fc7a0_810b4e56 PS6, Line 11: to To https://review.coreboot.org/c/flashrom/+/63832/comment/3f72e220_43eea6bf PS6, Line 12: - Add a dot at the end. File meson.build: https://review.coreboot.org/c/flashrom/+/63832/comment/0d6c5e4c_212972fd PS6, Line 509: # .allowed() is introduced in 0.59.0 Rather put this in the commit message instead here. -- To view, visit https://review.coreboot.org/c/flashrom/+/63832 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I384c904c577b265dfe36bf46bf07c641bc29de9b Gerrit-Change-Number: 63832 Gerrit-PatchSet: 6 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Felix Singer <felixsinger(a)posteo.net> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Thomas Heijligen <src(a)posteo.de> Gerrit-Attention: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Comment-Date: Tue, 03 May 2022 15:40:28 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: meson: link flashrom binary against static libflashrom
by Thomas Heijligen (Code Review) 03 May '22

03 May '22

Thomas Heijligen has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/64030 ) Change subject: meson: link flashrom binary against static libflashrom ...................................................................... Patch Set 1: (1 comment) File meson.build: https://review.coreboot.org/c/flashrom/+/64030/comment/bbfcec33_be7b301b PS1, Line 431: lib must stay 'flashrom', now it's liblibflashrom -- To view, visit https://review.coreboot.org/c/flashrom/+/64030 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ic522610f59e00299ebfa1bd29482ff92120ec52b Gerrit-Change-Number: 64030 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 03 May 2022 11:20:12 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: meson: use static library for tests
by Thomas Heijligen (Code Review) 03 May '22

03 May '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/64031 ) Change subject: meson: use static library for tests ...................................................................... meson: use static library for tests Change-Id: If10ed70e3bf629a6eb7e2c5bef3318d1e259dadd Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M meson.build M tests/chip.c M tests/flashrom.c M tests/helpers.c M tests/lifecycle.c M tests/meson.build M tests/tests.c 7 files changed, 39 insertions(+), 55 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/31/64031/1 diff --git a/meson.build b/meson.build index dbee971..aee07b7 100644 --- a/meson.build +++ b/meson.build @@ -492,32 +492,5 @@ subdir('util') if get_option('tests').auto() or get_option('tests').enabled() # .allowed() is introduced in 0.59.0 -# unit-test framework -cmocka_dep = dependency( - 'cmocka', - fallback: ['cmocka', 'cmocka_dep'], - required : get_option('tests') - ) - - flashrom_test_dep = declare_dependency( - include_directories : include_dir, - sources : [ - srcs, - 'cli_common.c', - 'cli_output.c', - 'flashrom.c', - ], - compile_args : [ - '-includestdlib.h', - '-includeunittest_env.h', - '-includehwaccess_x86_io_unittest.h' - ], - dependencies : [ - deps, - ], - ) - - if cmocka_dep.found() - subdir('tests') - endif + subdir('tests') endif diff --git a/tests/chip.c b/tests/chip.c index 6f03172..ce6b53c 100644 --- a/tests/chip.c +++ b/tests/chip.c @@ -27,6 +27,7 @@ #include <include/test.h> #include <stdio.h> +#include <stdlib.h> #include <string.h> #include "tests.h" diff --git a/tests/flashrom.c b/tests/flashrom.c index cc88831..ecab69c 100644 --- a/tests/flashrom.c +++ b/tests/flashrom.c @@ -12,7 +12,7 @@ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. */ - +#include <stdlib.h> #include <include/test.h> #include "tests.h" diff --git a/tests/helpers.c b/tests/helpers.c index e2e652e..81cb1cd 100644 --- a/tests/helpers.c +++ b/tests/helpers.c @@ -19,6 +19,7 @@ #include "flash.h" #include <stdint.h> +#include <stdlib.h> void address_to_bits_test_success(void **state) { diff --git a/tests/lifecycle.c b/tests/lifecycle.c index 63fcc3c..99c360a 100644 --- a/tests/lifecycle.c +++ b/tests/lifecycle.c @@ -14,6 +14,7 @@ */ #include <include/test.h> +#include <stdlib.h> #include <string.h> #include <linux/spi/spidev.h> diff --git a/tests/meson.build b/tests/meson.build index d3b46b1..a2ac78e 100644 --- a/tests/meson.build +++ b/tests/meson.build @@ -11,20 +11,6 @@ # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the # GNU General Public License for more details. -root_includes = include_directories('../subprojects') - -srcs = [ - 'io_mock.c', - 'tests.c', - 'libusb_wraps.c', - 'helpers.c', - 'flashrom.c', - 'spi25.c', - 'lifecycle.c', - 'layout.c', - 'chip.c', -] - mocks = [ '-Wl,--wrap=strdup', '-Wl,--wrap=physunmap', @@ -95,16 +81,37 @@ '-Wl,--gc-sections', ] -flashrom_tests = executable('flashrom_unit_tests', - srcs, - include_directories : root_includes, - c_args : [ - cargs, - '-ffunction-sections', - '-fdata-sections', - ], - export_dynamic : true, - link_args : mocks, - dependencies : [cmocka_dep, flashrom_test_dep], +cmocka = dependency( + 'cmocka', + fallback: ['cmocka', 'cmocka_dep'], + required : get_option('tests'), ) -test('cmocka test flashrom', flashrom_tests) + +if cmocka.found() + flashrom_tests = executable( + 'flashrom_unit_tests', + files( + 'io_mock.c', + 'tests.c', + 'libusb_wraps.c', + 'helpers.c', + 'flashrom.c', + 'spi25.c', + 'lifecycle.c', + 'layout.c', + 'chip.c', + ), + c_args : [ + cargs, + '-ffunction-sections', + '-fdata-sections', + ], + dependencies : [ cmocka, deps ], + export_dynamic : true, + include_directories : include_dir, + link_args : mocks, + link_with : libflashrom.get_static_lib(), + ) + + test('cmocka test flashrom', flashrom_tests) +endif \ No newline at end of file diff --git a/tests/tests.c b/tests/tests.c index 6aabd90..d0fc6af 100644 --- a/tests/tests.c +++ b/tests/tests.c @@ -20,6 +20,7 @@ #include <stdio.h> #include <string.h> +#include <stdlib.h> #include <stdint.h> void *not_null(void) -- To view, visit https://review.coreboot.org/c/flashrom/+/64031 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: If10ed70e3bf629a6eb7e2c5bef3318d1e259dadd Gerrit-Change-Number: 64031 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: meson: link flashrom binary against static libflashrom
by Thomas Heijligen (Code Review) 03 May '22

03 May '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/64030 ) Change subject: meson: link flashrom binary against static libflashrom ...................................................................... meson: link flashrom binary against static libflashrom Change-Id: Ic522610f59e00299ebfa1bd29482ff92120ec52b Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M meson.build 1 file changed, 11 insertions(+), 21 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/30/64030/1 diff --git a/meson.build b/meson.build index 3018c17..dbee971 100644 --- a/meson.build +++ b/meson.build @@ -7,7 +7,9 @@ 'c_std=c99', 'werror=true', 'optimization=s', - 'debug=false'], + 'debug=false', + 'default_library=both', + ], ) # libtool versioning @@ -425,8 +427,8 @@ mapfile = 'libflashrom.map' vflag = '-Wl,--version-script,@0@/@1@'.format(meson.current_source_dir(), mapfile) -flashrom = library( - 'flashrom', +libflashrom = library( + 'libflashrom', sources : [ srcs, ], @@ -455,7 +457,7 @@ pkgg = import('pkgconfig') pkgg.generate( - libraries : flashrom, + libraries : libflashrom, version : version, name : 'flashrom', filebase : 'flashrom', @@ -472,31 +474,19 @@ install_dir: join_paths(get_option('mandir'), 'man8'), ) -flashrom_dep = declare_dependency( - link_with : flashrom, - include_directories : include_dir, - dependencies : deps -) - # we can't just link_with libflashrom as we require all the internal symbols... executable( 'flashrom', - sources : [ - srcs, + files( 'cli_classic.c', 'cli_common.c', 'cli_output.c', - 'flashrom.c', - ], + ), + c_args : cargs, include_directories : include_dir, - dependencies : [ - deps, - ], - c_args : [ - cargs - ], install : true, - install_dir : get_option('sbindir') + install_dir : get_option('sbindir'), + link_with : libflashrom.get_static_lib(), # flashrom needs internal symbols of libflashrom ) subdir('util') -- To view, visit https://review.coreboot.org/c/flashrom/+/64030 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ic522610f59e00299ebfa1bd29482ff92120ec52b Gerrit-Change-Number: 64030 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: meson: relocate config_print_wiki & config_default_programmer_*
by Thomas Heijligen (Code Review) 03 May '22

03 May '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/64029 ) Change subject: meson: relocate config_print_wiki & config_default_programmer_* ...................................................................... meson: relocate config_print_wiki & config_default_programmer_* Change-Id: I9538b0aee31b294844d4f4ca0396334a81dfb498 Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M meson.build 1 file changed, 13 insertions(+), 13 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/29/64029/1 diff --git a/meson.build b/meson.build index cfb353f..3018c17 100644 --- a/meson.build +++ b/meson.build @@ -403,6 +403,19 @@ endif endif +if config_print_wiki + srcs += 'print_wiki.c' + cargs += '-DCONFIG_PRINT_WIKI=1' +endif + +if config_default_programmer_name != '' + cargs += '-DCONFIG_DEFAULT_PROGRAMMER_NAME=&programmer_' + config_default_programmer_name +else + cargs += '-DCONFIG_DEFAULT_PROGRAMMER_NAME=NULL' +endif + +cargs += '-DCONFIG_DEFAULT_PROGRAMMER_ARGS="' + config_default_programmer_args + '"' + install_headers([ 'include/libflashrom.h', ], @@ -465,19 +478,6 @@ dependencies : deps ) -if config_print_wiki - srcs += 'print_wiki.c' - cargs += '-DCONFIG_PRINT_WIKI=1' -endif - -if config_default_programmer_name != '' - cargs += '-DCONFIG_DEFAULT_PROGRAMMER_NAME=&programmer_' + config_default_programmer_name -else - cargs += '-DCONFIG_DEFAULT_PROGRAMMER_NAME=NULL' -endif - -cargs += '-DCONFIG_DEFAULT_PROGRAMMER_ARGS="' + config_default_programmer_args + '"' - # we can't just link_with libflashrom as we require all the internal symbols... executable( 'flashrom', -- To view, visit https://review.coreboot.org/c/flashrom/+/64029 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I9538b0aee31b294844d4f4ca0396334a81dfb498 Gerrit-Change-Number: 64029 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: meson: use build in options for install paths
by Thomas Heijligen (Code Review) 03 May '22

03 May '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/64028 ) Change subject: meson: use build in options for install paths ...................................................................... meson: use build in options for install paths The install functions of meson can take an relative path and join the prefix automaticly. Change-Id: I9cb9faf4bdbcfd66098478cc3a260eb3b664a2e6 Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M meson.build 1 file changed, 2 insertions(+), 7 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/28/64028/1 diff --git a/meson.build b/meson.build index 38354dd..cfb353f 100644 --- a/meson.build +++ b/meson.build @@ -403,11 +403,6 @@ endif endif -prefix = get_option('prefix') -sbindir = join_paths(prefix, get_option('sbindir')) -libdir = join_paths(prefix, get_option('libdir')) -mandir = join_paths(prefix, get_option('mandir')) - install_headers([ 'include/libflashrom.h', ], @@ -461,7 +456,7 @@ output : 'flashrom.8', configuration : conf, install: true, - install_dir: join_paths(mandir, 'man8'), + install_dir: join_paths(get_option('mandir'), 'man8'), ) flashrom_dep = declare_dependency( @@ -501,7 +496,7 @@ cargs ], install : true, - install_dir : sbindir, + install_dir : get_option('sbindir') ) subdir('util') -- To view, visit https://review.coreboot.org/c/flashrom/+/64028 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I9cb9faf4bdbcfd66098478cc3a260eb3b664a2e6 Gerrit-Change-Number: 64028 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: flashrom: Drop read_flash_to_file() usage
by Angel Pons (Code Review) 03 May '22

03 May '22

Attention is currently required from: Nico Huber, Edward O'Callaghan. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/59291 ) Change subject: flashrom: Drop read_flash_to_file() usage ...................................................................... Patch Set 14: Code-Review+1 -- To view, visit https://review.coreboot.org/c/flashrom/+/59291 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I4b690b688acf9d5deb46e8642a252a2132ea8c73 Gerrit-Change-Number: 59291 Gerrit-PatchSet: 14 Gerrit-Owner: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Daniel Campello <campello(a)chromium.org> Gerrit-Reviewer: Nikolai Artemiev <nartemiev(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Nico Huber <nico.h(a)gmx.de> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Thomas Heijligen <src(a)posteo.de> Gerrit-Attention: Nico Huber <nico.h(a)gmx.de> Gerrit-Attention: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Comment-Date: Tue, 03 May 2022 09:42:57 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit May 2022