summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* sandbox: Fix warnings in hashtable.cSimon Glass2011-11-221-5/+5
| | | | | | | | This fixes a few printf() strings for size_t which are missing the 'z' modifier. Signed-off-by: Simon Glass <sjg@chromium.org> Acked-by: Mike Frysinger <vapier@gentoo.org>
* Improve Power Management in SMC911X driver.bertrand.cachet@heig-vd.ch2011-11-221-2/+5
| | | | | | | | | device is already in *normal* (D0) mode => it doesn't need to be wake-up. With this patch, we only wake-up (writing on TEST_BYTE register) if PM_MODE bits of PM_CTRL register is in sleep (D1/D2) mode. Signed-off-by: Bertrand Cachet <bertrand.cachet@heig-vd.ch>
* env: clean env_common.c checkpatch and code styleIgor Grinberg2011-11-222-34/+26
| | | | | | Cleanup the env_common.c checkpatch warnings, errors and coding style. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_nand.c checkpatch and code styleIgor Grinberg2011-11-221-53/+46
| | | | | | Cleanup the env_nand.c checkpatch warnings, errors and coding style. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_flash.c checkpatch and code styleIgor Grinberg2011-11-221-91/+84
| | | | | | Cleanup the env_flash.c checkpatch warnings, errors and coding style. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_sf.c checkpatch and code styleIgor Grinberg2011-11-221-28/+20
| | | | | | Cleanup the env_sf.c checkpatch warnings, errors and coding style. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_eeprom.c checkpatch and code styleIgor Grinberg2011-11-221-58/+46
| | | | | | Cleanup the env_eeprom.c checkpatch warnings, errors and coding style. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_embedded.c checkpatch and code styleIgor Grinberg2011-11-221-24/+23
| | | | | | | | | | | | | | | | | Cleanup the env_embedded.c checkpatch warnings, errors and coding style. There is one error left though: ERROR: Macros with multiple statements should be enclosed in a do - while loop #79: FILE: u-boot/common/env_embedded.c:79: +#define GEN_ABS(name, value) \ + asm(".globl " GEN_SYMNAME(name)); \ + GEN_SET_VALUE(name, value) total: 1 errors, 0 warnings, 213 lines checked We cannot enclose that statement in a do - while loop, because these are a global assembly declarations. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_mmc.c checkpatch and code styleIgor Grinberg2011-11-221-49/+23
| | | | | | | | Cleanup the env_mmc.c checkpatch warnings, errors and coding style. Simplify env_relocate_spec() function implementation. Also mark internal functions as static. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_nvram.c checkpatch and code styleIgor Grinberg2011-11-221-12/+12
| | | | | | | | | | | | | | | | | | Cleanup the env_nvram.c checkpatch warnings, errors and coding style. There are 2 wanring left about the extern used in c file: WARNING: externs should be avoided in .c files #53: FILE: env_nvram.c:53: +extern void *nvram_read(void *dest, const long src, size_t count); WARNING: externs should be avoided in .c files #54: FILE: env_nvram.c:54: +extern void nvram_write(long dest, const void *src, size_t count); total: 0 errors, 2 warnings, 138 lines checked There is no common nvram header file to use instead of the externs. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_onenand.c checkpatch and code styleIgor Grinberg2011-11-221-7/+5
| | | | | | Cleanup the env_onenand.c checkpatch warnings, errors and coding style. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_dataflash.c checkpatch and code styleIgor Grinberg2011-11-222-20/+14
| | | | | | | Cleanup the env_dataflash.c checkpatch warnings, errors and coding style. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_mgdisk.c checkpatch and code styleIgor Grinberg2011-11-221-2/+2
| | | | | | | Cleanup the env_mgdisk.c checkpatch warnings, errors and coding style. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean env_nowhere.c checkpatch and code styleIgor Grinberg2011-11-221-5/+5
| | | | | | Cleanup the env_nowhere.c checkpatch warnings, errors and coding style. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: clean cmd_nvedit.c checkpatch and code styleIgor Grinberg2011-11-223-52/+53
| | | | | | | | | | | | | | Cleanup the cmd_nvedit.c checkpatch warnings, errors and coding style. There are 10 wanrings left about the simple_strtoul() function: WARNING: simple_strtoul is obsolete, use kstrtoul instead #359: FILE: cmd_nvedit.c:359: + load_addr = simple_strtoul(argv[2], NULL, 16); ... total: 0 errors, 10 warnings, 1043 lines checked Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: move extern environment[] to environment.hIgor Grinberg2011-11-226-14/+11
| | | | | | | Extract all extern declarations for environment out of c files into the environment.h header. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* env: move extern default_environment[] to environment.hIgor Grinberg2011-11-2211-24/+2
| | | | | | | | | Extract all extern declarations for default_environment[] out of c files into the environment.h header. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il> Cc: Stefan Roese <sr@denx.de> Acked-by: Stefan Roese <sr@denx.de>
* common: move extern char console_buffer[] to common.hIgor Grinberg2011-11-2214-17/+1
| | | | | | | | | Extract all extern declarations for console_buffer[] out of c files into the common.h header. Signed-off-by: Igor Grinberg <grinberg@compulab.co.il> Cc: Frank Gottschling <fgottschling@eltec.de> Cc: Murray Jensen <Murray.Jensen@csiro.au>
* env: clean environment.h checkpatch and code styleIgor Grinberg2011-11-221-21/+23
| | | | | | | | | | | Though one warning left: WARNING: do not add new typedefs #149: FILE: u-boot/include/environment.h:149: +typedef struct environment_s { total: 0 errors, 1 warnings, 181 lines checked Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
* net: rtl8109: drop unused !NET_MULTI driverMike Frysinger2011-11-224-390/+0
| | | | | | | No one uses this driver, and it isn't converted to the NET_MULTI framework (which we dropped recently), so drop this driver too. Signed-off-by: Mike Frysinger <vapier@gentoo.org>
* Fix warnings in cmd_nvedit.cSimon Glass2011-11-221-2/+2
| | | | | | | | This printf() string should be %ld now that uintptr_t is defined as long. Also fix a size_t error. Signed-off-by: Simon Glass <sjg@chromium.org> Acked-by: Mike Frysinger <vapier@gentoo.org>
* Define uintptr_t as long int to simplify printf() format stringsSimon Glass2011-11-221-9/+3
| | | | | | | | | | If uintptr_t can be either an unsigned int or an unsigned long int, it is tricky to use it in a printf() format string. This changes it to unsigned long int consistently. This should do the right thing on both 32-bit and 64-bit architectures. Signed-off-by: Simon Glass <sjg@chromium.org> Acked-by: Mike Frysinger <vapier@gentoo.org>
* new tool mkenvimage: generates an env image from an arbitrary config fileDavid Wagner2011-11-222-0/+275
| | | | | | | | | | | This tool takes a key=value configuration file (same as would a `printenv' show) and generates the corresponding environment image, ready to be flashed. use case: flash the environment with an external tool Signed-off-by: David Wagner <david.wagner@free-electrons.com> Acked-by; Mike Frysinger <vapier@gentoo.org> Tested-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
* smsc95xx: Debug message cleanupWolfgang Grandegger2011-11-221-7/+3
| | | | | | | | Cc: Simon Glass <sjg@chromium.org> Cc: Mike Frysinger <vapier@gentoo.org> Signed-off-by: Wolfgang Grandegger <wg@denx.de> Acked-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Simon Glass <sjg@chromium.org>
* smsc95xx: Fix MAC address programmingWolfgang Grandegger2011-11-221-3/+3
| | | | | | | | | | | | | Commit 79ad54400932d6484178a372fb3b659e3437473b broke MAC address programming in the SMSC95xx register set. Cc: Marek Vasut <marek.vasut@gmail.com> Cc: Wolfgang Denk <wd@denx.de> Cc: Simon Glass <sjg@chromium.org> Cc: Mike Frysinger <vapier@gentoo.org> Signed-off-by: Wolfgang Grandegger <wg@denx.de> Acked-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Simon Glass <sjg@chromium.org>
* Merge branch 'sr@denx.de' of git://git.denx.de/u-boot-stagingWolfgang Denk2011-11-213-26/+71
|\ | | | | | | | | | | * 'sr@denx.de' of git://git.denx.de/u-boot-staging: Makefile: Add the missing dependency for spl target gpio: Adapt PCA9698 to standard GPIO API
| * Makefile: Add the missing dependency for spl targetAneesh V2011-11-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Wolfgang's patch for build time improvement is bringing out issues due to missing dependencies in the top-level Makefile. I get errors such as the below while building with many threads. make[1]: /home/a0393566local/u-boot-denx/tools/mkimage: Command not found This also allows one to do: $ make spl/u-boot-spl.bin if you want to build only spl Cc: Wolfgang Denk <wd@denx.de> Signed-off-by: Aneesh V <aneesh@ti.com>
| * gpio: Adapt PCA9698 to standard GPIO APIDirk Eibach2011-11-182-25/+70
| | | | | | | | | | | | Signed-off-by: Dirk Eibach <eibach@gdsys.de> Acked-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Stefan Roese <sr@denx.de>
* | Merge branch 'master' of git://git.denx.de/u-boot-blackfinWolfgang Denk2011-11-213-7/+12
|\ \ | | | | | | | | | | | | | | | | | | * 'master' of git://git.denx.de/u-boot-blackfin: Blackfin: cache result of cpp check Blackfin: traps: fix up printf warnings from debug Blackfin: drop now unused local variable
| * | Blackfin: cache result of cpp checkMike Frysinger2011-11-181-1/+2
| | | | | | | | | | | | | | | | | | Avoid overhead of computing this value multiple times. Signed-off-by: Mike Frysinger <vapier@gentoo.org>
| * | Blackfin: traps: fix up printf warnings from debugMike Frysinger2011-11-181-5/+10
| | | | | | | | | | | | | | | | | | Cast uint32_t to void* for %p, and use %zu for size_t. Signed-off-by: Mike Frysinger <vapier@gentoo.org>
| * | Blackfin: drop now unused local variableMike Frysinger2011-11-181-1/+0
| | | | | | | | | | | | Signed-off-by: Mike Frysinger <vapier@gentoo.org>
* | | Merge branch 'master' of git://git.denx.de/u-boot-nds32Wolfgang Denk2011-11-213-3/+25
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | * 'master' of git://git.denx.de/u-boot-nds32: nds32/lib: add ide generic support nds32: enhance io.h for compatibility with periphals ftide020: fix incorrect information display format
| * | | nds32/lib: add ide generic supportMacpaul Lin2011-11-181-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | Add ide generic support. Signed-off-by: Macpaul Lin <macpaul@andestech.com>
| * | | nds32: enhance io.h for compatibility with periphalsMacpaul Lin2011-11-181-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Enhance io.h for periphals origin design on x86 systems. For example, pci, ide, etc. Signed-off-by: Macpaul Lin <macpaul@andestech.com>
| * | | ftide020: fix incorrect information display formatMacpaul Lin2011-11-181-3/+2
| |/ / | | | | | | | | | | | | | | | Fix incorrect information display format when probing the device. Signed-off-by: Macpaul Lin <macpaul@andestech.com>
* | | Merge branch 'master' of git://git.denx.de/u-boot-niosWolfgang Denk2011-11-2111-74/+394
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | * 'master' of git://git.denx.de/u-boot-nios: nios2: Offer ft_board_setup() capability and call fdt_fixup_ethernet(). board/nios2-generic: Use altera_pio driver and remove board specific driver gpio: Add driver for Altera's PIO core nios2: Pseudo implement dcache_status/enable/disable()
| * | nios2: Offer ft_board_setup() capability and call fdt_fixup_ethernet().Joachim Foerster2011-10-302-0/+54
| | | | | | | | | | | | | | | | | | | | | | | | The main motivation is, to have the local-mac-address property of the ethernet peripherals fixed/synced with U-Boot's environment settings. Signed-off-by: Joachim Foerster <joachim.foerster@missinglinkelectronics.com> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
| * | board/nios2-generic: Use altera_pio driver and remove board specific driverJoachim Foerster2011-10-283-72/+4
| | | | | | | | | | | | | | | Signed-off-by: Joachim Foerster <joachim.foerster@missinglinkelectronics.com> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
| * | gpio: Add driver for Altera's PIO coreJoachim Foerster2011-10-285-2/+320
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This driver may handle multiple PIO cores and thus needs to be setup by calling the altera_pio_init() function within the early board setup routine. The driver comes with some extras, see below the copyleft header. Signed-off-by: Joachim Foerster <joachim.foerster@missinglinkelectronics.com> Tested-by: Thomas Chou <thomas@wytron.com.tw> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
| * | nios2: Pseudo implement dcache_status/enable/disable()Joachim Foerster2011-10-281-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AFAIK the NIOS2 architecture does not have any possibility to dynamically switch off the data cache. So _status() always reports 1 and _enable/_disable() just issue a flush of everything. For example, common/cmd_elf.c depends on these symbols. Signed-off-by: Joachim Foerster <joachim.foerster@missinglinkelectronics.com> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
* | | tools: checkpatch.pl from Linux added to toolsJoe Hershberger2011-11-172-0/+3351
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added from Linux - commit 30ecad51849ae132dc6ef6ddb62d499c7257515b Include config file to ignore common false-positives Signed-off-by: Joe Hershberger <joe.hershberger@ni.com> Cc: Joe Hershberger <joe.hershberger@gmail.com> Cc: Wolfgang Denk <wd@denx.de>
* | | e1000: fix unused variable waring for e1000 driverZang Roy-R619112011-11-171-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the following build warning in drivers/net/e1000.c e1000.c: In function 'e1000_reset_hw': e1000.c:1373:11: warning: variable 'icr' set but not used [-Wunused-but-set-variable] e1000.c: In function 'e1000_phy_init_script': e1000.c:4395:11: warning: variable 'ret_val' set but not used [-Wunused-but-set-variable] Signed-off-by: Roy Zang <tie-fei.zang@freescale.com> Cc: Wolfgang Denk <wd@denx.de> Cc: Kyle Moffett <Kyle.D.Moffett@boeing.com>
* | | e1000: remove duplicate macros in e1000.hTimur Tabi2011-11-171-11/+0
| | | | | | | | | | | | | | | | | | | | | Some of the EEPROM Word Offset macros, and a few others, are defined twice in e1000.h. Signed-off-by: Timur Tabi <timur@freescale.com>
* | | Orphan EVB64260_750CX boardWolfgang Denk2011-11-161-5/+1
| | | | | | | | | | | | | | | | | | The last maintainer (Eran Man) cannot be reached any more. Signed-off-by: Wolfgang Denk <wd@denx.de>
* | | Revert "sandbox: Change md command to use map_physmem"Kumar Gala2011-11-161-7/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 355a835747c6f7c5071ead295a7dfc489c73cb03. The original commit broke long standing assumption that md commands work on effective addresses. This normally isn't an issue for most systems that map 1:1, however on systems with a 36-bit address map it breaks. Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
* | | common: tsi148 - fix gcc 4.6 compiler warningKim Phillips2011-11-161-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | Configuring for vme8349 board... cmd_tsi148.c: In function 'tsi148_init': cmd_tsi148.c:56:17: warning: variable 'lastError' set but not used [-Wunused-but-set-variable] Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
* | | mpc83xx: km83xx_i2c - fix gcc 4.6 compiler warningKim Phillips2011-11-161-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Configuring for kmeter1 board... km83xx_i2c.c: In function 'i2c_make_abort': km83xx_i2c.c:44:8: warning: variable 'dummy' set but not used [-Wunused-but-set-variable] Signed-off-by: Kim Phillips <kim.phillips@freescale.com> Acked-by: Heiko Schocher <hs@denx.de>
* | | mpc83xx: mpc8360emds - fix gcc 4.6 compiler warningKim Phillips2011-11-161-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | Configuring for MPC8360EMDS_66_HOST_33 - Board: MPC8360EMDS, Options: CLKIN_66MHZ,PCI,PCI_33M,PQ_MDS_PIB=1 mpc8360emds.c: In function 'board_eth_init': mpc8360emds.c:178:12: warning: array subscript is above array bounds [-Warray-bounds] Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
* | | mpc83xx: spd_sdram - fix gcc 4.6 compiler warningKim Phillips2011-11-161-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | Configuring for sbc8349 board... spd_sdram.c: In function 'spd_sdram': spd_sdram.c:152:41: warning: variable 'trfc_high' set but not used [-Wunused-but-set-variable] Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
OpenPOWER on IntegriCloud