Search Linux Wireless

Re: [RFC/WIP 16/22] ath9k_hw: Use CONFIG_ATH9K_BTCOEX_SUPPORT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux