Search Linux Wireless

Re: [PATCH 11/24] ath6kl: add htc.c

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

 



On Wed, 2011-07-13 at 10:46 +0300, Kalle Valo wrote:
> On 07/13/2011 07:30 AM, Joe Perches wrote:
> > On Wed, 2011-07-13 at 04:34 +0300, Kalle Valo wrote: 
> >> Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxxxx>
> >> diff --git a/drivers/net/wireless/ath/ath6kl/htc.c b/drivers/net/wireless/ath/ath6kl/htc.c
> > [] 
> >> +static void htc_tx_comp_update(struct htc_target *target,
> >> +			       struct htc_endpoint *endpoint,
> >> +			       struct htc_packet *packet)
> >> +{
> > [] 
> >> 	ath6kl_dbg(ATH6KL_DBG_HTC_SEND, "ctxt:0x%lx dist:0x%lx\n",
> >> +		(unsigned long) target->cred_dist_cntxt,
> >> +		(unsigned long *) &target->cred_dist_list);
> > compiler warning?
> I haven't seen any. But all these unsigned long casts will be removed
> anyway.

You need to update debug.h [08/24]

> diff --git a/drivers/net/wireless/ath/ath6kl/debug.h b/drivers/net/wireless/ath/ath6kl/debug.h
[]
> +extern int ath6kl_printk(const char *level, const char *fmt, ...);

You're not seeing compiler warnings for mismatched
arguments because this should to be:

extern __attribute__((format (printf, 2, 3)))
int ath6kl_printk(const char *level, const char *fmt, ...);



--
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 Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux