On Wed, 2010-03-17 at 17:20 +0100, ext Johannes Berg wrote:
> On Wed, 2010-03-17 at 15:01 +0200, Juuso Oikarinen wrote:
>
> > +void ieee80211_cqm_notify(struct ieee80211_vif *vif,
> > + enum nl80211_cqm_state rssi_state)
> > +{
> > + struct ieee80211_sub_if_data *sdata = vif_to_sdata(vif);
> > +
> > + cfg80211_cqm_notify(sdata->dev, rssi_state, GFP_KERNEL);
> > +}
> > +EXPORT_SYMBOL(ieee80211_cqm_notify);
>
> How do you know GFP_KERNEL is OK? The driver might call this from an
> interrupt? Seems better to me to ask the driver for the allocation flags
> too.
Good point, I'll change this.
-Juuso
> johannes
>
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
