Change in flashrom[staging]: spi: Add valid_addr_offset() to spi_master

[Nico Huber (Code Review)]

Nico Huber 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-1

The patch is fine, but doesn't solve the real problem:
spi_get_valid_read_addr() shouldn't exist. We are testing
something programmer specific right before giving control
to the programmer driver... ichspi should just test this.


-- 
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 <[e-mail]>
Gerrit-Reviewer: Nico Huber <[e-mail]>
Gerrit-Reviewer: Paul Menzel <[e-mail]>
Gerrit-Reviewer: Stefan Reinauer <[e-mail]>
Gerrit-Reviewer: build bot (Jenkins) <[e-mail]>
Gerrit-Comment-Date: Mon, 16 Oct 2017 17:03:14 +0000
Gerrit-HasComments: No