
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.zhao@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_clk_sel ? "sync" : "async");(pci1_speed == 66666666) ? "66" : "unknown",
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: wd@denx.de The only time the world beats a path to your door is when you are in the bathroom.