* avr.c: Unsigned variable was used for return code of paged_write/load

functions. So a negative return code led never to a fallback to byte
  functions.


git-svn-id: svn://svn.savannah.nongnu.org/avrdude/trunk/avrdude@1042 81a1dc3b-b13d-400b-aceb-764788c761c2
This commit is contained in:
rliebscher 2012-01-19 20:06:52 +00:00
parent 4976fcf88e
commit d342905ecb
2 changed files with 10 additions and 4 deletions

View File

@ -1,3 +1,9 @@
2012-01-19 Rene Liebscher <R.Liebscher@gmx.de>
* avr.c: Unsigned variable was used for return code of paged_write/load
functions. So a negative return code led never to a fallback to byte
functions.
2012-01-17 Rene Liebscher <R.Liebscher@gmx.de> 2012-01-17 Rene Liebscher <R.Liebscher@gmx.de>
bug #34302: Feature request : device configuration with parent classes bug #34302: Feature request : device configuration with parent classes

8
avr.c
View File

@ -289,9 +289,9 @@ int avr_read(PROGRAMMER * pgm, AVRPART * p, char * memtype,
break; break;
} }
if (need_read) { if (need_read) {
i = pgm->paged_load(pgm, p, mem, mem->page_size, rc = pgm->paged_load(pgm, p, mem, mem->page_size,
pageaddr, mem->page_size); pageaddr, mem->page_size);
if (i < 0) if (rc < 0)
/* paged load failed, fall back to byte-at-a-time read below */ /* paged load failed, fall back to byte-at-a-time read below */
failure = 1; failure = 1;
} else if (verbose >= 3) { } else if (verbose >= 3) {
@ -807,8 +807,8 @@ int avr_write(PROGRAMMER * pgm, AVRPART * p, char * memtype, int size,
break; break;
} }
if (need_write) { if (need_write) {
i = pgm->paged_write(pgm, p, m, m->page_size, pageaddr, m->page_size); rc = pgm->paged_write(pgm, p, m, m->page_size, pageaddr, m->page_size);
if (i < 0) if (rc < 0)
/* paged write failed, fall back to byte-at-a-time write below */ /* paged write failed, fall back to byte-at-a-time write below */
failure = 1; failure = 1;
} else if (verbose >= 3) { } else if (verbose >= 3) {