From mboxrd@z Thu Jan 1 00:00:00 1970 From: walter harms Date: Sun, 03 Oct 2010 13:06:14 +0000 Subject: Re: [PATCH v3 14/20] Staging: rtl8192u: ieee80211: Makefile: cleaned Message-Id: <4CA87FC6.40208@bfs.de> List-Id: References: <1286037489-28439-1-git-send-email-tdent48227@gmail.com> <1286037489-28439-14-git-send-email-tdent48227@gmail.com> In-Reply-To: <1286037489-28439-14-git-send-email-tdent48227@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Tracey Dent Cc: greg@kroah.com, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, sam@ravnborg.org Tracey Dent schrieb: > Changed to use the proper ccflags-y option > > Signed-off-by: Tracey Dent > --- > drivers/staging/rtl8192u/ieee80211/Makefile | 16 ++++++++-------- > 1 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/rtl8192u/ieee80211/Makefile b/drivers/staging/rtl8192u/ieee80211/Makefile > index 71ca5d9..45704f8 100644 > --- a/drivers/staging/rtl8192u/ieee80211/Makefile > +++ b/drivers/staging/rtl8192u/ieee80211/Makefile > @@ -5,22 +5,22 @@ MODDESTDIR := /lib/modules/$(KVER)/kernel/drivers/net/wireless/$(NIC_SELECT) > > CC = gcc does this mean we can compile with gcc only ? where are the icc users ? > ifneq ($(shell uname -r|cut -d. -f1,2), 2.4) I can guess what the author intended but this will only check *on* what system i am compiling, not what linux version i try *to conpile*. see: KERNEL_VERSION macro > -EXTRA_CFLAGS += -I$(TOPDIR)/drivers/net/wireless > -EXTRA_CFLAGS += -O2 no optimisation, please > -EXTRA_CFLAGS += -DJACKSON_NEW_8187 -DJACKSON_NEW_RX > +ccflags-y := -I$(TOPDIR)/drivers/net/wireless > +ccflags-y += -O2 > +ccflags-y += -DJACKSON_NEW_8187 -DJACKSON_NEW_RX > > #it will fail to compile in suse linux enterprise 10 sp2. This flag is to solve this problem. > ifeq ($(shell uname -r | cut -d. -f1,2,3,4), 2.6.16.60-0) > -EXTRA_CFLAGS += -DOPENSUSE_SLED > +ccflags-y := -DOPENSUSE_SLED > endif this should be something like ..... # if LINUX_VERSION_CODE = KERNEL_VERSION(2,6,16) #define OPENSUSE_SLED #endif > ifeq ($(NIC_SELECT),RTL8192U) > -#EXTRA_CFLAGS += -DUSB_TX_DRIVER_AGGREGATION_ENABLE > -#EXTRA_CFLAGS += -DUSB_RX_AGGREGATION_SUPPORT > +#ccflags-y := -DUSB_TX_DRIVER_AGGREGATION_ENABLE > +#ccflags-y := -DUSB_RX_AGGREGATION_SUPPORT > endif > -#EXTRA_CFLAGS += -DJOHN_NOCPY > +#ccflags-y := -DJOHN_NOCPY > #flags to enable or disble 80211D feature > -EXTRA_CFLAGS += -DENABLE_DOT11D > +ccflags-y += -DENABLE_DOT11D > ieee80211-rsl-objs := ieee80211_rx.o \ > ieee80211_softmac.o \ > ieee80211_tx.o \ just my 2 cents, wh