[U-Boot] [PATCH v2] net: ll_temac: Add LL TEMAC driver to u-boot

Marek Vasut marek.vasut at gmail.com
Wed Aug 31 22:01:59 CEST 2011


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


More information about the U-Boot mailing list