Hi Sujith, On Tue, Feb 21, 2012 at 20:23, Sujith Manoharan <c_manoha@xxxxxxxxxxxxxxxx> wrote: > Make use of CONFIG_ATH9K_BTCOEX_SUPPORT in ath9k_hw > to provide a clean way of compilation without BTCOEX > support. > > Signed-off-by: Sujith Manoharan <c_manoha@xxxxxxxxxxxxxxxx> > --- > drivers/net/wireless/ath/ath9k/Makefile | 6 +- > drivers/net/wireless/ath/ath9k/ar9003_mci.h | 63 ++++++++++++++++++++++++--- > drivers/net/wireless/ath/ath9k/btcoex.h | 1 - > drivers/net/wireless/ath/ath9k/hw.c | 3 +- > drivers/net/wireless/ath/ath9k/hw.h | 12 +++++- > 5 files changed, 71 insertions(+), 14 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath9k/ar9003_mci.h b/drivers/net/wireless/ath/ath9k/ar9003_mci.h > index 507527b..de94f69 100644 > --- a/drivers/net/wireless/ath/ath9k/ar9003_mci.h > +++ b/drivers/net/wireless/ath/ath9k/ar9003_mci.h > @@ -272,13 +288,46 @@ int ar9003_mci_end_reset(struct ath_hw *ah, struct ath9k_channel *chan, > struct ath9k_hw_cal_data *caldata); > void ar9003_mci_reset(struct ath_hw *ah, bool en_int, bool is_2g, > bool is_full_sleep); > -void ar9003_mci_get_interrupt(struct ath_hw *ah, u32 *raw_intr, > - u32 *rx_msg_intr); > void ar9003_mci_get_isr(struct ath_hw *ah, enum ath9k_int *masked); > > -static inline bool ar9003_mci_is_ready(struct ath_hw *ah) > +#else > + > +#define ar9003_mci_is_ready(...) false If I recall correctly, the recommended form here is to create a inline function that returns false, rather than the #define - this way the type checking of the function's arguments still happens when CONFIG_ATH9K_BTCOEX_SUPPORT is disabled. You've done this a couple of other times in this patch too. Thanks, -- Julian Calaby Email: julian.calaby@xxxxxxxxx Profile: http://www.google.com/profiles/julian.calaby/ .Plan: http://sites.google.com/site/juliancalaby/ -- 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