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

Reply via email to