[U-Boot] [PATCH 07/18] net: macb: Convert to driver model

[Joe Hershberger]

On Tue, May 3, 2016 at 1:40 AM, Simon Glass <sjg at chromium.org> wrote:
> Add driver-model support to this driver. The old code remains for now so
> that we can convert boards one at a time.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>

Acked-by: Joe Hershberger <joe.hershberger at ni.com>

But with a nit...

> ---

<snip>

> +static const struct eth_ops macb_eth_ops = {
> +       .start  = macb_start,
> +       .send   = macb_send,
> +       .recv   = macb_recv,
> +       .stop   = macb_stop,
> +       .free_pkt       = macb_free_pkt,
> +       .write_hwaddr   = macb_write_hwaddr,
> +};
> +
> +static int macb_eth_probe(struct udevice *dev)
> +{
> +       struct eth_pdata *pdata = dev_get_platdata(dev);
> +       struct macb_device *macb = dev_get_priv(dev);
> +
> +       macb->regs = (void *)pdata->iobase;
> +
> +       _macb_eth_initialize(macb);
> +#if defined(CONFIG_CMD_MII) || defined(CONFIG_PHYLIB)
> +       miiphy_register(dev->name, macb_miiphy_read, macb_miiphy_write);

It's unfortunate that you're proliferating the oldest phy API. My
semantic patch can come along and clean it up later as long as this is
in before I run it again.

> +       macb->bus = miiphy_get_dev_by_name(dev->name);
> +#endif
> +
> +       return 0;
> +}