On Tue, Sep 13, 2011 at 10:14:44PM +0200, Wolfgang Denk wrote: > Dear Zhao Chenhui, > > In message <1315898131-27710-2-git-send-email-chenhui.z...@freescale.com> you > wrote: > ... > > printf("PCI1: %d bit, %s MHz, %s\n", > > (pci1_32) ? 32 : 64, > > - (pci1_speed == 33000000) ? "33" : > > - (pci1_speed == 66000000) ? "66" : "unknown", > > + (pci1_speed == 33333333) ? "33" : > > + (pci1_speed == 66666666) ? "66" : "unknown", > > pci1_clk_sel ? "sync" : "async"); > > Why cannot you simply write: > > ... > char buf[32]; > ... > printf("PCI1: %d bit, %s MHz, %ssync\n", > (pci1_32) ? 32 : 64, > strmhz(buf, pci1_speed), > pci1_clk_sel ? "" : "a"); > > ? > > > Best regards, > > Wolfgang Denk >
Yes. I'll repost it. -chenhui > -- > 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 > The only time the world beats a path to your door is when you are in > the bathroom. > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot