| Message ID | 1431621821-8173-1-git-send-email-hofrat@osadl.org (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Kalle Valo |
| Headers | show |
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.
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 ? thx! hofrat -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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.
> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/wireless/ath/ath9k/wmi.c b/drivers/net/wireless/ath/ath9k/wmi.c index ca533b4..a1458db 100644 --- a/drivers/net/wireless/ath/ath9k/wmi.c +++ b/drivers/net/wireless/ath/ath9k/wmi.c @@ -299,7 +299,8 @@ int ath9k_wmi_cmd(struct wmi *wmi, enum wmi_cmd_id cmd_id, sizeof(struct wmi_cmd_hdr); struct sk_buff *skb; u8 *data; - int time_left, ret = 0; + unsigned long time_left; + int ret = 0; if (ah->ah_flags & AH_UNPLUGGED) return 0;
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> --- drivers/net/wireless/ath/ath9k/wmi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)