On Fri, Jun 13, 2008 at 10:56 AM, Johannes Berg
<johannes@xxxxxxxxxxxxxxxx> wrote:
> On Fri, 2008-06-13 at 15:44 +0800, Zhu Yi wrote:
>> From: Ester Kummer <ester.kummer@xxxxxxxxx>
>>
>> This patch returns channel list to sysfs.
>>
>> Signed-off-by: Ester Kummer <ester.kummer@xxxxxxxxx>
>> Signed-off-by: Tomas Winkler <tomas.winkler@xxxxxxxxx>
>> Signed-off-by: Zhu Yi <yi.zhu@xxxxxxxxx>
>> ---
>> drivers/net/wireless/iwlwifi/iwl4965-base.c | 58 ++++++++++++++++++++++++++-
>> 1 files changed, 56 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/net/wireless/iwlwifi/iwl4965-base.c b/drivers/net/wireless/iwlwifi/iwl4965-base.c
>> index 81a1c22..c179b21 100644
>> --- a/drivers/net/wireless/iwlwifi/iwl4965-base.c
>> +++ b/drivers/net/wireless/iwlwifi/iwl4965-base.c
>> @@ -4122,8 +4122,62 @@ static DEVICE_ATTR(power_level, S_IWUSR | S_IRUSR, show_power_level,
>> static ssize_t show_channels(struct device *d,
>> struct device_attribute *attr, char *buf)
>> {
>> - /* all this shit doesn't belong into sysfs anyway */
>> - return 0;
>
> Hehe. Do you really have to? You can query this information with
> nl80211. I don't know why you need all that sysfs stuff anyway.
I'm not sure why Yi has cut off the log message but original was
'This patch returns channel list to sysfs. This is requirement
for validation purposes'
Our automation is build around sysfs and wext and I'm not going to
rewrite it now.
Tomas
--
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
