Attention is currently required from: Thomas Heijligen, Edward O'Callaghan, Nikolai Artemiev, Peter Marheine.
4 comments:
Patchset:
Moving the map/unmap functions out of the programmer_entry struct is a good thing. […]
Yeah I think that would also work. I agree I don't think any of the SPI/opaque masters actually need this. I mainly kept it around to keep this change a strict refactor (thought it would be easier to review with the actual change in behavior called out separately) and because I couldn't test the others.
I tried the ICH9 swseq briefly but just got timeouts on the boards I tested (with no changes); didn't have time to dig into it yet. The only hardware programmers I have are ch341a_spi and ft2232_spi.
File atapromise.c:
Patch Set #2, Line 133: atapromise_map
This may not be needed any longer, because `atapromise_map` has the same implementation as `fallback […]
Agree, will tweak this when I'm back Tuesday.
File atavia.c:
Patch Set #2, Line 147: atavia_map
This may also not be needed, CB:67656 […]
If you can test that would be awesome, FWIW I could keep this around even if we move the mapper into par_master though.
File serprog.c:
Patch Set #2, Line 582: .map_flash_region = serprog_map,
Does anyone know if this is needed?
I'm pretty sure ICH7/9 swseq don't need mappers, but if this one does, and we want to move the mapper into the bus master structs, then I will add a mapper to spi_master too.
To view, visit change 67695. To unsubscribe, or for help writing mail filters, visit settings.