[U-Boot] [PATCH 2/9] powerpc/mpc85xxcds: Fix PCI speed

Zhao Chenhui b35336 at freescale.com
Wed Sep 14 12:34:19 CEST 2011


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 at 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: wd at denx.de
> The only time the world beats a path to your door is when you are  in
> the bathroom.
> 



More information about the U-Boot mailing list