)]}' { "log": [ { "commit": "dad3a16e252142df49694e9b5db9e002e41eb9a1", "tree": "166ada4dbef3c3f2f517bfd9f7a211bf79ca715e", "parents": [ "40407b60cccd64a655647d48a80d686ddafc4b13" ], "author": { "name": "Richard Hughes", "email": "richard@hughsie.com", "time": "Mon Feb 17 09:57:01 2020 +0000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Jan 13 01:34:15 2023 +0000" }, "message": "Install the man file when using meson as a buildsystem\n\nThis fixes a regression with the Fedora package.\n\nChange-Id: I881bd5002a842072ce9dadea033c51a2668f9e7c\nSigned-off-by: Richard Hughes \u003crichard@hughsie.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/38939\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: David Hendricks \u003cdavid.hendricks@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71738\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "40407b60cccd64a655647d48a80d686ddafc4b13", "tree": "cc03a5ce347e3cbbac4b829026ed2578e194b4a8", "parents": [ "332fdabea41b542c7ab3f56d9c1d9c5fdc492d63" ], "author": { "name": "Idwer Vollering", "email": "vidwer@gmail.com", "time": "Tue Dec 08 00:24:39 2020 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Jan 13 01:34:15 2023 +0000" }, "message": "cli_classic.c: fix minor cosmetic bug when support for wiki page generation is not compiled in\n\nChange-Id: I222fd7aa5d633fe81ef1894d67dcb25ba0dd8937\nSigned-off-by: Idwer Vollering \u003cvidwer@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/48430\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71736\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Elyes Haouas \u003cehaouas@noos.fr\u003e\n" }, { "commit": "332fdabea41b542c7ab3f56d9c1d9c5fdc492d63", "tree": "09325ad92002fdd2a6600a6215f81a6c6bc8badb", "parents": [ "c92f6876ed82488365c3ef1c59dd4669eb512527" ], "author": { "name": "Angel Pons", "email": "th3fanbus@gmail.com", "time": "Wed Dec 21 16:34:39 2022 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Jan 13 01:34:15 2023 +0000" }, "message": "amd_imc.c: Fix unusual typo in log message\n\nReplace a `)` with a `.` for consistency with other log messages.\n\nChange-Id: I977990237821f6aec8f127bc3994a1f3f3a0a350\nSigned-off-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/71184\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Elyes Haouas \u003cehaouas@noos.fr\u003e\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71735\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Elyes Haouas \u003cehaouas@noos.fr\u003e\n" }, { "commit": "c92f6876ed82488365c3ef1c59dd4669eb512527", "tree": "b1926b71b105e94f826c646e13d85d84a295e4e5", "parents": [ "a60f641a7f8b1a4cd602eecb8ddc247e40777400" ], "author": { "name": "Angel Pons", "email": "th3fanbus@gmail.com", "time": "Tue Mar 31 15:32:10 2020 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Jan 13 01:34:15 2023 +0000" }, "message": "spi25.c: Add a null check\n\nChange-Id: I5fcc23d81b8404af90768afa2954509bf334ab2c\nSigned-off-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/39974\nOriginal-Reviewed-by: HAOUAS Elyes \u003cehaouas@noos.fr\u003e\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71734\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\n" }, { "commit": "a60f641a7f8b1a4cd602eecb8ddc247e40777400", "tree": "91cb81687cbd41227f4c40ab2c07a75f210169a0", "parents": [ "721396876e9bca4307e46fe01bb9c0936e01d234" ], "author": { "name": "Angel Pons", "email": "th3fanbus@gmail.com", "time": "Tue Mar 31 15:02:02 2020 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Jan 13 01:34:15 2023 +0000" }, "message": "cli_classic.c: Prevent memory leak\n\nIf the `--flash-contents` switch is specified more than once, it will\nresult in a memory leak. Therefore, allow this option only once.\n\nChange-Id: I530933c9a6431580fe4645396bb363939472a80a\nSigned-off-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/39972\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71733\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\n" }, { "commit": "721396876e9bca4307e46fe01bb9c0936e01d234", "tree": "4109135dd8b818a0b3d49888dbb8616d7f1135f2", "parents": [ "ac90af6cdc747bfe3dc38c83c0b7272addf37659" ], "author": { "name": "Michael Niewöhner", "email": "foss@mniewoehner.de", "time": "Tue Sep 21 20:14:42 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Jan 13 01:34:15 2023 +0000" }, "message": "flashrom.8: replace `svn changelog` with `git history`\n\nChange-Id: If8659dd603cbabdb8e20d89f818072986373e24f\nSigned-off-by: Michael Niewöhner \u003cfoss@mniewoehner.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57811\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71732\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\n" }, { "commit": "ac90af6cdc747bfe3dc38c83c0b7272addf37659", "tree": "ec67fd7c4d01db82b5a1ffd8c8ed36a7229108dd", "parents": [ "bb4f3b06dcfb60a6ab84750c9b149482dc5ee579" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Sun Dec 18 00:22:47 2022 +0000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Jan 13 01:34:15 2023 +0000" }, "message": "Change references to flashrom-stable\n\nAdapt all mentions of the mailing list and also the version print.\n\nChange-Id: Ib4a3271422ee6cf4d0efb8c3fa858b66a22c0a33\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/70922\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "bb4f3b06dcfb60a6ab84750c9b149482dc5ee579", "tree": "d6fb2ecdbf18b14ab226e450f298f8976cc063ca", "parents": [ "f60f64fe5aa6c3f30a994b35d614a048f026e501" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Dec 30 14:28:06 2022 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Jan 13 01:34:15 2023 +0000" }, "message": "cli_common: Skip printing WP test status\n\nFirst version of flashrom-stable won\u0027t have CLI support for write\nprotection. Hence remove messages that could confuse the user.\n\nChange-Id: Ia654d3dd69be9778608d8f8762d6531893ff60d0\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71557\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "f60f64fe5aa6c3f30a994b35d614a048f026e501", "tree": "5893ca00ee1a07fae312c95ee963b4a358af63e7", "parents": [ "245973b44fe9702f74894068840128d8a11eb8ee" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Sat Nov 12 12:08:01 2022 +1100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:46:20 2023 +0000" }, "message": "flashrom.c: Position heap alloc along side check in compare_range()\n\nChange-Id: I0386ac4c09a541cb9a659b2410ce49c3292ecc6e\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/69473\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Alexander Goncharov \u003cchat@joursoir.net\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71499\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\n" }, { "commit": "245973b44fe9702f74894068840128d8a11eb8ee", "tree": "f3918b3c7fba90725ab38499a1dd2fcdd530a026", "parents": [ "a31a572b44f95841c74d48e692586a46595dd928" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Dec 02 03:08:27 2022 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:46:20 2023 +0000" }, "message": "cbtables.c/search_lb_records: Drop unused variable `count`\n\nClang 15 complains about it. Remove it.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I340208f513bed57a9cc2bba880a2400352c5cc42\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/70247\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71498\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "a31a572b44f95841c74d48e692586a46595dd928", "tree": "72fb17517f6e6084c4443d0aa7b9332b315519af", "parents": [ "bd27581be60e7c59b3d0f814467260196c261281" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Sat Nov 12 12:05:36 2022 +1100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:46:20 2023 +0000" }, "message": "tree/: Make heap alloc checks err msg consistent\n\nChange-Id: Id84a9f15c33781efc494ed36a1c7cec82a0333d6\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/69472\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71497\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "bd27581be60e7c59b3d0f814467260196c261281", "tree": "b444aee0a4bed45d7276377c89d57ddf852e16e3", "parents": [ "7db7059e0ebe1b655c352dec8cf99fe31cc3876b" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Mon Nov 28 11:20:44 2022 +1100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "internal.c: laptop_ok global state can become stale\n\nCraask and similar DUT\u0027s are erroneously probing random second chips.\n\n```\nFound chipset \"Intel Alder Lake-N\".\nEnabling flash write... SPI Configuration is locked down.\nFREG0: Flash Descriptor region (0x00000000-0x00000fff) is read-write.\nFREG1: BIOS region (0x003a0000-0x00ffffff) is read-write.\nFREG2: Management Engine region (0x00001000-0x0039ffff) is read-write.\nOK.\nFound Winbond flash chip \"W25Q128.V..M\" (16384 kB, Programmer-specific) on host.\nWarning: Setting BIOS Control at 0xdc from 0x8b to 0x89 failed.\nNew value is 0x8b.\nFound MoselVitelic flash chip \"V29C51000T\" (64 kB, Parallel) mapped at physical address 0x00000000ffff0000.\n```\n\nThis seems to be due to `laptop_ok` becoming a stale global state\nafter the first operation leading to probing on unrelated buses.\n\nTherefore unconditionally reset the global state upon entry into\nthe internal driver.\n\nTested: Craask reportly no longer finds duplicate chip.\n\nChange-Id: I2c00c351904307eeb1488c5dfaffc91d6468ee25\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/70026\nOriginal-Reviewed-by: Sam McNally \u003csammc@google.com\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71496\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\n" }, { "commit": "7db7059e0ebe1b655c352dec8cf99fe31cc3876b", "tree": "3136dc58205030072618b10f1b42cc2f0151e51c", "parents": [ "9441c6e36f4ad152125f110ed83a6d72a97a4b02" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Sat Nov 12 12:00:24 2022 +1100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "flashrom.c: Drop redundant chip read validation in verify_range()\n\nThe \u0027chip_safety_check()\u0027 already validates the chip structure\nwithin \u0027prepare_flash_access()\u0027 before all subsequent chip operations\nsuch as \u0027verify_range()\u0027 and therefore the chip structure is\nguaranteed to be valid in the domain of those operations.\n\nChange-Id: Ifc57dd89715115e03d013691352463a8b3c0dc52\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/69471\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71495\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "9441c6e36f4ad152125f110ed83a6d72a97a4b02", "tree": "8270188dfee8ed9ea9c9a304a11c6dd4d962b2a6", "parents": [ "0c8221bb4462d7410a4667ab3526502c5eaa241d" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Oct 21 12:47:11 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "Makefile: Don\u0027t install git hooks automatically\n\nThese specific git hooks are only needed when someone wants to push a\npatch to upstream and so it\u0027s not needed to run it in every make call.\nBeside that, we also don\u0027t know the environment in which this is\nexecuted and it might not be wanted.\n\nThus, add a new make target `gitconfig` and move the install command to\nit. It can be used by running `make gitconfig`.\n\nflashrom-stable: Add `gitconfig` to the `.PHONY` targets.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: Ib83568c7ff149a8ec34ad7e92720c36a89def7bd\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/68647\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71494\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "0c8221bb4462d7410a4667ab3526502c5eaa241d", "tree": "f2264704cde0950882873089f8c8d16154c401a1", "parents": [ "e28d8e4e2332054fb4b7f61e8ce03f316e3a63f3" ], "author": { "name": "Angel Pons", "email": "th3fanbus@gmail.com", "time": "Thu Oct 20 21:23:33 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "chipset_enable.c: Mark Intel C246 as DEP\n\nTested reading, writing and erasing the flash chip of a Prodrive\nHermes mainboard with an Intel C246 PCH. However, since ME-enabled\nchipsets are marked as DEP instead of OK, this one shall also be.\n\nChange-Id: I07d6c4a60e468c61eba836db91e1335f4a762048\nSigned-off-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/68594\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71493\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "e28d8e4e2332054fb4b7f61e8ce03f316e3a63f3", "tree": "dcd1df2c485d8da20c2845e584bd157e0e4f63fb", "parents": [ "6a2ebebfabff9ae72f8f03076b015a6b21bf8b90" ], "author": { "name": "Jean THOMAS", "email": "virgule@jeanthomas.me", "time": "Tue Oct 11 17:54:30 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "dirtyjtag: Add DirtyJTAG programmer\n\nAdd a new programmer driver for the DirtyJTAG project (a USB-JTAG\nfirmware for STM32 MCUs).\n\nSuccessfully tested with DirtyJTAG 1.4 running on an Olimex STM32-H103\ndevelopment board and a SST25VF020B SPI flash chip.\n\nChange-Id: Ic43e9a014ed7d04e429e73b30c9dcfdde1a78913\nSigned-off-by: Jean THOMAS \u003cvirgule@jeanthomas.me\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/67878\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71492\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "6a2ebebfabff9ae72f8f03076b015a6b21bf8b90", "tree": "1828bbf5c718a40a07c75eee81aa6df2062ed593", "parents": [ "dd5f69301cb36a00ce28868af9df1711eaf9fc17" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Fri Aug 26 11:36:48 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "programmer_init: Work on a mutable copy of programmer params\n\nThe signature of extract_param() was wrong all the time. It actually\nmodifies the passed, global `programmer_param` string. This only com-\npiled w/o warnings because of a deficiency of the strstr() API. It\ntakes a const string as argument but returns a mutable pointer to\na substring of it.\n\nAs we take a const parameter string in the libflashrom API and should\nnot change that, we create a copy in programmer_init() instead.\n\nNow that we free our copy of the programmer parameters at the end of\nprogrammer_init() it\u0027s more obvious that the string can only be used\nduring initialization. So also clear `programmer_param` inside\nprogrammer_init() instead of programmer_shutdown().\n\nChange-Id: If6bb2e5e4312b07f756615984bd3757e92b86b0a\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/67094\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71491\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "dd5f69301cb36a00ce28868af9df1711eaf9fc17", "tree": "21534da755e110c1dfc63b37dd03e5ba6d72cd3a", "parents": [ "2fb53b1f30ae7c811e3bfcf9fc96b8db9182e61f" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 03:33:27 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "dmi.c: Retype variable `anchored` with bool\n\nUse the bool type instead of an integer for the variable `anchored`,\nsince this represents its purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I5bedaf47cdd2fd5d152f69b6b02aab69fd1285c6\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66903\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71490\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "2fb53b1f30ae7c811e3bfcf9fc96b8db9182e61f", "tree": "17ce3c1dc60eae86942d8e9302f7c173acfa6304", "parents": [ "d68a0ece50b2250ba25be67d27e8606247d5306c" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 03:29:32 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "dummyflasher.c: Retype appropriate variables and attributes with bool\n\nUse the bool type instead of an integer for appropriate variables and\nattributes, since this represents their purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I712b1ef7e1ad74d3e004dcf36c82898c88072c63\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66901\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71489\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "d68a0ece50b2250ba25be67d27e8606247d5306c", "tree": "97d0a03f0b60954a0463ed2d1be9fc3237a867fb", "parents": [ "085572812789076e05faffd677264470dcea14c6" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 03:23:35 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "ich_descriptors.c: Retype variable `has_forbidden_opcode` with bool\n\nUse the bool type instead of an integer for the variable\n`has_forbidden_opcode`, since this represents its purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I6214956e377547b894fc76237832b6f7b2db41dd\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66899\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71488\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "085572812789076e05faffd677264470dcea14c6", "tree": "8c3f0e3c0afe36319ec98f0a1f1ba5e631b5c990", "parents": [ "7b83f3c2e3e201faf63c04b8ec2477eb3ebe9577" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 03:19:06 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "fmap.c: Retype appropriate variables with bool\n\nUse the bool type instead of an integer for appropriate variables, since\nthis represents their purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I323d40cabe9c580057d870e742b3b55942c78321\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66896\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71487\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "7b83f3c2e3e201faf63c04b8ec2477eb3ebe9577", "tree": "1e0857735df89e5508adbc7240d7c253081d2cfc", "parents": [ "d1ab7d2bb0c4ea7be2e251caa564ab91d27ee7ea" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 03:16:19 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "ft2232_spi.c: Retype variable `clock_5x` with bool\n\nUse the bool type instead of an integer for the variable `clock_5x`,\nsince this represents its purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: Ie3005f9dcb2dcfe63261c52a36d865d175a0a75c\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66895\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71486\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "d1ab7d2bb0c4ea7be2e251caa564ab91d27ee7ea", "tree": "905816b92fb3759fcb82767cb94b49d6e0f02594", "parents": [ "8cfc7377ffa880659660b344e97333986aba9130" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 03:03:47 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "tree: Retype variable `laptop_ok` with bool\n\nUse the bool type instead of an integer for the variable `laptop_ok`,\nsince this represents its purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I5d9fc3516bc2d29f11b056e35b3e5e324ce93423\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66891\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71485\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "8cfc7377ffa880659660b344e97333986aba9130", "tree": "3ab74fbb322c79a63fed60c20077c42332408280", "parents": [ "980d6b8d712d26711dcd5a71007e4626c7198cd5" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 03:10:29 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "ichspi.c: Retype appropriate variables with bool\n\nUse the bool type instead of an integer for appropriate variables, since\nthis represents their purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: If7eeacc44921f52aa593ab1302f17a5c5190f830\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66892\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71484\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "980d6b8d712d26711dcd5a71007e4626c7198cd5", "tree": "38e926243e6a10734971066fd1602d373fd04784", "parents": [ "f25447e5724bd8664338b69b12399f101abed76b" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 02:48:15 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "tree: Retype variable `programmer_may_write` with bool\n\nUse the bool type instead of an integer for the variable\n`programmer_may_write`, since this represents its purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I69958527ae018a92f1c42734a7990d0c532dee0c\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66885\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71483\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "f25447e5724bd8664338b69b12399f101abed76b", "tree": "98ffb81be1bf94c2f82dfe01a1441944c2b8be04", "parents": [ "2e003a0c9c0ce45fc7a34b68728227f1a83793ad" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 02:44:28 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "flashrom.c: Retype appropriate variables with bool\n\nUse the bool type instead of an integer for appropriate variables, since\nthis represents their purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I6629f391284c8f1266e4ba66c9976f3df43955d4\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66883\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71482\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "2e003a0c9c0ce45fc7a34b68728227f1a83793ad", "tree": "15e9940fd1e3553643ba3ce7a7658de6f8369454", "parents": [ "815eb796ae636641db4c7bc74d867d31b2be4a6c" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 02:36:28 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "82802ab.c: Retype appropriate variables with bool\n\nUse the bool type instead of an integer for appropriate variables, since\nthis represents their purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I5dfd9ed4856c37dd70706b2dd71fbb9a8acbdf4c\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66879\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71481\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "815eb796ae636641db4c7bc74d867d31b2be4a6c", "tree": "840ea6f02d716d4d54c3c5f3831dc8db39f5da27", "parents": [ "cf6b70883ea79aac874901184009a048bc46ec45" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Aug 19 01:02:42 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "serprog.c: Retype variable `have_device` with bool\n\nUse the bool type instead of an integer for the variable `have_device`,\nsince this represents its purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: Ia12f1503ae3f7be20a065f22b4ab84c4140cd23e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66878\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71480\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "cf6b70883ea79aac874901184009a048bc46ec45", "tree": "23c4e7584b0f5e744066ec548126b71b6629f902", "parents": [ "316ef019e04c402423bb4602aa9f5f231062b175" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Thu Aug 18 23:57:31 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "buspirate_spi.c: Retype appropriate variables with bool\n\nUse the bool type instead of integer for appropriate variables, since\nthis represents their purpose much better.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I245616168796f2c7fe99388688b0f606bd3405bf\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66868\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71479\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "316ef019e04c402423bb4602aa9f5f231062b175", "tree": "b99fb0f75c7c1943edcea0f1505f20d0a78b4594", "parents": [ "494fa833effc2de065a7beede123d1aa0d02afcb" ], "author": { "name": "Alexander Goncharov", "email": "chat@joursoir.net", "time": "Sun Aug 07 12:08:49 2022 +0300" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "nicintel_eeprom: handle errors using goto for i210\u0027s shutdown\n\nThis patch prepares the programmer to move global singleton states\ninto a struct.\n\nTOPIC\u003dregister_master_api\n\nChange-Id: Idb4f75fa92966a0fffd83dd0e297f5f51f6c0bd3\nSigned-off-by: Alexander Goncharov \u003cchat@joursoir.net\u003e\nTicket: https://ticket.coreboot.org/issues/391\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66491\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71478\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "494fa833effc2de065a7beede123d1aa0d02afcb", "tree": "99f6f93765e54509e3ee9d2d16868ad6ac322185", "parents": [ "d42f1f65b023a4845feb1f7022eb8f2081a39b3d" ], "author": { "name": "direstraits96", "email": "56371107+direstraits96@users.noreply.github.com", "time": "Wed Feb 16 02:26:51 2022 +0000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "Add support for motherboard Jetway P4MDPT\n\nAdd support for motherboard Jetway P4MDPT. According to the author of\nthis patch, reading and writing works.\n\nFound on https://github.com/flashrom/flashrom/pull/241.\n\nChange-Id: I494863389d2afdb745d5c83a99cd955e6a5c334d\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/63919\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71477\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "d42f1f65b023a4845feb1f7022eb8f2081a39b3d", "tree": "4c2b2af39bf959103a37efb8daa6c59270029927", "parents": [ "88e87c568678ab36c997c1c807d97b3b1e7091e6" ], "author": { "name": "Adrien Destugues", "email": "adrien.destugues@opensource.viveris.fr", "time": "Tue Aug 10 11:48:18 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "nic3com: mark PCI id 9006 as supported\n\nAccording to the author of this patch, this has been tested and it\nworked.\n\nFound on https://github.com/flashrom/flashrom/pull/219.\n\nChange-Id: I88712d81b291c7ef4212bf19e284d270219fa908\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/63920\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71476\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "88e87c568678ab36c997c1c807d97b3b1e7091e6", "tree": "fdd492a475759d50abca6c1fa5b61a7970bee032", "parents": [ "927c1f036fc6b3d9b6cb84c5cacc8e78ec7d28cc" ], "author": { "name": "Thomas Heijligen", "email": "thomas.heijligen@secunet.com", "time": "Fri Aug 05 17:56:20 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "tree: Check properly if libusb is initialized\n\nThe dediprog, developerbox_spi and stlinkv3_spi programmers only check\nif the libusb_context is not NULL after the initialization. But\nfollowing the API documentation from libusb, the context is undefined\nunless the init function returns 0. Fix this by checking the return\nvalue instead of the libusb_context to see if the initialization was\nsuccessful.\n\nhttps://libusb.sourceforge.io/api-1.0/group__libusb__lib.html\n\nChange-Id: Ia45ccd3fa2239dfccd821be46a09c86426cb22e5\nSigned-off-by: Thomas Heijligen \u003cthomas.heijligen@secunet.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66460\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71475\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "927c1f036fc6b3d9b6cb84c5cacc8e78ec7d28cc", "tree": "1393274311640dca4be11768ae87d71dfc9aec52", "parents": [ "48c3f1844888d7175106d2bd98262353e985e3cb" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Tue Aug 02 11:31:16 2022 +1000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "include/ich_descriptors.h: Add missing include\n\nAvoid transitively including the bool type for the ich_descriptor\nfunctionality.\n\nChange-Id: I2d9eb833ca9198c0586543d3f0074893aac5c6fb\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66348\nOriginal-Reviewed-by: Matei Dibu \u003cmatdibu@protonmail.com\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71474\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "48c3f1844888d7175106d2bd98262353e985e3cb", "tree": "19bd3d2df41fea256e2e3068c4e5d8f5abb7981b", "parents": [ "d07c4a40d376c77cab0073ec8bca6724c7c291d5" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Jul 29 03:16:19 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "pickit2_spi.c: Use a variable to store the total packetsize\n\nInstead of calculating the total packetsize multiple times, use a\nvariable instead.\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: I714054669e16dcf531a57174f9522b3af72d2754\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66251\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71473\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "d07c4a40d376c77cab0073ec8bca6724c7c291d5", "tree": "c4faf094b38ef81f91744ddba8f22ac84a5b7230", "parents": [ "75d6ec647ca0d398fb7ec33b7293e5f57294834e" ], "author": { "name": "Felix Singer", "email": "felixsinger@posteo.net", "time": "Fri Jul 29 03:09:02 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "pickit2_spi.c: Replace hard-coded value with define in msg_perr()\n\nSigned-off-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nChange-Id: Ia3fa89285d8761e51078afa486df50a7b8247fc5\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/66250\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71472\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "75d6ec647ca0d398fb7ec33b7293e5f57294834e", "tree": "0e6a9138d2bfe506b7f8bef0e9a22870833baace", "parents": [ "9beddb69184fa60473b0ba5b0d6ccadfde6157d3" ], "author": { "name": "Angel Pons", "email": "th3fanbus@gmail.com", "time": "Sat Jul 16 12:12:50 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "pickit2_spi.c: Add `pickit2_interrupt_transfer()` helper\n\nIntroduce the `pickit2_interrupt_transfer()` helper function to simplify\ncalls to the `libusb_interrupt_transfer()` function, as the last three\narguments are always the same: two constants and an unused output value.\n\nChange-Id: I7ff704243b63a7ea2872fbc6e596190573dc13f6\nSigned-off-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/65915\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Tested-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71471\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "9beddb69184fa60473b0ba5b0d6ccadfde6157d3", "tree": "f374e82af2aad56c96e9841a8f55fbaee90bd456", "parents": [ "24e1bbbbc48ae4ab51eed508d8e5288ad009193b" ], "author": { "name": "Miklós Márton", "email": "martonmiklosqdev@gmail.com", "time": "Tue Jun 21 23:34:35 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "stlinkv3_spi: add support for more product variants\n\nST released further STLINK-V3 variants with different PIDs:\n- STLINK-V3E\n- STLINK-V3S\n- STLINK-V3 With dual VCP\n- STLINK-V3 Without MSD\n\nTested with STLINK-V3S and STLINK-V3 With dual VCP\n\nCredits goes to the stlink project for collecting the the PID list:\nhttps://github.com/stlink-org/stlink/blob/develop/src/stlink-lib/\nusb.h#L22\n\nChange-Id: Ic9ad03316b7005aa35e6f2f710c86f48befd38f2\nSigned-off-by: Miklós Márton \u003cmartonmiklosqdev@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/65302\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71470\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "24e1bbbbc48ae4ab51eed508d8e5288ad009193b", "tree": "eae17460b9eab81a921725bd47434b885d857257", "parents": [ "5dc7d949366c84833d9bf84e0ff3bf7c347c9273" ], "author": { "name": "Jacek Naglak", "email": "jnaglak@tlen.pl", "time": "Wed May 18 02:25:13 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "ft2232_spi.c: Add support for kt-link jtag interface\n\nChange tested writing, reading and erasing spi flashes\npinout:\n jtag - spi\n 1 vcc - vcc, wp#, hold#\n 4 gnd - gnd\n 5 tdi - si\n 7 tms - cs#\n 9 tck - sck\n13 tdo - so\nConnect pins 9 and 12 in EXT connector for 3.3V power.\n\nSigned-off-by: Jacek Naglak \u003cjnaglak@tlen.pl\u003e\nChange-Id: Id58c675bc410ec3ef6d58603d13efc9ca53bb87c\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/64440\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71469\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "5dc7d949366c84833d9bf84e0ff3bf7c347c9273", "tree": "731e7610e4c796f23ce6d55465265a5f481a4407", "parents": [ "2ab6bba5602a8a2f7e0a99cc94ccc9fe6a412723" ], "author": { "name": "Pete Smith", "email": "zailawee@protonmail.com", "time": "Wed May 11 10:01:25 2022 +0000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "it87spi.c: Enable probing of IT8686E\n\nEnable probing for IT8686E allowing to use the `dualbiosindex`\nparameter. Dumped and verified both firmwares. Tested on GIGABYTE\nGA-H270N-WIFI.\n\nSigned-off-by: Pete Smith \u003czailawee@protonmail.com\u003e\nChange-Id: I5a1780275a92089c2d91c5da1c472f6d8bc39a56\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/64254\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Arthur Heymans \u003carthur@aheymans.xyz\u003e\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71468\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "2ab6bba5602a8a2f7e0a99cc94ccc9fe6a412723", "tree": "65cb4b8a6c7de64b6dbf1a41cc333ac4259831a1", "parents": [ "43040f297e68cd4d826d58f57566581ef902d179" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Fri Jun 24 16:48:09 2022 +1000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "it85spi: EOL support\n\nThis code was originally introduced by ITE for now exceedingly old\nChromebooks. The code has had very little attention to maintain it,\nunlikely tested for a long time and now seems to be just a technical\nburden to the flashrom project.\n\nIf someone is later interested it could be resurrected for reference\nfrom git history. However, it needs quite a bit of work to bring it back\ninto maintainable order.\n\nTested: tree builds under meson+make and unit tests pass.\n\nChange-Id: I5e8cafd73db837941c518f0e2d72d8192274fd79\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/65378\nOriginal-Reviewed-by: Stefan Reinauer \u003cstefan.reinauer@coreboot.org\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71467\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "43040f297e68cd4d826d58f57566581ef902d179", "tree": "a8357aba5a8dbfd43f3e7949c865f2682bf5962d", "parents": [ "4203a47a102e2622f404ee6567b240882d584116" ], "author": { "name": "Thomas Heijligen", "email": "thomas.heijligen@secunet.com", "time": "Thu Jun 23 14:38:35 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "tree: indent struct *_master consistently with tabs\n\nUse `\u003ctab\u003e.key\u003ctab\u003e*\u003d \u003cvalue\u003e,`\n\nTEST: `make VERSION\u003d0 MAN_DATE\u003d0` returns the same flashrom binary\nbefore and after the patch\n\nChange-Id: I1c45ea9804ca09e040d7ac98255042f58b01f8ef\nSigned-off-by: Thomas Heijligen \u003cthomas.heijligen@secunet.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/65363\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71466\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "4203a47a102e2622f404ee6567b240882d584116", "tree": "57656409f88916ec1c9c70cfcfb20007406ccf10", "parents": [ "bbccdb275537493ec252362fc5fb792a879ae3d7" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Sat May 28 17:28:05 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "dummyflasher: Add emulation for S25FL128L\n\nUsed to test WRSR_EXT2/3 support.\n\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nChange-Id: Ic3cbea87218c973331b9b83e809e7d438407bc13\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/64748\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71464\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "bbccdb275537493ec252362fc5fb792a879ae3d7", "tree": "3666bfc60d97ba51abcaa3c6c7d62be23c449cdc", "parents": [ "58cf5197a6f89fd146ce0fbc340d67ef7cbe191c" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Sat May 28 16:48:26 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "spi25_statusreg: Allow WRSR_EXT for Status Register 3 (dummy part)\n\nSpansion flash chips S25FL128L and S25FL256L use the WRSR instruction to\nwrite more than 2 registers. So align SR2 and SR3 support: The current\nFEATURE_WRSR_EXT is renamed to FEATURE_WRSR_EXT2 and FEATURE_WRSR_EXT3\nis added. Also, WRSR3 needs a separate flag now.\n\nVerified that FEATURE_WRSR_EXT2 still works using the `dummy_flasher`.\n\nflashrom-stable:\nOriginal patch was split, this is the `dummy_flasher.c` part.\n\nChange-Id: I81153cb758b9fc29b803959683ad7acf8fb771e4\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Change-Id: Ibdfc6eb3d2cfecbf8da0493d067031ddb079a094\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/64746\nOriginal-Reviewed-by: Nikolai Artemiev \u003cnartemiev@google.com\u003e\nOriginal-Reviewed-by: Arthur Heymans \u003carthur@aheymans.xyz\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71463\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "58cf5197a6f89fd146ce0fbc340d67ef7cbe191c", "tree": "f2bf0a793fbcfec3ac486bf1b4e30f90649d9364", "parents": [ "c50f4aed20b6d0d455cc0a1622ef683e5fcac2ce" ], "author": { "name": "Tasos Sahanidis", "email": "tasos@tasossah.com", "time": "Wed Apr 20 09:30:42 2022 +0300" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "board_enable: Add ASUS P5W DH Deluxe\n\nFlashrom can now write to the onboard SST49LF008 flash\n\nSigned-off-by: Tasos Sahanidis \u003ctasos@tasossah.com\u003e\nChange-Id: Iea4f858cb45c60a6180de07c8361a8a831635dfd\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/63736\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71462\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "c50f4aed20b6d0d455cc0a1622ef683e5fcac2ce", "tree": "078e3e4c51d02b4af427eda32fb249dc2b4db95e", "parents": [ "a1805097309f7d230f3596550fc673b109d5b19d" ], "author": { "name": "Alexander Goncharov", "email": "chat@joursoir.net", "time": "Fri Jun 17 08:32:47 2022 +0300" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "atavia: fix BYTE_OFFSET\u0027s macro argument value\n\nA macro value has to use a correct argument name.\n\nChange-Id: I666204ec92c6df625b34ca721b3e1af78772bccf\nSigned-off-by: Alexander Goncharov \u003cchat@joursoir.net\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/65193\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71461\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "a1805097309f7d230f3596550fc673b109d5b19d", "tree": "1a2772c9df3563a3ec5dc167dd900501b665744e", "parents": [ "2fc70dcfe988928e97e69e98d91e7e3999ecd352" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Mon May 16 11:10:36 2022 +1000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "flashrom.c: Make need_erase() helper static local\n\nThe need_erase() helper is only used within flashrom.c\n\nChange-Id: Ic0946bb109fca2fc18e15eefa11cccea284ded0b\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/64369\nOriginal-Reviewed-by: Arthur Heymans \u003carthur@aheymans.xyz\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71460\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "2fc70dcfe988928e97e69e98d91e7e3999ecd352", "tree": "b9d2b757606242debe22d8107fbae41c668937a9", "parents": [ "27835eaf1a0384293dcd1f3ad6ad506d919f3b43" ], "author": { "name": "Sergii Dmytruk", "email": "sergii.dmytruk@3mdeb.com", "time": "Mon Nov 08 01:38:52 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "dummyflasher: enforce write protection for W25Q128FV\n\nStart taking bits related to write protection into account.\n\nAlso add \"hwwp\" parameter for dummy programmer that sets state of WP\npin (not inverted value).\n\nTested: use command-line interface to run WP-related commands\n\ndummyflasher doesn\u0027t store state of the chip between runs and flashrom\nallows running only one command, so testing WP in this way is limited.\nHowever, WP options can be combined with other operations and are\nexecuted prior to them, so certain scenarios can be checked.\n\nList possible ranges:\n flashrom -p dummy:emulate\u003dW25Q128FV,hwwp\u003dyes --wp-list\n\nSet a particular range and check status is correct:\n flashrom -p dummy:emulate\u003dW25Q128FV,hwwp\u003dyes \\\n --wp-enable \\\n --wp-range\u003d0x00100000,0x00f00000 \\\n --wp-status\n\nEnable write protection and try erasing/writing (erasing here):\n # this fails\n flashrom -p dummy:emulate\u003dW25Q128FV,hwwp\u003dyes \\\n --wp-range\u003d0,0x00c00000 \\\n --wp-enable \\\n --erase\n\nWrite protecting empty range has no effect:\n # this succeeds\n flashrom -p dummy:emulate\u003dW25Q128FV,hwwp\u003dyes \\\n --wp-range\u003d0,0 \\\n --wp-enable \\\n --erase\n\nDisabling WP is possible if hwwp is off:\n # this fails\n flashrom -p dummy:emulate\u003dW25Q128FV,spi_status\u003d0x80,hwwp\u003dyes \\\n --wp-disable\n # this succeeds\n flashrom -p dummy:emulate\u003dW25Q128FV,spi_status\u003d0x80,hwwp\u003dno \\\n --wp-disable\n\nChange-Id: I9fd1417f941186391bd213bd355530143c8f04a0\nSigned-off-by: Sergii Dmytruk \u003csergii.dmytruk@3mdeb.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/59074\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71459\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "27835eaf1a0384293dcd1f3ad6ad506d919f3b43", "tree": "6f2043a3506fa43952717d16dffdcc0829987638", "parents": [ "59151a4b508e2b8037bc18f46232f6ca0ee90ea2" ], "author": { "name": "Sergii Dmytruk", "email": "sergii.dmytruk@3mdeb.com", "time": "Mon Nov 08 00:06:33 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "dummyflasher: emulate SR2 and SR3 for W25Q128FV\n\nEnable emulation of SR2 and SR3 for W25Q128FV and provide logic for\nupdating them (masks of read-only bits that can\u0027t be set from outside).\n\nTested: check how input value affects status registers of emulated chip\n\nflashrom -V -p dummy:emulate\u003dW25Q128FV,spi_status\u003d0x12 |\n grep -A3 \u0027Initial status registers\u0027\nflashrom -V -p dummy:emulate\u003dW25Q128FV,spi_status\u003d0x1234 |\n grep -A3 \u0027Initial status registers\u0027\nflashrom -V -p dummy:emulate\u003dW25Q128FV,spi_status\u003d0x123456 |\n grep -A3 \u0027Initial status registers\u0027\n\nChange-Id: I79f9b4a0b604663d3288ad70dcbe3ea4075dede5\nSigned-off-by: Sergii Dmytruk \u003csergii.dmytruk@3mdeb.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/59073\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71458\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "59151a4b508e2b8037bc18f46232f6ca0ee90ea2", "tree": "3b1c6092c3b9c501cba4600946725edc4399275a", "parents": [ "885917c1594fa68ddbf5213099d99167105c046b" ], "author": { "name": "Sergii Dmytruk", "email": "sergii.dmytruk@3mdeb.com", "time": "Mon Nov 08 00:05:12 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "dummyflasher: add SR2 and SR3 emulation harness\n\nPrepare everything for emulating SR2 and SR3 for chips that have it.\n\nThis is needed for accessing SRP1 and WPS bits which are involved in\nwrite protection. The emulated register doesn\u0027t affect anything yet\nand will be tested by write-protection tests.\n\nTested: check how input value affects status registers of emulated chip\n\nflashrom -V -p dummy:emulate\u003dW25Q128FV,spi_status\u003d0x12 |\n grep \u0027Initial status register\u0027\nflashrom -V -p dummy:emulate\u003dW25Q128FV,spi_status\u003d0x1234 |\n grep \u0027Initial status register\u0027\nflashrom -V -p dummy:emulate\u003dW25Q128FV,spi_status\u003d0x123456 |\n grep \u0027Initial status register\u0027\n\nMind that at this point there are no chips that emulate more than one\nstatus register.\n\nChange-Id: I177ae3f068f03380f5b3941d9996a07205672e59\nSigned-off-by: Sergii Dmytruk \u003csergii.dmytruk@3mdeb.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/59072\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71457\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "885917c1594fa68ddbf5213099d99167105c046b", "tree": "80e44186ee8a49d049673c79ba2395958b6662bc", "parents": [ "f6c1cb1a856e67b8cf7eaf7a90b09bc3923a3718" ], "author": { "name": "aarya", "email": "aarya.chaumal@gmail.com", "time": "Thu Mar 10 09:16:44 2022 +0530" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "serprog.c: Avoid calling memcpy with NULL pointer arguments\n\nIn function sp_stream_buffer_op, the variable parms might be NULL when\npassed to memcpy. Although, since parmlen is also 0 at that time I\ndon\u0027t think it would make much difference. Still, add a NULL check\nbefore calling memcpy to be safe.\n\nChange-Id: I850123237e328f9548ba7f77a01888be2cbc9e7b\nSigned-off-by: Aarya Chaumal \u003caarya.chaumal@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/62726\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71456\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "f6c1cb1a856e67b8cf7eaf7a90b09bc3923a3718", "tree": "9776b733a8f851fe90b582068baae97e64425e40", "parents": [ "137f02f887144eae222e44adb675cb299fd00337" ], "author": { "name": "Martin Roth", "email": "gaumless@gmail.com", "time": "Tue Mar 15 10:55:25 2022 -0600" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "Global cleanup: Fix a few spelling errors\n\nJust a trivial patch to fix a few errors found by codespell.\n\nHere\u0027s the command I used:\ncodespell -S subprojects,out \\\n-L fwe,dout,tast,crate,parms,claus,nt,nd,te,truns,trun\n\nSigned-off-by: Martin Roth \u003cgaumless@gmail.com\u003e\nChange-Id: I4e3b277f220fa70dcab21912c30f1d26d9bd8749\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/62840\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Elyes Haouas \u003cehaouas@noos.fr\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71455\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "137f02f887144eae222e44adb675cb299fd00337", "tree": "28638ab1b32d38f515a7b63a6c5dd3336093990b", "parents": [ "756b6b32e2a49c487d4bbc42d370e8826b41c922" ], "author": { "name": "Marc Schink", "email": "dev@zapb.de", "time": "Sun Aug 23 16:19:44 2020 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "jlink_spi: Add option to enable target power\n\nflashrom-stable: Adapted error paths and shutdown function.\n\nChange-Id: I026c22ae1c22541d0024f164c827909ca4a34cf4\nSigned-off-by: Marc Schink \u003cdev@zapb.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/48380\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71454\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "756b6b32e2a49c487d4bbc42d370e8826b41c922", "tree": "ce4db0712ab0fba0c5bed684ca67e012a2f2807d", "parents": [ "15095dbb7b9d7f1938d43e5ff3b99054b0798803" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Wed Dec 21 17:15:13 2022 +0000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "Add Intel Jasper Lake support\n\nLoosely based on commit b01d7e9f (ichspi: Add Jasper Lake support)\non flashrom master.\n\nBesides a little change in descriptor detection, no difference to\nCannon Point was found. Hence, add new PCI IDs as 300 series.\n\nChange-Id: I9c715c1a5f1ceea32dc51669453d89b315ba8ca2\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71453\nReviewed-by: Matt DeVillier \u003cmatt.devillier@gmail.com\u003e\nReviewed-by: Paul Menzel \u003cpaulepanter@mailbox.org\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "15095dbb7b9d7f1938d43e5ff3b99054b0798803", "tree": "ceb67bedd9544117f89b2e06504171d2cd42c4b3", "parents": [ "0ac29566c7a4a8977875b65497904f05e5ceeb47" ], "author": { "name": "Thomas Heijligen", "email": "thomas.heijligen@secunet.com", "time": "Wed Apr 06 20:45:02 2022 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "flashrom.c: remove unused include\n\ngetopt.h is only needed in cli_classic.c\n\nChange-Id: I09b3233a128582bc98c5af77b6c89bd49984800e\nSigned-off-by: Thomas Heijligen \u003cthomas.heijligen@secunet.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/63412\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71452\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "0ac29566c7a4a8977875b65497904f05e5ceeb47", "tree": "d66b2fc87a4bb4cda966e0c9b68b3e1d4338efd8", "parents": [ "c9c7d52a98558075b0e52d8ace559e0e5b0923e4" ], "author": { "name": "aarya", "email": "aarya.chaumal@gmail.com", "time": "Sun Mar 13 15:35:12 2022 +0530" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "ich_descriptors.c: Ensure unsigned types \u003e\u003d0 on to prevent underflow\n\nUnsigned types show undefined behaviour if they are subtracted by a\nvalue greater than their own (mostly it wraps to the max value). Using\nthis value for left shifting could be even more dangerous.\n\nflashrom-stable:\nThe actual issue here was that a static-analyzer didn\u0027t realize that\nthe second loop would only run, if ever, with `j \u003e\u003d 12`. So we made\nthat explicit.\n\nChange-Id: I5921cc571f3dca5188ca1973dba6ececbcbe2f39\nSigned-off-by: Aarya Chaumal \u003caarya.chaumal@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/62764\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71451\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "c9c7d52a98558075b0e52d8ace559e0e5b0923e4", "tree": "70e51db3189f4c7c48e4013ff2466048e8f816cb", "parents": [ "d12b6bcbb667c6c852b7d0b521bc0f2f3963f856" ], "author": { "name": "aarya", "email": "aarya.chaumal@gmail.com", "time": "Sun Mar 13 00:05:56 2022 +0530" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "board_enable.c: Remove unnecessary assignment\n\nIn function board_asus_p3b_f there were two consecutive lines which\nmodified the value of variable b\n\n // Do something with b\n\n b\u003dINB(0x80);\n b\u003dINB(smbba);\n\n //Do something with b\n\nSince the value of b is not used after first assignment, remove the\nfirst assignment.\n\nChange-Id: I7458b416a69fd5e2aa300ca49d1352b6074ad0bc\nOriginal-Tested-by: Keith Hui \u003cbuurin@gmail.com\u003e\nSigned-off-by: Aarya Chaumal \u003caarya.chaumal@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/62761\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Keith Hui \u003cbuurin@gmail.com\u003e\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71450\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "d12b6bcbb667c6c852b7d0b521bc0f2f3963f856", "tree": "a337cfd6d9e665484409922ec4f9a82a7a292676", "parents": [ "251e3cd04c311812cb81d8d96c95a392653e38c8" ], "author": { "name": "Daniel Campello", "email": "campello@chromium.org", "time": "Mon Mar 14 11:43:16 2022 -0600" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "flashrom.8.tmpl: Clarify man entries for -w/-v/-x\n\nThis change adds follow up changes to the man page:\n - Explain (-) argument for -w/-v operations\n - Expand on region name handling of -x operation\n\nAlso updates cli_classic.c to match with --help output.\n\nflashrom-stable:\n* Fixed -v description (reads from stdin)\n* s/ B/.B/\n\nChange-Id: I0cba593da3926c8587027789f4e1e89a2329ca7f\nSigned-off-by: Daniel Campello \u003ccampello@chromium.org\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/62794\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71556\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "251e3cd04c311812cb81d8d96c95a392653e38c8", "tree": "82811c929de9540f849faaafa474e4805bebd40d", "parents": [ "29c23dd08474b2457ffd8a72ee3d8587676cd999" ], "author": { "name": "Daniel Campello", "email": "campello@chromium.org", "time": "Wed Mar 16 07:05:48 2022 -0600" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "helpers.c: use unsigned int for bit shifts (ASAN)\n\nThis change addresses the following ASAN error detected in the chromium\ntree:\n\n * ASAN error detected:\n * ../flashrom-9999/helpers.c:28:13: runtime error: left shift of 1 by 31 places cannot be represented in type \u0027int\u0027\n * #0 0x5589a94bb284 in address_to_bits /build/amd64-generic/tmp/portage/sys-apps/flashrom-9999/work/flashrom-9999-build/../flashrom-9999/helpers.c:28:13\n *\n * SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior ../flashrom-9999/helpers.c:28:13 in\n\nTested: ./test_build.sh; FEATURES\u003dtest emerge-amd64-generic flashrom\n\nSigned-off-by: Daniel Campello \u003ccampello@chromium.org\u003e\nChange-Id: Ib595f13c29dd5c0775e074801756e4f920b4daaf\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/62862\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nOriginal-Reviewed-by: Paul Menzel \u003cpaulepanter@mailbox.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71449\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "29c23dd08474b2457ffd8a72ee3d8587676cd999", "tree": "73ccbc237df969f57ad5c9f7e044e0d4b9ca2656", "parents": [ "8efb0b337a46aaa5f2da902aa862d30e6a1305be" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Wed Dec 21 15:25:09 2022 +0000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:35:01 2023 +0000" }, "message": "Add Alder Lake support\n\nLoosely based on commit 11680db4 (ichspi: Add Alder Lake support)\non flashrom master.\n\nBeside a little change in descriptor detection, no difference to\nTiger Lake was found. Hence, add new PCI IDs as 500 series.\n\nChange-Id: Icc1278755ff64f03128d8faadbca85a4ff76864d\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71448\nReviewed-by: Matt DeVillier \u003cmatt.devillier@gmail.com\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "8efb0b337a46aaa5f2da902aa862d30e6a1305be", "tree": "66158792850ac6baa99ae42cfc0ac628d8c880dd", "parents": [ "6fb2f6224e03c4a0688b883a01defdd0a59bb46b" ], "author": { "name": "Charles Parent", "email": "charles.parent@orolia2s.com", "time": "Mon Feb 28 14:55:58 2022 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "ft2232_spi.c: Add FTDI FT4233H\n\nChange tested to probe, read, write and erase a MT25QU256 through a\nFT4233H FTDI\n\nChange-Id: I73cee8fd2a6613a8fbc26508d99bbe67da2b4f72\nSigned-off-by: Charles Parent \u003ccharles.parent@orolia2s.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/62480\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71447\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "6fb2f6224e03c4a0688b883a01defdd0a59bb46b", "tree": "6b67bcdf46f6ddfad4d9785991743cc7c962fd57", "parents": [ "8f0db7d4945d0e7c4d538337ce0dbf47c5356945" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Feb 24 18:17:40 2022 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "libflashrom: Allow NULL-pointer argument in flashrom_flash_release()\n\nfree() allows NULL and it makes error paths easier to handle when one\njust needs to write `free(x);` without needing to care if `x` was\nallocated already. Let\u0027s follow this rule in flashrom_flash_release().\n\nChange-Id: Id119c2e4f3aa1b11313059f11aac73c3e583185c\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/62340\nOriginal-Reviewed-by: Nikolai Artemiev \u003cnartemiev@google.com\u003e\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71446\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "8f0db7d4945d0e7c4d538337ce0dbf47c5356945", "tree": "bd7af29d6e036f189ee3f9a3a7033adc5da39dc9", "parents": [ "d3e8fd9dd50f45028d9cd987c649229c78b0e5e1" ], "author": { "name": "Julius Werner", "email": "jwerner@chromium.org", "time": "Mon Feb 14 17:07:39 2022 -0800" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "libflashrom/fmap: Don\u0027t use off_t for flash offsets\n\noff_t is a special POSIX type that is used to represent file offsets in\ncertain APIs (e.g. lseek(), mmap()), and should not be reused to\nrepresent anything else (such as flash offsets). In particular, the\nwidth of the type may change based on the definition of the\n_FILE_OFFSET_BITS macro. Using such a type at the libflashrom interface\nis particularly dangerous, because if a program is built with a\ndifferent _FILE_OFFSET_BITS value than libflashrom, the resulting ABI\ncorruption will cause very very nasty and confusing bugs. This patch\nreplaces all instances of off_t that are not related to file offsets\nwith (s)size_t.\n\nTested: `elogtool list` on cherry.\n\nSigned-off-by: Julius Werner \u003cjwerner@chromium.org\u003e\nChange-Id: I68a386973f79ea634f63dfcd7d95a63400e1fdee\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/61943\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nOriginal-Reviewed-by: Nikolai Artemiev \u003cnartemiev@google.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71445\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "d3e8fd9dd50f45028d9cd987c649229c78b0e5e1", "tree": "bc2932802bae29104495956ca3c9a4d3c4d64246", "parents": [ "e57d4e49fd2556f0fe267833d35cc57b7e252c06" ], "author": { "name": "Werner Zeh", "email": "werner.zeh@siemens.com", "time": "Tue Jan 25 07:02:49 2022 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "ich_descriptors.c Invert the meaning of \u0027dual_output\u0027 bit\n\nIn the Flash Component description register (FLCOMP) bit 30 reports the\ncapability of using dual output for fast read operation on the flash\ncomponent. According to various SPI Programming Guides (checked for\nPanther Point, Lewisburg C620, Apollo Lake and Elkhart Lake) the dual\noutput is enabled when this bit is set and disabled if not. Currently the\nlogic displays it the other way around when parsing the descriptor.\n\nThis patch changes this so now if bit 30 in FLCOMP is not set, dual read\nsupport for fast read operation is shown as disabled.\n\nChange-Id: If6282ac8326ab0b92e9c70c09dba0299bf0deb6f\nSigned-off-by: Werner Zeh \u003cwerner.zeh@siemens.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/61362\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71444\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "e57d4e49fd2556f0fe267833d35cc57b7e252c06", "tree": "ef260430367f0d2025fd6c5c12c101f37639613e", "parents": [ "672bdcfd4ffeb065b7056042769e3cc512d87c06" ], "author": { "name": "Werner Zeh", "email": "werner.zeh@siemens.com", "time": "Mon Jan 03 09:44:29 2022 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Add Elkhart Lake support\n\nElkhart Lake has a chipset called Mule Creek Canyon which is quite\ncompatible with 300 series chipsets. There are a few differences though,\ne.g. different encoding for the SPI clock values for read and write in\nthe FLCOMP register. In addition Elkhart Lake has a new PCI device ID\nfor the SPI controller which is added, too.\n\nTested: Read and flash complete flash on Siemens MC EHL1\n\nChange-Id: I711e39a3ec9cd7098389231eaa1cb864d615a475\nSigned-off-by: Werner Zeh \u003cwerner.zeh@siemens.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/60711\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71443\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "672bdcfd4ffeb065b7056042769e3cc512d87c06", "tree": "70ad68d799865d833ceab9dbca9259488be8be06", "parents": [ "4f9cd8bf947580bc87d947956aff9e28766405ce" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Wed Feb 02 17:15:05 2022 +1100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "fmap.c: Avoid undefined behaviour with fmap_lsearch([len:\u003d0])\n\nCalling libflashrom entry-points that internally dispatch to\nfmap_lsearch() can result in a integer overflow. Therefore\nvalidate the length paramter before attempting to use it.\n\nTested: `make`\n\nChange-Id: Ifb408c55c3b69ddff453dcc704b7389298050473\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nSpotted-by: Julius Werner \u003cjwerner@chromium.org\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/61545\nOriginal-Reviewed-by: Julius Werner \u003cjwerner@chromium.org\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71442\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "4f9cd8bf947580bc87d947956aff9e28766405ce", "tree": "bdd63b1889c1df94ae072904b18eb21d4d300b3f", "parents": [ "fafc3d8048d9e26b708323805868d127eeecc12e" ], "author": { "name": "Rick Altherr", "email": "rick@oxidecomputer.com", "time": "Mon Dec 13 17:10:00 2021 -0800" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "dediprog: wait for spi bulk read xfers to finish\n\ndediprog_bulk_read_poll()\u0027s finish argument allows it to be used in two\ndistinct cases: where dediprog_bulk_read_poll will be called as part of\na loop (finish\u003d0) and where dediprog_bulk_read_poll should wait for all\noutstanding transfers to finish (finish\u003d1). In both cases,\ndediprog_bulk_read_poll() calls libusb to process events with a 10\nsecond timeout.\n\nAfter dediprog_spi_bulk_read() has queued the last transfers, it calls\ndediprog_bulk_read_poll() with finish\u003d0 when it should be finish\u003d1.\nfinish\u003d0 just happens to work because frequently the transfers finish in\nthe 10 second timeout.\n\nSigned-off-by: Rick Altherr \u003crick@oxidecomputer.com\u003e\nChange-Id: If7cb541742c8620358c8e04275d8316131b2d1ab\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/60087\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71441\nReviewed-by: Rick Altherr \u003ckc8apf@kc8apf.net\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "fafc3d8048d9e26b708323805868d127eeecc12e", "tree": "dfe8d02595cde3456708d263a01033ba9754efac", "parents": [ "6ae640b1f33464969d401e00b44b3b5157f1ce09" ], "author": { "name": "Tim Crawford", "email": "tcrawford@system76.com", "time": "Wed Nov 17 06:23:25 2021 -0700" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "chipset_enable.c: Add TGP-H IDs\n\nAdd IDs for: H510, B560, H570, Q570, Z590, W580, HM570, QM570, WM590\n\nTested on system76/oryp8 (HM570). flashrom is able to read the image\nusing the internal programmer.\n\nChange-Id: I96f63253d42578151f99dcbb42347afecc03f49d\nSigned-off-by: Tim Crawford \u003ctcrawford@system76.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57533\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Benjamin Doron \u003cbenjamin.doron00@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71440\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "6ae640b1f33464969d401e00b44b3b5157f1ce09", "tree": "d8a5dd7c6fde8db17b2e14fb73577ef5f3c22adc", "parents": [ "69c324a7d4015c3ca4ba8d39a1c42670369f5afb" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Wed Nov 17 14:24:04 2021 +1100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "flashrom.c: Validate before allocate in verify_range()\n\nSimplify a goto away for free\u0027ing a buffer by validating\nbefore attempting to allocate.\n\nChange-Id: Iae886f203d1c59ae9a89421f7483a4ec3f747256\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/59372\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71439\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "69c324a7d4015c3ca4ba8d39a1c42670369f5afb", "tree": "c24c0bbe9512d67ed1daa9f919beeea00672441e", "parents": [ "5c9f542bf8ce514c628c59e42e35fbcb615d8937" ], "author": { "name": "melvyn2", "email": "melvyn2@brcok.tk", "time": "Sat Oct 30 16:02:22 2021 -0700" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "chipset_enable.c: Mark Intel Z390 as DEP\n\nTested read/write on GIGABYTE Z390 AORUS MASTER, incl. ME region with\nme_cleaner.\n\nChange-Id: If14d45c144bb32a1d1046185d4476ea29e4d0912\nSigned-off-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nSigned-off-by: melvyn2 \u003cmelvyn2@brcok.tk\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/58774\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71438\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "5c9f542bf8ce514c628c59e42e35fbcb615d8937", "tree": "bcc2215bccd5a34f07460ff0f680aa7fba224744", "parents": [ "cce1e5b8636ebef59dd509680594e17b0a207857" ], "author": { "name": "Michał Żygowski", "email": "michal.zygowski@3mdeb.com", "time": "Wed Jun 16 15:13:54 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Add Tiger Lake U Premium support\n\nTiger Lake has very low ICCRIBA (TGL\u003d0x11, CNL\u003d0x34 and CML\u003d0x34) and\ndetects as unknown chipset compatible with 300 series chipset. Add a\nnew enum CHIPSET_500_SERIES_TIGER_POINT and treat it identically to\nCHIPSET_400_SERIES_COMET_POINT. There are some exceptions though,\nICCRIBA is no longer present n descriptor content so a new union has\nbeen defined for new fields and used in descriptor guessing.\nfreq_read field is not present on Tiger Lake, moreover in CannonPoint\nand Comet Point this field is used as eSPI/EC frequency, so a new\nfunction to print read frequency has ben added. Finally Tiger lake\nboot straps include eSPI, so a new bus has been added for the new\nstraps.\n\nTested: Flash BIOS region on Intel i5-1135G7\n\nSigned-off-by: Michał Żygowski \u003cmichal.zygowski@3mdeb.com\u003e\nChange-Id: I28f3b6fe9f8ce9e976a6808683f46b6f4ec72bdd\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/55578\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71437\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "cce1e5b8636ebef59dd509680594e17b0a207857", "tree": "5f50e5444c3d46b785af7e086ff60159bdf72250", "parents": [ "fa959bacf06a1db31cc82b5d601cc208faf11859" ], "author": { "name": "Thomas Heijligen", "email": "thomas.heijligen@secunet.com", "time": "Tue Nov 02 20:33:35 2021 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "hwaccess.c: move function declarations from programmer.h to hwaccess.h\n\nMove declarations for functions implemented in hwaccess.c from\nprogrammer.h to hwaccess.h.\n\nChange-Id: I075fd86211c766ae3d5f29c76adbd7c5b9bdbd80\nSigned-off-by: Thomas Heijligen \u003cthomas.heijligen@secunet.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/58865\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71436\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "fa959bacf06a1db31cc82b5d601cc208faf11859", "tree": "ceec0521602a491188ae7df029b62691c7060b58", "parents": [ "bb37973298b95a28d36fa83017148cbe28971a94" ], "author": { "name": "Simon Buhrow", "email": "simon.buhrow@sieb-meyer.de", "time": "Fri Oct 29 08:28:51 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "flashrom.c: Delete obsolete comment\n\nChange-Id: Ibd53fe34c05f87d7ecc0d6eee6463f9da3a174d4\nSigned-off-by: Simon Buhrow \u003csimon.buhrow@posteo.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/58742\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71435\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "bb37973298b95a28d36fa83017148cbe28971a94", "tree": "aaba12f6600f5fb215f69985c096cc6d662b7335", "parents": [ "100be2cd19fe0024df82e64b52e24d3fd087720c" ], "author": { "name": "Thomas Heijligen", "email": "thomas.heijligen@secunet.com", "time": "Thu Oct 21 10:41:00 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Makefile: remove NEED_LIBUSB1 from FEATURE_CFLAGS\n\nNEED_LIBUSB1 is not used outside of the Makefile. No need to pass it to\nthe compiler.\n\nChange-Id: Ie7cb3df39daf22cb954186d38ba32812b05d92f9\nSigned-off-by: Thomas Heijligen \u003cthomas.heijligen@secunet.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/58518\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71434\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "100be2cd19fe0024df82e64b52e24d3fd087720c", "tree": "32d00dba126e46499c46c4316eb233bbfc95589b", "parents": [ "dd547c7de5402af4ecf0b1513a9302b95711c0eb" ], "author": { "name": "Felix Singer", "email": "felix.singer@secunet.com", "time": "Wed Oct 13 13:40:07 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "serprog: Move content of programmer specific header to C file\n\nMove the programmer specific code into serprog.c and remove its related\nheader file.\n\nChange-Id: I5c65f43db200129489d487ff948daa08bd0e92fa\nSigned-off-by: Felix Singer \u003cfelix.singer@secunet.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/58300\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71433\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "dd547c7de5402af4ecf0b1513a9302b95711c0eb", "tree": "f9eb84bb9c8d273966c700b46f555e523ed6f694", "parents": [ "ece63c829acde03ed66ee00824d8b371af542333" ], "author": { "name": "Felix Singer", "email": "felix.singer@secunet.com", "time": "Wed Oct 13 13:56:38 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "gfxnvidia: Fix indents\n\nChange-Id: I18f916923d3574dd8c68019e4db3985444dd7bee\nSigned-off-by: Felix Singer \u003cfelix.singer@secunet.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/58301\nOriginal-Reviewed-by: Thomas Heijligen \u003csrc@posteo.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71432\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "ece63c829acde03ed66ee00824d8b371af542333", "tree": "5a944aff6b54520105fd6c72892743ad3968c842", "parents": [ "1da06355905f403a0206ca817ece534155b3c3c4" ], "author": { "name": "Michael Niewöhner", "email": "foss@mniewoehner.de", "time": "Tue Sep 21 20:15:32 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "ft2232_spi: reintroduce generic GPIOL control\n\nThis reintroduces a reworked version of the GPIOL pin control first\nintroduced in commit 3207844 (CB:49637), which was reverted in commit\n6518cf3 (CB:55692) due to breakage.\n\nThis change introduces a new argument `gpiolX` to allow use of the four\nGPIOL pins either as generic gpios or as additional CS# signal(s). `X`\nspecifies the GPIOL pin (0-3) to be set to one of [HLC] with the\nfollowing meaning:\n\n * H - set the pin as output high\n * L - set the pin as output low\n * C - use the pin as additional CS# signal\n\nThe third value, `C`, aims to replace the parameter `csgpiol`, that is\nnow marked as deprecated and can be removed at some point in the future.\n`gpiol` and `csgpiol` are mutually exclusive and use of both results in\nan error.\n\nMultiple pins may be set by specifying the parameter multiple times.\n\nDocumentation was updated/added accordingly.\n\nTest: All pin levels/modes have been verified to behave correctly with a\n logic analyzer.\n\nChange-Id: I3989f0f9596c090de52dca67183b1363dae59d3a\nSigned-off-by: Alan Green \u003cavg@google.com\u003e\nSigned-off-by: Michael Niewöhner \u003cfoss@mniewoehner.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57810\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71431\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\n" }, { "commit": "1da06355905f403a0206ca817ece534155b3c3c4", "tree": "a9a268f6956d6117290faa0cfa5be9f15b5d71b0", "parents": [ "b50d9eb29c61993cae7c55e39d14c42406263328" ], "author": { "name": "Michael Niewöhner", "email": "foss@mniewoehner.de", "time": "Thu Sep 23 21:25:03 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "flashrom.8: carve out `csgpiol` into its own section\n\nDocumentation for `csgpiol` was put into the generic programmer options\nsection. Move it to its own section.\n\nChange-Id: Ic7379331d36b3068eacde5a983b4ccb3afc56c51\nSigned-off-by: Michael Niewöhner \u003cfoss@mniewoehner.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57893\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71430\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "b50d9eb29c61993cae7c55e39d14c42406263328", "tree": "0f504b5eab82e2932a98cb2f2ac0018f7a3257b1", "parents": [ "f0c03fb87d75a3be3a10dcd6abfc24a193dd2f93" ], "author": { "name": "Michael Niewöhner", "email": "foss@mniewoehner.de", "time": "Tue Sep 21 19:43:04 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "ft2232_spi: prevent use of reserved pins on some programmers\n\nOn some programmers an output buffer needs to be enabled by pulling a\ngpio high/low. This gpio can not be used for `csgpiol`. Prevent this by\nprinting an error.\n\nChange-Id: Ied450fa5ef358153adefec3beabc63a62c9f60cd\nSigned-off-by: Michael Niewöhner \u003cfoss@mniewoehner.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57809\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71429\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "f0c03fb87d75a3be3a10dcd6abfc24a193dd2f93", "tree": "7a49430da020ee693a00653c0c0e052175185a0c", "parents": [ "116d56d2265cbe97bd5b97f008c0c6ef3cadce63" ], "author": { "name": "Angel Pons", "email": "th3fanbus@gmail.com", "time": "Mon Jun 21 13:01:45 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "nicintel_spi.c: Implement `set_sck_set_mosi` and `set_sck_get_miso`\n\nTested on a 8086:1533 (i210 GbE), reads still return the same data.\nThis cuts the time to read a Winbond W25Q80.V (1 MiB, SPI) from 66\nseconds down to 48 seconds, i.e. a 37.5% increase in speed.\n\nflashrom-stable: backported to use globals\n\nChange-Id: I52a0ae5ff331ee2ed41652eb5c2ed7ebe7253d74\nSigned-off-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/49267\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71428\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "116d56d2265cbe97bd5b97f008c0c6ef3cadce63", "tree": "bd88f18ce3bb89f4f2da3a0e43d55fbf45ad7f6c", "parents": [ "4b918a132c00c8a7075cab9a21e74f90e37ff489" ], "author": { "name": "Angel Pons", "email": "th3fanbus@gmail.com", "time": "Mon Mar 22 11:28:00 2021 +0100" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "dediprog.c: Split up compound conditional and swap two operations\n\nregister_spi_master now becomes the last operation in init function,\nwhich is consistent with other spi masters.\n\nIn addition, the patch fixes propagation of register_spi_master return\nvalues, which is also consistent with other spi masters.\n\nTested: ninja test\n\nChange-Id: Ib7e0179da39279e32a8497466b044b69ec836da8\nSigned-off-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nSigned-off-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/51706\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71427\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "4b918a132c00c8a7075cab9a21e74f90e37ff489", "tree": "f556fb1d401cbf199b573bfd1755300008922c2f", "parents": [ "140c1261b202aa0e1bf7fff55d6e96cf6bf44d0a" ], "author": { "name": "Thomas Heijligen", "email": "thomas.heijligen@secunet.de", "time": "Sun Sep 26 13:42:39 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "internal.c: unify the macro for x86 only code\n\nThe #if defined(__i386__) || defined(__x86_64__) guard is commonly used\nfor x86 only code across flashrom.\nOnly platform.h and hwaccess.* use the IS_X86 macro.\n\nChange-Id: I94a599431f58666189c8cd601286e9b30c8bf62b\nSigned-off-by: Thomas Heijligen \u003cthomas.heijligen@secunet.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57942\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Felix Singer \u003cfelixsinger@posteo.net\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71426\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "140c1261b202aa0e1bf7fff55d6e96cf6bf44d0a", "tree": "510192d64f21244426e821c146ee369a8e68c5a3", "parents": [ "c2b303bba8bbb99bda6fee0383def382bb4db24d" ], "author": { "name": "Thomas Heijligen", "email": "thomas.heijligen@secunet.de", "time": "Mon Sep 27 15:12:26 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "custom_baud: move Linux specific code into own file\n\nHandle system specific code in an own file like i2c_helper_linux.c.\nThe build system decides when to build it.\n\nChange-Id: I0744e769dcc6000483e7256105903a87e927ee77\nSigned-off-by: Thomas Heijligen \u003cthomas.heijligen@secunet.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57990\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71425\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "c2b303bba8bbb99bda6fee0383def382bb4db24d", "tree": "16ca0b80f727be7feb6d41ee12d52f98dc307985", "parents": [ "7d973885e5c323ac64d5414f88289efdd4221753" ], "author": { "name": "Michael Niewöhner", "email": "foss@mniewoehner.de", "time": "Tue Sep 21 18:03:29 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "ft2232_spi: clarify the comment about gpio configuration\n\nThe comment explaining gpio levels might be easily misunderstood when\nthe reader misses the word `output`. Add an explicit description of\nhandling of the GPIOL* pins to avoid that and make things even more\nclear.\n\nChange-Id: Iaceec889a65ead8cdde917f61b2a9695d440f781\nSigned-off-by: Michael Niewöhner \u003cfoss@mniewoehner.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57808\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71424\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "7d973885e5c323ac64d5414f88289efdd4221753", "tree": "0a505688f6931ea8605b8fdf7f821c6c0b845716", "parents": [ "b31f7ac1a04c9c90256723531aa36001d5399a0a" ], "author": { "name": "Anastasia Klimchuk", "email": "aklm@chromium.org", "time": "Wed Jul 14 09:33:50 2021 +1000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "dediprog.c: Drop dediprog_ prefix for spi data struct members\n\nThe name of the struct type already contains dp_ prefix, so\nprefix doesn\u0027t need to be repeated in members name.\n\nTested: builds and ninja test\n\nChange-Id: I688d50926b78a6c3f1c5a8ba4ef88a0d5b495bd0\nSigned-off-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/56415\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71423\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "b31f7ac1a04c9c90256723531aa36001d5399a0a", "tree": "8010942dbcc00249755853d5cf4be5071df4d5f1", "parents": [ "eec477f1a6f4ba718e713438532c81d9a9b7db72" ], "author": { "name": "Anastasia Klimchuk", "email": "aklm@chromium.org", "time": "Mon Jul 12 14:18:37 2021 +1000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "dediprog.c: Refactor singleton states into reentrant pattern\n\nMove global singleton states into a struct and store within\nthe spi_master data field for the life-time of the driver.\n\nThis is one of the steps on the way to move spi_master data\nmemory management behind the initialisation API, for more\ncontext see other patches under the same topic \"register_master_api\".\n\nTested: builds and ninja test\n\nflashrom-stable: backported to old master API\n\nChange-Id: I72085e750af97b94dfa94f2ebf2a134e41a2ec8d\nSigned-off-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/56414\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71422\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "eec477f1a6f4ba718e713438532c81d9a9b7db72", "tree": "ee51926458b3f9fcc18fc81a74f6fc3be2fc90a5", "parents": [ "a4e63e7957c5eb41d9e0dc237949102795494f31" ], "author": { "name": "Sophie van Soest", "email": "sophie@entropie.rocks", "time": "Sun Jul 04 13:54:26 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "chipset_enable.c: Mark Z97 as DEP\n\nTested on GIGABYTE GA-Z97-HD3.\n\nSigned-off-by: Sophie van Soest \u003csophie@entropie.rocks\u003e\nChange-Id: I73bdd9afefae8e7c013d400e17a15e56d84322f4\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/56060\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Michael Niewöhner \u003cfoss@mniewoehner.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71421\nReviewed-by: Michael Niewöhner \u003cfoss@mniewoehner.de\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "a4e63e7957c5eb41d9e0dc237949102795494f31", "tree": "92d1708f56f2a4c8faeff9729f4d6dee08a1bc63", "parents": [ "3bf7cfba9fca9b18d4ad5b843399bc00a3be1f49" ], "author": { "name": "Miklós Márton", "email": "martonmiklosqdev@gmail.com", "time": "Tue Aug 17 22:53:40 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "ni845x_spi: add missing const specifier to the spi_transmit function\n\nChange-Id: I2c14361283b2da3725d9ba2cae0e1007e6be3d66\nSigned-off-by: Miklós Márton \u003cmartonmiklosqdev@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57003\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71420\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\n" }, { "commit": "3bf7cfba9fca9b18d4ad5b843399bc00a3be1f49", "tree": "cc4c65492591a94835ad9a0c9617084ec8ab006e", "parents": [ "66565a7953b27a55cd963bb4f608c08f7d5237f0" ], "author": { "name": "Jonathan Zhang", "email": "jonzhang@fb.com", "time": "Mon Aug 30 23:25:06 2021 -0700" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Add support for Intel Emmitsburg PCH\n\nThis patch does the following:\n- Add PCIe ID for Intel Emmitsburg PCH\n- Based on ICH descriptor content, choose CHIPSET_C620_SERIES_LEWISBURG\n if ISL/PSL is 80.\n\nTested: tried on a server with Intel Emmitsburg PCH, flash update\nwas successful. This server, however, does not have flash chip\ninstalled, it instead has em100 emulator connected.\n\nChange-Id: I2a1bb7467e693d1583aa885fa0e277075edd4a3e\nSigned-off-by: Jonathan Zhang \u003cjonzhang@fb.com\u003e\nSigned-off-by: David Hendricks \u003cddaveh@amazon.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/54965\nOriginal-Reviewed-by: Christian Walter \u003cchristian.walter@9elements.com\u003e\nOriginal-Reviewed-by: Johnny Lin \u003cJohnny_Lin@wiwynn.com\u003e\nOriginal-Reviewed-by: Tim Chu \u003cTim.Chu@quantatw.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71419\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\n" }, { "commit": "66565a7953b27a55cd963bb4f608c08f7d5237f0", "tree": "69f52540e49bbb94456a1961a308b3ab133c78f9", "parents": [ "fea5aa13d2845835f798c4709ee074433568d714" ], "author": { "name": "David Hendricks", "email": "ddaveh@amazon.com", "time": "Mon Sep 20 21:56:40 2021 -0700" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "ich_descriptors: Add explicit checks for all chipsets\n\nThis partially undoes changes made in commit cd9b7b427\n(ich_descriptors: Normalize chipset detection) to re-add explicit\nmatching of each chipset with one or more strap length values.\n\nSince ranges are checked explicitly, the `warn_if` parameter to\nwarn_peculiar_desc() is no longer necessary and is removed.\n\nChange-Id: Ica49477492876810a6fa212768b1ab9e8c12001f\nSigned-off-by: David Hendricks \u003cddaveh@amazon.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/57793\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71418\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "fea5aa13d2845835f798c4709ee074433568d714", "tree": "d442defee69eb841cb08ef8e0572473d80c4c15e", "parents": [ "27fdfd751ee0d4ed919a23d934b9271568377fa6" ], "author": { "name": "Miklós Márton", "email": "martonmiklosqdev@gmail.com", "time": "Tue Jul 27 14:29:16 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "ni845x_spi: handle PROGRAMFILES(X86) env var properly\n\nThe PROGRAMFILES(X86) envvar contains brackets which\ncould not be interpreted by the Makefile\u0027s interpreter.\nA sed based tweak have been added to extract the variable\nvalue from the env command output. The prefixed include\nand linker path with this (now correctly extracted) prefix\nonly added to the compilation flags if it differ from the\nPROGRAMFILES variable.\n\nChange-Id: I397619a5038567d649a417ce6b9d8ac9e1c8c67b\nSigned-off-by: Miklós Márton \u003cmartonmiklosqdev@gmail.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/56636\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71417\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "27fdfd751ee0d4ed919a23d934b9271568377fa6", "tree": "fe26d2ce85d796031e89178efc59cc7b2e59a4e1", "parents": [ "9fc8e258914a1e891fd4025a3d50edfa8153a141" ], "author": { "name": "Anastasia Klimchuk", "email": "aklm@chromium.org", "time": "Tue Aug 03 10:41:50 2021 +1000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "nicintel_eeprom: Delete extra parentheses from if condition\n\nChange-Id: I068bd227668a79f14e38e703a5a49db18d5de8c7\nSigned-off-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/56820\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nOriginal-Reviewed-by: Edward O\u0027Callaghan \u003cquasisec@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71416\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "9fc8e258914a1e891fd4025a3d50edfa8153a141", "tree": "9636cc3c6b8dd8408740e0c6f0beb7905a98bf4f", "parents": [ "d75e1b1fda75e3573140912cb0af246ccf86c8ee" ], "author": { "name": "Harry Johnson", "email": "johnsonh@waymo.com", "time": "Thu Jul 08 14:28:39 2021 -0700" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "ft2232_spi: Add FTDI search by description.\n\nThis adds to the search-by-serial functionality with\nsearch-by-description (product string). This is useful when e.g. one has\nmultiple FTDIs in a system and wants the serial numbers to reflect the\nsystem-level serial number, and the description to reflect the\nsubcomponent names.\n\nTested manually by running with both serial and description searches, on\na machine with multiple FTDIs plugged in. Ensured that when two devices\nwith the same vid/pid/serial number are plugged in, description can\nbe used to differentiate.\n\nVerifed no-description, no-serial, one FTDI plugged in base case works.\n\nOriginal version of this code used the original single \"arg\" char*, but\non further thought, this wasn\u0027t worth the readability and functionality\nlosses. The new version with arg2 gets rid of several lines of code, the\ngotos, and adds the ability to filter by both description and serial\nsimultaneously.\n\nChange-Id: Ib4be23247995710900175f5f16e38db577ef08fa\nSigned-off-by: Harry Johnson \u003cjohnsonh@waymo.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/56164\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71415\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "d75e1b1fda75e3573140912cb0af246ccf86c8ee", "tree": "0b20aa11090ab199fe70055e964f78aa5a0fe096", "parents": [ "f55ca20fcf7fcc8a8c95d034c7126217a117e88e" ], "author": { "name": "Edward O\u0027Callaghan", "email": "quasisec@google.com", "time": "Wed Jul 14 15:06:04 2021 +1000" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "flashrom.c: Make extract_param() static local\n\nThe function is only ever used within flashrom.c.\n\nChange-Id: I81f1cdb9df98c151201390edeb69c74defe7881f\nSigned-off-by: Edward O\u0027Callaghan \u003cquasisec@google.com\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/56295\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71414\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\n" }, { "commit": "f55ca20fcf7fcc8a8c95d034c7126217a117e88e", "tree": "4c2723b04c1b441112907430b76690beda41fb2c", "parents": [ "29c6834b22a39017725d5dacb69befb4b52fcc5f" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Sun Jun 27 14:36:06 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Makefile: Use dependency lists to generate NEED_\u003clibrary\u003e vars\n\nChange-Id: I30e6430a7e833a7f74eb973fb55bf1947ae2d87b\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/55881\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71413\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "29c6834b22a39017725d5dacb69befb4b52fcc5f", "tree": "1ec976b16d33c5002c205fce3571d845a41cb548", "parents": [ "a54f5a41e65c9ae26b067b7062bb0cbad56bcc93" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Sun Jun 27 14:34:16 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Makefile: Enable bitbang_spi based on its dependency list\n\nChange-Id: I2a22667cfe14e4c5816b3bce13de632941690d17\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/55880\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71412\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "a54f5a41e65c9ae26b067b7062bb0cbad56bcc93", "tree": "f293e04a563f32b8328a8426851c76912a1b2c8e", "parents": [ "8945b815f16be92615875c881cbfcaa80f72ccb5" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Sun Jun 27 14:27:41 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Makefile: Disable libusb/libpci programmers via dependency lists\n\nChange-Id: Ic2e56059a5cd7bd9f3d74d27866ded97e86d6ef1\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/55879\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71411\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "8945b815f16be92615875c881cbfcaa80f72ccb5", "tree": "69ae0d2e6875af66c59d5677b087a57e0ebcabcc", "parents": [ "dc8e54ad9d86d227bdeb8462e2154833ded38cce" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Sun Jun 27 14:24:53 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Makefile: Deflate per-architecture handling of unsupported programmers\n\nChange-Id: Ia54d9e7a2602af6e8725851d01758985b36c8461\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/55878\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71410\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "dc8e54ad9d86d227bdeb8462e2154833ded38cce", "tree": "92133471fd7940fb648eaa85bea8823686a90b8e", "parents": [ "713310385794e1f0d28d5879f736bde325426e1c" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Sun Jun 27 14:17:34 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Makefile: Deflate handling of Linux-only programmers\n\nAlso one case that doesn\u0027t work on Android.\n\nChange-Id: I17381172f8df213bbddccedec2ef288326e49ecf\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/55877\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71409\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" }, { "commit": "713310385794e1f0d28d5879f736bde325426e1c", "tree": "da7d5390a3e8fec6cec5dada6e56966b060c5232", "parents": [ "ca6217827f3ed1223db6b29dcc5b3088e3858a8e" ], "author": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Sun Jun 27 14:00:39 2021 +0200" }, "committer": { "name": "Nico Huber", "email": "nico.h@gmx.de", "time": "Thu Jan 05 16:15:22 2023 +0000" }, "message": "Makefile: Deflate handling of unsupported programmers for libpayload\n\nChange-Id: I78ab1e3217aa6362b4c3525763e15140aa4c204f\nSigned-off-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/55876\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71408\nReviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\n" } ], "next": "ca6217827f3ed1223db6b29dcc5b3088e3858a8e" }