[PATCH] ppc32: Support for Freescale PQ2FADS

Wade Farnsworth wfarnsworth at mvista.com
Fri Apr 7 02:44:44 EST 2006


On Thu, 2006-04-06 at 09:06, Wade Farnsworth wrote:
> On Thu, 2006-04-06 at 07:05, Vitaly Bordug wrote:
> > 
> > Though I doubt if new boards are accepting to ppc/ currently, it's nice to have a patch 
> > available. BTW, I suggest to use phy abstraction and updated fs_enet (submitted here recently)
<snip>
> > > +CONFIG_USB_ARCH_HAS_HCD=y
> > > +CONFIG_USB_ARCH_HAS_OHCI=y
> > 
> > Not sure this is required as well.
> 
> I'll disable this as well.

Looking at drivers/usb/Kconfig, these are actually enabled by default if
CONFIG_PCI is enabled.  CONFIG_USB then depends on
CONFIG_USB_ARCH_HAS_HCD.

> 
> <snip>
> 
> > > +#ifdef CONFIG_PQ2FADS
> > > +#define F2_RXCLK	13
> > > +#define F2_TXCLK	14
> > > +#else
> > >  #define F2_RXCLK	15
> > >  #define F2_TXCLK	16
> > > +#endif
> > >  
> > Hmm, this should go into the board-specific header - here should exist just a "reasonable default" (pq2ads.h in this context). 
> > 
> > 
> > >  /* FCC3 Clock Source Configuration.  These can be redefined in the board specific file.
> > >     Can only choose from CLK13-16 */
> > > +#ifdef CONFIG_PQ2FADS
> > > +#define F3_RXCLK	15
> > > +#define F3_TXCLK	16
> > > +#else
> > >  #define F3_RXCLK	13
> > >  #define F3_TXCLK	14
> > > +#endif
> > >  
> > ditto.
> 
> Ok, I'll move these defines into pq2ads.h
> 

Also, moving the defines into pq2ads.h results in some nasty warning
messages.  I'll put some #ifndef's around the #define's in pq2ads_pd.h
to squelch them.

-Wade




More information about the Linuxppc-embedded mailing list