Re: finished run driver for CURRENT

----- Original Message ----
> Subject: Re: finished run driver for CURRENT
> 
> On Sunday 08 November 2009 11:22:50 PseudoCylon wrote:
> > Hello everyone,
> >
> > I finished porting run driver to CURRENT (sort of). I posted it at freebsd
> > fourums
> > http://forums.freebsd.org/showpost.php?s=a3756a43cb6eca54dea97673ac8424e7&p
> >=47936&postcount=28 Anyone interested in please try it out.
> 
> Just some comments:
> 
> There is an ep_index field you can set, instead of specifying exactly which 
> endpoint to use.
> 
> Eg.
> 
> UE_BULK_IN
> UE_ADDR_ANY
> 
> ep_index = 0, /* will match first bulk_in + addr_any */
> 
> 
> UE_BULK_OUT
> UE_ADDR_ANY
> 
> ep_index = 2, /* will match third bulk_out + addr_any */
> 
> I'm not sure if the manufacturer will change those values?
> 
> Else, do you have any other feedback on the new USB stack? Was it difficult?
> 
> --HPS

> 
> Compile fails at line 2338 of if_run.c.
> 
> /home/tsgan/run-CUR/sys/modules/usb/run/../../../dev/usb/wlan/if_run.c:
> In function 'run_tx':
> /home/tsgan/run-CUR/sys/modules/usb/run/../../../dev/usb/wlan/if_run.c:2338:
> warning: comparison between pointer and integer
> *** Error code 1
> 
> Stop in /home/tsgan/run-CUR/sys/modules/usb/run.
> 
> I guess it should either:
> 
>         struct ieee80211_channel *chan;
>         chan = ni->ni_chan == IEEE80211_CHAN_ANYC ? ic->ic_curchan :
> ni->ni_chan;
> 
> or:
> 
>         struct ieee80211_channel *chan;
>         chan = ni->ni_chan == (ieee80211_channel *)IEEE80211_CHAN_ANY ?
> ic->ic_curchan : ni->ni_chan;
> 

Thank you to everyone testing the driver. I've implemented all suggestions.


      __________________________________________________________________
Get a sneak peak at messages with a handy reading pane with All new Yahoo! Mail: http://ca.promos.yahoo.com/newmail/overview2/