Date: Fri, 4 May 2012 11:31:48 -0400 From: Patrick Kelsey <kelsey@ieee.org> To: Adrian Chadd <adrian@freebsd.org> Cc: freebsd-mips@freebsd.org Subject: Re: heads up - ar71xx changes Message-ID: <CAD44qMUubHchQJbjpvJdSutbSQZdCCjiKXV9a8hqPx1p0Vi=8A@mail.gmail.com> In-Reply-To: <CAJ-VmonZbXe3yUaBn3-giaz7aiMYsk3D4u%2BghbZUZkLWw595mQ@mail.gmail.com> References: <CAJ-VmonZbXe3yUaBn3-giaz7aiMYsk3D4u%2BghbZUZkLWw595mQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, May 2, 2012 at 5:37 PM, Adrian Chadd <adrian@freebsd.org> wrote: > Hi, > > there's been a few ar71xx changes in the last few days. I'd appreciate > it if it got decent testing. > Hi Adrian, I've finally gotten back to doing a bit of testing with an RB450G (AR71XX + AR8316), and I found that I needed to use the following diff to get arg1 to initialize properly on that platform: Index: sys/mips/atheros/if_arge.c =================================================================== --- sys/mips/atheros/if_arge.c (revision 234999) +++ sys/mips/atheros/if_arge.c (working copy) @@ -600,7 +605,8 @@ &sc->arge_miibus, sc->arge_ifp, arge_ifmedia_upd, arge_ifmedia_sts, BMSR_DEFCAPMASK, i, MII_OFFSET_ANY, 0); - if (error != 0) { + if ((error != 0) && + (sc->arge_miibus != NULL)) { device_printf(sc->arge_dev, "unable to attach" " PHY %d: %d\n", i, error); goto fail; Without the above diff, things wind up at goto fail on arge1 (which has no miibus and a phymask of 0xf) instead of in the multiphy logic that follows. -Patrick
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAD44qMUubHchQJbjpvJdSutbSQZdCCjiKXV9a8hqPx1p0Vi=8A>