On 1/18/2010 12:49 PM, Ben Warren wrote: >>>> >>>> if (chip->cpufreq == -1) >>>> @@ -156,6 +215,13 @@ int do_chip_config(cmd_tbl_t *cmdtp, int flag, >>>> int argc, char *argv[]) >>>> else >>>> printf("DDR Type = Not Known\n"); >>>> >>>> + if (!i2c_read_mac(mac)) { >>>> + sprintf(i2c_mac, "%pM", mac); >>>> + printf("Ethaddr (from i2c mem) = %s\n", i2c_mac); >>>> >>> Maybe I'm missing some context, but why's the sprintf needed? >>> >> >> Yes, sprintf is not needed here >> Thanks for pointing out >>
Hello Tom, Should I send the whole patch set again or only the subpatch after removing the sprintf Regards Vipin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot