- flashrom-gerrit - mail.coreboot.org

Change in flashrom[staging]: spi25: Fix layering violation in probe_spi_rdid4()
by David Hendricks (Code Review) 07 Jun '17

07 Jun '17

David Hendricks has posted comments on this change. ( https://review.coreboot.org/19420 ) Change subject: spi25: Fix layering violation in probe_spi_rdid4() ...................................................................... Patch Set 2: Code-Review-1 (1 comment) I'm not sure how this will affect wbsio_spi.c. Can you please explain the impact of setting max_data_read to 3 a bit more? I have no experience with that hardware, so there might be something you know that I don't. https://review.coreboot.org/#/c/19420/2/wbsio_spi.c File wbsio_spi.c: https://review.coreboot.org/#/c/19420/2/wbsio_spi.c@73 PS2, Line 73: .max_data_read = 3, The table above wbsio_spi_send_command() implies that this is 4. It seems you set it to 3 here since there isn't a mode that supports 4-byte RDID (1 command byte + 4 data read). I'm wondering if it's cleaner to leave this as MAX_DATA_UNSPECIFIED and let it fail in wbsio_spi_send_command(). -- To view, visit https://review.coreboot.org/19420 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: Idd21d20465cb214f3ff5bf3267b9014f8beee3f3 Gerrit-Change-Number: 19420 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 07 Jun 2017 19:55:17 +0000 Gerrit-HasComments: Yes

1 0

Change in flashrom[staging]: bitbang_spi: Drop bitbang_spi_master_type
by David Hendricks (Code Review) 07 Jun '17

07 Jun '17

David Hendricks has posted comments on this change. ( https://review.coreboot.org/19417 ) Change subject: bitbang_spi: Drop bitbang_spi_master_type ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://review.coreboot.org/19417 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: I7a3b5d9ff1e99d2d4f873c6f19fb318f93762037 Gerrit-Change-Number: 19417 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 07 Jun 2017 19:20:44 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[staging]: Adapt CLI to use new libflashrom interface' print callback
by Philippe Mathieu-Daudé (Code Review) 06 Jun '17

06 Jun '17

Philippe Mathieu-Daudé has posted comments on this change. ( https://review.coreboot.org/17948 ) Change subject: Adapt CLI to use new libflashrom interface' print callback ...................................................................... Patch Set 14: Code-Review+1 -- To view, visit https://review.coreboot.org/17948 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: Idf19978eb8e340d258199193d2978f37409e9983 Gerrit-Change-Number: 17948 Gerrit-PatchSet: 14 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philippe Mathieu-Daudé <philippe.mathieu.daude(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 06 Jun 2017 21:17:42 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[staging]: spi: Add valid_addr_offset() to spi_master
by build bot (Jenkins) (Code Review) 06 Jun '17

06 Jun '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/19418 ) Change subject: spi: Add valid_addr_offset() to spi_master ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom_gerrit/353/ : SUCCESS https://qa.coreboot.org/job/flashrom-customrules/355/ : SUCCESS -- To view, visit https://review.coreboot.org/19418 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: Icbac513c5339e8aff624870252133284ef85ab73 Gerrit-Change-Number: 19418 Gerrit-PatchSet: 3 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 06 Jun 2017 17:51:02 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[staging]: bitbang_spi: Drop bitbang_spi_master_type
by build bot (Jenkins) (Code Review) 06 Jun '17

06 Jun '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/19417 ) Change subject: bitbang_spi: Drop bitbang_spi_master_type ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom_gerrit/352/ : SUCCESS https://qa.coreboot.org/job/flashrom-customrules/354/ : SUCCESS -- To view, visit https://review.coreboot.org/19417 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: I7a3b5d9ff1e99d2d4f873c6f19fb318f93762037 Gerrit-Change-Number: 19417 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 06 Jun 2017 17:50:48 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[staging]: udelay: Use clock_gettime() if available and precise
by Stefan Reinauer (Code Review) 06 Jun '17

06 Jun '17

Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/19391 ) Change subject: udelay: Use clock_gettime() if available and precise ...................................................................... Patch Set 3: Code-Review+2 -- To view, visit https://review.coreboot.org/19391 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: I85ad359823875237ada9cd027af3017d62e9a235 Gerrit-Change-Number: 19391 Gerrit-PatchSet: 3 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 06 Jun 2017 17:49:15 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[staging]: spi: Drop spi_controller type
by Stefan Reinauer (Code Review) 06 Jun '17

06 Jun '17

Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/19421 ) Change subject: spi: Drop spi_controller type ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://review.coreboot.org/19421 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: I8eda93b429e3ebaef79e22aba76be62987e496f4 Gerrit-Change-Number: 19421 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 06 Jun 2017 17:47:46 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[staging]: spi25: Fix layering violation in probe_spi_rdid4()
by Stefan Reinauer (Code Review) 06 Jun '17

06 Jun '17

Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/19420 ) Change subject: spi25: Fix layering violation in probe_spi_rdid4() ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://review.coreboot.org/19420 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: Idd21d20465cb214f3ff5bf3267b9014f8beee3f3 Gerrit-Change-Number: 19420 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 06 Jun 2017 17:46:04 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[staging]: spi25: Fix layering violation in default_spi_write_aai()
by Stefan Reinauer (Code Review) 06 Jun '17

06 Jun '17

Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/19419 ) Change subject: spi25: Fix layering violation in default_spi_write_aai() ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://review.coreboot.org/19419 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: I8aa3e2992f64906edc669060936f9522d32637fb Gerrit-Change-Number: 19419 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 06 Jun 2017 17:45:15 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[staging]: spi: Add valid_addr_offset() to spi_master
by Stefan Reinauer (Code Review) 06 Jun '17

06 Jun '17

Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/19418 ) Change subject: spi: Add valid_addr_offset() to spi_master ...................................................................... Patch Set 3: Code-Review+2 -- To view, visit https://review.coreboot.org/19418 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: staging Gerrit-MessageType: comment Gerrit-Change-Id: Icbac513c5339e8aff624870252133284ef85ab73 Gerrit-Change-Number: 19418 Gerrit-PatchSet: 3 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 06 Jun 2017 17:39:49 +0000 Gerrit-HasComments: No

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit