Arend, Henry,
One problem I noticed with your patch:
On Tue, May 10, 2011 at 00:33, Arend van Spriel <arend@xxxxxxxxxxxx> wrote:
> From: Henry Ptasinski <henryp@xxxxxxxxxxxx>
>
> Move utility functions shared by the brcmsmac and brcmfmac drivers into a new
> module, brcmutil.ko. This eliminates the need to compile and link the shared
> functions into both drivers. Prefix all exported symbols with "bcm_".
>
> Cc: devel@xxxxxxxxxxxxxxxxxxxxxx
> Cc: linux-wireless@xxxxxxxxxxxxxxx
> Reviewed-by: Brett Rudley <brudley@xxxxxxxxxxxx>
> Signed-off-by: Arend van Spriel <arend@xxxxxxxxxxxx>
> ---
> drivers/staging/brcm80211/Makefile | 1 +
>
> diff --git a/drivers/staging/brcm80211/Makefile b/drivers/staging/brcm80211/Makefile
> index c064cdf..287ad0e 100644
> --- a/drivers/staging/brcm80211/Makefile
> +++ b/drivers/staging/brcm80211/Makefile
> @@ -19,5 +19,6 @@
> subdir-ccflags-y := -DBCMDMA32
> subdir-ccflags-$(CONFIG_BRCMDBG) += -DBCMDBG -DBCMDBG_ASSERT
>
> +obj-m += util/
> obj-$(CONFIG_BRCMFMAC) += brcmfmac/
> obj-$(CONFIG_BRCMSMAC) += brcmsmac/
You should not use obj-m directly - the way this is set up, if module
support is disabled, then the driver will fail as the brcmutil module
will not be built. This might also happen if the drivers are built
into the kernel ('y' rather than 'm')
You should create a new Kconfig variable (say BRCMWLUTIL) and use some
Kconfig trickery to ensure that it's built properly for the drivers
that need it, then replace this line with something like
obj-$(CONFIG_BRCMWLUTIL)
The LIB80211 symbol used by the Intel 2x00 (old non-mac80211) wireless
drivers uses similar trickery to ensure that everything is built
properly.
--
Julian Calaby
Email: julian.calaby@xxxxxxxxx
Profile: http://www.google.com/profiles/julian.calaby/
.Plan: http://sites.google.com/site/juliancalaby/
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
