On 20/08/12 19:28, Andreas Fritiofson wrote: > On Mon, Aug 20, 2012 at 8:15 PM, Freddie Chopin <[email protected]> wrote: >> W dniu 2012-08-20 16:06, Spencer Oliver pisze: >> >>> On 20 August 2012 14:04, Andreas Fritiofson >>> <[email protected]> wrote: >>>> >>>> >>>> What is the problem here? Why is the current code destroying data? Do >>>> we have to scan all 32 bits, even though the client only requests >>>> less? And we don't have anywhere to store all 32 bits since Øyvind >>>> removed the "unnecessary" API functions in 991ed5a? And a temporary >>>> stack buffer won't work because the transfer is performed after the >>>> return so your patch is broken? >>>> >>> >>> seems a 32bit read is required for all read sizes. >>> I have just dragged out an arm7 target and indeed current master is >>> broken. >> >> >> So what shall we do? I guess that's a show-stopper for going further with >> the release, right? >> >> 4\/3!! >> > > Looks serious enough to hold rc2 for a while, yes. > > I'll have a look at it this evening if nothing else comes up. I think > I have some str750/str710 boards I can try with. > > Spencer, have you got a simple test case, or will just about everything fail? >
for the target i have the cfi probe fails - so the issue is related to any < 32 bit reads. reverting c3074f377c1da33ca8ba8493826e1b52351eebc6 is not enough, as 16bit reads still fail. Spen ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ OpenOCD-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openocd-devel
