summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeLines
* board/svm_sc8xx/flash.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-3/+1
| | | | | | | | | Fix: flash.c: In function 'write_word': flash.c:542:15: warning: variable 'last' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/svm_sc8xx/flash.c: CodingStyle cleanupWolfgang Denk2011-11-07-470/+357
| | | | | | | | Make (mostly) checkpatch clean (don't convert to use I/O accessors yet, so there will be "Use of volatile is usually wrong" warnings left. Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/svm_sc8xx/svm_sc8xx.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-4/+1
| | | | | | | | | Fix: svm_sc8xx.c: In function 'checkboard': svm_sc8xx.c:83:6: warning: variable 'board_type' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/rbc823/flash.c: Fix flash initalizationWolfgang Denk2011-11-07-9/+3
| | | | | | | | | | | | | | | The RBC823 has only one NOR flash bank. Remove all code trying to deal with a second bank. This also fixes a number of GCC 4.6 build warnings: flash.c: In function 'flash_init': flash.c:62:12: warning: array subscript is above array bounds [-Warray-bounds] flash.c:63:12: warning: array subscript is above array bounds [-Warray-bounds] flash.c:66:12: warning: array subscript is above array bounds [-Warray-bounds] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/rbc823/flash.c: CodingStyle cleanupWolfgang Denk2011-11-07-182/+180
| | | | | | | | Make (mostly) checkpatch clean (don't convert to use I/O accessors yet, so there will be "Use of volatile is usually wrong" warnings left. Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/fads/fads.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-1/+3
| | | | | | | | | Fix: fads.c: In function 'initdram': fads.c:606:7: warning: variable 'base' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/mbx8xx/mbx8xx.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-5/+12
| | | | | | | | | | | | | | | Fix: mbx8xx.c: In function 'mbx_init': mbx8xx.c:127:15: warning: variable 'refclock' set but not used [-Wunused-but-set-variable] mbx8xx.c: In function 'board_get_cpufreq': mbx8xx.c:231:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] mbx8xx.c: In function 'get_reffreq': mbx8xx.c:242:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] Signed-off-by: Wolfgang Denk <wd@denx.de>
* mpc8xx/spi.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-4/+0
| | | | | | | | | | | Fix: spi.c: In function 'spi_init_f': spi.c:144:21: warning: variable 'iop' set but not used [-Wunused-but-set-variable] spi.c:142:22: warning: variable 'cpi' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* net/net.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-3/+1
| | | | | | | | | Fix: net.c: In function 'CDPHandler': net.c:1083:8: warning: variable 'applid' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/genietv/flash.c: Fix flash initializationWolfgang Denk2011-11-07-10/+4
| | | | | | | | | | | | | | | | The GENIETV has only one NOR flash bank. Remove all code trying to deal with a second bank. This also fixes a number of GCC 4.6 build warnings: flash.c:65:12: warning: array subscript is above array bounds [-Warray-bounds] flash.c:66:12: warning: array subscript is above array bounds [-Warray-bounds] flash.c:69:12: warning: array subscript is above array bounds [-Warray-bounds] flash.c:183:2: warning: format '%x' expects argument of type 'unsigned int', but argument 2 has type ' Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/genietv/flash.c: CodingStyle cleanupWolfgang Denk2011-11-07-177/+179
| | | | | | | Make (mostly) checkpatch clean (don't convert to use I/O accessors yet, so there will be "Use of volatile is usually wrong" warnings left. Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/etx094/flash.c: Fix flash initializationWolfgang Denk2011-11-07-51/+3
| | | | | | | | | | | | | | | | | | | | | The ETX094 has only one NOR flash bank. Remove all code trying to deal with a second bank. This also fixes a number of GCC 4.6 build warnings: flash.c: In function 'flash_init': flash.c:68:13: warning: array subscript is above array bounds [-Warray-bounds] flash.c:70:13: warning: array subscript is above array bounds [-Warray-bounds] flash.c:72:13: warning: array subscript is above array bounds [-Warray-bounds] flash.c:128:13: warning: array subscript is above array bounds [-Warray-bounds] flash.c:129:13: warning: array subscript is above array bounds [-Warray-bounds] flash.c:133:12: warning: array subscript is above array bounds [-Warray-bounds] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/etx094/flash.c: CodingStyle cleanupWolfgang Denk2011-11-07-187/+178
| | | | | | | Make (mostly) checkpatch clean (don't convert to use I/O accessors yet, so there will be "Use of volatile is usually wrong" warnings left. Signed-off-by: Wolfgang Denk <wd@denx.de>
* mpc8xx/i2c.c: replace PRINTD() by debug()Wolfgang Denk2011-11-07-48/+46
| | | | | | | | | This also fixes some GCC 4.6 build warnings like: warning: variable 'txbd' set but not used [-Wunused-but-set-variable] warning: variable 'rxbd' set but not used [-Wunused-but-set-variable] Signed-off-by: WOlfgang Denk <wd@denx.de> Cc: Heiko Schocher <hs@denx.de>
* mpc8xx/i2c.c: CodungStyle cleanupWolfgang Denk2011-11-07-149/+154
| | | | | | | | | | | Make (mostly) checkpatch-clean We don't acctually change the code (like convert to use I/O accessors), so there will be some remaining "Use of volatile" warnings from checkpatch. Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Heiko Schocher <hs@denx.de>
* board/eltec/mhpc/flash.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-4/+3
| | | | | | | | | | Fix: flash.c: In function 'write_buff': flash.c:314:6: warning: variable 'count' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Frank Gottschling <fgottschling@eltec.de>
* board/kup/kup4k/kup4k.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-2/+1
| | | | | | | | | | Fix: kup4k.c: In function 'initdram': kup4k.c:155:19: warning: variable 'mod' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Klaus Heydeck <heydeck@kieback-peter.de>
* ICU862: Fix incorrect flash handlingWolfgang Denk2011-11-07-29/+3
| | | | | | | | | | | | | | | | There is only one bank of NOR flash memory on ICU862 boards. Remove code that attempts to deal with a second bank. This also fixes build warnings: flash.c: In function 'flash_init': flash.c:120:12: warning: array subscript is above array bounds [-Warray-bounds] flash.c:121:12: warning: array subscript is above array bounds [-Warray-bounds] flash.c:124:12: warning: array subscript is above array bounds [-Warray-bounds] Signed-off-by: Wolfgang Denk <wd@denx.de>
* mpc8xx/cpu.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-1/+2
| | | | | | | | | Fix: cpu.c: In function 'check_CPU': cpu.c:188:8: warning: variable 'mid' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/r360mpi/pcmcia.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-4/+0
| | | | | | | | | | | | Fix: pcmcia.c: In function 'pcmcia_hardware_enable': pcmcia.c:22:21: warning: variable 'cp' set but not used [-Wunused-but-set-variable] pcmcia.c: In function 'pcmcia_hardware_disable': pcmcia.c:130:25: warning: variable 'pcmp' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/netta/pcmcia.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-50/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: pcmcia.c: In function 'cfg_ports': pcmcia.c:151:21: warning: variable 'cp' set but not used [-Wunused-but-set-variable] pcmcia.c:150:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] pcmcia.c: In function 'pcmcia_hardware_enable': pcmcia.c:179:25: warning: variable 'sysp' set but not used [-Wunused-but-set-variable] pcmcia.c:177:21: warning: variable 'cp' set but not used [-Wunused-but-set-variable] pcmcia.c:176:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] pcmcia.c: In function 'pcmcia_hardware_disable': pcmcia.c:271:25: warning: variable 'pcmp' set but not used [-Wunused-but-set-variable] pcmcia.c:270:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] pcmcia.c: In function 'pcmcia_voltage_set': pcmcia.c:303:9: warning: variable 'sreg' set but not used [-Wunused-but-set-variable] pcmcia.c:300:21: warning: variable 'cp' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/netta/codec.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-6/+3
| | | | | | | | | | | | | | | Fix: codec.c: In function 's_write_BR': codec.c:1389:15: warning: variable 'v' set but not used [-Wunused-but-set-variable] codec.c: In function 's_write_OR': codec.c:1400:15: warning: variable 'v' set but not used [-Wunused-but-set-variable] codec.c: In function 's_write_NR': codec.c:1411:15: warning: variable 'v' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/mbx8xx/pcmcia.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-6/+0
| | | | | | | | | | | | | Fix: pcmcia.c: In function 'pcmcia_hardware_enable': pcmcia.c:83:25: warning: variable 'sysp' set but not used [-Wunused-but-set-variable] pcmcia.c:81:21: warning: variable 'cp' set but not used [-Wunused-but-set-variable] pcmcia.c:80:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/manroland/uc100/pcmcia.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-28/+22
| | | | | | | | | | | Fix: pcmcia.c: In function 'pcmcia_voltage_set': pcmcia.c:158:25: warning: variable 'pcmp' set but not used [-Wunused-but-set-variable] pcmcia.c:157:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/lwmon/pcmcia.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-6/+0
| | | | | | | | | | | | | | Fix: pcmcia.c: In function 'pcmcia_hardware_enable': pcmcia.c:33:21: warning: variable 'cp' set but not used [-Wunused-but-set-variable] pcmcia.c:32:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] pcmcia.c: In function 'pcmcia_voltage_set': pcmcia.c:174:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/kup/common/pcmcia.c: Fix GCC 4.6 compiler warningsWolfgang Denk2011-11-07-23/+19
| | | | | | | | | | | | | Fix: ../common/pcmcia.c: In function 'pcmcia_hardware_enable': ../common/pcmcia.c:23:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] ../common/pcmcia.c: In function 'pcmcia_voltage_set': ../common/pcmcia.c:152:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Klaus Heydeck <heydeck@kieback-peter.de>
* board/icu862/pcmcia.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-10/+4
| | | | | | | | | | | | | | | Fix: pcmcia.c: In function 'cfg_port_B': pcmcia.c:21:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] pcmcia.c: In function 'pcmcia_hardware_enable': pcmcia.c:50:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] pcmcia.c: In function 'pcmcia_voltage_set': pcmcia.c:191:20: warning: variable 'immap' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/c2mon/pcmcia.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-5/+3
| | | | | | | | | Fix: pcmcia.c: In function 'pcmcia_voltage_set': pcmcia.c:202:21: warning: variable 'cp' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* drivers/pcmcia/mpc8xx_pcmcia.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-1/+2
| | | | | | | | | Fix: mpc8xx_pcmcia.c: In function 'pcmcia_on': mpc8xx_pcmcia.c:76:8: warning: variable 'slotbit' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* mpc8xx/fec.c: Fix GCC 4.6 build warningsWolfgang Denk2011-11-07-22/+25
| | | | | | | | | | | | | | | Fix: fec.c: In function 'fec_pin_init': fec.c:381:18: warning: variable 'fecp' set but not used [-Wunused-but-set-variable] fec.c: In function 'fec8xx_miiphy_write': fec.c:1013:8: warning: variable 'rdreg' set but not used [-Wunused-but-set-variable] Note: The code was slightly rearranged, but no functional changes attempted, i. e. no conversion to use I/O accessors. Signed-off-by: Wolfgang Denk <wd@denx.de>
* drivers/video/cfb_console.c:Fix GCC 4.6 build warningWolfgang Denk2011-11-07-1/+3
| | | | | | | | | | | Silence this warning: cfb_console.c: In function 'video_logo': cfb_console.c:1563:18: warning: variable 'y_off' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Anatolij Gustschin <agust@denx.de> Acked-by: Anatolij Gustschin <agust@denx.de>
* board/tqc/tqm5200/cam5200_flash.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-8/+2
| | | | | | | | | | | | Fix: cam5200_flash.c: In function 'flash_erase_32': cam5200_flash.c:282:24: warning: variable 'l_sect' set but not used [-Wunused-but-set-variable] cam5200_flash.c: In function 'flash_erase_16': cam5200_flash.c:612:24: warning: variable 'l_sect' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/esd/pf5200/pf5200.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-6/+4
| | | | | | | | | | Fix: pf5200.c: In function 'do_phypower': pf5200.c:330:6: warning: variable 'status' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Reinhard Arlt <reinhard.arlt@esd-electronics.com>
* board/esd/cpci5200/strataflash.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-3/+1
| | | | | | | | | | Fix: strataflash.c: In function 'flash_write_cfiword': strataflash.c:681:11: warning: variable 'ctladdr' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Reinhard Arlt <reinhard.arlt@esd-electronics.com>
* PM520: add missing enable_interrupts()Wolfgang Denk2011-11-07-5/+12
| | | | | | | | | | | | | The code disabled interrupts in several locations, without re-enabling them again. Fix this. While we are at it, also fix a GCC 4.6 build warning: flash.c: In function 'flash_erase': flash.c:373:21: warning: variable 'last' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Josef Wagner <Wagner@Microsys.de>
* fs/fat/fat.c: Fix GCC 4.6 build warningWolfgang Denk2011-11-07-16/+19
| | | | | | | | | Fix: fat.c: In function 'fat_register_device': fat.c:74:19: warning: variable 'info' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* drivers/video/videomodes.c: fix GCC 4.6 build warningWolfgang Denk2011-11-07-5/+11
| | | | | | | | | | Fix: videomodes.c: In function 'video_get_params': videomodes.c:162:13: warning: variable 't' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Acked-by: Anatolij Gustschin <agust@denx.de>
* drivers/rtc/rv3029.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-07-16/+11
| | | | | | | | | | | | | | | | | | Fix: rv3029.c: In function 'rtc_set': rv3029.c:98:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable] rv3029.c: In function 'set_eere_bit': rv3029.c:131:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable] rv3029.c: In function 'wait_eebusy': rv3029.c:149:9: warning: variable 'ret' set but not used [-Wunused-but-set-variable] rv3029.c: In function 'rtc_reset': rv3029.c:165:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* drivers/rtc/ds3231.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-07-13/+3
| | | | | | | | | | | Fix: ds3231.c: In function 'rtc_get': ds3231.c:90:52: warning: variable 'control' set but not used [-Wunused-but-set-variable] Fix is done by switching to standard debug() instead of custom macro. Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/mcc200/auto_update.c: fixx GCC 4.6 build warningsWolfgang Denk2011-11-07-6/+1
| | | | | | | | | | | Fix: auto_update.c: In function 'do_auto_update': auto_update.c:344:48: warning: variable 'got_ctrlc' set but not used [-Wunused-but-set-variable] auto_update.c:344:18: warning: variable 'bitmap_first' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* arch/powerpc/cpu/mpc5xxx/usb_ohci.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-07-2/+1
| | | | | | | | | Fix: usb_ohci.c: In function 'dl_transfer_length': usb_ohci.c:751:8: warning: variable 'tdINFO' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/tqc/tqm5200/cmd_stk52xx.c: fix GC 4.6 build warningsWolfgang Denk2011-11-07-11/+12
| | | | | | | | | | | | Fix: cmd_stk52xx.c: In function 'spi_transmit': cmd_stk52xx.c:85:6: warning: variable 'dummy' set but not used [-Wunused-but-set-variable] cmd_stk52xx.c: In function 'i2s_play_wave': cmd_stk52xx.c:199:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] Signed-off-by: Wolfgang Denk <wd@denx.de>
* arch/powerpc/lib/board.c: fix build warningWolfgang Denk2011-11-07-3/+0
| | | | | | | | Fix: board.c: In function 'board_init_r': board.c:633:8: warning: unused variable 's' [-Wunused-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* arch/powerpc/lib/board.c: make (mostly) checkpatch-cleanWolfgang Denk2011-11-07-221/+234
| | | | | | | | | | | | | | | | | | CodingStyle cleanup. Accepted (in this cleanup) checkpatch messages: - externs should be avoided (to be cleaned up later) - no spaces at the start of a line (accepted in multi-line #if's) - Macros with complex values (false reports) - do not use assignment in if condition (accepted in one place, where avoiding it would have required an additional level of nesting, resulting in less readable code) Signed-off-by: Wolfgang Denk <wd@denx.de>
* arch/powerpc/cpu/mpc5xxx/i2c.c: fix GC 4.6 build warningsWolfgang Denk2011-11-07-4/+1
| | | | | | | | | | Fix: i2c.c: In function 'wait_for_bb': i2c.c:104:16: warning: variable 'temp' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Heiko Schocher <hs@denx.de>
* board/mpl/common/flash.c: fix GCC 4.6 build warningWolfgang Denk2011-11-07-14/+1
| | | | | | | | | | Fix: ../common/flash.c: In function 'flash_erase': ../common/flash.c:603:24: warning: variable 'l_sect' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Denis Peter <d.peter@mpl.ch>
* config.mk: use memoization in cc-option macro to speed up compilationDaniel Schwierzeck2011-11-07-2/+18
| | | | | | | | | | | Apply memoization to cc-option macro by caching the results of the gcc calls. This macro is called very often so using cached results leads to faster compilation times. The old behaviour can be restored by defining the config option CONFIG_CC_OPT_CACHE_DISABLE=y. Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@googlemail.com>
* sandbox: drop unused returnMike Frysinger2011-11-03-2/+0
| | | | | Signed-off-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Simon Glass <sjg@chromium.org>
* sandbox: put stdin into raw modeMike Frysinger2011-11-03-0/+40
| | | | | | | | This allows us to act like a serial device: we get tab chars and CTRL+C and respond appropriately. Signed-off-by: Mike Frysinger <vapier@gentoo.org> Tested-by: Simon Glass <sjg@chromium.org>
* serial: sandbox: use ssize_t to match os_readMike Frysinger2011-11-03-1/+1
| | | | | Signed-off-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Simon Glass <sjg@chromium.org>