Fix a bunch of warnings.

* avr910.c (avr910_paged_load): possible unitialized use of
rd_size
* jtagmkI.c (jtagmkI_initialize): pointer signedness mixup
* jtagmkII.c (jtagmkII_print_parms1): propagate const'ness
of parameter
* usbasp.c (usbasp_transmit): poiner signedness mixup


git-svn-id: svn://svn.savannah.nongnu.org/avrdude/trunk/avrdude@775 81a1dc3b-b13d-400b-aceb-764788c761c2
This commit is contained in:
Joerg Wunsch 2008-07-25 21:14:43 +00:00
parent 6ca3ef6517
commit 6a633ea9f1
5 changed files with 15 additions and 5 deletions

View File

@ -1,3 +1,13 @@
2008-07-25 Joerg Wunsch <j.gnu@uriah.heep.sax.de>
Fix a bunch of warnings.
* avr910.c (avr910_paged_load): possible unitialized use of
rd_size
* jtagmkI.c (jtagmkI_initialize): pointer signedness mixup
* jtagmkII.c (jtagmkII_print_parms1): propagate const'ness
of parameter
* usbasp.c (usbasp_transmit): poiner signedness mixup
2008-07-25 Joerg Wunsch <j.gnu@uriah.heep.sax.de> 2008-07-25 Joerg Wunsch <j.gnu@uriah.heep.sax.de>
Contributed by Ville Voipio: Contributed by Ville Voipio:

View File

@ -666,7 +666,7 @@ static int avr910_paged_load(PROGRAMMER * pgm, AVRPART * p, AVRMEM * m,
int page_size, int n_bytes) int page_size, int n_bytes)
{ {
char cmd; char cmd;
int rd_size; int rd_size = 1;
unsigned int addr = 0; unsigned int addr = 0;
unsigned int max_addr; unsigned int max_addr;
char buf[2]; char buf[2];

View File

@ -550,7 +550,7 @@ static unsigned char jtagmkI_get_baud(long baud)
static int jtagmkI_initialize(PROGRAMMER * pgm, AVRPART * p) static int jtagmkI_initialize(PROGRAMMER * pgm, AVRPART * p)
{ {
AVRMEM hfuse; AVRMEM hfuse;
char cmd[1], resp[5]; unsigned char cmd[1], resp[5];
unsigned char b; unsigned char b;
if (!(p->flags & AVRPART_HAS_JTAG)) { if (!(p->flags & AVRPART_HAS_JTAG)) {

View File

@ -128,7 +128,7 @@ static int jtagmkII_reset(PROGRAMMER * pgm, unsigned char flags);
static int jtagmkII_set_sck_period(PROGRAMMER * pgm, double v); static int jtagmkII_set_sck_period(PROGRAMMER * pgm, double v);
static int jtagmkII_setparm(PROGRAMMER * pgm, unsigned char parm, static int jtagmkII_setparm(PROGRAMMER * pgm, unsigned char parm,
unsigned char * value); unsigned char * value);
static void jtagmkII_print_parms1(PROGRAMMER * pgm, char * p); static void jtagmkII_print_parms1(PROGRAMMER * pgm, const char * p);
static void jtagmkII_setup(PROGRAMMER * pgm) static void jtagmkII_setup(PROGRAMMER * pgm)
{ {
@ -2166,7 +2166,7 @@ static void jtagmkII_display(PROGRAMMER * pgm, const char * p)
} }
static void jtagmkII_print_parms1(PROGRAMMER * pgm, char * p) static void jtagmkII_print_parms1(PROGRAMMER * pgm, const char * p)
{ {
unsigned char vtarget[4], jtag_clock[4]; unsigned char vtarget[4], jtag_clock[4];
char clkbuf[20]; char clkbuf[20];

View File

@ -82,7 +82,7 @@ static int usbasp_transmit(PROGRAMMER * pgm,
functionid, functionid,
(send[1] << 8) | send[0], (send[1] << 8) | send[0],
(send[3] << 8) | send[2], (send[3] << 8) | send[2],
buffer, buffersize, (char *)buffer, buffersize,
5000); 5000);
if(nbytes < 0){ if(nbytes < 0){
fprintf(stderr, "%s: error: usbasp_transmit: %s\n", progname, usb_strerror()); fprintf(stderr, "%s: error: usbasp_transmit: %s\n", progname, usb_strerror());