Dear Roy, dear Kyle, you have been modifyong the E1000 driver lately so I hope you are in the best position to help and fix a number of build warnings.
For example when building for the MVBC_P board, I get this: e1000.c: In function 'e1000_read_mac_addr': e1000.c:1149:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] e1000.c:1149:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] e1000.c: In function 'e1000_reset_hw': e1000.c:1373:11: warning: variable 'icr' set but not used [-Wunused-but-set-variable] e1000.c: In function 'e1000_phy_init_script': e1000.c:4395:11: warning: variable 'ret_val' set but not used [-Wunused-but-set-variable] I'm especially concerned about the last warning (the others should be straightforward to fix). The code reads: ... 4402 /* Save off the current value of register 0x2F5B to be 4403 * restored at the end of this routine. */ 4404 ret_val = e1000_read_phy_reg(hw, 0x2F5B, &phy_saved_data); ... However, no such restore gets ever done. AFAICT this is your code, Roy. Please check and fix. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de One possible reason that things aren't going according to plan is that there never was a plan in the first place. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot