Vipin KUMAR wrote: > 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
Let us wait for feed back from nand and usb subsystem so we can a accumulate them in just one patch set. Tom > > Regards > Vipin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot