April 2025 - flashrom-gerrit - mail.coreboot.org

[M] Change in flashrom[main]: libflashrom: Add probing v2 which can find all mathching chips
by Anastasia Klimchuk (Code Review) 16 Apr '25

16 Apr '25

Attention is currently required from: Dmitry Zhadinets, Peter Marheine. Anastasia Klimchuk has posted comments on this change by Anastasia Klimchuk. ( https://review.coreboot.org/c/flashrom/+/87341?usp=email ) Change subject: libflashrom: Add probing v2 which can find all mathching chips ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: Finally I did what I was talking about, probing v2 It still has few things to do, I added comments for myself, plus polish - but I wanted to share early so that we can discuss! My main goal was to have the same logic (and code) for probing in cli and libflashrom, but as an additional consideration, in the context of CB:87251 I am thinking this might also solve the programmer capabilities question - since it now returns info of all matches. -- To view, visit https://review.coreboot.org/c/flashrom/+/87341?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: Idfcf377a8071e22028ba98515f08495ed2a6e9f0 Gerrit-Change-Number: 87341 Gerrit-PatchSet: 1 Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Comment-Date: Wed, 16 Apr 2025 13:20:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[M] Change in flashrom[main]: libflashrom: Add probing v2 which can find all mathching chips
by Anastasia Klimchuk (Code Review) 16 Apr '25

16 Apr '25

Anastasia Klimchuk has posted comments on this change by Anastasia Klimchuk. ( https://review.coreboot.org/c/flashrom/+/87341?usp=email ) Change subject: libflashrom: Add probing v2 which can find all mathching chips ...................................................................... Patch Set 1: (3 comments) File cli_classic.c: https://review.coreboot.org/c/flashrom/+/87341/comment/e63e4059_68640d19?us… : PS1, Line 1050: struct flashctx flashes[8] = {{0}}; This array won't be needed after this patch, in the next commit it can be replaced with just one flash context. It was only one context that was used, the rest was needed for probing only. The probing (and temp contexts) is now inside libflashrom. https://review.coreboot.org/c/flashrom/+/87341/comment/dd130a42_9d8b13ce?us… : PS1, Line 1215: flashrom_flash_probe_v2 Check the return value File tests/lifecycle.c: https://review.coreboot.org/c/flashrom/+/87341/comment/71579532_f4ea7535?us… : PS1, Line 27: flashrom_flash_probe_v2 Tests needs to be added (not replaced), leave existing probing tests and add new for v2. -- To view, visit https://review.coreboot.org/c/flashrom/+/87341?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: Idfcf377a8071e22028ba98515f08495ed2a6e9f0 Gerrit-Change-Number: 87341 Gerrit-PatchSet: 1 Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 16 Apr 2025 13:03:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[M] Change in flashrom[main]: libflashrom: Add probing v2 which can find all mathching chips
by Anastasia Klimchuk (Code Review) 16 Apr '25

16 Apr '25

Anastasia Klimchuk has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/87341?usp=email ) Change subject: libflashrom: Add probing v2 which can find all mathching chips ...................................................................... libflashrom: Add probing v2 which can find all mathching chips Probing v2 can (if requested) go through all flashchips and find all the matching chip definitions. This is the way cli behaves, so cli becomes a client of probing v2. Previously cli and libflashrom had different probing logic, and different code in different source files. TODO here is to create double set of tests. In the first version of the patch all probing tests are force-redirected to probe_v2, just to quickly get test coverage. Testing from the cli: ./flashrom -p dummy:emulate=W25Q128FV -r dump.rom ./flashrom -p dummy:emulate=MX25L6436 -r dump.rom ./flashrom -p dummy:emulate=MX25L6436 -c "MX25L6405" -r dump.rom ./flashrom -p dummy:emulate=SST25VF032B -E ./flashrom -p dummy:emulate=S25FL128L -r dump.rom Change-Id: Idfcf377a8071e22028ba98515f08495ed2a6e9f0 Signed-off-by: Anastasia Klimchuk <aklm(a)flashrom.org> --- M cli_classic.c M include/libflashrom.h M libflashrom.c M libflashrom.map M tests/lifecycle.c 5 files changed, 101 insertions(+), 25 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/41/87341/1 diff --git a/cli_classic.c b/cli_classic.c index 3e5dab9..798c218 100644 --- a/cli_classic.c +++ b/cli_classic.c @@ -1050,7 +1050,7 @@ struct flashctx flashes[8] = {{0}}; struct flashctx *fill_flash; char *tempstr = NULL; - int startchip = -1, chipcount = 0; + int startchip = -1; int i, j; int ret = 0; @@ -1209,26 +1209,21 @@ msg_pdbg("The following protocols are supported: %s.\n", tempstr ? tempstr : "?"); free(tempstr); - for (j = 0; j < registered_master_count; j++) { - startchip = 0; - while (chipcount < (int)ARRAY_SIZE(flashes)) { - startchip = probe_flash(&registered_masters[j], startchip, &flashes[chipcount], 0, options.chip_to_probe); - if (startchip == -1) - break; - chipcount++; - startchip++; - } - } + const char *all_matched_names[20]; + unsigned int all_matched_count; - if (chipcount > 1) { + flashrom_flash_probe_v2(&flashes[0], all_matched_names, &all_matched_count, + NULL, options.chip_to_probe); + + if (all_matched_count > 1) { msg_cinfo("Multiple flash chip definitions match the detected chip(s): \"%s\"", flashes[0].chip->name); - for (i = 1; i < chipcount; i++) - msg_cinfo(", \"%s\"", flashes[i].chip->name); + for (unsigned int ind = 1; ind < all_matched_count; ind++) + msg_cinfo(", \"%s\"", all_matched_names[ind]); msg_cinfo("\nPlease specify which chip definition to use with the -c <chipname> option.\n"); ret = 1; goto out_shutdown; - } else if (!chipcount) { + } else if (!all_matched_count) { msg_cinfo("No EEPROM/flash device found.\n"); if (!options.force || !options.chip_to_probe) { msg_cinfo("Note: flashrom can never write if the flash chip isn't found " @@ -1535,9 +1530,9 @@ out_shutdown: flashrom_programmer_shutdown(NULL); out: - for (i = 0; i < chipcount; i++) { - flashrom_layout_release(flashes[i].default_layout); - free(flashes[i].chip); + for (unsigned int ind = 0; ind < all_matched_count; ind++) { + flashrom_layout_release(flashes[ind].default_layout); + free(flashes[ind].chip); } free_options(&options); diff --git a/include/libflashrom.h b/include/libflashrom.h index f30ad6b..19e5b97 100644 --- a/include/libflashrom.h +++ b/include/libflashrom.h @@ -279,6 +279,42 @@ * or 1 on any other error. */ int flashrom_flash_probe(struct flashrom_flashctx **flashctx, const struct flashrom_programmer *flashprog, const char *chip_name); + + +/** + * @brief Probe for a flash chip, v2 + * + * Probes for a flash chip and returns a flash context, that can be used + * later with flash chip and @ref flashrom-ops "image operations", if + * exactly one matching chip is found. + * + * Returns the list of names for all chips that matched, and the count of + * how many chips matched. + * + * @param[out] flashctx Points to a struct flashrom_flashctx + * that will be set if exactly one chip is found. *flashctx + * has to be freed by the caller with @ref flashrom_flash_release. + * @param[out] all_matched_names list of all names of chips that were probed + * successfully during probing process, or empty + if no chips found. + * @param[out] all_matched_count the total number of chip that were probed + successfully during probing process, or 0 + if no chips found. + * @param[in] flashprog The flash programmer used to access the chip, + currently unused. + * @param[in] chip_name Name of a chip to probe for, or NULL to probe for + * all known chips. + * @return 0 on success, + * 3 if multiple chips were found, + * 2 if no chip was found, + * or 1 on any other error. + */ +int flashrom_flash_probe_v2(struct flashrom_flashctx *flashctx, + const char **all_matched_names, + unsigned int *all_matched_count, + const struct flashrom_programmer *flashprog, + const char *chip_name); + /** * @brief Returns the size of the specified flash chip in bytes. * diff --git a/libflashrom.c b/libflashrom.c index d6bd098..c72c7a7 100644 --- a/libflashrom.c +++ b/libflashrom.c @@ -329,8 +329,6 @@ return programmer_shutdown(); } -/* TODO: flashrom_programmer_capabilities()? */ - int flashrom_flash_probe(struct flashrom_flashctx **const flashctx, const struct flashrom_programmer *const flashprog, const char *const chip_name) @@ -363,6 +361,47 @@ return ret; } +int flashrom_flash_probe_v2(struct flashrom_flashctx *flashctx, + const char **all_matched_names, + unsigned int *all_matched_count, + const struct flashrom_programmer *flashprog, + const char *chip_name) +{ + int startchip; + int ret = 2; // return code 2 means chip not found, start with this + + *all_matched_count = 0; + + for (int i = 0; i < registered_master_count; i++) { + startchip = 0; + while (*all_matched_count < flashchips_size) { + struct flashrom_flashctx second_flashctx = { 0, }; // used for second and more matches + struct flashctx *context_for_probing = (*all_matched_count > 0) ? &second_flashctx : flashctx; + startchip = probe_flash(&registered_masters[i], startchip, context_for_probing, 0, chip_name); + + if (startchip == -1) + break; + + all_matched_names[*all_matched_count] = context_for_probing->chip->name; + (*all_matched_count)++; + startchip++; + ret = 0; + + if (*all_matched_count > 1) { + ret = 3; + /* It's used for the second and subsequent probing. */ + flashrom_layout_release(second_flashctx.default_layout); + free(second_flashctx.chip); + } + } + } + + all_matched_names[*all_matched_count] = 0; + + return ret; +} + + size_t flashrom_flash_getsize(const struct flashrom_flashctx *const flashctx) { return flashctx->chip->total_size * 1024; diff --git a/libflashrom.map b/libflashrom.map index fc6724a..2b757e5 100644 --- a/libflashrom.map +++ b/libflashrom.map @@ -6,6 +6,7 @@ flashrom_flash_erase; flashrom_flash_getsize; flashrom_flash_probe; + flashrom_flash_probe_v2; flashrom_flash_release; flashrom_image_read; flashrom_image_verify; diff --git a/tests/lifecycle.c b/tests/lifecycle.c index 95c4249..f5509c8 100644 --- a/tests/lifecycle.c +++ b/tests/lifecycle.c @@ -19,13 +19,18 @@ struct flashrom_programmer *flashprog, const char *const chip_name) { - struct flashrom_flashctx *flashctx; + struct flashrom_flashctx flashctx = { 0 }; + unsigned int all_matched_count; + const char *all_matched_names[20]; - printf("Testing flashrom_flash_probe for programmer=%s, chip=%s ... \n", prog->name, chip_name); - assert_int_equal(0, flashrom_flash_probe(&flashctx, flashprog, chip_name)); - printf("... flashrom_flash_probe for programmer=%s successful\n", prog->name); + printf("Testing flashrom_flash_probe_v2 for programmer=%s, chip=%s ... \n", prog->name, chip_name); + assert_int_equal(0, flashrom_flash_probe_v2(&flashctx, all_matched_names, &all_matched_count, + flashprog, chip_name)); + printf("... flashrom_flash_probe_v2 for programmer=%s successful\n", prog->name); - flashrom_flash_release(flashctx); /* cleanup */ + /* cleanup */ + flashrom_layout_release(flashctx.default_layout); + free(flashctx.chip); } static void run_lifecycle(void **state, const struct io_mock *io, const struct programmer_entry *prog, -- To view, visit https://review.coreboot.org/c/flashrom/+/87341?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: Idfcf377a8071e22028ba98515f08495ed2a6e9f0 Gerrit-Change-Number: 87341 Gerrit-PatchSet: 1 Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>

1 0

[S] Change in flashrom[main]: cli: Use flashrom_supported_programmers API in cli
by Anastasia Klimchuk (Code Review) 12 Apr '25

12 Apr '25

Attention is currently required from: Dmitry Zhadinets, Peter Marheine. Anastasia Klimchuk has posted comments on this change by Dmitry Zhadinets. ( https://review.coreboot.org/c/flashrom/+/87195?usp=email ) Change subject: cli: Use flashrom_supported_programmers API in cli ...................................................................... Patch Set 2: (1 comment) Patchset: PS2: I just thought about something, it's for the separate commit. I am writing here but it's for separate patch. It would be great if you could add information about your new APIs (3 already submitted!) to our Recent development doc. We accumulate ongoing development between releases, so that people know what's going on and then it makes it easier to write release notes (and not forget anything). Often it's included in the same patch, but it's fine to add later. The doc is in `doc/release_notes/devel.rst` and it will be displayed here https://flashrom.org/release_notes/devel.html Also a doc how to add docs https://flashrom.org/how_to_add_docs.html Thank you so much! -- To view, visit https://review.coreboot.org/c/flashrom/+/87195?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: I86ac83ee0ef4e850e69aa4e7b607e28ebab9d3d5 Gerrit-Change-Number: 87195 Gerrit-PatchSet: 2 Gerrit-Owner: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Comment-Date: Sat, 12 Apr 2025 12:33:45 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[M] Change in flashrom[main]: libflashrom: Add API for capabilitible chips
by Anastasia Klimchuk (Code Review) 12 Apr '25

12 Apr '25

Attention is currently required from: Dmitry Zhadinets, Peter Marheine. Anastasia Klimchuk has posted comments on this change by Dmitry Zhadinets. ( https://review.coreboot.org/c/flashrom/+/87251?usp=email ) Change subject: libflashrom: Add API for capabilitible chips ...................................................................... Patch Set 4: (1 comment) File tests/tests.c: https://review.coreboot.org/c/flashrom/+/87251/comment/588af163_e5e4dbcd?us… : PS4, Line 496: const struct CMUnitTest libflashrom_tests[] = { : cmocka_unit_test(flashrom_set_log_callback_test_success), : cmocka_unit_test(flashrom_set_log_callback_v2_test_success), : cmocka_unit_test(flashrom_set_log_level_test_success), : cmocka_unit_test(flashrom_supported_programmers_test_success), : cmocka_unit_test(flashrom_programmer_capabilities_test_success), : }; : ret |= cmocka_run_group_tests_name("libflashrom.c tests", libflashrom_tests, NULL, NULL); : I could repro the failing test/spi25.c when I downloaded the patch and reverted the diffs in tests/tests.c (moved the lines back). 4 errors, they look similar like this: ``` [ ERROR ] --- %s() has remaining non-returned values. : __wrap_spi_send_command../tests/spi25.c:167: note: remaining item was declared here ../tests/spi25.c:168: note: remaining item was declared here ../tests/spi25.c:169: note: remaining item was declared here __wrap_spi_send_command: '%s' parameter still has values that haven't been checked. : flash../tests/spi25.c:164: note: remaining item was declared here ``` I am not sure which test has a bug, need to have a look. The failing sequence has libflashrom_tests first and spi25_tests after that. -- To view, visit https://review.coreboot.org/c/flashrom/+/87251?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: Id33fac2935b4098586526bda87b25231c9a6ab39 Gerrit-Change-Number: 87251 Gerrit-PatchSet: 4 Gerrit-Owner: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Comment-Date: Sat, 12 Apr 2025 11:46:57 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[M] Change in flashrom[main]: libflashrom: Add API for capabilitible chips
by Anastasia Klimchuk (Code Review) 12 Apr '25

12 Apr '25

Attention is currently required from: Dmitry Zhadinets, Peter Marheine. Anastasia Klimchuk has posted comments on this change by Dmitry Zhadinets. ( https://review.coreboot.org/c/flashrom/+/87251?usp=email ) Change subject: libflashrom: Add API for capabilitible chips ...................................................................... Patch Set 4: (1 comment) Patchset: PS2: > It turns out that DUMMY is disabled in some cases. […] Sorry for some delay, while you asked for help. I read the comment and info in commit message, it seems there are few questions? 1) what happened in spi test, you said there is a bug? do you have error logs? I will try to repro. It's good if you found bug in another test, it can go in a separate commit 2) dummy has several options what to emulate (see https://github.com/flashrom/flashrom/blob/main/dummyflasher.c#L31, string values in the code https://github.com/flashrom/flashrom/blob/main/dummyflasher.c#L1216) and one of them has duplicate IDs as of now. I forgot which one, probably EMULATE_MACRONIX_MX25L6436. If you emulate it should find two matches in flashchips. 3) I still want to create a patch for probing v2, but it will need test as well, so tests are useful anyway. -- To view, visit https://review.coreboot.org/c/flashrom/+/87251?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: Id33fac2935b4098586526bda87b25231c9a6ab39 Gerrit-Change-Number: 87251 Gerrit-PatchSet: 4 Gerrit-Owner: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Comment-Date: Sat, 12 Apr 2025 10:43:16 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Dmitry Zhadinets <dzhadinets(a)gmail.com>

1 0

[S] Change in flashrom[main]: cli: Use flashrom_supported_programmers API in cli
by Anastasia Klimchuk (Code Review) 12 Apr '25

12 Apr '25

Attention is currently required from: Dmitry Zhadinets, Peter Marheine. Anastasia Klimchuk has posted comments on this change by Dmitry Zhadinets. ( https://review.coreboot.org/c/flashrom/+/87195?usp=email ) Change subject: cli: Use flashrom_supported_programmers API in cli ...................................................................... Patch Set 2: (2 comments) Patchset: PS2: > I think that this change is not needed because the code has become more complex […] I can see what you mean by saying the code is more complex. But I think we should be using our own API wherever it's applicable: it gives an example of correct usage to people, and also can help to detect breakage. I just looked and it's a bit sad that the other functions print_supported_* are not using libflashrom :( I think it's better to gradually change them one by one and maybe in future have them all using libflashrom. Commit Message: https://review.coreboot.org/c/flashrom/+/87195/comment/333a962b_31f16a1e?us… : PS2, Line 9: Testing: Both unit tests and CLI tools serve as libflashrom clients. : All unit tests run successfully. I have a suggestion instead of these two lines (which are true but very generic) add testing info which is specific to the patch. Concretely, in this case running `flashrom -h` (help) or `flashrom -L` (info on what's supported) will run the code which this patch is changing. So instead I suggest something like: Testing: `flashrom -h` and `flashrom -L` both print the list of supported programmers successfully -- To view, visit https://review.coreboot.org/c/flashrom/+/87195?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: I86ac83ee0ef4e850e69aa4e7b607e28ebab9d3d5 Gerrit-Change-Number: 87195 Gerrit-PatchSet: 2 Gerrit-Owner: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Comment-Date: Sat, 12 Apr 2025 09:42:14 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Dmitry Zhadinets <dzhadinets(a)gmail.com>

1 0

[XS] Change in flashrom[main]: doc: Update supported flashchips page because we split the large file
by Anastasia Klimchuk (Code Review) 10 Apr '25

10 Apr '25

Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/86953?usp=email ) Change subject: doc: Update supported flashchips page because we split the large file ...................................................................... doc: Update supported flashchips page because we split the large file Change-Id: Ic6179517d0f951a32c0c4e0baf32677398224542 Signed-off-by: Anastasia Klimchuk <aklm(a)flashrom.org> Reviewed-on: https://review.coreboot.org/c/flashrom/+/86953 Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org> --- M doc/supported_hw/supported_flashchips.rst 1 file changed, 6 insertions(+), 3 deletions(-) Approvals: build bot (Jenkins): Verified Stefan Reinauer: Looks good to me, approved diff --git a/doc/supported_hw/supported_flashchips.rst b/doc/supported_hw/supported_flashchips.rst index 8e881d2..9af2601 100644 --- a/doc/supported_hw/supported_flashchips.rst +++ b/doc/supported_hw/supported_flashchips.rst @@ -2,10 +2,13 @@ Supported flash chips ===================== -The list of all supported flash chips is in ``flashchips.c`` file in the source tree. -If you have a flashrom repo cloned locally, you can look at the file in your repo. +The info about all supported flash chips is in the ``/flashchips`` directory in the source tree. +If you have a flashrom repo cloned locally, you can look at this directory in your repo. -Alternatively inspect the file on the `web UI of our GitHub mirror <https://github.com/flashrom/flashrom/blob/main/flashchips.c>`_. +Alternatively inspect it on the `web UI of our GitHub mirror <https://github.com/flashrom/flashrom/tree/main/flashchips>`_. + +All the files in the ``/flashchips`` directory are included in parent file ``flashchips.c``. You can inspect the source +`here <https://github.com/flashrom/flashrom/blob/main/flashchips.c>`_. If you can run flashrom locally, the command ``flashrom -L`` prints the list of all supported flash chips (see :doc:`/classic_cli_manpage` for more details on command line options). The output of this command is long, so you might -- To view, visit https://review.coreboot.org/c/flashrom/+/86953?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: Ic6179517d0f951a32c0c4e0baf32677398224542 Gerrit-Change-Number: 86953 Gerrit-PatchSet: 4 Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

[XS] Change in flashrom[main]: flashchips: Add GD25B127D to the models
by Anastasia Klimchuk (Code Review) 10 Apr '25

10 Apr '25

Anastasia Klimchuk has abandoned this change. ( https://review.coreboot.org/c/flashrom/+/86977?usp=email ) Change subject: flashchips: Add GD25B127D to the models ...................................................................... Abandoned This patch has newer version CB:86996 -- To view, visit https://review.coreboot.org/c/flashrom/+/86977?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: abandon Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: Ifa96a4d5c9cee6b74d561622664d52835caafa3f Gerrit-Change-Number: 86977 Gerrit-PatchSet: 1 Gerrit-Owner: Victor Lim <vlim(a)gigadevice.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Nikolai Artemiev <nartemiev(a)google.com> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

[M] Change in flashrom[main]: libflashrom: Add API for capabilitible chips
by Dmitry Zhadinets (Code Review) 10 Apr '25

10 Apr '25

Attention is currently required from: Anastasia Klimchuk. Dmitry Zhadinets has posted comments on this change by Dmitry Zhadinets. ( https://review.coreboot.org/c/flashrom/+/87251?usp=email ) Change subject: libflashrom: Add API for capabilitible chips ...................................................................... Patch Set 4: (1 comment) Patchset: PS2: > I need help with the test. […] It turns out that DUMMY is disabled in some cases. I could check the return value of programmer_init(), but… to be honest, I don’t like this test anyway—there’s no variability in it. Do you have any ideas on how to test compatibility more effectively? -- To view, visit https://review.coreboot.org/c/flashrom/+/87251?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: Id33fac2935b4098586526bda87b25231c9a6ab39 Gerrit-Change-Number: 87251 Gerrit-PatchSet: 4 Gerrit-Owner: Dmitry Zhadinets <dzhadinets(a)gmail.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Comment-Date: Thu, 10 Apr 2025 05:31:27 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Dmitry Zhadinets <dzhadinets(a)gmail.com>

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit April 2025