[ath9k-devel] [PATCH] ath9k_htc: wmi: match wait_for_completion_timeout return type

[ath9k-devel] [PATCH] ath9k_htc: wmi: match wait_for_completion_timeout return type

[Kalle Valo]

Nicholas Mc Guire <hofrat@osadl.org> writes:

> Patch is against 4.1-rc3 (localversion-next is -next-20150514)

BTW, this info should be under "---" line so that it doesn't get stored
to the actual commit log.

-- 
Kalle Valo

[ath9k-devel] [PATCH] ath9k_htc: wmi: match wait_for_completion_timeout return type

[Kalle Valo]

Nicholas Mc Guire <der.herr@hofr.at> writes:

> On Thu, 14 May 2015, Kalle Valo wrote:
>
>> Nicholas Mc Guire <hofrat@osadl.org> writes:
>> 
>> > Patch is against 4.1-rc3 (localversion-next is -next-20150514)
>> 
>> BTW, this info should be under "---" line so that it doesn't get stored
>> to the actual commit log.
>>
> that is where I had been putting it until ask by Josh Triplett and 
> Steven Rostedt to put it above the "---" line explicitly
> see: http://lkml.org/lkml/2015/5/11/552 
>
> So now I'm a bit lost where to actually put this buildtest info ?

Different maintainers have different opinions, there is no one answer,
but if Steven is okay to have this in the commit log so am I. So no need
to change anything.

-- 
Kalle Valo

[ath9k-devel] ath9k_htc: wmi: match wait_for_completion_timeout return type

[Kalle Valo]

> Return type of wait_for_completion_timeout is unsigned long not int.
> As time_left is exclusively used for wait_for_completion_timeout here its
> type is simply changed to unsigned long.
> 
> API conformance testing for completions with coccinelle spatches are being
> used to locate API usage inconsistencies:
> ./drivers/net/wireless/ath/ath9k/wmi.c:331
> 	int return assigned to unsigned long
> 
> Patch was compile tested with x86_64_defconfig + CONFIG_ATH_CARDS=m,
> CONFIG_ATH9K_HTC=m
> 
> Patch is against 4.1-rc3 (localversion-next is -next-20150514)
> 
> Signed-off-by: Nicholas Mc Guire <hofrat@osadl.org>

Thanks, applied to wireless-drivers-next.git.

Kalle Valo