On Sunday 19 November 2006 20:31, Johannes Berg wrote:
> --- wireless-dev.orig/drivers/net/wireless/d80211/bcm43xx/bcm43xx_debugfs.c
> 2006-11-19 20:12:47.629275208 +0100
> +++ wireless-dev/drivers/net/wireless/d80211/bcm43xx/bcm43xx_debugfs.c
> 2006-11-19 20:14:01.189275208 +0100
> @@ -285,7 +285,7 @@ void bcm43xx_debugfs_add_device(struct b
> {
> struct bcm43xx_dfsentry *e;
> struct bcm43xx_txstatus_log *log;
> - char devdir[IFNAMSIZ];
> + char devdir[16];
What's the purpose of this?
> assert(bcm);
> e = kzalloc(sizeof(*e), GFP_KERNEL);
> @@ -308,7 +308,7 @@ void bcm43xx_debugfs_add_device(struct b
>
> bcm->dfsentry = e;
>
> - strncpy(devdir, bcm->net_dev->name, ARRAY_SIZE(devdir));
> + snprintf(devdir, sizeof(devdir), "wiphy%d", bcm->ieee->index);
> e->subdir = debugfs_create_dir(devdir, fs.root);
> e->dentry_tsf = debugfs_create_file("tsf", 0666, e->subdir,
> bcm, &tsf_fops);
> +static struct ieee80211_ops bcm43xx_hw_ops = {
> + .tx = bcm43xx_net_hard_start_xmit,
> + .open = bcm43xx_net_open,
> + .stop = bcm43xx_net_stop,
> + .add_interface = bcm43xx_add_interface,
> + .remove_interface = bcm43xx_remove_interface,
> + .reset = bcm43xx_net_reset,
> + .config = bcm43xx_net_config,
> + .config_interface = bcm43xx_config_interface,
> + .set_multicast_list = bcm43xx_set_multicast_list,
> + .set_key = bcm43xx_net_set_key,
> + .get_stats = bcm43xx_net_get_stats,
> + .get_tx_stats = bcm43xx_net_get_tx_stats,
> + .conf_tx = bcm43xx_net_conf_tx,
> +};
I like that being a seperate and static struct.
--
Greetings Michael.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
