Re: converted ral(4) Was: [Testers needed!] WiFi drivers changes

On Fri, Jun 05, 2015 at 11:16:06AM +0800, Kevin Lo wrote:
> 
> On Thu, Jun 04, 2015 at 02:18:08PM +0300, Gleb Smirnoff wrote:
> > 
> >   Hi Kevin and Olivier,
> 
> Hi Gleb,
> 
> > 
> >   you signed up as testers of ral(4). I have converted ral(4)
> > and uploaded new patch at:
> > 
> > https://reviews.freebsd.org/D2655
> > 
> > Please try, report and update the project page.
> > 
> > https://wiki.freebsd.org/projects/ifnet/net80211
> > 
> > Thanks a lot for your help.
> 
> It works fine for me.  Tested on RT5390, thanks.

As I mentioned in a private message, the diff below changes ifp->if_drv_flags
to sc->sc_flags, thanks.

--- D2655.diff.orig	2015-06-08 10:20:12.220844488 +0800
+++ D2655.diff	2015-06-08 10:20:38.020180596 +0800
_at__at_ -6279,7 +6279,7 _at__at_
  	RAL_LOCK_ASSERT(sc);
  
 -	KASSERT(ifp->if_drv_flags & IFF_DRV_RUNNING, ("not running"));
-+	KASSERT(ifp->if_drv_flags & RAL_RUNNING, ("not running"));
++	KASSERT(sc->sc_flags & RAL_RUNNING, ("not running"));
  
  	if (sc->sc_invalid)		/* card ejected */
  		return;
_at__at_ -7109,7 +7109,7 _at__at_
  	RAL_LOCK_ASSERT(sc);
  
 -	KASSERT(ifp->if_drv_flags & IFF_DRV_RUNNING, ("not running"));
-+	KASSERT(ifp->if_drv_flags & RT2860_RUNNNING, ("not running"));
++	KASSERT(sc->sc_flags & RT2860_RUNNNING, ("not running"));
  
  	if (sc->sc_invalid)		/* card ejected */
  		return;