March 2023 - flashrom-gerrit - mail.coreboot.org

[S] Change in flashrom[master]: MAINTAINERS: Add Nikolai Artemiev for raiden_debug_spi
by Nikolai Artemiev (Code Review) 13 Mar '23

13 Mar '23

Attention is currently required from: Felix Singer, Anastasia Klimchuk. Nikolai Artemiev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/73697 ) Change subject: MAINTAINERS: Add Nikolai Artemiev for raiden_debug_spi ...................................................................... Patch Set 3: (2 comments) Commit Message: https://review.coreboot.org/c/flashrom/+/73697/comment/0402e6fe_1df0dec9 PS2, Line 7: maintainer > Add Nikolai Artemiev for raiden_debug_spi Done File MAINTAINERS: https://review.coreboot.org/c/flashrom/+/73697/comment/8a6337a2_76f89432 PS2, Line 157: RAIDEN DEBUG SPI > We keep it in abc order, so this one should go between Linux mtd and Stlink. […] Done -- To view, visit https://review.coreboot.org/c/flashrom/+/73697 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ie70eafb301b3363caa8e670b67a16cd25c7e15a8 Gerrit-Change-Number: 73697 Gerrit-PatchSet: 3 Gerrit-Owner: Nikolai Artemiev <nartemiev(a)google.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Felix Singer <felixsinger(a)posteo.net> Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Comment-Date: Mon, 13 Mar 2023 02:32:57 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-MessageType: comment

1 0

[S] Change in flashrom[master]: MAINTAINERS: Add Nikolai Artemiev for raiden_debug_spi
by Nikolai Artemiev (Code Review) 13 Mar '23

13 Mar '23

Attention is currently required from: Felix Singer, Nikolai Artemiev. Hello Felix Singer, build bot (Jenkins), Anastasia Klimchuk, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/flashrom/+/73697 to look at the new patch set (#3). Change subject: MAINTAINERS: Add Nikolai Artemiev for raiden_debug_spi ...................................................................... MAINTAINERS: Add Nikolai Artemiev for raiden_debug_spi BUG=none BRANCH=none TEST=none Change-Id: Ie70eafb301b3363caa8e670b67a16cd25c7e15a8 Signed-off-by: Nikolai Artemiev <nartemiev(a)google.com> --- M MAINTAINERS 1 file changed, 19 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/97/73697/3 -- To view, visit https://review.coreboot.org/c/flashrom/+/73697 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ie70eafb301b3363caa8e670b67a16cd25c7e15a8 Gerrit-Change-Number: 73697 Gerrit-PatchSet: 3 Gerrit-Owner: Nikolai Artemiev <nartemiev(a)google.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Felix Singer <felixsinger(a)posteo.net> Gerrit-Attention: Nikolai Artemiev <nartemiev(a)google.com> Gerrit-MessageType: newpatchset

1 0

[S] Change in flashrom[master]: MAINTAINERS: Add maintainer for raiden_debug_spi
by Anastasia Klimchuk (Code Review) 13 Mar '23

13 Mar '23

Attention is currently required from: Felix Singer, Nikolai Artemiev. Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/73697 ) Change subject: MAINTAINERS: Add maintainer for raiden_debug_spi ...................................................................... Patch Set 2: Code-Review+1 (3 comments) Commit Message: https://review.coreboot.org/c/flashrom/+/73697/comment/ba955a15_72a99dd2 PS2, Line 7: maintainer Add Nikolai Artemiev for raiden_debug_spi Patchset: PS2: Thank you so much Nikolai! File MAINTAINERS: https://review.coreboot.org/c/flashrom/+/73697/comment/ae51f8f0_5abedb84 PS2, Line 157: RAIDEN DEBUG SPI We keep it in abc order, so this one should go between Linux mtd and Stlink. Could you please move? -- To view, visit https://review.coreboot.org/c/flashrom/+/73697 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ie70eafb301b3363caa8e670b67a16cd25c7e15a8 Gerrit-Change-Number: 73697 Gerrit-PatchSet: 2 Gerrit-Owner: Nikolai Artemiev <nartemiev(a)google.com> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Felix Singer <felixsinger(a)posteo.net> Gerrit-Attention: Nikolai Artemiev <nartemiev(a)google.com> Gerrit-Comment-Date: Mon, 13 Mar 2023 01:42:19 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in flashrom[master]: MAINTAINERS: Add maintainer for raiden_debug_spi
by Nikolai Artemiev (Code Review) 13 Mar '23

13 Mar '23

