[U-Boot-Users] [RFC][PATCH 1/1] Add board_eth_init() function
    Haavard Skinnemoen 
    haavard.skinnemoen at atmel.com
       
    Wed Mar 26 11:00:12 CET 2008
    
    
  
On Sun, 23 Mar 2008 15:19:55 +0900
Shinya Kuribayashi <skuribay at ruby.dti.ne.jp> wrote:
>  int board_eth_init(bd_t *bis)
>  {
> +	extern int tsec_initialize(bd_t * bis, int index, char *devname);
Eww...no, please don't do that. Function prototypes belong in header
files. By adding extern declarations all over the place, the compiler
won't notice if the caller and callee get out of sync.
Having a header file full of driver-specific declarations does increase
the chances of conflicts, but I think it's worth it. It's definitely an
improvement over the old code. And the conflicts will be absolutely
trivial.
Haavard
    
    
More information about the U-Boot
mailing list