Re: [avrdude-dev] bug in usbasp.c

[jasonkotzin]

Yes, looks like you are correct. I'm not sure the different of the two
functions, but being that they are identical, I would say it's missing it
there as well.

Thanks.

Sincerely,
Jason


Joerg Wunsch wrote:
> 
> (Better subscribe to the list, in order to not miss followups.)
> 
> As Jason Kotzin wrote:
> 
>>  680   unsigned char *buffer = (m->buf)+address;
>>  681   unsigned char blockflags = USBASP_BLOCKFLAG_FIRST;
>> 
>> 
>> Above is the new code, line 680 used to be as follows:
>> 
>> 680   unsigned char *buffer = m->buf;
> 
> Thanks a lot for debugging this, as well as for the patch.  Normally,
> I'd ask you to file it in the patch tracker, but I'm just about to
> apply it right now, so that's not necessary this time.
> 
> One question though: wouldn't usbasp_spi_paged_load() require a
> similar change?
> -- 
> cheers, J"org               .-.-.   --... ...--   -.. .  DL8DTL
> 
> http://www.sax.de/~joerg/                        NIC: JW11-RIPE
> Never trust an operating system you don't have sources for. ;-)
> 
> _______________________________________________
> avrdude-dev mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/avrdude-dev
> 
> 

-- 
View this message in context: 
http://old.nabble.com/bug-in-usbasp.c-tp33071691p33075496.html
Sent from the AVR - AVRdude - Dev mailing list archive at Nabble.com.