12 Sep, 2011
1 commit
-
Signed-off-by: Wolfgang Denk
11 Sep, 2011
1 commit
-
Fix build problem:
nand_spl/board/davinci/da8xxevm/hawkboard_nand_spl.c: In function 'board_init_f':
nand_spl/board/davinci/da8xxevm/hawkboard_nand_spl.c:132: warning: implicit declaration of function 'nand_boot'
nand_spl/board/davinci/da8xxevm/hawkboard_nand_spl.c:133: warning: 'noreturn' function does returnSigned-off-by: Wolfgang Denk
Cc: Syed Mohammed Khasim
Cc: Sughosh Ganu
Cc: Albert ARIBAUD
10 Sep, 2011
27 commits
-
Fix these:
cm4008.c: In function 'board_eth_init':
cm4008.c:79: warning: implicit declaration of function 'ks8695_eth_initialize'cm41xx.c: In function 'board_eth_init':
cm41xx.c:79: warning: implicit declaration of function 'ks8695_eth_initialize'While we are at it, sort include list in netdev.h
Signed-off-by: Wolfgang Denk
Cc: Greg Ungerer -
Commit 21726a7 "Add assert() for debug assertions" caused build
warnings for all tegra2 based boards:clock.c:36:1: warning: "assert" redefined
In file included from clock.c:29:
include/common.h:144:1: warning: this is the location of the previous definitionSigned-off-by: Wolfgang Denk
Cc: Simon Glass -
get_ram_size() is called, but memsize.c is not compiled.
Signed-off-by: Stefano Babic
-
Trivial conversion of the ks8695eth driver to a CONFIG_MULTI_ETH type
driver.Signed-off-by: Greg Ungerer
-
The pointer to the flash based bootargs should be a "char *", not unsigned.
Fixes:cm41xx.c: In function ‘env_flash_cmdline’:
cm41xx.c:67: warning: pointer targets in passing argument 2 of ‘setenv’ differ in signednessSigned-off-by: Greg Ungerer
-
The pointer to the flash based bootargs should be a "char *", not unsigned.
Fixes:cm4008.c: In function ‘env_flash_cmdline’:
cm4008.c:67: warning: pointer targets in passing argument 2 of ‘setenv’ differ in signednessSigned-off-by: Greg Ungerer
-
Commit 21726a7 "Add assert() for debug assertions" broke building the
utx8245 board:dlmalloc.c: In function 'do_check_chunk':
dlmalloc.c:1660: error: 'sz' undeclared (first use in this function)
dlmalloc.c:1660: error: (Each undeclared identifier is reported only once
dlmalloc.c:1660: error: for each function it appears in.)
dlmalloc.c: In function 'do_check_free_chunk':
dlmalloc.c:1689: error: 'next' undeclared (first use in this function)
dlmalloc.c: In function 'do_check_malloced_chunk':
dlmalloc.c:1748: error: 'sz' undeclared (first use in this function)
dlmalloc.c:1750: error: 'room' undeclared (first use in this function)Signed-off-by: Wolfgang Denk
Cc: Simon Glass -
Commit 46d7274 "UBIFS: Change ubifsload to set the filesize variable"
introduced the follwing compiler warning:ubifs.c: In function 'ubifs_load':
ubifs.c:742: warning: format '%lX' expects type 'long unsigned int', but argument 3 has type 'u32'Signed-off-by: Wolfgang Denk
Cc: Bastian Ruppert -
Commit 21726a7 "Add assert() for debug assertions" caused build
warnings for many systems:In file included from bedbug.c:6:
/home/wd/git/u-boot/work/include/bedbug/bedbug.h:24:1: warning: "assert" redefined
In file included from bedbug.c:3:
/home/wd/git/u-boot/work/include/common.h:144:1: warning: this is the location of the previous definition
In file included from cmd_bedbug.c:10:
/home/wd/git/u-boot/work/include/bedbug/bedbug.h:24:1: warning: "assert" redefined
In file included from cmd_bedbug.c:5:
/home/wd/git/u-boot/work/include/common.h:144:1: warning: this is the location of the previous definitionSigned-off-by: Wolfgang Denk
Cc: Simon Glass
Acked-by: Simon Glass -
* 'master' of git://git.denx.de/u-boot-mpc85xx:
powerpc/mpc8610hpcd: set pci1_hose.config_table after fsl_setup_hose
powerpc/mpc8548cds: set pci1_hose.config_table after fsl_setup_hose
powerpc/mpc8568mds: set pci1_hose.config_table after fsl_setup_hose -
Use _bss_start_ofs as the size of the boot loader code+data that we want
to protect in the flash. This replaces use of the no longer defined
_armboot_start.Fixes:
flash.c: In function ‘flash_init’:
flash.c:75: error: ‘_bss_start’ undeclared (first use in this function)
flash.c:75: error: (Each undeclared identifier is reported only once
flash.c:75: error: for each function it appears in.)
flash.c:75: error: ‘_armboot_start’ undeclared (first use in this function)Signed-off-by:
-
The OpenGear boards CM4008, CM4116 and CM4148 need their DRAM base
and RAM stack base addresses defined.Fixes:
board.c: In function ‘__dram_init_banksize’:
board.c:227: error: ‘CONFIG_SYS_SDRAM_BASE’ undeclared (first use in this function)
board.c:227: error: (Each undeclared identifier is reported only once
board.c:227: error: for each function it appears in.)
board.c: In function ‘board_init_f’:
board.c:270: error: ‘CONFIG_SYS_INIT_SP_ADDR’ undeclared (first use in this function)
board.c:303: error: ‘CONFIG_SYS_SDRAM_BASE’ undeclared (first use in this function)Signed-off-by: Greg Ungerer
-
Move the TIMER_ definitions before they are used in KS8695 timer.c code.
Fixes:timer.c: In function ‘timer_init’:
timer.c:37: error: ‘TIMER_COUNT’ undeclared (first use in this function)
timer.c:37: error: (Each undeclared identifier is reported only once
timer.c:37: error: for each function it appears in.)
timer.c:38: error: ‘TIMER_PULSE’ undeclared (first use in this function)Signed-off-by: Greg Ungerer
-
This change slightly improves readability of the phydev speed/duplex
assignment logic.Signed-off-by: Vladimir Zapolskiy
Acked-by: Detlev Zundel -
This change allows to cope with a mii bus device registered using
miiphy_register(), which doesn't assign a default reset handler.Signed-off-by: Vladimir Zapolskiy
Acked-by: Detlev Zundel -
This is the same behaviour like tftp or fatload command.
Signed-off-by: Bastian Ruppert
CC: kmpark@infradead.org
Acked-by: Detlev Zundel -
assert() is like BUG_ON() but compiles to nothing unless DEBUG is defined.
This is useful when a condition is an error but a board reset is unlikely
to fix it, so it is better to soldier on in hope. Assertion failures should
be caught during development/test.It turns out that assert() is defined separately in a few places in U-Boot
with various meanings. This patch cleans up some of these.Build errors exposed by this change (and defining DEBUG) are also fixed in
this patch.Signed-off-by: Simon Glass
-
Pick them up automatically using $(boards_by_arch ...)
Signed-off-by: Wolfgang Denk
-
Signed-off-by: Wolfgang Denk
Cc: Albert ARIBAUD -
Fix these:
yaffs_guts.c: In function 'yaffs_ReadDataFromFile':
yaffs_guts.c:4622: warning: pointer targets in passing argument 3 of 'yaffs_AddrToChunk' differ in signedness
yaffs_guts.c:4622: warning: pointer targets in passing argument 4 of 'yaffs_AddrToChunk' differ in signedness
yaffs_guts.c: In function 'yaffs_WriteDataToFile':
yaffs_guts.c:4745: warning: pointer targets in passing argument 3 of 'yaffs_AddrToChunk' differ in signedness
yaffs_guts.c:4745: warning: pointer targets in passing argument 4 of 'yaffs_AddrToChunk' differ in signedness
yaffs_guts.c: In function 'yaffs_ResizeFile':
yaffs_guts.c:4968: warning: pointer targets in passing argument 3 of 'yaffs_AddrToChunk' differ in signedness
yaffs_guts.c:4968: warning: pointer targets in passing argument 4 of 'yaffs_AddrToChunk' differ in signedness
yaffs_guts.c: In function 'yaffs_GutsInitialise':
yaffs_guts.c:7235: warning: assignment from incompatible pointer type
yaffs_guts.c: In function 'yaffs_CreateNewObject':
yaffs_guts.c:2143: warning: 'tn' may be used uninitialized in this function
yaffs_guts.c: In function 'yaffs_MknodObject':
yaffs_guts.c:2258: warning: 'str' may be used uninitialized in this functionSigned-off-by: Wolfgang Denk
-
Fix these:
yaffs_guts.c: At top level:
yaffs_guts.c:400: warning: 'yaffs_SkipFullVerification' defined but not usedTesting shows no changes of the image sizes.
Signed-off-by: Wolfgang Denk
-
Fix these:
yaffs_guts.c: In function 'yaffs_Scan':
yaffs_guts.c:5436: warning: pointer targets in passing argument 4 of 'yaffs_QueryInitialBlockState' differ in signedness
yaffs_guts.c: In function 'yaffs_ScanBackwards':
yaffs_guts.c:6017: warning: pointer targets in passing argument 4 of 'yaffs_QueryInitialBlockState' differ in signedness
yaffs_nand.c: In function 'yaffs_QueryInitialBlockState':
yaffs_nand.c:109: warning: pointer targets in passing argument 4 of 'dev->queryNANDBlock' differ in signedness
yaffs_nand.c:113: warning: pointer targets in passing argument 4 of 'yaffs_TagsCompatabilityQueryNANDBlock' differ in signednessSigned-off-by: Wolfgang Denk
-
Drop the "-DNO_Y_INLINE" setting to fix these:
yaffs_guts.h:806: warning: 'yaffs_GetBlockInfo' defined but not usedImpact on image size is negligible - for the VCMA9 board the text
segment size grew from 496353 to 496357 bytes (i. e. 0.0008%);
total image size even remained constant.Signed-off-by: Wolfgang Denk
-
Fix these:
yaffscfg.c: In function 'cmd_yaffs_mread_file':
yaffscfg.c:316: warning: format '%08x' expects type 'unsigned int', but argument 3 has type 'char *'
yaffscfg.c: In function 'cmd_yaffs_ls': yaffscfg.c:371: warning: format '%7d' expects type 'int', but argument 3 has type 'off_t'Signed-off-by: Wolfgang Denk
-
Fix these:
cmd_yaffs2.c: In function 'do_ywr':
cmd_yaffs2.c:69: warning: format '%x' expects type 'unsigned int', but argument 2 has type 'ulong'
cmd_yaffs2.c:69: warning: format '%x' expects type 'unsigned int', but argument 3 has type 'ulong'Signed-off-by: Wolfgang Denk
-
eth_get_dev_by_name() is not safe to use for devname being NULL
as it uses strcmp. This patch makes it fail with a BUG().Signed-off-by: Helmut Raiger
Acked-by: Mike Frysinger -
Commit 6af1d41 "smc911x MII made available" was missing a few "const"
qualifiers. Fix the resulting in build warnings:smc911x.c: In function 'smc911x_initialize':
smc911x.c:297: warning: passing argument 2 of 'miiphy_register' from incompatible pointer type
smc911x.c:297: warning: passing argument 3 of 'miiphy_register' from incompatible pointer typeSigned-off-by: Wolfgang Denk
Cc: Helmut Raiger
09 Sep, 2011
3 commits
-
The function fsl_setup_hose clears the variable pci1_hose.
Set pci1_hose.config_table after it.Signed-off-by: Zhao Chenhui
Signed-off-by: Kumar Gala -
The function fsl_setup_hose clears the variable pci1_hose.
Set pci1_hose.config_table after it.Signed-off-by: Zhao Chenhui
Signed-off-by: Kumar Gala -
The function fsl_setup_hose clears the variable pci1_hose.
Set pci1_hose.config_table after it.Signed-off-by: Zhao Chenhui
Signed-off-by: Kumar Gala
08 Sep, 2011
8 commits
-
Signed-off-by: Stefano Babic
-
FIX for a typo-bug: The address is stored in PSPR, not PSSR.
Signed-off-by: Marek Vasut
-
The driver already had the MII functions, but they have not been
registered using miiphy_register().Signed-off-by: Helmut Raiger
-
Currently when you call ROUND with a value that is already a
multiple of the second parameter it will return a value that is
one multiple larger, instead of returning the value passed in.There are only two types of usage of ROUND currently, one in
various config files to round CONFIG_SYS_MALLOC_LEN to a multiple
of 4096 bytes. The other in cmd_sf.c where the incorrect behavior
of ROUND is worked around be subtracting one from the length argument
before passing it to ROUND.This patch fixes ROUND and removes the workaround from cmd_sf. It
also results in all of the malloc pools that use ROUND to compute
their size shrinking by 4KB.Cc: Lukasz Majewski
Cc: Mike Frysinger -
* 'master' of git://git.denx.de/u-boot-mmc:
ftsdc010: add support of ftsdc010 mmc controller
mmc: Fix mmc_send_status() -
* 'master' of git://git.denx.de/u-boot-video:
VIDEO: mb86r0xgdc.c: fix warning: unused variable 'i' -
Signed-off-by: Stefano Babic
CC: Heiko Schocher
Acked-by: Heiko Schocher -
Add missing case to be used in common MXC code.
Signed-off-by: Stefano Babic
