On Wed, Feb 06, 2013 at 08:35:40PM +0100, Sascha Hauer wrote: > On Tue, Feb 05, 2013 at 11:15:04AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote: > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@xxxxxxxxxxxx> > > --- > > drivers/net/macb.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/net/macb.c b/drivers/net/macb.c > > index 15b1e93..8eac4a8 100644 > > --- a/drivers/net/macb.c > > +++ b/drivers/net/macb.c > > @@ -476,8 +476,8 @@ static int macb_probe(struct device_d *dev) > > > > macb_writel(macb, NCFGR, ncfgr); > > > > - mdiobus_register(&macb->miibus); > > eth_register(edev); > > + mdiobus_register(&macb->miibus); > > Why? > > Most other drivers do it the other way round and you do not provide > information why this is done. Applied the other patches. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox