Attention is currently required from: Nico Huber, Angel Pons. Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/62282 )
Change subject: ichspi: Add Jasper Lake support ......................................................................
Patch Set 4:
(2 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/62282/comment/af2bdd40_80265b64 PS4, Line 10: TEST=dedede with `flashrom -p internal --flash-size`.
The test I've suggested will most likely not work because the IFD chipset detection is not complete. […]
Thanks Angel, a rw was tested to validate it doesn't regress the chromeos fw update mechanism.
I'll try to get some time to check your particular example above, thanks for the idea of running that!
File ich_descriptors.c:
https://review.coreboot.org/c/flashrom/+/62282/comment/a8b6b65d_a9ba5ca5 PS4, Line 1064: case CHIPSET_JASPER_LAKE:
It's possible to test the detection code with just a firmware image, using ich_descriptors_tool. […]
Just FYI Angel, I cannot control how public the data sheets are however I can have Intel collaborate with us here to move things forwards in a positive way over someone spending endless hours in their personal time reverse engineering and guessing.
I'll get someone to follow up here who is working on the platform.