summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeLines
* 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>
* serial: sandbox: optimize putsMike Frysinger2011-11-03-2/+1
| | | | | Signed-off-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Simon Glass <sjg@chromium.org>
* sandbox/tegra2: fix typo in autocomplete defineMike Frysinger2011-11-03-2/+2
| | | | | Signed-off-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Simon Glass <sjg@chromium.org>
* sandbox: Change md command to use map_physmemSimon Glass2011-11-03-2/+7
| | | | | | | | | | Sandbox wants to support commands which use memory. The map_physmen() call provides this feature, so should be used more consistently in U-Boot. Signed-off-by: Simon Glass <sjg@chromium.org> Acked-by: Mike Frysinger <vapier@gentoo.org> Tested-by: Matthias Weisser <weisserm@arcor.de>
* sandbox: Adjust Makefile so that standalone/api are not builtSimon Glass2011-11-03-3/+4
| | | | | | | | | | The check for sandbox architecture is too early in the Makefile, so standalone and api are built regardless. This moves the check until after autoconf.mk has been read. This fixes a build breakage. Signed-off-by: Simon Glass <sjg@chromium.org>
* sandbox: Add asm/cache.h for sandboxAnton staaf2011-11-03-0/+33
| | | | | | | | | This file is required by the new DMA buffer alignment macro. Signed-off-by: Anton Staaf <robotboy@chromium.org> Cc: Simon Glass <sjg@chromium.org> Tested-by: Simon Glass <sjg@chromium.org> Acked-by: Mike Frysinger <vapier@gentoo.org>
* Reduce build timesWolfgang Denk2011-11-03-43/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | U-Boot Makefiles contain a number of tests for compiler features etc. which so far are executed again and again. On some architectures (especially ARM) this results in a large number of calls to gcc. This patch makes sure to run such tests only once, thus largely reducing the number of "execve" system calls. Example: number of "execve" system calls for building the "P2020DS" (Power Architecture) and "qong" (ARM) boards, measured as: -> strace -f -e trace=execve -o /tmp/foo ./MAKEALL <board> -> grep execve /tmp/foo | wc -l Before: After: Reduction: ================================== P2020DS 20555 15205 -26% qong 31692 14490 -54% As a result, built times are significantly reduced, typically by 30...50%. Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Andy Fleming <afleming@gmail.com> Cc: Kumar Gala <galak@kernel.crashing.org> Cc: Albert Aribaud <albert.aribaud@free.fr> cc: Graeme Russ <graeme.russ@gmail.com> cc: Mike Frysinger <vapier@gentoo.org> Tested-by: Graeme Russ <graeme.russ@gmail.com> Tested-by: Matthias Weisser <weisserm@arcor.de> Tested-by: Sanjeev Premi <premi@ti.com> Tested-by: Simon Glass <sjg@chromium.org> Tested-by: Macpaul Lin <macpaul@gmail.com> Acked-by: Mike Frysinger <vapier@gentoo.org>
* post/post.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-2/+1
| | | | | | | | | Fix: post.c: In function 'post_log': post.c:425:7: warning: variable 'i' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* post/post.c: CodingStyle cleanupWolfgang Denk2011-11-03-32/+27
| | | | | | | Make checkpatch-clean.. Signed-off-by: Wolfgang Denk <wd@denx.de> Acked-by: Marek Vasut <marek.vasut@gmail.com>
* common/cmd_ide.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-2/+0
| | | | | | | | | | | Fix: cmd_ide.c: In function 'ide_ident': cmd_ide.c:988:6: warning: variable 'do_retry' set but not used [-Wunused-but-set-variable] Delete the unused variable. Signed-off-by: Wolfgang Denk <wd@denx.de>
* common/cmd_ide.c: CodingStyle cleanupWolfgang Denk2011-11-03-840/+874
| | | | | | | | | | | | Make file acceptable to checkpatch. This is only a basic clean up to the extend possible without any real changes to the source code. Warnings due to line over 80 characters were accepted because these affect only printf()s with user visible strings. No attempts were made to fix warnings about volatile and externs - these need a more thorough cleanup. Signed-off-by: Wolfgang Denk <wd@denx.de>
* drivers/rtc/ds1337.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-13/+3
| | | | | | | | | Fix: ds1337.c: In function 'rtc_get': ds1337.c:88:52: warning: variable 'control' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* 4xx_pci.c: add error checking, fix GCC 4.6 build warningWolfgang Denk2011-11-03-2/+4
| | | | | | | | | | | Fix: 4xx_pci.c: In function 'pci_init_board': 4xx_pci.c:855:6: warning: variable 'busno' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Stefan Roese <sr@denx.de> Acked-by: Stefan Roese <sr@denx.de>
* board/prodrive/p3mx/mv_eth.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-27/+6
| | | | | | | | | | | | | | | | | | | | | | | Fix: mv_eth.c: In function 'mv64460_eth_real_open': mv_eth.c:471:6: warning: variable 'port_status' set but not used [-Wunused-but-set-variable] mv_eth.c: In function 'mv64460_eth_stop': mv_eth.c:722:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable] mv_eth.c: In function 'mv64460_eth_xmit': mv_eth.c:796:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable] mv_eth.c: In function 'mv64460_eth_receive': mv_eth.c:877:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable] mv_eth.c: In function 'mv64460_eth_get_stats': mv_eth.c:979:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable] mv_eth.c: In function 'mv64460_eth_update_stat': mv_eth.c:1006:24: warning: variable 'dummy' set but not used [-Wunused-but-set-variable] mv_eth.c:1005:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable] mv_eth.c: In function 'mv64460_eth_print_stat': mv_eth.c:1087:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable] mv_eth.c: In function 'eth_clear_mib_counters': mv_eth.c:2141:15: warning: variable 'dummy' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* 4xx_enet.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-3/+0
| | | | | | | | | | Fix: 4xx_enet.c: In function 'enet_rcv': 4xx_enet.c:1772:21: warning: variable 'ef_ptr' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Stefan Roese <sr@denx.de>
* 4xx_uart.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-12/+14
| | | | | | | | | | Fix: 4xx_uart.c: In function 'get_serial_clock': 4xx_uart.c:204:6: warning: variable 'tmp' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: Stefan Roese <sr@denx.de>
* drivers/net/eepro100.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-2/+0
| | | | | | | | | Fix: eepro100.c: In function 'read_hw_addr': eepro100.c:926:6: warning: variable 'eeprom' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/g2000/strataflash.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-3/+0
| | | | | | | | | Fix: strataflash.c: In function 'flash_write_cfiword': strataflash.c:669:11: warning: variable 'ctladdr' set but not used [-Wunused-but-set-variable] Signed-off-by: Wolfgang Denk <wd@denx.de>
* board/jse/flash.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-14/+1
| | | | | | | | | | | Fix: flash.c: In function 'flash_erase': flash.c:304:24: warning: variable 'l_sect' set but not used [-Wunused-but-set-variable] In addition, remove some dead code. Signed-off-by: Wolfgang Denk <wd@denx.de>
* drivers/net/ns8382x.c: fix GCC 4.6 build warningsWolfgang Denk2011-11-03-48/+37
| | | | | | | | | | | | | | | | | | Fix: ns8382x.c: In function 'ns8382x_check_duplex': ns8382x.c:704:6: warning: variable 'hun' set but not used [-Wunused-but-set-variable] To fix this, we get rid of the NS8382X_DEBUG code and use standard debug() instead. This will now trigger a so far undetected warning: ns8382x.c:780:2: warning: format '%X' expects argument of type 'unsigned int', but argument 2 has type 'long unsigned int' [-Wformat] Fix that, too. Signed-off-by: Wolfgang Denk <wd@denx.de>
* arch/powerpc/lib/board.c: fix build warningWolfgang Denk2011-11-03-1/+3
| | | | | | | | | | | | | | Commit 1272592 "powerpc: Use getenv_ulong() in place of getenv(), strtoul" instroduced a build warning for some PPC systems: board.c: In function 'board_init_r': board.c:626: warning: unused variable 's' Fix it. Signed-off-by: Wolfgang Denk <wd@denx.de> Acked-by: Simon Glass <sjg@chromium.org> Acked-by: Heiko Schocher <hs@denx.de>
* Merge branch 'master' of git://git.denx.de/u-boot-mmcWolfgang Denk2011-11-03-55/+719
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 'master' of git://git.denx.de/u-boot-mmc: tegra2: Move MMC clock initialization into MMC driver mmc: sdhci: fix sdma bug for large file transfer mmc: sdhci: add timeout for data transfer mmc: sdhci: add mmc structure for host mmc: sdhci: fix build warning mmc: sdhci: fix cache flush mmc: CMD7:MMC_CMD_SELECT_CARD response fix mmc: test mmc bus width on startup mmc: change magic number to macro define mmc: mv_sdhci: fix 8bus width access for 88SV331xV5 mmc: retry the cmd8 to meet 74 clocks requirement in the spec PXA: Add MMC driver using the generic MMC framework
| * tegra2: Move MMC clock initialization into MMC driverStephen Warren2011-11-03-15/+10
| | | | | | | | | | | | | | | | | | | | | | This centralizes knowledge of MMC clocking into the MMC driver. This also removes clock setup from the board files, which will simplify later changes that modify the Harmony board to support the correct set of MMC controllers. Signed-off-by: Stephen Warren <swarren@nvidia.com> Cc: Andy Fleming <afleming@gmail.com> Acked-by: Simon Glass <sjg@chromium.org> Tested-by: Simon Glass <sjg@chromium.org>
| * mmc: sdhci: fix sdma bug for large file transferLei Wen2011-11-03-1/+1
| | | | | | | | | | | | | | | | | | SDHCI spec need to reset the sdma base address while the software try to accorss the 512k bytes address boundary. When meet such accross behavior, sdhci controller would generate a interrupt automatically, and software need handle this. Signed-off-by: Lei Wen <leiwen@marvell.com>
| * mmc: sdhci: add timeout for data transferLei Wen2011-11-03-1/+8
| | | | | | | | Signed-off-by: Lei Wen <leiwen@marvell.com>