On Friday, September 09, 2011 02:48:46 AM Mike Frysinger wrote:
> On Thursday, September 08, 2011 16:37:16 Marek Vasut wrote:
> > --- a/drivers/net/fec_mxc.c
> > +++ b/drivers/net/fec_mxc.c
> > @@ -38,6 +38,10 @@ DECLARE_GLOBAL_DATA_PTR;
> > 
> >  #error "CONFIG_MII has to be defined!"
> >  #endif
> > 
> > +#ifndef    CONFIG_FEC_XCV_TYPE
> > +#define    CONFIG_FEC_XCV_TYPE     MII100
> > +#endif
> > +
> > 
> >  #undef DEBUG
> >  
> >  struct nbuf {
> > 
> > @@ -711,7 +715,7 @@ static int fec_probe(bd_t *bd)
> > 
> >     fec->eth = (struct ethernet_regs *)IMX_FEC_BASE;
> >     fec->bd = bd;
> > 
> > -   fec->xcv_type = MII100;
> > +   fec->xcv_type = CONFIG_FEC_XCV_TYPE;
> 
> shouldnt this be a parameter to fec_probe() ?
> -mike

We plan to add a driver model eventually. Until then, let's not polute this 
more 
than necessary.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to