On 3 October 2012 22:26, Freddie Chopin <[email protected]> wrote: > W dniu 2012-10-03 20:12, Olivier Schonken pisze: >> Hi >> >> Could http://openocd.zylin.com/#/c/830/ also be included in the 0.6.1 >> bugfix release? As it is a bug fix on the sector erase of the SAM4S >> >> Thanks and regards >> >> Olivier Schonken > > I have no opinion on this patch - I cannot test... Could you point us to > the place where the actual shift (or validation of the value to be 4, 8, > etc.) happens? If there's no such place in OpenOCD's code than I guess > that the fix is not valid... >
I cannot validate the change in hardware, however looking at the atmel asf src suggest that it is correct and the docs are just a bit misleading. no objections from me to cherrypick for 0.6.1. I have already merged the change into master. Spen ------------------------------------------------------------------------------ Don't let slow site performance ruin your business. Deploy New Relic APM Deploy New Relic app performance management and know exactly what is happening inside your Ruby, Python, PHP, Java, and .NET app Try New Relic at no cost today and get our sweet Data Nerd shirt too! http://p.sf.net/sfu/newrelic-dev2dev _______________________________________________ OpenOCD-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openocd-devel
