On Tue, Dec 05, 2017 at 10:15:31PM +0100, Jean THOMAS wrote:
> Hi !
> 
> I came up with some modifications on UrJTAG source code to support
> SST39VF160x and SST39VF320x flash chips. Those chips needs some
> special unlock addresses (0x5555/0x2AAA instead of the regular CFI
> 0x555/0x2AAA). The applied patches shouldn't break existing CFI
> flash compatibility.
> 
> The source code has been uploaded here :
> https://github.com/jeanthom/urjtag-sst39vf160x-sst39vf320x
> 

Quick first scan done.

I'm not happy with changes in flash/amd.c for SST chips.

And CFI_VENDOR_SST_WEIRD should be something like CFI_VENDOR_SST_MPFP
or another acronym that matches "Multi-Purpose Flash Plus" from the datasheet.


> Could this be merged into mainline ?

Not as it currently is.



Cheers
Geert Stappers

P.S.
Thanks for the reminder on "dirty JTAG"

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
UrJTAG-development mailing list
UrJTAG-development@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/urjtag-development

Reply via email to