
25 Mar
2010
25 Mar
'10
7:39 a.m.
Hello Frans,
Frans Meulenbroeks wrote:
While grepping some sources I bumped upon i2c_mux_add_device This one is implemented in cmd_i2c.c and exported through i2c.h. However no-one outside cmd_i2c.c is using it:
frans@linux-suse:~/u-boot> grep -r i2c_mux_add_device . ./common/cmd_i2c.c:int i2c_mux_add_device(I2C_MUX_DEVICE *dev) ./common/cmd_i2c.c: i2c_mux_add_device (device); ./include/i2c.h:int i2c_mux_add_device(I2C_MUX_DEVICE *dev); frans@linux-suse:~/u-boot>
Should we make this function static and remove from the .h file?
Yep, good catch. Please post a patch.
bye Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany