reduce warnings with W=1
Subsystems (23)
cover reduce warnings with W=1
When compiling u-boot with W=1 there are numerous of warnings. This patchset attempts to reduce it a bit. One source of false warnings are the aliases missing a prototype, most of them are replaced with __weak functions. Others are cause by missing includes / local functions not be marked as such. At last some actual missing prototypes are added. These are (hopefully) just trivial patches, some less than trial is ahead of us.
Jeroen Hofstee (49):
arm: board: use __weak
common: board: use __weak
cti_flash.c: use __weak when requested
i2c: use __weak
misc: use __weak
net: phy: fix warnings with W=1
pci: use __weak
samsung: board: use __weak
usb: use __weak
video: use __weak
common: cmd_elf: make do_bootelf_exec static
common: board_r: make local functions static disk/part.c: make local function static ehci-hcd.c: make local functions static
env_nand: make local functions static
eth: make eth_address_set static
gpio: make local functions static
image-fit: make local function static
lmb: make local functions static
net: fec_mxc: make local functions static
omap3: make local functions static
omap_gpmc: fix warnigs with W=1
sdhci: make local functions static
serial: make local functions static
spi: make local functions static
tegra: make local functions static
video: ipu: prevent warnings with W=1 common:console: add missing include string.h: add missing prototypes
arch-mx: add missing include common/cmd_elf.c: add missing include
imx: add missing includes
leds: missing include
lib: bootm: add missing include
mmc: bcm2835_sdhci: add missing include
net: davince: add missing include
omap3: board: add missing include and proto
twister: add missing includes
arm: vectors: provide protypes from vectors.S
bootm: add prototype for arch_preboot_os
elf: add prototype for valid_elf_image
fdt_support: add prototypes for __weak functions
gpio: add protype for name_to_gpio
ide: add missing prototype
mmc: add prototype for mmc_get_env_addr
serial: add prototypes for init functions
spl: add prototype for jump_to_image_no_args
tegra: add proto for pin_mux_mmc
i2c: designware: add an implement i2c protos
arch/arm/cpu/arm926ejs/at91/led.c | 1 + arch/arm/cpu/arm926ejs/cache.c | 5 +- arch/arm/cpu/armv7/mx6/soc.c | 1 + arch/arm/cpu/armv7/omap3/board.c | 3 +- arch/arm/cpu/armv7/omap3/emif4.c | 2 +- arch/arm/cpu/armv7/omap3/sys_info.c | 4 +- arch/arm/cpu/armv7/tegra20/display.c | 3 +- arch/arm/cpu/tegra-common/sys_info.c | 2 +- arch/arm/imx-common/cpu.c | 2 + arch/arm/imx-common/misc.c | 1 + arch/arm/include/asm/arch-mxs/sys_proto.h | 2 + arch/arm/include/asm/arch-omap3/sys_proto.h | 1 + arch/arm/include/asm/arch-tegra/board.h | 11 +- arch/arm/include/asm/u-boot-arm.h | 15 +++ arch/arm/lib/board.c | 41 +++----- arch/arm/lib/bootm.c | 1 + arch/arm/lib/interrupts.c | 1 + board/atmel/at91rm9200ek/led.c | 1 + board/atmel/at91sam9260ek/led.c | 1 + board/nvidia/common/board.c | 43 ++------ board/nvidia/common/emc.c | 1 + board/nvidia/seaboard/seaboard.c | 1 + board/samsung/common/board.c | 8 +- board/technexion/twister/twister.c | 2 + board/w7o/fsboot.c | 3 +- common/board_f.c | 10 +- common/board_r.c | 18 ++-- common/cmd_elf.c | 4 +- common/cmd_gpio.c | 2 +- common/console.c | 3 +- common/env_nand.c | 4 +- common/image-fit.c | 2 +- common/menu.c | 5 +- common/stdio.c | 14 +-- disk/part.c | 2 +- drivers/gpio/bcm2835_gpio.c | 2 +- drivers/gpio/kw_gpio.c | 4 +- drivers/i2c/designware_i2c.c | 17 ++-- drivers/i2c/i2c_core.c | 4 +- drivers/i2c/tegra_i2c.c | 7 +- drivers/input/tegra-kbc.c | 2 +- drivers/mmc/bcm2835_sdhci.c | 1 + drivers/mmc/sdhci.c | 6 +- drivers/mmc/tegra_mmc.c | 3 +- drivers/mtd/cfi_flash.c | 50 +++------- drivers/mtd/nand/omap_gpmc.c | 19 +--- drivers/net/davinci_emac.c | 1 + drivers/net/fec_mxc.c | 8 +- drivers/net/phy/phy.c | 7 +- drivers/pci/pci.c | 8 +- drivers/power/power_spi.c | 4 +- drivers/serial/serial.c | 149 ++++++++++++++-------------- drivers/serial/serial_sh.c | 4 +- drivers/usb/host/ehci-hcd.c | 17 +--- drivers/video/cfb_console.c | 14 +-- drivers/video/exynos_fb.c | 32 ++---- drivers/video/ipu_common.c | 2 +- drivers/video/ipu_disp.c | 17 +--- drivers/video/mxc_ipuv3_fb.c | 2 +- include/asm-generic/gpio.h | 2 + include/bootm.h | 2 + include/elf.h | 2 + include/fdt_support.h | 2 + include/ide.h | 1 + include/linux/string.h | 7 ++ include/mmc.h | 1 + include/serial.h | 49 +++++++++ include/spl.h | 2 + lib/lmb.c | 2 +- net/eth.c | 2 +- post/post.c | 9 +- 71 files changed, 320 insertions(+), 361 deletions(-)
Signature · 1 line
2.1.0
PATCH 1/49 arm: board: use __weak
--- arch/arm/lib/board.c | 40 ++++++++++++---------------------------- 1 file changed, 12 insertions(+), 28 deletions(-)
arch/arm/lib/board.c +12 -28
@@ -63,25 +63,15 @@ extern void dataflash_print_info(void);************************************************************************ * May be supplied by boards if desired */ -inline void __coloured_LED_init (void ) {} -void coloured_LED_init (void ) - __attribute__((weak,alias ("__coloured_LED_init" ))); -inline void __red_led_on (void ) {} -void red_led_on (void ) __attribute__((weak,alias ("__red_led_on" ))); -inline void __red_led_off (void ) {} -void red_led_off (void ) __attribute__((weak,alias ("__red_led_off" ))); -inline void __green_led_on (void ) {} -void green_led_on (void ) __attribute__((weak,alias ("__green_led_on" ))); -inline void __green_led_off (void ) {} -void green_led_off (void ) __attribute__((weak,alias ("__green_led_off" ))); -inline void __yellow_led_on (void ) {} -void yellow_led_on (void ) __attribute__((weak,alias ("__yellow_led_on" ))); -inline void __yellow_led_off (void ) {} -void yellow_led_off (void ) __attribute__((weak,alias ("__yellow_led_off" ))); -inline void __blue_led_on (void ) {} -void blue_led_on (void ) __attribute__((weak,alias ("__blue_led_on" ))); -inline void __blue_led_off (void ) {} -void blue_led_off (void ) __attribute__((weak,alias ("__blue_led_off" ))); +__weak void coloured_LED_init (void ) {} +__weak void red_led_on (void ) {} +__weak void red_led_off (void ) {} +__weak void green_led_on (void ) {} +__weak void green_led_off (void ) {} +__weak void yellow_led_on (void ) {} +__weak void yellow_led_off (void ) {} +__weak void blue_led_on (void ) {} +__weak void blue_led_off (void ) {}/* ************************************************************************
@@ -198,27 +188,21 @@ static int arm_pci_init(void)*/ typedef int (init_fnc_t ) (void ); -void __dram_init_banksize (void ) +__weak void dram_init_banksize (void ) { gd->bd->bi_dram[0 ].start =CONFIG_SYS_SDRAM_BASE ; gd->bd->bi_dram[0 ].size = gd->ram_size; } -void dram_init_banksize (void ) - __attribute__((weak,alias ("__dram_init_banksize" ))); -int __arch_cpu_init (void ) +__weak int arch_cpu_init (void ) {return 0 ; } -int arch_cpu_init (void ) - __attribute__((weak,alias ("__arch_cpu_init" ))); -int __power_init_board (void ) +__weak int power_init_board (void ) {return 0 ; } -int power_init_board (void ) - __attribute__((weak,alias ("__power_init_board" )));/* Record the board_init_f() bootstage (after arch_cpu_init()) */ static int mark_bootstage (void )
Signature · 1 line
2.1.0
PATCH 1/49 arm: board: use __weak
Hi Jeroen,
Quoted 65 lines · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>--- arch/arm/lib/board.c | 40 ++++++++++++---------------------------- 1 file changed, 12 insertions(+), 28 deletions(-)arch/arm/lib/board.c +11
-25@@ -63,25 +63,15 @@ extern void dataflash_print_info(void);************************************************************************ * May be supplied by boards if desired */ -inline void __coloured_LED_init (void ) {} -void coloured_LED_init (void ) - __attribute__((weak,alias ("__coloured_LED_init" ))); -inline void __red_led_on (void ) {} -void red_led_on (void ) __attribute__((weak,alias ("__red_led_on" ))); -inline void __red_led_off (void ) {} -void red_led_off (void ) __attribute__((weak,alias ("__red_led_off" ))); -inline void __green_led_on (void ) {} -void green_led_on (void ) __attribute__((weak,alias ("__green_led_on" ))); -inline void __green_led_off (void ) {} -void green_led_off (void ) __attribute__((weak,alias ("__green_led_off" ))); -inline void __yellow_led_on (void ) {} -void yellow_led_on (void ) __attribute__((weak,alias ("__yellow_led_on" ))); -inline void __yellow_led_off (void ) {} -void yellow_led_off (void ) __attribute__((weak,alias ("__yellow_led_off" ))); -inline void __blue_led_on (void ) {} -void blue_led_on (void ) __attribute__((weak,alias ("__blue_led_on" ))); -inline void __blue_led_off (void ) {} -void blue_led_off (void ) __attribute__((weak,alias ("__blue_led_off" ))); +__weak void coloured_LED_init (void ) {} +__weak void red_led_on (void ) {} +__weak void red_led_off (void ) {} +__weak void green_led_on (void ) {} +__weak void green_led_off (void ) {} +__weak void yellow_led_on (void ) {} +__weak void yellow_led_off (void ) {} +__weak void blue_led_on (void ) {} +__weak void blue_led_off (void ) {}/* ************************************************************************ @@ -198,27 +188,21 @@ static int arm_pci_init(void)*/ typedef int (init_fnc_t ) (void ); -void __dram_init_banksize (void ) +__weak void dram_init_banksize (void ) { gd->bd->bi_dram[0 ].start =CONFIG_SYS_SDRAM_BASE ; gd->bd->bi_dram[0 ].size = gd->ram_size; } -void dram_init_banksize (void ) - __attribute__((weak,alias ("__dram_init_banksize" ))); -int __arch_cpu_init (void ) +__weak int arch_cpu_init (void ) {return 0 ; } -int arch_cpu_init (void ) - __attribute__((weak,alias ("__arch_cpu_init" )));
arch/arm/lib/board.c static int arm_pci_init(void) · line 208
Quoted 10 lines
arch/arm/lib/board.c +1
-3-int __power_init_board (void ) +__weak int power_init_board (void ) {return 0 ; } -int power_init_board (void ) - __attribute__((weak,alias ("__power_init_board" )));/* Record the board_init_f() bootstage (after arch_cpu_init()) */ static int mark_bootstage (void )
Amicalement,
Signature · 1 line
Albert.
PATCH 1/49 arm: board: use __weak
In reply to Jeroen HofsteeQuoted 2 lines · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>Acked-by Albert ARIBAUD <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/fcd49784/attachment.pgp>
PATCH 2/49 common: board: use __weak
In reply to Jeroen Hofstee--- common/board_f.c | 10 ++-------- common/board_r.c | 10 ++-------- 2 files changed, 4 insertions(+), 16 deletions(-)
common/board_f.c +2 -8
@@ -128,14 +128,11 @@ int init_func_watchdog_reset(void) }#endif /* CONFIG_WATCHDOG */ -void __board_add_ram_info (int use_default) +__weak void board_add_ram_info (int use_default) {/* please define platform specific board_add_ram_info() */ } -void board_add_ram_info (int ) - __attribute__ ((weak,alias ("__board_add_ram_info" ))); -static int init_baud_rate (void ) { gd->baudrate =getenv_ulong ("baudrate" ,10 ,CONFIG_BAUDRATE );
@@ -221,7 +218,7 @@ static int show_dram_config(void)return 0 ; } -void __dram_init_banksize (void ) +__weak void dram_init_banksize (void ) {#if defined(CONFIG_NR_DRAM_BANKS ) && defined(CONFIG_SYS_SDRAM_BASE ) gd->bd->bi_dram[0 ].start =CONFIG_SYS_SDRAM_BASE ;
@@ -229,9 +226,6 @@ void __dram_init_banksize(void)#endif } -void dram_init_banksize (void ) - __attribute__((weak,alias ("__dram_init_banksize" ))); -#if defined(CONFIG_HARD_I2C ) || defined(CONFIG_SYS_I2C )static int init_func_i2c (void ) {
common/board_r.c +2 -8
@@ -60,7 +60,7 @@ DECLARE_GLOBAL_DATA_PTR;ulong monitor_flash_len; -int __board_flash_wp_on (void ) +__weak int board_flash_wp_on (void ) {/* * Most flashes can't be detected when write protection is enabled,
@@ -70,16 +70,10 @@ int __board_flash_wp_on(void)return 0 ; } -int board_flash_wp_on (void ) - __attribute__ ((weak,alias ("__board_flash_wp_on" ))); - -void __cpu_secondary_init_r (void ) +__weak void cpu_secondary_init_r (void ) { } -void cpu_secondary_init_r (void ) - __attribute__ ((weak,alias ("__cpu_secondary_init_r" ))); -static int initr_secondary_cpu (void ) {/*
Signature · 1 line
2.1.0
PATCH 2/49 common: board: use __weak
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/aa73c3c3/attachment.pgp>
PATCH 3/49 cti_flash.c: use __weak when requested
In reply to Jeroen HofsteeFor performance reason there seems to be a CONFIG_xyz. Introduce a __maybe_weak to allow using __weak directly instead of relying on aliases.
--- drivers/mtd/cfi_flash.c | 50 ++++++++++++++++--------------------------------- 1 file changed, 16 insertions(+), 34 deletions(-)
drivers/mtd/cfi_flash.c +16 -34
@@ -63,6 +63,12 @@ flash_info_t flash_info[CFI_MAX_FLASH_BANKS]; /* FLASH chips info */#define CONFIG_SYS_FLASH_CFI_WIDTH FLASH_CFI_8BIT#endif +#ifdef CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS +#define __maybe_weak __weak +#else +#define __maybe_weak static +#endif +/* * 0xffff is an undefined value for the configuration register. When * this value is returned, the configuration register shall not be
@@ -81,14 +87,12 @@ static u16 cfi_flash_config_reg(int i)int cfi_flash_num_flash_banks =CONFIG_SYS_MAX_FLASH_BANKS_DETECT ;#endif -static phys_addr_t __cfi_flash_bank_addr (int i) +__weak phys_addr_t cfi_flash_bank_addr (int i) {return ((phys_addr_t [])CONFIG_SYS_FLASH_BANKS_LIST )[i]; } -phys_addr_t cfi_flash_bank_addr (int i) - __attribute__((weak,alias ("__cfi_flash_bank_addr" ))); -static unsigned long __cfi_flash_bank_size (int i) +__weak unsigned long cfi_flash_bank_size (int i) {#ifdef CONFIG_SYS_FLASH_BANKS_SIZES return ((unsigned long [])CONFIG_SYS_FLASH_BANKS_SIZES )[i];
@@ -96,71 +100,49 @@ static unsigned long __cfi_flash_bank_size(int i)return 0 ;#endif } -unsigned long cfi_flash_bank_size (int i) - __attribute__((weak,alias ("__cfi_flash_bank_size" ))); -static void __flash_write8 (u8 value,void *addr) +__maybe_weak void flash_write8(u8 value, void *addr) {__raw_writeb (value, addr); } -static void __flash_write16 (u16 value,void *addr) +__maybe_weak void flash_write16(u16 value, void *addr) {__raw_writew (value, addr); } -static void __flash_write32 (u32 value,void *addr) +__maybe_weak void flash_write32(u32 value, void *addr) {__raw_writel (value, addr); } -static void __flash_write64 (u64 value,void *addr) +__maybe_weak void flash_write64(u64 value, void *addr) {/* No architectures currently implement __raw_writeq() */ *(volatile u64 *)addr = value; } -static u8 __flash_read8 (void *addr) +__maybe_weak u8 flash_read8(void *addr) {return __raw_readb (addr); } -static u16 __flash_read16 (void *addr) +__maybe_weak u16 flash_read16(void *addr) {return __raw_readw (addr); } -static u32 __flash_read32 (void *addr) +__maybe_weak u32 flash_read32(void *addr) {return __raw_readl (addr); } -static u64 __flash_read64 (void *addr) +__maybe_weak u64 flash_read64(void *addr) {/* No architectures currently implement __raw_readq() */ return *(volatile u64 *)addr; } -#ifdef CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS -void flash_write8 (u8 value,void *addr)__attribute__((weak,alias ("__flash_write8" ))); -void flash_write16 (u16 value,void *addr)__attribute__((weak,alias ("__flash_write16" ))); -void flash_write32 (u32 value,void *addr)__attribute__((weak,alias ("__flash_write32" ))); -void flash_write64 (u64 value,void *addr)__attribute__((weak,alias ("__flash_write64" ))); -u8 flash_read8 (void *addr)__attribute__((weak,alias ("__flash_read8" ))); -u16 flash_read16 (void *addr)__attribute__((weak,alias ("__flash_read16" ))); -u32 flash_read32 (void *addr)__attribute__((weak,alias ("__flash_read32" ))); -u64 flash_read64 (void *addr)__attribute__((weak,alias ("__flash_read64" ))); -#else -#define flash_write8 __flash_write8 -#define flash_write16 __flash_write16 -#define flash_write32 __flash_write32 -#define flash_write64 __flash_write64 -#define flash_read8 __flash_read8 -#define flash_read16 __flash_read16 -#define flash_read32 __flash_read32 -#define flash_read64 __flash_read64 -#endif -/*----------------------------------------------------------------------- */ #if defined(CONFIG_ENV_IS_IN_FLASH ) || defined(CONFIG_ENV_ADDR_REDUND ) || (CONFIG_SYS_MONITOR_BASE >=CONFIG_SYS_FLASH_BASE )
Signature · 1 line
2.1.0
PATCH 3/49 cti_flash.c: use __weak when requested
Quoted 8 lines · Jeroen Hofstee
For performance reason there seems to be a
CONFIG_xyz. Introduce a__maybe_weakto allow using__weakdirectly instead of relying on aliases.Signed-off-by Jeroen Hofstee <[email protected]>--- drivers/mtd/cfi_flash.c | 50 ++++++++++++++++--------------------------------- 1 file changed, 16 insertions(+), 34 deletions(-)
As a reminder to myself, CONFIG_xyz should beCONFIG_CFI_FLASH_USE_WEAK_ACCESSOR. And you seem
to need to opt-in for it, not opt-out.
PATCH 1/1 cti_flash.c: use __weak for flash_(read|write){8, 16, 32, 64}
In reply to Jeroen HofsteeFrom: Jeroen Hofstee <[email protected]>
For various reasons (flash access design, errata work-around) some platforms need to implement their own flash_read/write functions, so mark them as __weak. A follow-up cleanup here would be to remove CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS if this is always known at compile-time, as it appears to be.
--- drivers/mtd/cfi_flash.c | 45 +++++++++++---------------------------------- include/mtd/cfi_flash.h | 2 -- 2 files changed, 11 insertions(+), 36 deletions(-)
drivers/mtd/cfi_flash.c +11 -34
@@ -24,6 +24,7 @@#include <asm/unaligned.h> #include <environment.h> #include <mtd/cfi_flash.h> +#include <linux/compiler.h> #include <watchdog.h> /*
@@ -81,14 +82,12 @@ static u16 cfi_flash_config_reg(int i)int cfi_flash_num_flash_banks =CONFIG_SYS_MAX_FLASH_BANKS_DETECT ;#endif -static phys_addr_t __cfi_flash_bank_addr (int i) +__weak phys_addr_t cfi_flash_bank_addr (int i) {return ((phys_addr_t [])CONFIG_SYS_FLASH_BANKS_LIST )[i]; } -phys_addr_t cfi_flash_bank_addr (int i) - __attribute__((weak,alias ("__cfi_flash_bank_addr" ))); -static unsigned long __cfi_flash_bank_size (int i) +__weak unsigned long cfi_flash_bank_size (int i) {#ifdef CONFIG_SYS_FLASH_BANKS_SIZES return ((unsigned long [])CONFIG_SYS_FLASH_BANKS_SIZES )[i];
@@ -96,71 +95,49 @@ static unsigned long __cfi_flash_bank_size(int i)return 0 ;#endif } -unsigned long cfi_flash_bank_size (int i) - __attribute__((weak,alias ("__cfi_flash_bank_size" ))); -static void __flash_write8 (u8 value,void *addr) +__weak void flash_write8 (u8 value,void *addr) {__raw_writeb (value, addr); } -static void __flash_write16 (u16 value,void *addr) +__weak void flash_write16 (u16 value,void *addr) {__raw_writew (value, addr); } -static void __flash_write32 (u32 value,void *addr) +__weak void flash_write32 (u32 value,void *addr) {__raw_writel (value, addr); } -static void __flash_write64 (u64 value,void *addr) +__weak void flash_write64 (u64 value,void *addr) {/* No architectures currently implement __raw_writeq() */ *(volatile u64 *)addr = value; } -static u8 __flash_read8 (void *addr) +__weak u8 flash_read8 (void *addr) {return __raw_readb (addr); } -static u16 __flash_read16 (void *addr) +__weak u16 flash_read16 (void *addr) {return __raw_readw (addr); } -static u32 __flash_read32 (void *addr) +__weak u32 flash_read32 (void *addr) {return __raw_readl (addr); } -static u64 __flash_read64 (void *addr) +__weak u64 flash_read64 (void *addr) {/* No architectures currently implement __raw_readq() */ return *(volatile u64 *)addr; } -#ifdef CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS -void flash_write8 (u8 value,void *addr)__attribute__((weak,alias ("__flash_write8" ))); -void flash_write16 (u16 value,void *addr)__attribute__((weak,alias ("__flash_write16" ))); -void flash_write32 (u32 value,void *addr)__attribute__((weak,alias ("__flash_write32" ))); -void flash_write64 (u64 value,void *addr)__attribute__((weak,alias ("__flash_write64" ))); -u8 flash_read8 (void *addr)__attribute__((weak,alias ("__flash_read8" ))); -u16 flash_read16 (void *addr)__attribute__((weak,alias ("__flash_read16" ))); -u32 flash_read32 (void *addr)__attribute__((weak,alias ("__flash_read32" ))); -u64 flash_read64 (void *addr)__attribute__((weak,alias ("__flash_read64" ))); -#else -#define flash_write8 __flash_write8 -#define flash_write16 __flash_write16 -#define flash_write32 __flash_write32 -#define flash_write64 __flash_write64 -#define flash_read8 __flash_read8 -#define flash_read16 __flash_read16 -#define flash_read32 __flash_read32 -#define flash_read64 __flash_read64 -#endif -/*----------------------------------------------------------------------- */ #if defined(CONFIG_ENV_IS_IN_FLASH ) || defined(CONFIG_ENV_ADDR_REDUND ) || (CONFIG_SYS_MONITOR_BASE >=CONFIG_SYS_FLASH_BASE )
include/mtd/cfi_flash.h +0 -2
@@ -171,7 +171,6 @@ phys_addr_t cfi_flash_bank_addr(int i);unsigned long cfi_flash_bank_size (int i);void flash_cmd_reset (flash_info_t *info); -#ifdef CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS void flash_write8 (u8 value,void *addr);void flash_write16 (u16 value,void *addr);void flash_write32 (u32 value,void *addr);
@@ -180,6 +179,5 @@ u8 flash_read8(void *addr);u16 flash_read16 (void *addr);u32 flash_read32 (void *addr);u64 flash_read64 (void *addr); -#endif #endif /* __CFI_FLASH_H__ */
Signature · 1 line
1.7.9.5
PATCH 1/1 cti_flash.c: use __weak for flash_(read|write){8, 16, 32, 64}
Quoted 10 lines · Tom Rini
From: Jeroen Hofstee <[email protected]>
For various reasons (flash access design, errata work-around) some platforms need to implement their own flash_read/write functions, so mark them as
__weak. A follow-up cleanup here would be to removeCONFIG_CFI_FLASH_USE_WEAK_ACCESSORSif this is always known at compile-time, as it appears to be.Signed-off-by Jeroen Hofstee <[email protected]>Signed-off-by Tom Rini <[email protected]>
On IRC, Jeroen pointed out that this way does change behavior and potentially performance since in the normal case these functions no longer get inlined. So I'm withdrawing this patch.
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/a809626a/attachment.pgp>
PATCH 3/49 cti_flash.c: use __weak when requested
In reply to Jeroen HofsteeQuoted 8 lines · Jeroen Hofstee
For various reasons (design, errata) boards may need to implement their own versions of these accessors. So in the case of
CONFIG_CFI_FLASH_USE_WEAK_ACCESSORmark the functions as weak. In the normal case mark them as static to allow for better optimization.Signed-off-by Jeroen Hofstee <[email protected]>[trini: Reword commit message]Signed-off-by Tom Rini <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/d06c504a/attachment.pgp>
PATCH 4/49 i2c: use __weak
In reply to Jeroen Hofstee--- arch/arm/cpu/arm926ejs/cache.c | 5 +---- drivers/i2c/i2c_core.c | 4 +--- 2 files changed, 2 insertions(+), 7 deletions(-)
arch/arm/cpu/arm926ejs/cache.c +1 -4
@@ -99,7 +99,4 @@ void flush_cache(unsigned long start, unsigned long size)/* * Stub implementations for l2 cache operations */ -void __l2_cache_disable (void ) {} - -void l2_cache_disable (void ) - __attribute__((weak,alias ("__l2_cache_disable" ))); +__weak void l2_cache_disable (void ) {}
drivers/i2c/i2c_core.c +1 -3
@@ -395,9 +395,7 @@ void i2c_reg_write(uint8_t addr, uint8_t reg, uint8_t val)i2c_write (addr, reg,1 , &val,1 ); } -void __i2c_init (int speed,int slaveaddr) +__weak void i2c_init (int speed,int slaveaddr) {i2c_init_bus (i2c_get_bus_num (), speed, slaveaddr); } -void i2c_init (int speed,int slaveaddr) - __attribute__((weak,alias ("__i2c_init" )));
Signature · 1 line
2.1.0
PATCH 4/49 i2c: use __weak
Hello Jeroen,
first, thanks for this clean up!
Am 08.10.2014 22:57, schrieb Jeroen Hofstee:
Quoted 3 lines
Signed-off-by Jeroen Hofstee<[email protected]>--- arch/arm/cpu/arm926ejs/cache.c | 5 +----
This change seems to have nothing to do with the subject ... could you please split this?
Quoted 2 lines
drivers/i2c/i2c_core.c | 4 +--- 2 files changed, 2 insertions(+), 7 deletions(-)
Beside of this nitpick:
bye,
Heiko
Signature · 2 lines
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
PATCH 4/49 i2c: use __weak
Hello Heiko,
Quoted 9 lines · Heiko Schocher
Am 08.10.2014 22:57, schrieb Jeroen Hofstee:
Quoted 3 lines
Signed-off-by Jeroen Hofstee<[email protected]>--- arch/arm/cpu/arm926ejs/cache.c | 5 +----This change seems to have nothing to do with the subject ... could you please split this?
No problem, I likely accidentally squashed them. I will repost this patch with out it and append a new one.
Regards,
Jeroen
PATCH 4/49 i2c: use __weak
Quoted 9 lines · Jeroen Hofstee
Hello Heiko,
Quoted 6 lines · Heiko Schocher
Am 08.10.2014 22:57, schrieb Jeroen Hofstee:
Quoted 3 lines
Signed-off-by Jeroen Hofstee<[email protected]>--- arch/arm/cpu/arm926ejs/cache.c | 5 +----
Quoted 6 lines
Quoted 3 lines
This change seems to have nothing to do with the subject ... could you please split this?
No problem, I likely accidentally squashed them. I will repost this patch with out it and append a new one.
Did I miss it? Thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/89d496a4/attachment.pgp>
PATCH 4/49 i2c: use __weak
Hello Tom,
Quoted 9 lines · Tom Rini
Quoted 7 lines · Jeroen Hofstee
Hello Heiko,
Quoted 4 lines · Heiko Schocher
Am 08.10.2014 22:57, schrieb Jeroen Hofstee:
Quoted 3 lines
Signed-off-by Jeroen Hofstee<[email protected]>--- arch/arm/cpu/arm926ejs/cache.c | 5 +----
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
This change seems to have nothing to do with the subject ... could you please split this?
No problem, I likely accidentally squashed them. I will repost this patch with out it and append a new one.
Did I miss it? Thanks!
Thanks for applying most of them. I reposted the remaining ones.
Regards,
Jeroen
PATCH 5/49 misc: use __weak
In reply to Jeroen Hofstee--- common/menu.c | 5 +---- post/post.c | 9 ++------- 2 files changed, 3 insertions(+), 11 deletions(-)
common/menu.c +1 -4
@@ -105,12 +105,9 @@ static inline void *menu_item_destroy(struct menu *m,return NULL ; } -void __menu_display_statusline (struct menu *m) +__weak void menu_display_statusline (struct menu *m) { -return ; } -void menu_display_statusline (struct menu *m) - __attribute__ ((weak,alias ("__menu_display_statusline" )));/* * Display a menu so the user can make a choice of an item. First display its
post/post.c +2 -7
@@ -52,7 +52,7 @@ int post_init_f(void)* Boards with hotkey support can override this weak default function * by defining one in their board specific code. */ -int __post_hotkeys_pressed (void ) +__weak int post_hotkeys_pressed (void ) {#ifdef CONFIG_SYS_POST_HOTKEYS_GPIO int ret;
@@ -73,9 +73,6 @@ int __post_hotkeys_pressed(void)return 0 ;/* No hotkeys supported */ } -int post_hotkeys_pressed (void ) - __attribute__((weak,alias ("__post_hotkeys_pressed" ))); -void post_bootmode_init (void ) {
@@ -236,11 +233,9 @@ static void post_get_flags(int *test_flags) test_flags[j] |=POST_SLOWTEST ; } -void __show_post_progress (unsigned int test_num,int before,int result) +__weak void show_post_progress (unsigned int test_num,int before,int result) { } -void show_post_progress (unsigned int ,int ,int ) - __attribute__((weak,alias ("__show_post_progress" )));static int post_run_single (struct post_test *test,int test_flags,int flags,unsigned int i)
Signature · 1 line
2.1.0
PATCH 5/49 misc: use __weak
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/4da3471e/attachment.pgp>
PATCH 6/49 net: phy: fix warnings with W=1
In reply to Jeroen Hofsteeget_phy_id is marked weak but has no protype nor a strong version, just make it static. Use __weak for board_phy_config.
--- drivers/net/phy/phy.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)
drivers/net/phy/phy.c +2 -5
@@ -575,7 +575,7 @@ static struct phy_device *phy_device_create(struct mii_dev *bus, int addr,* Description: Reads the ID registers of the PHY at @addr on the * @bus, stores it in @phy_id and returns zero on success. */ -int __weak get_phy_id (struct mii_dev *bus,int addr,int devad,u32 *phy_id) +static int get_phy_id (struct mii_dev *bus,int addr,int devad,u32 *phy_id) {int phy_reg;
@@ -785,16 +785,13 @@ int phy_startup(struct phy_device *phydev)return 0 ; } -static int __board_phy_config (struct phy_device *phydev) +__weak int board_phy_config (struct phy_device *phydev) {if (phydev->drv->config)return phydev->drv->config (phydev);return 0 ; } -int board_phy_config (struct phy_device *phydev) - __attribute__((weak,alias ("__board_phy_config" ))); -int phy_config (struct phy_device *phydev) {/* Invoke an optional board-specific helper */
Signature · 1 line
2.1.0
PATCH 6/49 net: phy: fix warnings with W=1
Quoted 5 lines · Jeroen Hofstee
get_phy_idis marked weak but has no protype nor a strong version, just make it static. Use__weakforboard_phy_config.Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/a2d58fcb/attachment.pgp>
PATCH 7/49 pci: use __weak
In reply to Jeroen Hofstee--- drivers/pci/pci.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-)
drivers/pci/pci.c +2 -6
@@ -572,7 +572,7 @@ const char * pci_class_str(u8 class) }#endif /* CONFIG_CMD_PCI || CONFIG_PCI_SCAN_SHOW */ -int __pci_skip_dev (struct pci_controller *hose,pci_dev_t dev) +__weak int pci_skip_dev (struct pci_controller *hose,pci_dev_t dev) {/* * Check if pci device should be skipped in configuration
@@ -591,19 +591,15 @@ int __pci_skip_dev(struct pci_controller *hose, pci_dev_t dev)return 0 ; } -int pci_skip_dev (struct pci_controller *hose,pci_dev_t dev) - __attribute__((weak,alias ("__pci_skip_dev" )));#ifdef CONFIG_PCI_SCAN_SHOW -int __pci_print_dev (struct pci_controller *hose,pci_dev_t dev) +__weak int pci_print_dev (struct pci_controller *hose,pci_dev_t dev) {if (dev ==PCI_BDF (hose->first_busno,0 ,0 ))return 0 ;return 1 ; } -int pci_print_dev (struct pci_controller *hose,pci_dev_t dev) - __attribute__((weak,alias ("__pci_print_dev" )));#endif /* CONFIG_PCI_SCAN_SHOW */ int pci_hose_scan_bus (struct pci_controller *hose,int bus)
Signature · 1 line
2.1.0
PATCH 7/49 pci: use __weak
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/dae5f758/attachment.pgp>
PATCH 8/49 samsung: board: use __weak
In reply to Jeroen Hofstee--- board/samsung/common/board.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-)
board/samsung/common/board.c +2 -6
@@ -28,19 +28,15 @@DECLARE_GLOBAL_DATA_PTR ; -int __exynos_early_init_f (void ) +__weak int exynos_early_init_f (void ) {return 0 ; } -int exynos_early_init_f (void ) - __attribute__((weak,alias ("__exynos_early_init_f" ))); -int __exynos_power_init (void ) +__weak int exynos_power_init (void ) {return 0 ; } -int exynos_power_init (void ) - __attribute__((weak,alias ("__exynos_power_init" )));#if definedCONFIG_EXYNOS_TMU /* Boot Time Thermal Analysis for SoC temperature threshold breach */
Signature · 1 line
2.1.0
PATCH 8/49 samsung: board: use __weak
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/01917213/attachment.pgp>
PATCH 9/49 usb: use __weak
In reply to Jeroen Hofstee--- drivers/usb/host/ehci-hcd.c | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-)
drivers/usb/host/ehci-hcd.c +3 -12
@@ -119,15 +119,12 @@ static struct descriptor {#define ehci_is_TDI () (0)#endif -int __ehci_get_port_speed (struct ehci_hcor *hcor,uint32_t reg) +__weak int ehci_get_port_speed (struct ehci_hcor *hcor,uint32_t reg) {return PORTSC_PSPD (reg); } -int ehci_get_port_speed (struct ehci_hcor *hcor,uint32_t reg) - __attribute__((weak,alias ("__ehci_get_port_speed" ))); - -void __ehci_set_usbmode (int index) +__weak void ehci_set_usbmode (int index) {uint32_t tmp;uint32_t *reg_ptr;
@@ -141,17 +138,11 @@ void __ehci_set_usbmode(int index)ehci_writel (reg_ptr, tmp); } -void ehci_set_usbmode (int index) - __attribute__((weak,alias ("__ehci_set_usbmode" ))); - -void __ehci_powerup_fixup (uint32_t *status_reg,uint32_t *reg) +__weak void ehci_powerup_fixup (uint32_t *status_reg,uint32_t *reg) {mdelay (50 ); } -void ehci_powerup_fixup (uint32_t *status_reg,uint32_t *reg) - __attribute__((weak,alias ("__ehci_powerup_fixup" ))); -static int handshake (uint32_t *ptr,uint32_t mask,uint32_t done,int usec) {uint32_t result;
Signature · 1 line
2.1.0
PATCH 9/49 usb: use __weak
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/cd1170c5/attachment.pgp>
PATCH 10/49 video: use __weak
In reply to Jeroen Hofstee--- drivers/video/cfb_console.c | 10 ++-------- drivers/video/exynos_fb.c | 32 ++++++++------------------------ 2 files changed, 10 insertions(+), 32 deletions(-)
drivers/video/cfb_console.c +2 -8
@@ -1171,14 +1171,11 @@ void video_puts(struct stdio_dev *dev, const char *s)* video_set_lut() if they do not support 8 bpp format. * Implement weak default function instead. */ -void __video_set_lut (unsigned int index,unsigned char r, +__weak void video_set_lut (unsigned int index,unsigned char r,unsigned char g,unsigned char b) { } -void video_set_lut (unsigned int ,unsigned char ,unsigned char ,unsigned char ) - __attribute__ ((weak,alias ("__video_set_lut" ))); -#if defined(CONFIG_CMD_BMP ) || defined(CONFIG_SPLASH_SCREEN )#define FILL_8BIT_332RGB (r,g,b) { \
@@ -2240,15 +2237,12 @@ static int video_init(void)* Implement a weak default function for boards that optionally * need to skip the video initialization. */ -int __board_video_skip (void ) +__weak int board_video_skip (void ) {/* As default, don't skip test */ return 0 ; } -int board_video_skip (void ) - __attribute__ ((weak,alias ("__board_video_skip" ))); -int drv_video_init (void ) {int skip_dev_init;
drivers/video/exynos_fb.c +8 -24
@@ -58,54 +58,38 @@ static void exynos_lcd_init(vidinfo_t *vid)lcd_set_flush_dcache (1 ); } -void __exynos_cfg_lcd_gpio (void ) +__weak void exynos_cfg_lcd_gpio (void ) { } -void exynos_cfg_lcd_gpio (void ) - __attribute__((weak,alias ("__exynos_cfg_lcd_gpio" ))); -void __exynos_backlight_on (unsigned int onoff) +__weak void exynos_backlight_on (unsigned int onoff) { } -void exynos_backlight_on (unsigned int onoff) - __attribute__((weak,alias ("__exynos_cfg_lcd_gpio" ))); -void __exynos_reset_lcd (void ) +__weak void exynos_reset_lcd (void ) { } -void exynos_reset_lcd (void ) - __attribute__((weak,alias ("__exynos_reset_lcd" ))); -void __exynos_lcd_power_on (void ) +__weak void exynos_lcd_power_on (void ) { } -void exynos_lcd_power_on (void ) - __attribute__((weak,alias ("__exynos_lcd_power_on" ))); -void __exynos_cfg_ldo (void ) +__weak void exynos_cfg_ldo (void ) { } -void exynos_cfg_ldo (void ) - __attribute__((weak,alias ("__exynos_cfg_ldo" ))); -void __exynos_enable_ldo (unsigned int onoff) +__weak void exynos_enable_ldo (unsigned int onoff) { } -void exynos_enable_ldo (unsigned int onoff) - __attribute__((weak,alias ("__exynos_enable_ldo" ))); -void __exynos_backlight_reset (void ) +__weak void exynos_backlight_reset (void ) { } -void exynos_backlight_reset (void ) - __attribute__((weak,alias ("__exynos_backlight_reset" ))); -int __exynos_lcd_misc_init (vidinfo_t *vid) +__weak int exynos_lcd_misc_init (vidinfo_t *vid) {return 0 ; } -int exynos_lcd_misc_init (vidinfo_t *vid) - __attribute__((weak,alias ("__exynos_lcd_misc_init" )));static void lcd_panel_on (vidinfo_t *vid) {
Signature · 1 line
2.1.0
PATCH 10/49 video: use __weak
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/01a11273/attachment.pgp>
PATCH 11/49 common: cmd_elf: make do_bootelf_exec static
In reply to Jeroen Hofsteedo_bootelf_exec was a weak function without a prototype nor and strong version. Just make it static.
--- common/cmd_elf.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
common/cmd_elf.c +1 -2
@@ -28,8 +28,7 @@ static unsigned long load_elf_image_phdr(unsigned long addr);static unsigned long load_elf_image_shdr (unsigned long addr);/* Allow ports to override the default behavior */ -__attribute__((weak)) -unsigned long do_bootelf_exec (ulong (*entry)(int ,char *const []), +static unsigned long do_bootelf_exec (ulong (*entry)(int ,char *const []),int argc,char *const argv[]) {unsigned long ret;
Signature · 1 line
2.1.0
PATCH 11/49 common: cmd_elf: make do_bootelf_exec static
Quoted 4 lines · Jeroen Hofstee
do_bootelf_execwas a weak function without a prototype nor and strong version. Just make it static.Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/1d67b3b4/attachment.pgp>
PATCH 12/49 common: board_r: make local functions static
In reply to Jeroen Hofstee--- common/board_r.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
common/board_r.c +4 -4
@@ -364,7 +364,7 @@ static int initr_spi(void)#ifdef CONFIG_CMD_NAND /* go init the NAND */ -int initr_nand (void ) +static int initr_nand (void ) {puts ("NAND: " );nand_init ();
@@ -374,7 +374,7 @@ int initr_nand(void)#if defined(CONFIG_CMD_ONENAND )/* go init the NAND */ -int initr_onenand (void ) +static int initr_onenand (void ) {puts ("NAND: " );onenand_init ();
@@ -383,7 +383,7 @@ int initr_onenand(void)#endif #ifdef CONFIG_GENERIC_MMC -int initr_mmc (void ) +static int initr_mmc (void ) {puts ("MMC: " );mmc_initialize (gd->bd);
@@ -392,7 +392,7 @@ int initr_mmc(void)#endif #ifdef CONFIG_HAS_DATAFLASH -int initr_dataflash (void ) +static int initr_dataflash (void ) {AT91F_DataflashInit ();dataflash_print_info ();
Signature · 1 line
2.1.0
PATCH 12/49 common: board_r: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/170f23c4/attachment.pgp>
PATCH 13/49 disk/part.c: make local function static
In reply to Jeroen Hofstee--- disk/part.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
disk/part.c +1 -1
@@ -57,7 +57,7 @@ static const struct block_drvr block_drvr[] = {DECLARE_GLOBAL_DATA_PTR ;#ifdef HAVE_BLOCK_DEVICE -block_dev_desc_t *get_dev_hwpart (const char *ifname,int dev,int hwpart) +static block_dev_desc_t *get_dev_hwpart (const char *ifname,int dev,int hwpart) {const struct block_drvr *drvr = block_drvr;block_dev_desc_t * (*reloc_get_dev)(int dev);
Signature · 1 line
2.1.0
PATCH 13/49 disk/part.c: make local function static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/36cef6e9/attachment.pgp>
PATCH 14/49 ehci-hcd.c: make local functions static
In reply to Jeroen Hofstee--- drivers/usb/host/ehci-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/usb/host/ehci-hcd.c +1 -1
@@ -1314,7 +1314,7 @@ void *poll_int_queue(struct usb_device *dev, struct int_queue *queue) }/* Do not free buffers associated with QHs, they're owned by someone else */ -int +static int destroy_int_queue (struct usb_device *dev,struct int_queue *queue) {struct ehci_ctrl *ctrl = dev->controller;
Signature · 1 line
2.1.0
PATCH 14/49 ehci-hcd.c: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/7f186210/attachment.pgp>
PATCH 15/49 env_nand: make local functions static
In reply to Jeroen Hofstee--- common/env_nand.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
common/env_nand.c +2 -2
@@ -124,7 +124,7 @@ int env_init(void)* The legacy NAND code saved the environment in the first NAND device i.e., * nand_dev_desc + 0. This is also the behaviour using the new NAND code. */ -int writeenv (size_t offset,u_char *buf) +static int writeenv (size_t offset,u_char *buf) {size_t end = offset +CONFIG_ENV_RANGE ;size_t amount_saved =0 ;
@@ -233,7 +233,7 @@ int saveenv(void) }#endif /* CMD_SAVEENV */ -int readenv (size_t offset,u_char *buf) +static int readenv (size_t offset,u_char *buf) {size_t end = offset +CONFIG_ENV_RANGE ;size_t amount_loaded =0 ;
Signature · 1 line
2.1.0
PATCH 15/49 env_nand: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/7cce3945/attachment.pgp>
PATCH 16/49 eth: make eth_address_set static
In reply to Jeroen Hofstee--- net/eth.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
net/eth.c +1 -1
@@ -153,7 +153,7 @@ static void eth_current_changed(void)setenv ("ethact" ,NULL ); } -int eth_address_set (unsigned char *addr) +static int eth_address_set (unsigned char *addr) {return memcmp (addr,"\0\0\0\0\0\0" ,6 ); }
Signature · 1 line
2.1.0
PATCH 16/49 eth: make eth_address_set static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/9e50625b/attachment.pgp>
PATCH 17/49 gpio: make local functions static
In reply to Jeroen Hofstee--- drivers/gpio/bcm2835_gpio.c | 2 +- drivers/gpio/kw_gpio.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-)
drivers/gpio/bcm2835_gpio.c +1 -1
@@ -9,7 +9,7 @@#include <asm/gpio.h> #include <asm/io.h> -inline int gpio_is_valid (unsigned gpio) +static inline int gpio_is_valid (unsigned gpio) {return (gpio <BCM2835_GPIO_COUNT ); }
drivers/gpio/kw_gpio.c +2 -2
@@ -36,7 +36,7 @@ void __set_direction(unsigned pin, int input) u =readl (GPIO_IO_CONF (pin)); } -void __set_level (unsigned pin,int high) +static void __set_level (unsigned pin,int high) {u32 u;
@@ -48,7 +48,7 @@ void __set_level(unsigned pin, int high)writel (u,GPIO_OUT (pin)); } -void __set_blinking (unsigned pin,int blink) +static void __set_blinking (unsigned pin,int blink) {u32 u;
Signature · 1 line
2.1.0
PATCH 17/49 gpio: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/528e442f/attachment.pgp>
PATCH 18/49 image-fit: make local function static
In reply to Jeroen Hofstee--- common/image-fit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
common/image-fit.c +1 -1
@@ -1434,7 +1434,7 @@ void fit_conf_print(const void *fit, int noffset, const char *p)printf ("%s FDT: %s\n" , p, uname); } -int fit_image_select (const void *fit,int rd_noffset,int verify) +static int fit_image_select (const void *fit,int rd_noffset,int verify) {fit_image_print (fit, rd_noffset," " );
Signature · 1 line
2.1.0
PATCH 18/49 image-fit: make local function static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/3dbe23ff/attachment.pgp>
PATCH 19/49 lmb: make local functions static
In reply to Jeroen Hofstee--- lib/lmb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
lib/lmb.c +1 -1
@@ -236,7 +236,7 @@ long lmb_reserve(struct lmb *lmb, phys_addr_t base, phys_size_t size)return lmb_add_region (_rgn, base, size); } -long lmb_overlaps_region (struct lmb_region *rgn,phys_addr_t base, +static long lmb_overlaps_region (struct lmb_region *rgn,phys_addr_t base,phys_size_t size) {unsigned long i;
Signature · 1 line
2.1.0
PATCH 19/49 lmb: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/b3494029/attachment.pgp>
PATCH 20/49 net: fec_mxc: make local functions static
In reply to Jeroen Hofstee--- drivers/net/fec_mxc.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)
drivers/net/fec_mxc.c +5 -3
@@ -11,6 +11,7 @@#include <common.h> #include <malloc.h> #include <net.h> +#include <netdev.h> #include <miiphy.h> #include "fec_mxc.h"
@@ -179,13 +180,14 @@ static int fec_mdio_write(struct ethernet_regs *eth, uint8_t phyAddr,return 0 ; } -int fec_phy_read (struct mii_dev *bus,int phyAddr,int dev_addr,int regAddr) +static int fec_phy_read (struct mii_dev *bus,int phyAddr,int dev_addr, +int regAddr) {return fec_mdio_read (bus->priv, phyAddr, regAddr); } -int fec_phy_write (struct mii_dev *bus,int phyAddr,int dev_addr,int regAddr, -u16 data) +static int fec_phy_write (struct mii_dev *bus,int phyAddr,int dev_addr, +int regAddr,u16 data) {return fec_mdio_write (bus->priv, phyAddr, regAddr, data); }
Signature · 1 line
2.1.0
PATCH 20/49 net: fec_mxc: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/2fe21367/attachment.pgp>
PATCH 21/49 omap3: make local functions static
In reply to Jeroen Hofstee--- arch/arm/cpu/armv7/omap3/board.c | 2 +- arch/arm/cpu/armv7/omap3/emif4.c | 2 +- arch/arm/cpu/armv7/omap3/sys_info.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-)
arch/arm/cpu/armv7/omap3/board.c +1 -1
@@ -266,7 +266,7 @@ int __weak misc_init_r(void)* Routine: wait_for_command_complete * Description: Wait for posting to finish on watchdog *****************************************************************************/ -void wait_for_command_complete (struct watchdog *wd_base) +static void wait_for_command_complete (struct watchdog *wd_base) {int pending =1 ; do {
arch/arm/cpu/armv7/omap3/emif4.c +1 -1
@@ -61,7 +61,7 @@ u32 get_sdr_cs_offset(u32 cs)* - Init the emif4 module for DDR access * - Early init routines, called from flash or SRAM. */ -void do_emif4_init (void ) +static void do_emif4_init (void ) {unsigned int regval;/* Set the DDR PHY parameters in PHY ctrl registers */
arch/arm/cpu/armv7/omap3/sys_info.c +1 -1
@@ -202,7 +202,7 @@ u32 __weak get_board_rev(void)/******************************************************** * get_base(); get upper addr of current execution *******************************************************/ -u32 get_base (void ) +static u32 get_base (void ) {u32 val;
Signature · 1 line
2.1.0
PATCH 21/49 omap3: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/9999e86b/attachment.pgp>
PATCH 22/49 omap_gpmc: fix warnigs with W=1
In reply to Jeroen Hofstee- make omap_spl_dev_ready static
- remove unused omap_reverse_list
--- drivers/mtd/nand/omap_gpmc.c | 19 +------------------ 1 file changed, 1 insertion(+), 18 deletions(-)
drivers/mtd/nand/omap_gpmc.c +1 -18
@@ -75,7 +75,7 @@ static void omap_nand_hwcontrol(struct mtd_info *mtd, int32_t cmd,#ifdef CONFIG_SPL_BUILD /* Check wait pin as dev ready indicator */ -int omap_spl_dev_ready (struct mtd_info *mtd) +static int omap_spl_dev_ready (struct mtd_info *mtd) {return gpmc_cfg->status & (1 <<8 ); }
@@ -162,23 +162,6 @@ static int __maybe_unused omap_correct_data(struct mtd_info *mtd, uint8_t *dat, }/* -* omap_reverse_list - re-orders list elements in reverse order [internal] -* @list: pointer to start of list -* @length: length of list -*/ -void omap_reverse_list (u8 *list,unsigned int length) -{ -unsigned int i, j; -unsigned int half_length = length /2 ; -u8 tmp; -for (i =0 , j = length -1 ; i < half_length; i++, j--) { - tmp = list[i]; - list[i] = list[j]; - list[j] = tmp; - } -} - -/* * omap_enable_hwecc - configures GPMC as per ECC scheme before read/write * @mtd: MTD device structure * @mode: Read/Write mode
Signature · 1 line
2.1.0
PATCH 22/49 omap_gpmc: fix warnigs with W=1
Quoted 4 lines · Jeroen Hofstee
- make
omap_spl_dev_readystatic
- remove unusedomap_reverse_listSigned-off-by Jeroen Hofstee <[email protected]>
After moving omap_reverse_list to be under CONFIG_NAND_OMAP_ELM instead, applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/6764b28a/attachment.pgp>
PATCH 23/49 sdhci: make local functions static
In reply to Jeroen Hofstee--- drivers/mmc/sdhci.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
drivers/mmc/sdhci.c +3 -3
@@ -124,7 +124,7 @@ static int sdhci_transfer_data(struct sdhci_host *host, struct mmc_data *data,#endif #define CONFIG_SDHCI_CMD_DEFAULT_TIMEOUT 100 -int sdhci_send_command (struct mmc *mmc,struct mmc_cmd *cmd, +static int sdhci_send_command (struct mmc *mmc,struct mmc_cmd *cmd,struct mmc_data *data) {struct sdhci_host *host = mmc->priv;
@@ -355,7 +355,7 @@ static void sdhci_set_power(struct sdhci_host *host, unsigned short power)sdhci_writeb (host, pwr,SDHCI_POWER_CONTROL ); } -void sdhci_set_ios (struct mmc *mmc) +static void sdhci_set_ios (struct mmc *mmc) {u32 ctrl;struct sdhci_host *host = mmc->priv;
@@ -393,7 +393,7 @@ void sdhci_set_ios(struct mmc *mmc)sdhci_writeb (host, ctrl,SDHCI_HOST_CONTROL ); } -int sdhci_init (struct mmc *mmc) +static int sdhci_init (struct mmc *mmc) {struct sdhci_host *host = mmc->priv;
Signature · 1 line
2.1.0
PATCH 23/49 sdhci: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/93a98c67/attachment.pgp>
PATCH 24/49 serial: make local functions static
In reply to Jeroen Hofstee--- common/stdio.c | 14 +++++++------- drivers/serial/serial.c | 8 ++++---- drivers/serial/serial_sh.c | 4 ++-- drivers/video/cfb_console.c | 4 ++-- 4 files changed, 15 insertions(+), 15 deletions(-)
common/stdio.c +7 -7
@@ -39,39 +39,39 @@ char *stdio_names[MAX_FILES] = { "stdin", "stdout", "stderr" };#endif #ifdef CONFIG_SYS_DEVICE_NULLDEV -void nulldev_putc (struct stdio_dev *dev,const char c) +static void nulldev_putc (struct stdio_dev *dev,const char c) {/* nulldev is empty! */ } -void nulldev_puts (struct stdio_dev *dev,const char *s) +static void nulldev_puts (struct stdio_dev *dev,const char *s) {/* nulldev is empty! */ } -int nulldev_input (struct stdio_dev *dev) +static int nulldev_input (struct stdio_dev *dev) {/* nulldev is empty! */ return 0 ; }#endif -void stdio_serial_putc (struct stdio_dev *dev,const char c) +static void stdio_serial_putc (struct stdio_dev *dev,const char c) {serial_putc (c); } -void stdio_serial_puts (struct stdio_dev *dev,const char *s) +static void stdio_serial_puts (struct stdio_dev *dev,const char *s) {serial_puts (s); } -int stdio_serial_getc (struct stdio_dev *dev) +static int stdio_serial_getc (struct stdio_dev *dev) {return serial_getc (); } -int stdio_serial_tstc (struct stdio_dev *dev) +static int stdio_serial_tstc (struct stdio_dev *dev) {return serial_tstc (); }
drivers/serial/serial.c +4 -4
@@ -254,28 +254,28 @@ void serial_initialize(void)serial_assign (default_serial_console ()->name); } -int serial_stub_start (struct stdio_dev *sdev) +static int serial_stub_start (struct stdio_dev *sdev) {struct serial_device *dev = sdev->priv;return dev->start (); } -int serial_stub_stop (struct stdio_dev *sdev) +static int serial_stub_stop (struct stdio_dev *sdev) {struct serial_device *dev = sdev->priv;return dev->stop (); } -void serial_stub_putc (struct stdio_dev *sdev,const char ch) +static void serial_stub_putc (struct stdio_dev *sdev,const char ch) {struct serial_device *dev = sdev->priv; dev->putc (ch); } -void serial_stub_puts (struct stdio_dev *sdev,const char *str) +static void serial_stub_puts (struct stdio_dev *sdev,const char *str) {struct serial_device *dev = sdev->priv;
drivers/serial/serial_sh.c +2 -2
@@ -122,7 +122,7 @@ static void handle_error(void)sci_out (&sh_sci,SCLSR ,0x00 ); } -void serial_raw_putc (const char c) +static void serial_raw_putc (const char c) {while (1 ) {/* Tx fifo is empty */
@@ -152,7 +152,7 @@ static int sh_serial_tstc(void) } -int serial_getc_check (void ) +static int serial_getc_check (void ) {unsigned short status;
drivers/video/cfb_console.c +2 -2
@@ -944,7 +944,7 @@ static void parse_putc(const char c)CURSOR_SET ; } -void video_putc (struct stdio_dev *dev,const char c) +static void video_putc (struct stdio_dev *dev,const char c) {#ifdef CONFIG_CFB_CONSOLE_ANSI int i;
@@ -1158,7 +1158,7 @@ void video_putc(struct stdio_dev *dev, const char c)flush_cache (VIDEO_FB_ADRS ,VIDEO_SIZE ); } -void video_puts (struct stdio_dev *dev,const char *s) +static void video_puts (struct stdio_dev *dev,const char *s) {int count =strlen (s);
Signature · 1 line
2.1.0
PATCH 24/49 serial: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/1ca1bc15/attachment.pgp>
PATCH 25/49 spi: make local functions static
In reply to Jeroen Hofstee--- drivers/power/power_spi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/power/power_spi.c +2 -2
@@ -17,13 +17,13 @@static struct spi_slave *slave; -void pmic_spi_free (struct spi_slave *slave) +static void pmic_spi_free (struct spi_slave *slave) {if (slave)spi_free_slave (slave); } -struct spi_slave *pmic_spi_probe (struct pmic *p) +static struct spi_slave *pmic_spi_probe (struct pmic *p) { return spi_setup_slave(p->bus, p->hw.spi.cs,
Signature · 1 line
2.1.0
PATCH 25/49 spi: make local functions static
Quoted 15 lines · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>--- drivers/power/power_spi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)drivers/power/power_spi.c +1
-1@@ -17,13 +17,13 @@static struct spi_slave *slave; -void pmic_spi_free (struct spi_slave *slave) +static void pmic_spi_free (struct spi_slave *slave)
drivers/power/power_spi.c line 29
Quoted 10 lines
drivers/power/power_spi.c +1
-1{if (slave)spi_free_slave (slave); } -struct spi_slave *pmic_spi_probe (struct pmic *p) +static struct spi_slave *pmic_spi_probe (struct pmic *p) { return spi_setup_slave(p->bus, p->hw.spi.cs,
So, this exposes some problems with the driver. pmic_spi_free isn't ever used, and shouldn't be. I'm going to take a stab at fixing things.
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141025/5aa77c21/attachment.pgp>
PATCH 26/49 tegra: make local functions static
In reply to Jeroen Hofstee--- arch/arm/cpu/armv7/tegra20/display.c | 3 ++- arch/arm/cpu/tegra-common/sys_info.c | 2 +- board/nvidia/common/board.c | 43 ++++++------------------------------ board/nvidia/common/emc.c | 1 + board/nvidia/seaboard/seaboard.c | 1 + drivers/i2c/tegra_i2c.c | 7 +++--- drivers/input/tegra-kbc.c | 2 +- drivers/mmc/tegra_mmc.c | 3 ++- 8 files changed, 19 insertions(+), 43 deletions(-)
arch/arm/cpu/armv7/tegra20/display.c +2 -1
@@ -194,7 +194,8 @@ static void rgb_enable(struct dc_com_reg *com)writel (rgb_sel_tab[i], &com->pin_output_sel[i]); } -int setup_window (struct disp_ctl_win *win,struct fdt_disp_config *config) +static int setup_window (struct disp_ctl_win *win, +struct fdt_disp_config *config) { win->x =0 ; win->y =0 ;
arch/arm/cpu/tegra-common/sys_info.c +1 -1
@@ -8,7 +8,7 @@#include <common.h> #include <linux/ctype.h> -void upstring (char *s) +static void upstring (char *s) {while (*s) { *s =toupper (*s);
board/nvidia/common/board.c +7 -36
65 lines not on this page · raw message
board/nvidia/common/emc.c +1 -0
@@ -5,6 +5,7 @@*/ #include <common.h> +#include "emc.h" #include <asm/io.h> #include <asm/arch/clock.h> #include <asm/arch/emc.h>
board/nvidia/seaboard/seaboard.c +1 -0
@@ -8,6 +8,7 @@#include <common.h> #include <asm/io.h> #include <asm/arch/tegra.h> +#include <asm/arch-tegra/board.h> #include <asm/arch/clock.h> #include <asm/arch/funcmux.h> #include <asm/arch/gpio.h>
drivers/i2c/tegra_i2c.c +4 -3
@@ -471,8 +471,8 @@ static void tegra_i2c_init(struct i2c_adapter *adap, int speed, int slaveaddr) }/* i2c write version without the register address */ -int i2c_write_data (struct i2c_bus *bus,uchar chip,uchar *buffer,int len, -bool end_with_repeated_start) +static int i2c_write_data (struct i2c_bus *bus,uchar chip,uchar *buffer, +int len,bool end_with_repeated_start) {int rc;
@@ -493,7 +493,8 @@ int i2c_write_data(struct i2c_bus *bus, uchar chip, uchar *buffer, int len, }/* i2c read version without the register address */ -int i2c_read_data (struct i2c_bus *bus,uchar chip,uchar *buffer,int len) +static int i2c_read_data (struct i2c_bus *bus,uchar chip,uchar *buffer, +int len) {int rc;
drivers/input/tegra-kbc.c +1 -1
@@ -181,7 +181,7 @@ static void kbd_wait_for_fifo_init(struct keyb *config)* @param input Input configuration * @return 1, to indicate that we have something to look at */ -int tegra_kbc_check (struct input_config *input) +static int tegra_kbc_check (struct input_config *input) {kbd_wait_for_fifo_init (&config);check_for_keys (&config);
drivers/mmc/tegra_mmc.c +2 -1
@@ -13,6 +13,7 @@#include <asm/io.h> #include <asm/arch/clock.h> #include <asm/arch-tegra/clk_rst.h> +#include <asm/arch-tegra/mmc.h> #include <asm/arch-tegra/tegra_mmc.h> #include <mmc.h>
@@ -508,7 +509,7 @@ static int tegra_mmc_core_init(struct mmc *mmc)return 0 ; } -int tegra_mmc_getcd (struct mmc *mmc) +static int tegra_mmc_getcd (struct mmc *mmc) {struct mmc_host *host = mmc->priv;
Signature · 1 line
2.1.0
PATCH 26/49 tegra: make local functions static
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/6d9f770a/attachment.pgp>
PATCH 27/49 video: ipu: prevent warnings with W=1
In reply to Jeroen Hofstee- make local functions static
- remove unused is_interlaced function
--- drivers/video/ipu_common.c | 2 +- drivers/video/ipu_disp.c | 17 +++-------------- drivers/video/mxc_ipuv3_fb.c | 2 +- 3 files changed, 5 insertions(+), 16 deletions(-)
drivers/video/ipu_common.c +1 -1
@@ -379,7 +379,7 @@ static struct clk pixel_clk[] = {/* * This function resets IPU */ -void ipu_reset (void ) +static void ipu_reset (void ) {u32 *reg;u32 value;
drivers/video/ipu_disp.c +3 -14
@@ -377,7 +377,7 @@ static struct dp_csc_param_t dp_csc_array[CSC_NUM][CSC_NUM] = {static enum csc_type_t fg_csc_type =CSC_NONE , bg_csc_type =CSC_NONE ;static int color_key_4rgb =1 ; -void ipu_dp_csc_setup (int dp,struct dp_csc_param_t dp_csc_param, +static void ipu_dp_csc_setup (int dp,struct dp_csc_param_t dp_csc_param,unsigned char srm_mode_update) {u32 reg;
@@ -605,17 +605,6 @@ void ipu_dc_uninit(int dc_chan) } } -int ipu_chan_is_interlaced (ipu_channel_t channel) -{ -if (channel ==MEM_DC_SYNC ) -return !!(__raw_readl (DC_WR_CH_CONF_1 ) & -DC_WR_CH_CONF_FIELD_MODE ); -else if ((channel ==MEM_BG_SYNC ) || (channel ==MEM_FG_SYNC )) -return !!(__raw_readl (DC_WR_CH_CONF_5 ) & -DC_WR_CH_CONF_FIELD_MODE ); -return 0 ; -} -void ipu_dp_dc_enable (ipu_channel_t channel) {int di;
@@ -782,7 +771,7 @@ void ipu_init_dc_mappings(void)ipu_dc_map_config (4 ,2 ,21 ,0xFC ); } -int ipu_pixfmt_to_map (uint32_t fmt) +static int ipu_pixfmt_to_map (uint32_t fmt) {switch (fmt) {case IPU_PIX_FMT_GENERIC :
@@ -804,7 +793,7 @@ int ipu_pixfmt_to_map(uint32_t fmt)/* * This function is called to adapt synchronous LCD panel to IPU restriction. */ -void adapt_panel_to_ipu_restricitions(uint32_t *pixel_clk, +static void adapt_panel_to_ipu_restricitions(uint32_t *pixel_clk, uint16_t width, uint16_t height, uint16_t h_start_width, uint16_t h_end_width,
drivers/video/mxc_ipuv3_fb.c +1 -1
@@ -36,7 +36,7 @@ static struct fb_videomode const *gmode;static uint8_t gdisp;static uint32_t gpixfmt; -void fb_videomode_to_var (struct fb_var_screeninfo *var, +static void fb_videomode_to_var (struct fb_var_screeninfo *var,const struct fb_videomode *mode) { var->xres = mode->xres;
Signature · 1 line
2.1.0
PATCH 27/49 video: ipu: prevent warnings with W=1
Quoted 4 lines · Jeroen Hofstee
- make local functions static
- remove unusedis_interlacedfunctionSigned-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/f98bdd0f/attachment.pgp>
PATCH 28/49 common:console: add missing include
In reply to Jeroen Hofsteesearch_device is declared in iomux, but console only had the definition. This prevents a warning.
--- common/console.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
common/console.c +2 -1
@@ -7,6 +7,7 @@#include <common.h> #include <stdarg.h> +#include <iomux.h> #include <malloc.h> #include <os.h> #include <serial.h>
@@ -621,7 +622,7 @@ inline void dbg(const char *fmt, ...) }#else -inline void dbg (const char *fmt, ...) +static inline void dbg (const char *fmt, ...) { }#endif
Signature · 1 line
2.1.0
PATCH 28/49 common:console: add missing include
Quoted 4 lines · Jeroen Hofstee
search_deviceis declared in iomux, but console only had the definition. This prevents a warning.Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/93f23d93/attachment.pgp>
PATCH 29/49 string.h: add missing prototypes
In reply to Jeroen Hofstee--- include/linux/string.h | 7 +++++++ 1 file changed, 7 insertions(+)
include/linux/string.h +7 -0
@@ -20,6 +20,10 @@ extern __kernel_size_t strspn(const char *,const char *);*/ #include <asm/string.h> +#ifndef __HAVE_ARCH_BCOPY +char *bcopy (const char *src,char *dest,int count); +#endif +#ifndef __HAVE_ARCH_STRCPYextern char *strcpy (char *,const char *);#endif
@@ -89,6 +93,9 @@ extern void * memchr(const void *,int,__kernel_size_t);void *memchr_inv (const void *,int ,size_t );#endif +unsigned long ustrtoul (const char *cp,char **endp,unsigned int base); +unsigned long long ustrtoull (const char *cp,char **endp,unsigned int base); + #ifdef __cplusplus }#endif
Signature · 1 line
2.1.0
PATCH 29/49 string.h: add missing prototypes
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/e7e4da1c/attachment.pgp>
PATCH 30/49 arch-mx: add missing include
In reply to Jeroen Hofsteemxs_wait_mask_set and friends need a declaration
of struct mxs_register_32.
--- arch/arm/include/asm/arch-mxs/sys_proto.h | 2 ++ 1 file changed, 2 insertions(+)
arch/arm/include/asm/arch-mxs/sys_proto.h +2 -0
@@ -10,6 +10,8 @@#ifndef __SYS_PROTO_H__#define __SYS_PROTO_H__ +#include <asm/imx-common/regs-common.h> +int mxs_reset_block (struct mxs_register_32 *reg); int mxs_wait_mask_set(struct mxs_register_32 *reg, uint32_t mask,
Signature · 1 line
2.1.0
PATCH 30/49 arch-mx: add missing include
Quoted 4 lines · Jeroen Hofstee
mxs_wait_mask_setand friends need a declaration
of structmxs_register_32.Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/7f6b39be/attachment.pgp>
PATCH 31/49 common/cmd_elf.c: add missing include
In reply to Jeroen Hofstee--- common/cmd_elf.c | 1 + 1 file changed, 1 insertion(+)
common/cmd_elf.c +1 -0
@@ -13,6 +13,7 @@* purpose. */ +#include <bootm.h> #include <common.h> #include <command.h> #include <linux/ctype.h>
Signature · 1 line
2.1.0
PATCH 31/49 common/cmd_elf.c: add missing include
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/c0e9bcf1/attachment.pgp>
PATCH 32/49 imx: add missing includes
In reply to Jeroen Hofstee--- arch/arm/cpu/armv7/mx6/soc.c | 1 + arch/arm/imx-common/cpu.c | 2 ++ arch/arm/imx-common/misc.c | 1 + 3 files changed, 4 insertions(+)
arch/arm/cpu/armv7/mx6/soc.c +1 -0
@@ -9,6 +9,7 @@#include <common.h> #include <asm/armv7.h> +#include <asm/bootm.h> #include <asm/pl310.h> #include <asm/errno.h> #include <asm/io.h>
arch/arm/imx-common/cpu.c +2 -0
@@ -7,7 +7,9 @@* SPDX-License-Identifier: GPL-2.0+ */ +#include <bootm.h> #include <common.h> +#include <netdev.h> #include <asm/errno.h> #include <asm/io.h> #include <asm/arch/imx-regs.h>
arch/arm/imx-common/misc.c +1 -0
@@ -5,6 +5,7 @@*/ #include <common.h> +#include <asm/arch/sys_proto.h> #include <asm/errno.h> #include <asm/io.h> #include <asm/imx-common/regs-common.h>
Signature · 1 line
2.1.0
PATCH 32/49 imx: add missing includes
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/04f9144a/attachment.pgp>
PATCH 33/49 leds: missing include
In reply to Jeroen Hofstee--- arch/arm/cpu/arm926ejs/at91/led.c | 1 + arch/arm/lib/board.c | 1 + board/atmel/at91rm9200ek/led.c | 1 + board/atmel/at91sam9260ek/led.c | 1 + 4 files changed, 4 insertions(+)
arch/arm/cpu/arm926ejs/at91/led.c +1 -0
@@ -9,6 +9,7 @@#include <common.h> #include <asm/gpio.h> #include <asm/arch/gpio.h> +#include <status_led.h> #ifdef CONFIG_RED_LED void red_led_on (void )
arch/arm/lib/board.c +1 -0
@@ -34,6 +34,7 @@#include <onenand_uboot.h> #include <mmc.h> #include <scsi.h> +#include <status_led.h> #include <libfdt.h> #include <fdtdec.h> #include <post.h>
board/atmel/at91rm9200ek/led.c +1 -0
@@ -14,6 +14,7 @@#include <asm/arch/hardware.h> #include <asm/arch/at91_pmc.h> #include <asm/arch/at91_pio.h> +#include <status_led.h> /* bit mask in PIO port B */ #define GREEN_LED (1<<0)
board/atmel/at91sam9260ek/led.c +1 -0
@@ -9,6 +9,7 @@#include <common.h> #include <asm/io.h> #include <asm/arch/gpio.h> +#include <status_led.h> void coloured_LED_init (void ) {
Signature · 1 line
2.1.0
PATCH 33/49 leds: missing include
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/30846a41/attachment.pgp>
PATCH 34/49 lib: bootm: add missing include
In reply to Jeroen Hofsteesince the vxworks weaks are reimplement make
sure their prototypes are visible.
--- arch/arm/lib/bootm.c | 1 + 1 file changed, 1 insertion(+)
arch/arm/lib/bootm.c +1 -0
@@ -15,6 +15,7 @@#include <common.h> #include <command.h> #include <image.h> +#include <vxworks.h> #include <u-boot/zlib.h> #include <asm/byteorder.h> #include <libfdt.h>
Signature · 1 line
2.1.0
PATCH 34/49 lib: bootm: add missing include
Quoted 4 lines · Jeroen Hofstee
since the vxworks weaks are reimplement make
sure their prototypes are visible.Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/5f5098fa/attachment.pgp>
PATCH 35/49 mmc: bcm2835_sdhci: add missing include
In reply to Jeroen Hofstee--- drivers/mmc/bcm2835_sdhci.c | 1 + 1 file changed, 1 insertion(+)
drivers/mmc/bcm2835_sdhci.c +1 -0
@@ -40,6 +40,7 @@#include <malloc.h> #include <sdhci.h> #include <asm/arch/timer.h> +#include <asm/arch-bcm2835/sdhci.h> /* 400KHz is max freq for card ID etc. Use that as min */ #define MIN_FREQ 400000
Signature · 1 line
2.1.0
PATCH 35/49 mmc: bcm2835_sdhci: add missing include
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/1366feaa/attachment.pgp>
PATCH 36/49 net: davince: add missing include
In reply to Jeroen Hofstee--- drivers/net/davinci_emac.c | 1 + 1 file changed, 1 insertion(+)
drivers/net/davinci_emac.c +1 -0
@@ -27,6 +27,7 @@#include <net.h> #include <miiphy.h> #include <malloc.h> +#include <netdev.h> #include <linux/compiler.h> #include <asm/arch/emac_defs.h> #include <asm/io.h>
Signature · 1 line
2.1.0
PATCH 36/49 net: davince: add missing include
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/e47c8731/attachment.pgp>
PATCH 37/49 omap3: board: add missing include and proto
In reply to Jeroen Hofstee--- arch/arm/cpu/armv7/omap3/board.c | 1 + arch/arm/cpu/armv7/omap3/sys_info.c | 2 ++ arch/arm/include/asm/arch-omap3/sys_proto.h | 1 + 3 files changed, 4 insertions(+)
arch/arm/cpu/armv7/omap3/board.c +1 -0
@@ -17,6 +17,7 @@* SPDX-License-Identifier: GPL-2.0+ */ #include <common.h> +#include <mmc.h> #include <spl.h> #include <asm/io.h> #include <asm/arch/sys_proto.h>
arch/arm/cpu/armv7/omap3/sys_info.c +2 -0
@@ -16,6 +16,8 @@#include <asm/io.h> #include <asm/arch/mem.h> /* get mem tables */ #include <asm/arch/sys_proto.h> +#include <asm/bootm.h> +#include <i2c.h> #include <linux/compiler.h>
arch/arm/include/asm/arch-omap3/sys_proto.h +1 -0
@@ -64,6 +64,7 @@ void try_unlock_memory(void);u32 get_boot_type (void );void invalidate_dcache (u32 );u32 wait_on_value (u32 ,u32 ,void *,u32 ); +void cancel_out (u32 *num,u32 *den,u32 den_limit);void sdelay (unsigned long );void make_cs1_contiguous (void );void omap_nand_switch_ecc (uint32_t ,uint32_t );
Signature · 1 line
2.1.0
PATCH 37/49 omap3: board: add missing include and proto
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/b9d43b17/attachment.pgp>
PATCH 38/49 twister: add missing includes
In reply to Jeroen Hofstee--- board/technexion/twister/twister.c | 2 ++ 1 file changed, 2 insertions(+)
board/technexion/twister/twister.c +2 -0
@@ -16,6 +16,8 @@#include <asm/omap_gpio.h> #include <asm/arch/mmc_host_def.h> #include <i2c.h> +#include <spl.h> +#include <mmc.h> #include <asm/gpio.h> #ifdef CONFIG_USB_EHCI #include <usb.h>
Signature · 1 line
2.1.0
PATCH 38/49 twister: add missing includes
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/55586275/attachment.pgp>
PATCH 39/49 arm: vectors: provide protypes from vectors.S
In reply to Jeroen Hofstee--- arch/arm/include/asm/u-boot-arm.h | 15 +++++++++++++++ arch/arm/lib/interrupts.c | 1 + 2 files changed, 16 insertions(+)
arch/arm/include/asm/u-boot-arm.h +15 -0
@@ -45,4 +45,19 @@ void reset_timer_masked (void);ulong get_timer_masked (void );void udelay_masked (unsigned long usec); +/* calls to c from vectors.S */ +void bad_mode (void ); +void do_undefined_instruction (struct pt_regs *pt_regs); +void do_software_interrupt (struct pt_regs *pt_regs); +void do_prefetch_abort (struct pt_regs *pt_regs); +void do_data_abort (struct pt_regs *pt_regs); +void do_not_used (struct pt_regs *pt_regs); +#ifdef CONFIG_ARM64 +void do_fiq (struct pt_regs *pt_regs,unsigned int esr); +void do_irq (struct pt_regs *pt_regs,unsigned int esr); +#else +void do_fiq (struct pt_regs *pt_regs); +void do_irq (struct pt_regs *pt_regswq); +#endif +#endif /* _U_BOOT_ARM_H_ */
arch/arm/lib/interrupts.c +1 -0
@@ -21,6 +21,7 @@#include <common.h> #include <asm/proc-armv/ptrace.h> +#include <asm/u-boot-arm.h> DECLARE_GLOBAL_DATA_PTR ;
Signature · 1 line
2.1.0
PATCH 39/49 arm: vectors: provide protypes from vectors.S
Hi Jeroen,
Quoted 35 lines · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>--- arch/arm/include/asm/u-boot-arm.h | 15 +++++++++++++++ arch/arm/lib/interrupts.c | 1 + 2 files changed, 16 insertions(+)arch/arm/include/asm/u-boot-arm.h +15
-0@@ -45,4 +45,19 @@ void reset_timer_masked (void);ulong get_timer_masked (void );void udelay_masked (unsigned long usec); +/* calls to c from vectors.S */ +void bad_mode (void ); +void do_undefined_instruction (struct pt_regs *pt_regs); +void do_software_interrupt (struct pt_regs *pt_regs); +void do_prefetch_abort (struct pt_regs *pt_regs); +void do_data_abort (struct pt_regs *pt_regs); +void do_not_used (struct pt_regs *pt_regs); +#ifdef CONFIG_ARM64 +void do_fiq (struct pt_regs *pt_regs,unsigned int esr); +void do_irq (struct pt_regs *pt_regs,unsigned int esr); +#else +void do_fiq (struct pt_regs *pt_regs); +void do_irq (struct pt_regs *pt_regswq); +#endif +#endif /* _U_BOOT_ARM_H_ */ arch/arm/lib/interrupts.c +0
-0
arch/arm/lib/interrupts.c line 27
Quoted 7 lines
arch/arm/lib/interrupts.c +1
-0@@ -21,6 +21,7 @@#include <common.h> #include <asm/proc-armv/ptrace.h> +#include <asm/u-boot-arm.h> DECLARE_GLOBAL_DATA_PTR ;
Amicalement,
Signature · 1 line
Albert.
PATCH 39/49 arm: vectors: provide protypes from vectors.S
In reply to Jeroen HofsteeQuoted 2 lines · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>Acked-by Albert ARIBAUD <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/daafbe44/attachment.pgp>
PATCH 40/49 bootm: add prototype for arch_preboot_os
In reply to Jeroen Hofstee--- include/bootm.h | 2 ++ 1 file changed, 2 insertions(+)
include/bootm.h +2 -0
@@ -54,4 +54,6 @@ int bootm_find_ramdisk_fdt(int flag, int argc, char * const argv[]);int do_bootm_states (cmd_tbl_t *cmdtp,int flag,int argc,char *const argv[],int states,bootm_headers_t *images,int boot_progress); +void arch_preboot_os (void ); +#endif
Signature · 1 line
2.1.0
PATCH 40/49 bootm: add prototype for arch_preboot_os
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/edaf8c35/attachment.pgp>
PATCH 41/49 elf: add prototype for valid_elf_image
In reply to Jeroen Hofstee--- board/w7o/fsboot.c | 3 +-- include/elf.h | 2 ++ 2 files changed, 3 insertions(+), 2 deletions(-)
board/w7o/fsboot.c +1 -2
@@ -8,12 +8,11 @@#include <common.h> #include <config.h> #include <command.h> +#include <elf.h> /* * FIXME: Add code to test image and it's header. */ -extern int valid_elf_image (unsigned long addr); -static int image_check (ulong addr) {
include/elf.h +2 -0
@@ -570,4 +570,6 @@ unsigned long elf_hash(const unsigned char *name);that may still be in object files. */ #define R_PPC_TOC16 255 +int valid_elf_image (unsigned long addr); +#endif /* _ELF_H */
Signature · 1 line
2.1.0
PATCH 41/49 elf: add prototype for valid_elf_image
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/d9dc9cf4/attachment.pgp>
PATCH 42/49 fdt_support: add prototypes for __weak functions
In reply to Jeroen Hofstee--- include/fdt_support.h | 2 ++ 1 file changed, 2 insertions(+)
include/fdt_support.h +2 -0
@@ -144,6 +144,8 @@ static inline u64 of_read_number(const fdt32_t *cell, int size)void of_bus_default_count_cells (void *blob,int parentoffset,int *addrc,int *sizec); +int ft_verify_fdt (void *fdt); +int arch_fixup_memory_node (void *blob);#endif /* ifdef CONFIG_OF_LIBFDT */
Signature · 1 line
2.1.0
PATCH 42/49 fdt_support: add prototypes for __weak functions
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/c66371a3/attachment.pgp>
PATCH 43/49 gpio: add protype for name_to_gpio
In reply to Jeroen Hofstee--- common/cmd_gpio.c | 2 +- include/asm-generic/gpio.h | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-)
common/cmd_gpio.c +1 -1
@@ -12,7 +12,7 @@#include <dm.h> #include <asm/gpio.h> -int __weak name_to_gpio (const char *name) +__weak int name_to_gpio (const char *name) {return simple_strtoul (name,NULL ,10 ); }
include/asm-generic/gpio.h +2 -0
@@ -193,4 +193,6 @@ const char *gpio_get_bank_info(struct udevice *dev, int *offset_count);int gpio_lookup_name (const char *name,struct udevice **devp,unsigned int *offsetp,unsigned int *gpiop); +int name_to_gpio (const char *name); +#endif /* _ASM_GENERIC_GPIO_H_ */
Signature · 1 line
2.1.0
PATCH 43/49 gpio: add protype for name_to_gpio
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/31ddf0de/attachment-0001.pgp>
PATCH 44/49 ide: add missing prototype
In reply to Jeroen Hofstee--- include/ide.h | 1 + 1 file changed, 1 insertion(+)
include/ide.h +1 -0
@@ -23,6 +23,7 @@ extern ulong ide_bus_offset[];#define LED_IDE2 0x02#define DEVICE_LED (d) ((d & 2) | ((d & 2) == 0))/* depends on bit positions! */ +void ide_led (uchar led,uchar status);#endif /* CONFIG_IDE_LED */ #ifdef CONFIG_SYS_64BIT_LBA
Signature · 1 line
2.1.0
PATCH 44/49 ide: add missing prototype
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/20ac6c41/attachment-0001.pgp>
PATCH 45/49 mmc: add prototype for mmc_get_env_addr
In reply to Jeroen Hofstee--- include/mmc.h | 1 + 1 file changed, 1 insertion(+)
include/mmc.h +1 -0
@@ -387,6 +387,7 @@ int mmc_legacy_init(int verbose);int board_mmc_init (bd_t *bis);int cpu_mmc_init (bd_t *bis); +int mmc_get_env_addr (struct mmc *mmc,int copy,u32 *env_addr);/* Set block count limit because of 16 bit register limit on some hardware*/ #ifndef CONFIG_SYS_MMC_MAX_BLK_COUNT
Signature · 1 line
2.1.0
PATCH 45/49 mmc: add prototype for mmc_get_env_addr
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/a5e2ef3e/attachment-0001.pgp>
PATCH 46/49 serial: add prototypes for init functions
In reply to Jeroen HofsteeWhile at it, sort them.
--- drivers/serial/serial.c | 141 ++++++++++++++++++++++++------------------------ include/serial.h | 49 +++++++++++++++++ 2 files changed, 120 insertions(+), 70 deletions(-)
drivers/serial/serial.c +71 -70
@@ -109,54 +109,55 @@ U_BOOT_ENV_CALLBACK(baudrate, on_baudrate);void name (void ) \ __attribute__((weak,alias ("serial_null" ))); -serial_initfunc (mpc8xx_serial_initialize); -serial_initfunc (ns16550_serial_initialize); -serial_initfunc (pxa_serial_initialize); -serial_initfunc (s3c24xx_serial_initialize); -serial_initfunc (s5p_serial_initialize); -serial_initfunc (zynq_serial_initialize); -serial_initfunc (bfin_serial_initialize); -serial_initfunc (bfin_jtag_initialize); -serial_initfunc (mpc512x_serial_initialize); -serial_initfunc (uartlite_serial_initialize); -serial_initfunc (au1x00_serial_initialize); -serial_initfunc (asc_serial_initialize); -serial_initfunc (jz_serial_initialize); -serial_initfunc (mpc5xx_serial_initialize); -serial_initfunc (mpc8260_scc_serial_initialize); -serial_initfunc (mpc8260_smc_serial_initialize); -serial_initfunc (mpc85xx_serial_initialize); -serial_initfunc (iop480_serial_initialize); -serial_initfunc (leon2_serial_initialize); -serial_initfunc (leon3_serial_initialize); -serial_initfunc (marvell_serial_initialize); +serial_initfunc (altera_jtag_serial_initialize); +serial_initfunc (altera_serial_initialize);serial_initfunc (amirix_serial_initialize); +serial_initfunc (arc_serial_initialize); +serial_initfunc (arm_dcc_initialize); +serial_initfunc (asc_serial_initialize); +serial_initfunc (atmel_serial_initialize); +serial_initfunc (au1x00_serial_initialize); +serial_initfunc (bfin_jtag_initialize); +serial_initfunc (bfin_serial_initialize);serial_initfunc (bmw_serial_initialize); +serial_initfunc (clps7111_serial_initialize);serial_initfunc (cogent_serial_initialize);serial_initfunc (cpci750_serial_initialize);serial_initfunc (evb64260_serial_initialize); -serial_initfunc (ml2_serial_initialize); -serial_initfunc (sconsole_serial_initialize); -serial_initfunc (p3mx_serial_initialize); -serial_initfunc (altera_jtag_serial_initialize); -serial_initfunc (altera_serial_initialize); -serial_initfunc (atmel_serial_initialize); -serial_initfunc (lpc32xx_serial_initialize); -serial_initfunc (mcf_serial_initialize); -serial_initfunc (oc_serial_initialize); -serial_initfunc (sandbox_serial_initialize); -serial_initfunc (clps7111_serial_initialize);serial_initfunc (imx_serial_initialize); +serial_initfunc (iop480_serial_initialize); +serial_initfunc (jz_serial_initialize);serial_initfunc (ks8695_serial_initialize); +serial_initfunc (leon2_serial_initialize); +serial_initfunc (leon3_serial_initialize);serial_initfunc (lh7a40x_serial_initialize); +serial_initfunc (lpc32xx_serial_initialize); +serial_initfunc (marvell_serial_initialize);serial_initfunc (max3100_serial_initialize); +serial_initfunc (mcf_serial_initialize); +serial_initfunc (ml2_serial_initialize); +serial_initfunc (mpc512x_serial_initialize); +serial_initfunc (mpc5xx_serial_initialize); +serial_initfunc (mpc8260_scc_serial_initialize); +serial_initfunc (mpc8260_smc_serial_initialize); +serial_initfunc (mpc85xx_serial_initialize); +serial_initfunc (mpc8xx_serial_initialize);serial_initfunc (mxc_serial_initialize); +serial_initfunc (mxs_auart_initialize); +serial_initfunc (ns16550_serial_initialize); +serial_initfunc (oc_serial_initialize); +serial_initfunc (p3mx_serial_initialize);serial_initfunc (pl01x_serial_initialize); +serial_initfunc (pxa_serial_initialize); +serial_initfunc (s3c24xx_serial_initialize); +serial_initfunc (s5p_serial_initialize);serial_initfunc (sa1100_serial_initialize); +serial_initfunc (sandbox_serial_initialize); +serial_initfunc (sconsole_serial_initialize);serial_initfunc (sh_serial_initialize); -serial_initfunc (arm_dcc_initialize); -serial_initfunc (mxs_auart_initialize); -serial_initfunc (arc_serial_initialize); +serial_initfunc (uartlite_serial_initialize); +serial_initfunc (zynq_serial_initialize); +/** * serial_register() - Register serial driver with serial driver core
@@ -202,54 +203,54 @@ void serial_register(struct serial_device *dev)*/ void serial_initialize (void ) { -mpc8xx_serial_initialize (); -ns16550_serial_initialize (); -pxa_serial_initialize (); -s3c24xx_serial_initialize (); -s5p_serial_initialize (); -mpc512x_serial_initialize (); -bfin_serial_initialize (); -bfin_jtag_initialize (); -uartlite_serial_initialize (); -zynq_serial_initialize (); -au1x00_serial_initialize (); -asc_serial_initialize (); -jz_serial_initialize (); -mpc5xx_serial_initialize (); -mpc8260_scc_serial_initialize (); -mpc8260_smc_serial_initialize (); -mpc85xx_serial_initialize (); -iop480_serial_initialize (); -leon2_serial_initialize (); -leon3_serial_initialize (); -marvell_serial_initialize (); +altera_jtag_serial_initialize (); +altera_serial_initialize ();amirix_serial_initialize (); +arc_serial_initialize (); +arm_dcc_initialize (); +asc_serial_initialize (); +atmel_serial_initialize (); +au1x00_serial_initialize (); +bfin_jtag_initialize (); +bfin_serial_initialize ();bmw_serial_initialize (); +clps7111_serial_initialize ();cogent_serial_initialize ();cpci750_serial_initialize ();evb64260_serial_initialize (); -ml2_serial_initialize (); -sconsole_serial_initialize (); -p3mx_serial_initialize (); -altera_jtag_serial_initialize (); -altera_serial_initialize (); -atmel_serial_initialize (); -lpc32xx_serial_initialize (); -mcf_serial_initialize (); -oc_serial_initialize (); -sandbox_serial_initialize (); -clps7111_serial_initialize ();imx_serial_initialize (); +iop480_serial_initialize (); +jz_serial_initialize ();ks8695_serial_initialize (); +leon2_serial_initialize (); +leon3_serial_initialize ();lh7a40x_serial_initialize (); +lpc32xx_serial_initialize (); +marvell_serial_initialize ();max3100_serial_initialize (); +mcf_serial_initialize (); +ml2_serial_initialize (); +mpc512x_serial_initialize (); +mpc5xx_serial_initialize (); +mpc8260_scc_serial_initialize (); +mpc8260_smc_serial_initialize (); +mpc85xx_serial_initialize (); +mpc8xx_serial_initialize ();mxc_serial_initialize (); +mxs_auart_initialize (); +ns16550_serial_initialize (); +oc_serial_initialize (); +p3mx_serial_initialize ();pl01x_serial_initialize (); +pxa_serial_initialize (); +s3c24xx_serial_initialize (); +s5p_serial_initialize ();sa1100_serial_initialize (); +sandbox_serial_initialize (); +sconsole_serial_initialize ();sh_serial_initialize (); -arm_dcc_initialize (); -mxs_auart_initialize (); -arc_serial_initialize (); +uartlite_serial_initialize (); +zynq_serial_initialize ();serial_assign (default_serial_console ()->name); }
include/serial.h +49 -0
53 lines not on this page · raw message
Signature · 1 line
2.1.0
PATCH 46/49 serial: add prototypes for init functions
Quoted 3 lines · Jeroen Hofstee
While at it, sort them.
Signed-off-by Jeroen Hofstee <[email protected]>
This doesn't apply currently, please re-base.
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/bc3285d0/attachment.pgp>
PATCH 47/49 spl: add prototype for jump_to_image_no_args
In reply to Jeroen Hofstee--- include/spl.h | 2 ++ 1 file changed, 2 insertions(+)
include/spl.h +2 -0
@@ -72,6 +72,8 @@ void spl_sata_load_image(void);int spl_load_image_fat (block_dev_desc_t *block_dev,int partition,const char *filename);int spl_load_image_fat_os (block_dev_desc_t *block_dev,int partition); +void __noreturn jump_to_image_no_args (struct spl_image_info *spl_image); +#ifdef CONFIG_SPL_BOARD_INIT void spl_board_init (void );#endif
Signature · 1 line
2.1.0
PATCH 47/49 spl: add prototype for jump_to_image_no_args
Quoted 1 line · Jeroen Hofstee
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/c2c24b21/attachment-0001.pgp>
PATCH 48/49 tegra: add proto for pin_mux_mmc
In reply to Jeroen Hofsteewhile at it, fix a typo
--- arch/arm/include/asm/arch-tegra/board.h | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-)
arch/arm/include/asm/arch-tegra/board.h +6 -5
@@ -24,10 +24,11 @@ void gpio_early_init(void); /* overrideable GPIO config */* an empty stub function will be called. */ -void pinmux_init (void );/* overrideable general pinmux setup */ -void pin_mux_usb (void );/* overrideable USB pinmux setup */ -void pin_mux_spi (void );/* overrideable SPI pinmux setup */ -void pin_mux_nand (void );/* overrideable NAND pinmux setup */ -void pin_mux_display (void );/* overrideable DISPLAY pinmux setup */ +void pinmux_init (void );/* overridable general pinmux setup */ +void pin_mux_usb (void );/* overridable USB pinmux setup */ +void pin_mux_spi (void );/* overridable SPI pinmux setup */ +void pin_mux_nand (void );/* overridable NAND pinmux setup */ +void pin_mux_mmc (void );/* overridable mmc pinmux setup */ +void pin_mux_display (void );/* overridable DISPLAY pinmux setup */ #endif
Signature · 1 line
2.1.0
PATCH 48/49 tegra: add proto for pin_mux_mmc
Quoted 3 lines · Jeroen Hofstee
while at it, fix a typo
Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/cf8599b6/attachment-0001.pgp>
PATCH 49/49 i2c: designware: add an implement i2c protos
In reply to Jeroen HofsteeInclude the i2c header and change the non confirming functions to do so.
--- drivers/i2c/designware_i2c.c | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-)
drivers/i2c/designware_i2c.c +11 -6
@@ -8,6 +8,7 @@#include <common.h> #include <asm/io.h> #include "designware_i2c.h" +#include <i2c.h> #ifdef CONFIG_I2C_MULTI_BUS static unsigned int bus_initialized[CONFIG_SYS_I2C_BUS_MAX ];
@@ -76,16 +77,20 @@ static void set_speed(int i2c_spd)* * Set the i2c speed. */ -int i2c_set_bus_speed (int speed) +int i2c_set_bus_speed (unsigned int speed) { +int i2c_spd; +if (speed >=I2C_MAX_SPEED ) -set_speed (IC_SPEED_MODE_MAX ); + i2c_spd =IC_SPEED_MODE_MAX ;else if (speed >=I2C_FAST_SPEED ) -set_speed (IC_SPEED_MODE_FAST ); + i2c_spd =IC_SPEED_MODE_FAST ;else -set_speed (IC_SPEED_MODE_STANDARD ); + i2c_spd =IC_SPEED_MODE_STANDARD ; -return 0 ; +set_speed (i2c_spd); + +return i2c_spd; }/*
@@ -93,7 +98,7 @@ int i2c_set_bus_speed(int speed)* * Gets the i2c speed. */ -int i2c_get_bus_speed (void ) +unsigned int i2c_get_bus_speed (void ) {u32 cntl;
Signature · 1 line
2.1.0
PATCH 49/49 i2c: designware: add an implement i2c protos
Hello Jeroen,
Am 08.10.2014 22:58, schrieb Jeroen Hofstee:
Quoted 8 lines
Include the i2c header and change the non confirming functions to do so.
Cc Heiko Schocher<[email protected]>Signed-off-by Jeroen Hofstee<[email protected]>--- drivers/i2c/designware_i2c.c | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-)
(It would be nice to see this driver also moved to the
CONFIG_SYS_I2C framework ... seeing a "CONFIG_I2C_MULTI_BUS" define in it is very bad ...)
bye,
Heiko
Signature · 2 lines
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
PATCH 49/49 i2c: designware: add an implement i2c protos
Hello Heiko,
Quoted 12 lines · Heiko Schocher
Hello Jeroen,
Am 08.10.2014 22:58, schrieb Jeroen Hofstee:
Quoted 8 lines
Include the i2c header and change the non confirming functions to do so.
Cc Heiko Schocher<[email protected]>Signed-off-by Jeroen Hofstee<[email protected]>--- drivers/i2c/designware_i2c.c | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-)
Quoted 6 lines
Acked-by Heiko Schocher <[email protected]>(It would be nice to see this driver also moved to the
CONFIG_SYS_I2Cframework ... seeing a "CONFIG_I2C_MULTI_BUS" define in it is very bad ...)
Likely, but I happily leave that to someone who can actually test the code and has a better understanding of the (new?) i2c model ;)
Regards,
Jeroen
PATCH 49/49 i2c: designware: add an implement i2c protos
In reply to Jeroen HofsteeQuoted 6 lines · Jeroen Hofstee
Include the i2c header and change the non confirming functions to do so.
Cc Heiko Schocher <[email protected]>Signed-off-by Jeroen Hofstee <[email protected]>Acked-by Heiko Schocher <[email protected]>
After also fixing the i2c_get_bus_num prototype, applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141026/93f7d7f1/attachment.pgp>
v2 cover reduce warnings with W=1
In reply to Jeroen HofsteeWhen compiling u-boot with W=1 there are numerous of warnings. This patchset attempts to reduce it a bit. One source of false warnings are the aliases missing a prototype, most of them are replaced with __weak functions. Others are cause by missing includes / local functions not be marked as such. At last some actual missing prototypes are added. These are (hopefully) just trivial patches, some less than trial is ahead of us.
v2:
- rebase on master (most of this serie is applied, so remaining patches are numbered differently.
- drop "PATCH 25/49 spi: make local functions static". Tom posted a more complete cleanup.
- Drop common/cmd_elf.c: add missing include (a slightly adjusted version was applied).
- split i2c: use __weak (included i2c and l2_cache)
- redo serial: add prototypes for init functions
Jeroen Hofstee (3):
i2c: use __weak
arm926ejs: cache: use __weak
serial: add prototypes for init functions
arch/arm/cpu/arm926ejs/cache.c | 5 +- drivers/i2c/i2c_core.c | 4 +- drivers/serial/serial.c | 140 ++++++++++++++++++++--------------------- include/serial.h | 49 +++++++++++++++ 4 files changed, 121 insertions(+), 77 deletions(-)
Signature · 1 line
2.1.0
PATCH v2 1/3 i2c: use __weak
--- drivers/i2c/i2c_core.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
drivers/i2c/i2c_core.c +1 -3
@@ -395,9 +395,7 @@ void i2c_reg_write(uint8_t addr, uint8_t reg, uint8_t val)i2c_write (addr, reg,1 , &val,1 ); } -void __i2c_init (int speed,int slaveaddr) +__weak void i2c_init (int speed,int slaveaddr) {i2c_init_bus (i2c_get_bus_num (), speed, slaveaddr); } -void i2c_init (int speed,int slaveaddr) - __attribute__((weak,alias ("__i2c_init" )));
Signature · 1 line
2.1.0
PATCH v2 2/3 arm926ejs: cache: use __weak
In reply to Jeroen Hofstee--- arch/arm/cpu/arm926ejs/cache.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-)
arch/arm/cpu/arm926ejs/cache.c +1 -4
@@ -99,7 +99,4 @@ void flush_cache(unsigned long start, unsigned long size)/* * Stub implementations for l2 cache operations */ -void __l2_cache_disable (void ) {} - -void l2_cache_disable (void ) - __attribute__((weak,alias ("__l2_cache_disable" ))); +__weak void l2_cache_disable (void ) {}
Signature · 1 line
2.1.0
PATCH v2 2/3 arm926ejs: cache: use __weak
Quoted 2 lines · Jeroen Hofstee
Cc Albert Aribaud <[email protected]>Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141104/cfee8991/attachment.pgp>
PATCH v2 3/3 serial: add prototypes for init functions
In reply to Jeroen HofsteeWhile at it, sort them.
--- drivers/serial/serial.c | 140 ++++++++++++++++++++++++------------------------ include/serial.h | 49 +++++++++++++++++ 2 files changed, 119 insertions(+), 70 deletions(-)
drivers/serial/serial.c +70 -70
@@ -109,54 +109,54 @@ U_BOOT_ENV_CALLBACK(baudrate, on_baudrate);void name (void ) \ __attribute__((weak,alias ("serial_null" ))); -serial_initfunc (mpc8xx_serial_initialize); -serial_initfunc (ns16550_serial_initialize); -serial_initfunc (pxa_serial_initialize); -serial_initfunc (s3c24xx_serial_initialize); -serial_initfunc (s5p_serial_initialize); -serial_initfunc (zynq_serial_initialize); -serial_initfunc (bfin_serial_initialize); -serial_initfunc (bfin_jtag_initialize); -serial_initfunc (mpc512x_serial_initialize); -serial_initfunc (uartlite_serial_initialize); -serial_initfunc (au1x00_serial_initialize); -serial_initfunc (asc_serial_initialize); -serial_initfunc (jz_serial_initialize); -serial_initfunc (mpc5xx_serial_initialize); -serial_initfunc (mpc8260_scc_serial_initialize); -serial_initfunc (mpc8260_smc_serial_initialize); -serial_initfunc (mpc85xx_serial_initialize); -serial_initfunc (iop480_serial_initialize); -serial_initfunc (leon2_serial_initialize); -serial_initfunc (leon3_serial_initialize); -serial_initfunc (marvell_serial_initialize); +serial_initfunc (altera_jtag_serial_initialize); +serial_initfunc (altera_serial_initialize);serial_initfunc (amirix_serial_initialize); +serial_initfunc (arc_serial_initialize); +serial_initfunc (arm_dcc_initialize); +serial_initfunc (asc_serial_initialize); +serial_initfunc (atmel_serial_initialize); +serial_initfunc (au1x00_serial_initialize); +serial_initfunc (bfin_jtag_initialize); +serial_initfunc (bfin_serial_initialize);serial_initfunc (bmw_serial_initialize); +serial_initfunc (clps7111_serial_initialize);serial_initfunc (cogent_serial_initialize);serial_initfunc (cpci750_serial_initialize);serial_initfunc (evb64260_serial_initialize); -serial_initfunc (ml2_serial_initialize); -serial_initfunc (sconsole_serial_initialize); -serial_initfunc (p3mx_serial_initialize); -serial_initfunc (altera_jtag_serial_initialize); -serial_initfunc (altera_serial_initialize); -serial_initfunc (atmel_serial_initialize); -serial_initfunc (lpc32xx_serial_initialize); -serial_initfunc (mcf_serial_initialize); -serial_initfunc (oc_serial_initialize); -serial_initfunc (sandbox_serial_initialize); -serial_initfunc (clps7111_serial_initialize);serial_initfunc (imx_serial_initialize); +serial_initfunc (iop480_serial_initialize); +serial_initfunc (jz_serial_initialize);serial_initfunc (ks8695_serial_initialize); +serial_initfunc (leon2_serial_initialize); +serial_initfunc (leon3_serial_initialize);serial_initfunc (lh7a40x_serial_initialize); +serial_initfunc (lpc32xx_serial_initialize); +serial_initfunc (marvell_serial_initialize);serial_initfunc (max3100_serial_initialize); +serial_initfunc (mcf_serial_initialize); +serial_initfunc (ml2_serial_initialize); +serial_initfunc (mpc512x_serial_initialize); +serial_initfunc (mpc5xx_serial_initialize); +serial_initfunc (mpc8260_scc_serial_initialize); +serial_initfunc (mpc8260_smc_serial_initialize); +serial_initfunc (mpc85xx_serial_initialize); +serial_initfunc (mpc8xx_serial_initialize);serial_initfunc (mxc_serial_initialize); +serial_initfunc (mxs_auart_initialize); +serial_initfunc (ns16550_serial_initialize); +serial_initfunc (oc_serial_initialize); +serial_initfunc (p3mx_serial_initialize);serial_initfunc (pl01x_serial_initialize); +serial_initfunc (pxa_serial_initialize); +serial_initfunc (s3c24xx_serial_initialize); +serial_initfunc (s5p_serial_initialize);serial_initfunc (sa1100_serial_initialize); +serial_initfunc (sandbox_serial_initialize); +serial_initfunc (sconsole_serial_initialize);serial_initfunc (sh_serial_initialize); -serial_initfunc (arm_dcc_initialize); -serial_initfunc (mxs_auart_initialize); -serial_initfunc (arc_serial_initialize); +serial_initfunc (uartlite_serial_initialize); +serial_initfunc (zynq_serial_initialize);/** * serial_register() - Register serial driver with serial driver core
@@ -202,54 +202,54 @@ void serial_register(struct serial_device *dev)*/ void serial_initialize (void ) { -mpc8xx_serial_initialize (); -ns16550_serial_initialize (); -pxa_serial_initialize (); -s3c24xx_serial_initialize (); -s5p_serial_initialize (); -mpc512x_serial_initialize (); -bfin_serial_initialize (); -bfin_jtag_initialize (); -uartlite_serial_initialize (); -zynq_serial_initialize (); -au1x00_serial_initialize (); -asc_serial_initialize (); -jz_serial_initialize (); -mpc5xx_serial_initialize (); -mpc8260_scc_serial_initialize (); -mpc8260_smc_serial_initialize (); -mpc85xx_serial_initialize (); -iop480_serial_initialize (); -leon2_serial_initialize (); -leon3_serial_initialize (); -marvell_serial_initialize (); +altera_jtag_serial_initialize (); +altera_serial_initialize ();amirix_serial_initialize (); +arc_serial_initialize (); +arm_dcc_initialize (); +asc_serial_initialize (); +atmel_serial_initialize (); +au1x00_serial_initialize (); +bfin_jtag_initialize (); +bfin_serial_initialize ();bmw_serial_initialize (); +clps7111_serial_initialize ();cogent_serial_initialize ();cpci750_serial_initialize ();evb64260_serial_initialize (); -ml2_serial_initialize (); -sconsole_serial_initialize (); -p3mx_serial_initialize (); -altera_jtag_serial_initialize (); -altera_serial_initialize (); -atmel_serial_initialize (); -lpc32xx_serial_initialize (); -mcf_serial_initialize (); -oc_serial_initialize (); -sandbox_serial_initialize (); -clps7111_serial_initialize ();imx_serial_initialize (); +iop480_serial_initialize (); +jz_serial_initialize ();ks8695_serial_initialize (); +leon2_serial_initialize (); +leon3_serial_initialize ();lh7a40x_serial_initialize (); +lpc32xx_serial_initialize (); +marvell_serial_initialize ();max3100_serial_initialize (); +mcf_serial_initialize (); +ml2_serial_initialize (); +mpc512x_serial_initialize (); +mpc5xx_serial_initialize (); +mpc8260_scc_serial_initialize (); +mpc8260_smc_serial_initialize (); +mpc85xx_serial_initialize (); +mpc8xx_serial_initialize ();mxc_serial_initialize (); +mxs_auart_initialize (); +ns16550_serial_initialize (); +oc_serial_initialize (); +p3mx_serial_initialize ();pl01x_serial_initialize (); +pxa_serial_initialize (); +s3c24xx_serial_initialize (); +s5p_serial_initialize ();sa1100_serial_initialize (); +sandbox_serial_initialize (); +sconsole_serial_initialize ();sh_serial_initialize (); -arm_dcc_initialize (); -mxs_auart_initialize (); -arc_serial_initialize (); +uartlite_serial_initialize (); +zynq_serial_initialize ();serial_assign (default_serial_console ()->name); }
include/serial.h +49 -0
53 lines not on this page · raw message
Signature · 1 line
2.1.0
PATCH v2 3/3 serial: add prototypes for init functions
Quoted 5 lines · Jeroen Hofstee
While at it, sort them.
Cc Tom Rini <[email protected]>Cc Simon Glass <[email protected]>Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141104/5e3c1a5a/attachment.pgp>
v3 cover reduce warnings with W=1
In reply to Jeroen HofsteeWhen compiling u-boot with W=1 there are numerous of warnings. This patchset attempts to reduce it a bit. One source of false warnings are the aliases missing a prototype, most of them are replaced with __weak functions. Others are cause by missing includes / local functions not be marked as such. At last some actual missing prototypes are added. These are (hopefully) just trivial patches, some less than trial is ahead of us.
v3:
- i2c did contain another alias function, i2c_init_board. This time the correct patch is squashed with it. Only this patch resend (1/3).
v2:
- rebase on master (most of this serie is applied, so remaining patches are numbered differently.
- drop "PATCH 25/49 spi: make local functions static". Tom posted a more complete cleanup.
- Drop common/cmd_elf.c: add missing include (a slightly adjusted version was applied).
- split i2c: use __weak (included i2c and l2_cache)
- redo serial: add prototypes for init functions
Jeroen Hofstee (3):
i2c: use __weak
arm926ejs: cache: use __weak
serial: add prototypes for init functions
arch/arm/cpu/arm926ejs/cache.c | 5 +- drivers/i2c/i2c_core.c | 8 +-- drivers/serial/serial.c | 140 ++++++++++++++++++++--------------------- include/serial.h | 49 +++++++++++++++ 4 files changed, 122 insertions(+), 80 deletions(-)
Signature · 1 line
2.1.0
PATCH v3 1/3 i2c: use __weak
--- drivers/i2c/i2c_core.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-)
drivers/i2c/i2c_core.c +2 -6
@@ -229,11 +229,9 @@ static void i2c_init_bus(unsigned int bus_no, int speed, int slaveaddr) }/* implement possible board specific board init */ -static void __def_i2c_init_board (void ) +__weak void i2c_init_board (void ) { } -void i2c_init_board (void ) - __attribute__((weak,alias ("__def_i2c_init_board" )));/* * i2c_init_all():
@@ -395,9 +393,7 @@ void i2c_reg_write(uint8_t addr, uint8_t reg, uint8_t val)i2c_write (addr, reg,1 , &val,1 ); } -void __i2c_init (int speed,int slaveaddr) +__weak void i2c_init (int speed,int slaveaddr) {i2c_init_bus (i2c_get_bus_num (), speed, slaveaddr); } -void i2c_init (int speed,int slaveaddr) - __attribute__((weak,alias ("__i2c_init" )));
Signature · 1 line
2.1.0
PATCH v3 1/3 i2c: use __weak
Quoted 2 lines · Jeroen Hofstee
Cc Heiko Schocher <[email protected]>Signed-off-by Jeroen Hofstee <[email protected]>
Applied to u-boot/master, thanks!
Signature · 1 line
Tom