[U-Boot] [PATCH V2] Allow PHY addresses on kirkwood egiga to be non continuous.

Tor Krill tor at excito.com
Mon Jun 28 11:20:32 CEST 2010


On Thu, 2010-06-24 at 18:37 +0200, Mike Frysinger wrote:
> On Thursday, June 24, 2010 05:31:47 Tor Krill wrote:
> >  void reset_phy(void)
> >  {
> >  	char *name = "egiga0";
> > +	u8 port_addr[] = CONFIG_PHY_ADDRS;
> > 
> >  	if (miiphy_set_current_dev(name))
> >  		return;
> > 
> >  	/* reset the phy */
> > -	miiphy_reset(name, CONFIG_PHY_BASE_ADR);
> > +	miiphy_reset(name, port_addr[0]);
> >  }
> 
> hmm, this cant possibly be correct.  how do you know which of the phys is to 
> be reset ?  but i guess that's just a bug in the ARM tree you cant do much 
> about without fixing everyone.

In the original the base adress was used to adress the PHY being reset
and as long as you don't enter the addresses in reverse in the array the
above construction should work. (And since this is for a specific board,
only using one PHY(?), it should be quite isolated.)

/Tor



More information about the U-Boot mailing list