On Tue, 2009-11-10 at 17:23 +0100, Holger Schurig wrote: > This patch implements the NL80211_CMD_GET_SURVEY command and an get_survey() > ops that a driver can implement. The goal of this command is to allow a > driver to report back channel survey data (e.g. channel noise, channel > occupation) back to cfg80211 and thus via nl80211 to user-space. > > In future, they will either be a survey-trigger command --- or the existing > scan-trigger command will be enhanced. However, the get_survey() operation > is usable even in absence of this: a driver can report the channel noise in > mBm with the implemented mechanism. > > get_survey() is currently modelled like get_key(). I hope that's right. I don't think this last sentence is true any more? > struct survey_info is modelled like struct station_info. This allows different > drivers to fill in different fields. Guess the details don't really belong into the commit log anyway. Anyway, apart from the two things we talked about on IRC (error codes and missing "cb->args[0] = ifidx;") it looks good, thanks. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part