On Tue, 2011-04-05 at 21:57 +0200, RafaÅ MiÅecki wrote:
> Signed-off-by: RafaÅ MiÅecki <zajec5@xxxxxxxxx>
Just some trivia.
> diff --git a/drivers/bcmai/b43_pci_ai_bridge.c b/drivers/bcmai/b43_pci_ai_bridge.c
[]
> +#include <linux/bcmai/bcmai.h>
> +#include <linux/bcmai/bcmai_regs.h>
> +#include <linux/pci.h>
> +
> +static const struct pci_device_id b43_pci_ai_bridge_tbl[] = {
DEFINE_PCI_DEVICE_TABLE
> diff --git a/drivers/bcmai/core.c b/drivers/bcmai/core.c
[]
> + bcmai_err("Power control not implemented!\n");
[]
> diff --git a/include/linux/bcmai/bcmai.h b/include/linux/bcmai/bcmai.h
[]
> +#define bcmai_info(fmt, args...) printk(KERN_INFO "bcmai: " fmt, ##args)
> +#ifdef CONFIG_BCMAI_DEBUG
> +#define bcmai_dbg(fmt, args...) printk(KERN_DEBUG "bcmai debug: " fmt, ##args)
> +#else
> +#define bcmai_dbg(fmt, args...) do { } while (0)
> +#endif
> +#define bcmai_err(fmt, args...) printk(KERN_ERR "bcmai error: " fmt, ##args)
I think there's very little value in prefixing
"error" and "debug" in front of equivalent
KERN_<level> message levels.
I think you might as well just use pr_<level>
and pr_fmt.
> diff --git a/include/linux/mod_devicetable.h b/include/linux/mod_devicetable.h
[]
> +/* AI core, see drivers/bcmai/ */
> +struct bcmai_device_id {
> + __u16 manuf;
> + __u16 id;
> + __u8 rev;
> +};
Do some of these structs need __packed declarations?
--
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
