[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