Nikolai Artemiev has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/73697 ) Change subject: MAINTAINERS: Add maintainer for raiden_debug_spi ...................................................................... MAINTAINERS: Add maintainer for raiden_debug_spi BUG=none BRANCH=none TEST=none Change-Id: Ie70eafb301b3363caa8e670b67a16cd25c7e15a8 Signed-off-by: Nikolai Artemiev <nartemiev(a)google.com> --- M MAINTAINERS 1 file changed, 19 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/97/73697/1 diff --git a/MAINTAINERS b/MAINTAINERS index 03f12b8..8509467 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -149,6 +149,11 @@ S: Maintained F: ni845x_spi.c +RAIDEN DEBUG SPI +M: Nikolai Artemiev <nartemiev(a)google.com> +S: Maintained +F: raiden_debug_spi.c + ################################################################################ # TESTS ################################################################################ -- To view, visit https://review.coreboot.org/c/flashrom/+/73697 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ie70eafb301b3363caa8e670b67a16cd25c7e15a8 Gerrit-Change-Number: 73697 Gerrit-PatchSet: 1 Gerrit-Owner: Nikolai Artemiev <nartemiev(a)google.com> Gerrit-MessageType: newchange

1 0

[S] Change in flashrom[master]: meson_cross: Introduce meson cross file for DJGPP/DOS
by Edward O'Callaghan (Code Review) 11 Mar '23

11 Mar '23

Attention is currently required from: Thomas Heijligen. Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/73439 ) Change subject: meson_cross: Introduce meson cross file for DJGPP/DOS ...................................................................... Patch Set 3: Code-Review+2 (1 comment) File meson_cross/i586_djgpp_dos.txt: https://review.coreboot.org/c/flashrom/+/73439/comment/50b5cbf4_2e3c5a47 PS3, Line 22: gnu99 are gnu extensions actually needed or can one consider just std c99 ? -- To view, visit https://review.coreboot.org/c/flashrom/+/73439 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Iabda73942c1e64cd46604c78533982374f78e1a4 Gerrit-Change-Number: 73439 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Thomas Heijligen <src(a)posteo.de> Gerrit-Comment-Date: Sat, 11 Mar 2023 10:32:51 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in flashrom[master]: meson.build: Add 'dos' as option for pci_based programmer
by Edward O'Callaghan (Code Review) 11 Mar '23

11 Mar '23

Attention is currently required from: Thomas Heijligen. Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/73156 ) Change subject: meson.build: Add 'dos' as option for pci_based programmer ...................................................................... Patch Set 5: Code-Review+2 -- To view, visit https://review.coreboot.org/c/flashrom/+/73156 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: If4907a3fd1852ef60d06ef1625ad8a106d89a126 Gerrit-Change-Number: 73156 Gerrit-PatchSet: 5 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Thomas Heijligen <src(a)posteo.de> Gerrit-Comment-Date: Sat, 11 Mar 2023 10:31:50 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in flashrom[master]: meson.build: Use library() to build libflashrom
by Edward O'Callaghan (Code Review) 11 Mar '23

11 Mar '23

Attention is currently required from: Thomas Heijligen, Angel Pons. Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/73101 ) Change subject: meson.build: Use library() to build libflashrom ...................................................................... Patch Set 4: Code-Review+2 -- To view, visit https://review.coreboot.org/c/flashrom/+/73101 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I27f10fdf1227795a9a3b4e050a2d708b58f10ee7 Gerrit-Change-Number: 73101 Gerrit-PatchSet: 4 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Attention: Thomas Heijligen <src(a)posteo.de> Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Comment-Date: Sat, 11 Mar 2023 10:30:55 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[L] Change in flashrom[master]: doc: Add build instructions
by Edward O'Callaghan (Code Review) 11 Mar '23

11 Mar '23

Attention is currently required from: Thomas Heijligen, Stefan Reinauer, Alexander Goncharov. Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/73359 ) Change subject: doc: Add build instructions ...................................................................... Patch Set 5: Code-Review+2 -- To view, visit https://review.coreboot.org/c/flashrom/+/73359 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I96771e98b313a6d26dd2be940ff37998d4124324 Gerrit-Change-Number: 73359 Gerrit-PatchSet: 5 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-Reviewer: Alexander Goncharov <chat(a)joursoir.net> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com> Gerrit-CC: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Attention: Thomas Heijligen <src(a)posteo.de> Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Attention: Alexander Goncharov <chat(a)joursoir.net> Gerrit-Comment-Date: Sat, 11 Mar 2023 10:30:28 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[M] Change in flashrom[master]: ch341a_spi: Refactor singleton states into reentrant pattern
by Edward O'Callaghan (Code Review) 11 Mar '23

11 Mar '23

