Skip to contentSkip to conversation

ARM breakage

U-Boot Tom 3 participants 3 messages Jan 18, 2010

lore
Share & tools
lore

Tom

author
#0 lore raw

After merging with u-boot/master
Multiple targets are fixed by getenv_IPaddr

However, i believe this may have broken an equal number of arm targets On regression testing there are multiple failures with this error

smc91111_eeprom.c:33: error: redefinition of 'struct eth_device' make[1]: *** [ .. build/examples/standalone/smc91111_eeprom.o] Error 1

Sample of targets
omap3_evm
versatile
cerg250

#1 lore raw
Quoted 10 lines · Tom

After merging with u-boot/master
Multiple targets are fixed by getenv_IPaddr

However, i believe this may have broken an equal number of arm targets On regression testing there are multiple failures with this error

smc91111_eeprom.c:33: error: redefinition of 'struct eth_device' make[1]: *** [ .. build/examples/standalone/smc91111_eeprom.o] Error 1

Hmmm... The getenv_IPaddr patch added '"#include net.h"', which brings this new error. I'll come up with a workaround.

Quoted 11 lines

Sample of targets
omap3_evm
versatile
cerg250

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

regards,
Ben

#2 lore raw
Quoted 7 lines · Ben Warren
Quoted 6 lines · Tom

After merging with u-boot/master
Multiple targets are fixed by getenv_IPaddr

However, i believe this may have broken an equal number of arm targets On regression testing there are multiple failures with this error

Quoted 7 lines
Quoted 4 lines

smc91111_eeprom.c:33: error: redefinition of 'struct eth_device' make[1]: *** [ .. build/examples/standalone/smc91111_eeprom.o] Error 1

Hmmm... The getenv_IPaddr patch added '"#include net.h"', which brings this new error. I'll come up with a workaround.

Does replacing

#include <net.h>

with just something like

extern IPaddr_t string_to_ip(char *s);

help?

Quoted 2 lines
Quoted 2 lines

Sample of targets
omap3_evm

Hmm, I could bet that I tested at least with ./MAKEALL ARM_CORTEX_A8 (?).

Best regards

Dirk