From: <oh...@ma...> - 2009-04-19 10:17:06
|
Author: oharboe Date: 2009-04-19 10:16:58 +0200 (Sun, 19 Apr 2009) New Revision: 1466 Modified: trunk/src/flash/aduc702x.c trunk/src/flash/at91sam7.c trunk/src/flash/cfi.c trunk/src/flash/ecos.c trunk/src/flash/flash.c trunk/src/flash/lpc2000.c trunk/src/flash/lpc2000.h trunk/src/flash/lpc288x.c trunk/src/flash/lpc3180_nand_controller.c trunk/src/flash/nand.c trunk/src/flash/ocl.c trunk/src/flash/pic32mx.c trunk/src/flash/stellaris.c trunk/src/flash/stm32x.c trunk/src/flash/str7x.c trunk/src/flash/str9x.c trunk/src/flash/str9xpec.c trunk/src/flash/tms470.c Log: Zach Welch <zw...@su...> fix signed/unsigned comparisons Modified: trunk/src/flash/aduc702x.c =================================================================== --- trunk/src/flash/aduc702x.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/aduc702x.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -331,7 +331,7 @@ * working area available. */ static int aduc702x_write_single(struct flash_bank_s *bank, u8 *buffer, u32 offset, u32 count) { - int x; + u32 x; u8 b; target_t *target = bank->target; Modified: trunk/src/flash/at91sam7.c =================================================================== --- trunk/src/flash/at91sam7.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/at91sam7.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -937,7 +937,8 @@ static int at91sam7_protect(struct flash_bank_s *bank, int set, int first, int last) { u32 cmd; - u32 sector, pagen; + int sector; + u32 pagen; at91sam7_flash_bank_t *at91sam7_info = bank->driver_priv; Modified: trunk/src/flash/cfi.c =================================================================== --- trunk/src/flash/cfi.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/cfi.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -1113,7 +1113,7 @@ }; u8 target_code[4*CFI_MAX_INTEL_CODESIZE]; const u32 *target_code_src; - int target_code_size; + u32 target_code_size; int retval = ERROR_OK; @@ -1969,7 +1969,7 @@ bufferwsize/=(bank->bus_width / bank->chip_width); /* fall back to memory writes */ - while (count >= bank->bus_width) + while (count >= (u32)bank->bus_width) { int fallback; if ((write_p & 0xff) == 0) @@ -2268,7 +2268,7 @@ LOG_DEBUG("size: 0x%x, interface desc: %i, max buffer write size: %x", 1 << cfi_info->dev_size, cfi_info->interface_desc, (1 << cfi_info->max_buf_write_size)); - if (((1 << cfi_info->dev_size) * bank->bus_width / bank->chip_width) != bank->size) + if ((u32)((1 << cfi_info->dev_size) * bank->bus_width / bank->chip_width) != bank->size) { LOG_WARNING("configuration specifies 0x%x size, but a 0x%x size flash was found", bank->size, 1 << cfi_info->dev_size); } @@ -2361,7 +2361,7 @@ for (i = 0; i < cfi_info->num_erase_regions; i++) { - int j; + u32 j; for (j = 0; j < (cfi_info->erase_region_info[i] & 0xffff) + 1; j++) { bank->sectors[sector].offset = offset; Modified: trunk/src/flash/ecos.c =================================================================== --- trunk/src/flash/ecos.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/ecos.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -328,7 +328,7 @@ return retval; - int i; + u32 i; for (i=0; i<len; i+=chunk) { int t=len-i; Modified: trunk/src/flash/flash.c =================================================================== --- trunk/src/flash/flash.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/flash.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -346,7 +346,7 @@ static int handle_flash_info_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc) { flash_bank_t *p; - int i = 0; + u32 i = 0; int j = 0; int retval; @@ -726,12 +726,12 @@ u8 chunk[1024]; u32 wrote = 0; u32 cur_size = 0; - int chunk_count; + u32 chunk_count; char *duration_text; duration_t duration; target_t *target = get_current_target(cmd_ctx); u32 i; - int wordsize; + u32 wordsize; if (argc != 3) { @@ -1129,7 +1129,7 @@ u8 buffer[1024]; int buffer_size = sizeof(buffer); int i; - int nBytes; + u32 nBytes; if (bank->target->state != TARGET_HALTED) { @@ -1139,12 +1139,12 @@ for (i = 0; i < bank->num_sectors; i++) { - int j; + u32 j; bank->sectors[i].is_erased = 1; for (j = 0; j < bank->sectors[i].size; j += buffer_size) { - int chunk; + u32 chunk; int retval; chunk = buffer_size; if (chunk > (j - bank->sectors[i].size)) Modified: trunk/src/flash/lpc2000.c =================================================================== --- trunk/src/flash/lpc2000.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/lpc2000.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -587,7 +587,7 @@ else { u8 *last_buffer = malloc(thisrun_bytes); - int i; + u32 i; memcpy(last_buffer, buffer + bytes_written, bytes_remaining); for (i = bytes_remaining; i < thisrun_bytes; i++) last_buffer[i] = 0xff; Modified: trunk/src/flash/lpc2000.h =================================================================== --- trunk/src/flash/lpc2000.h 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/lpc2000.h 2009-04-19 08:16:58 UTC (rev 1466) @@ -32,7 +32,7 @@ int cmd51_can_256b; int cmd51_can_8192b; int calc_checksum; - int cmd51_max_buffer; + u32 cmd51_max_buffer; } lpc2000_flash_bank_t; enum lpc2000_status_codes Modified: trunk/src/flash/lpc288x.c =================================================================== --- trunk/src/flash/lpc288x.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/lpc288x.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -328,10 +328,11 @@ static int lpc288x_write(struct flash_bank_s *bank, u8 *buffer, u32 offset, u32 count) { u8 page_buffer[FLASH_PAGE_SIZE]; - u32 i, status, source_offset,dest_offset; + u32 status, source_offset,dest_offset; target_t *target = bank->target; u32 bytes_remaining = count; u32 first_sector, last_sector, sector, page; + int i; /* probed? halted? */ status = lpc288x_system_ready(bank); Modified: trunk/src/flash/lpc3180_nand_controller.c =================================================================== --- trunk/src/flash/lpc3180_nand_controller.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/lpc3180_nand_controller.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -546,7 +546,7 @@ return ERROR_NAND_OPERATION_NOT_SUPPORTED; } - if (data_size > device->page_size) + if (data_size > (u32)device->page_size) { LOG_ERROR("data size exceeds page size"); return ERROR_NAND_OPERATION_NOT_SUPPORTED; @@ -681,7 +681,7 @@ } #endif - if (data_size > device->page_size) + if (data_size > (u32)device->page_size) { LOG_ERROR("data size exceeds page size"); return ERROR_NAND_OPERATION_NOT_SUPPORTED; @@ -741,7 +741,7 @@ target_write_u32(target, 0x200b8000, NAND_CMD_READSTART); } - while (page_bytes_done < device->page_size) + while (page_bytes_done < (u32)device->page_size) { /* MLC_ECC_AUTO_DEC_REG = dummy */ target_write_u32(target, 0x200b8014, 0xaa55aa55); Modified: trunk/src/flash/nand.c =================================================================== --- trunk/src/flash/nand.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/nand.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -834,7 +834,7 @@ int nand_read_page_raw(struct nand_device_s *device, u32 page, u8 *data, u32 data_size, u8 *oob, u32 oob_size) { - int i; + u32 i; if (!device->device) return ERROR_NAND_DEVICE_NOT_PROBED; @@ -948,7 +948,7 @@ int nand_write_page_raw(struct nand_device_s *device, u32 page, u8 *data, u32 data_size, u8 *oob, u32 oob_size) { - int i; + u32 i; int retval; u8 status; @@ -1400,7 +1400,7 @@ if (oob_format & NAND_OOB_SW_ECC) { - int i, j; + u32 i, j; u8 ecc[3]; memset(oob, 0xff, oob_size); for (i = 0, j = 0; i < page_size; i += 256) { Modified: trunk/src/flash/ocl.c =================================================================== --- trunk/src/flash/ocl.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/ocl.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -66,8 +66,8 @@ typedef struct ocl_priv_s { arm_jtag_t *jtag_info; - int buflen; - int bufalign; + unsigned int buflen; + unsigned int bufalign; } ocl_priv_t; static int ocl_register_commands(struct command_context_s *cmd_ctx) Modified: trunk/src/flash/pic32mx.c =================================================================== --- trunk/src/flash/pic32mx.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/pic32mx.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -526,7 +526,7 @@ u32 address = bank->base + offset; u32 bytes_written = 0; u32 status; - u32 retval; + int retval; if (bank->target->state != TARGET_HALTED) { Modified: trunk/src/flash/stellaris.c =================================================================== --- trunk/src/flash/stellaris.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/stellaris.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -570,12 +570,12 @@ return ERROR_FLASH_OPERATION_FAILED; } - if ((first < 0) || (last < first) || (last >= stellaris_info->num_pages)) + if ((first < 0) || (last < first) || (last >= (int)stellaris_info->num_pages)) { return ERROR_FLASH_SECTOR_INVALID; } - if ((first == 0) && (last == (stellaris_info->num_pages-1))) + if ((first == 0) && (last == ((int)stellaris_info->num_pages-1))) { return stellaris_mass_erase(bank); } @@ -821,7 +821,7 @@ u32 words_remaining = (count / 4); u32 bytes_remaining = (count & 0x00000003); u32 bytes_written = 0; - u32 retval; + int retval; if (bank->target->state != TARGET_HALTED) { Modified: trunk/src/flash/stm32x.c =================================================================== --- trunk/src/flash/stm32x.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/stm32x.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -603,7 +603,7 @@ u32 address = bank->base + offset; u32 bytes_written = 0; u8 status; - u32 retval; + int retval; if (bank->target->state != TARGET_HALTED) { Modified: trunk/src/flash/str7x.c =================================================================== --- trunk/src/flash/str7x.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/str7x.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -478,7 +478,7 @@ u32 address = bank->base + offset; u32 bytes_written = 0; u32 cmd; - u32 retval; + int retval; u32 check_address = offset; int i; Modified: trunk/src/flash/str9x.c =================================================================== --- trunk/src/flash/str9x.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/str9x.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -509,7 +509,7 @@ u32 address = bank->base + offset; u32 bytes_written = 0; u8 status; - u32 retval; + int retval; u32 check_address = offset; u32 bank_adr; int i; Modified: trunk/src/flash/str9xpec.c =================================================================== --- trunk/src/flash/str9xpec.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/str9xpec.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -687,8 +687,8 @@ scan_field_t field; u8 *scanbuf; int i; - u32 first_sector = 0; - u32 last_sector = 0; + int first_sector = 0; + int last_sector = 0; tap = str9xpec_info->tap; Modified: trunk/src/flash/tms470.c =================================================================== --- trunk/src/flash/tms470.c 2009-04-19 08:15:32 UTC (rev 1465) +++ trunk/src/flash/tms470.c 2009-04-19 08:16:58 UTC (rev 1466) @@ -955,7 +955,8 @@ { target_t *target = bank->target; u32 glbctrl, fmbac2, orig_fmregopt, fmbsea, fmbseb, fmmaxpp, fmmstat; - int i, result = ERROR_OK; + int result = ERROR_OK; + u32 i; if (target->state != TARGET_HALTED) { |