Attention is currently required from: Anastasia Klimchuk, Alexander Goncharov. Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/72807 ) Change subject: ch341a_spi: Refactor singleton states into reentrant pattern ...................................................................... Patch Set 5: (17 comments) Patchset: PS5: I added many cleanups you can do as a follow up patch to make type usage consistent, const correct and modern. File ch341a_spi.c: https://review.coreboot.org/c/flashrom/+/72807/comment/664c0111_1089e45e PS3, Line 100: i = 0 `size_t i = 0;` https://review.coreboot.org/c/flashrom/+/72807/comment/27bf8c37_a025c100 PS3, Line 234: unsigned int i; delete https://review.coreboot.org/c/flashrom/+/72807/comment/02bb8f27_97c52287 PS3, Line 235: i probably should be `size_t i = 0;` https://review.coreboot.org/c/flashrom/+/72807/comment/91193b4e_014f5a73 PS3, Line 268: uint8_t `const`? https://review.coreboot.org/c/flashrom/+/72807/comment/5c30bcf0_8f017d2d PS3, Line 311: stored_delay_us `delay_us` https://review.coreboot.org/c/flashrom/+/72807/comment/810fdff2_aed49088 PS3, Line 315: int `unsigned int` ? https://review.coreboot.org/c/flashrom/+/72807/comment/1caaccb1_b84b2414 PS3, Line 323: i = 0 `unsigned int i = 0;` https://review.coreboot.org/c/flashrom/+/72807/comment/178d376f_cd8784cf PS3, Line 365: nsigned int p; delete https://review.coreboot.org/c/flashrom/+/72807/comment/bac944ba_d7931173 PS3, Line 366: p actually should be `size_t p = 0;` https://review.coreboot.org/c/flashrom/+/72807/comment/6aa60df7_80cda1ce PS3, Line 387: i `unsigned int i` https://review.coreboot.org/c/flashrom/+/72807/comment/a46a66f2_64abd509 PS3, Line 419: 4 * 1024 `4*KiB` in another patch? https://review.coreboot.org/c/flashrom/+/72807/comment/c7541d2b_05b52209 PS3, Line 460: ret != 0 `ret`, C will auto-cast to make the predicate boolean. https://review.coreboot.org/c/flashrom/+/72807/comment/9ad093f7_751ad0eb PS3, Line 465: (ret != 0) `(ret)` is canonical. https://review.coreboot.org/c/flashrom/+/72807/comment/4568ca26_e7dcad08 PS3, Line 495: i `int i` in another patch? https://review.coreboot.org/c/flashrom/+/72807/comment/7cec1773_775c1af9 PS3, Line 504: i = 0 ditto https://review.coreboot.org/c/flashrom/+/72807/comment/571daf87_385f8c94 PS3, Line 513: i = 0 ditto -- To view, visit https://review.coreboot.org/c/flashrom/+/72807 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I9fe72bff88b7f8778c1199bdab8ba43bf32e1c8c Gerrit-Change-Number: 72807 Gerrit-PatchSet: 5 Gerrit-Owner: Alexander Goncharov <chat(a)joursoir.net> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Attention: Alexander Goncharov <chat(a)joursoir.net> Gerrit-Comment-Date: Sat, 11 Mar 2023 10:29:15 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[M] Change in flashrom[master]: ch341a_spi: Refactor singleton states into reentrant pattern
by Edward O'Callaghan (Code Review) 11 Mar '23

11 Mar '23

Attention is currently required from: Anastasia Klimchuk, Alexander Goncharov. Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/72807 ) Change subject: ch341a_spi: Refactor singleton states into reentrant pattern ...................................................................... Patch Set 5: (1 comment) File ch341a_spi.c: https://review.coreboot.org/c/flashrom/+/72807/comment/012f1d28_321b3ba1 PS3, Line 80: struct libusb_transfer *transfer_out; : struct libusb_transfer *transfer_ins[USB_IN_TRANSFERS]; : : /* Accumulate delays to be plucked between CS deassertion and CS assertions. */ : unsigned int stored_delay_us; : : struct libusb_device_handle *handle; > Done I recommend putting the handle at the start of the struct so that overflows in the above transfer buffers should they happen do not overflow into execution flow memory where function pointers exist. -- To view, visit https://review.coreboot.org/c/flashrom/+/72807 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I9fe72bff88b7f8778c1199bdab8ba43bf32e1c8c Gerrit-Change-Number: 72807 Gerrit-PatchSet: 5 Gerrit-Owner: Alexander Goncharov <chat(a)joursoir.net> Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org> Gerrit-Attention: Alexander Goncharov <chat(a)joursoir.net> Gerrit-Comment-Date: Sat, 11 Mar 2023 10:18:48 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Anastasia Klimchuk <aklm(a)chromium.org> Comment-In-Reply-To: Alexander Goncharov <chat(a)joursoir.net> Gerrit-MessageType: comment

1 0

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit March 2023