On Wednesday, August 31, 2011 09:26:27 PM Mike Frysinger wrote:
> On Tuesday, August 30, 2011 08:05:18 Michal Simek wrote:
> > +   volatile struct temac_reg *regs = priv->regs;
> > +
> > +   regs->lsw = (phy_addr << 5) | reg_addr;
> > +   regs->ctl = MIIMAI | (emac << 10);
> 
> you should still be going through read/write i/o helper funcs rather than
> relying on the volatile markings

Oh, I didn't comment on the TEMAC. I'll have to go through it.

Cheers

> 
> the rest looks sane
> -mike
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to