Search Linux Wireless

Re: [PATCH 4/7] ath6kl: Add new functions to handle wow suspend/resume operations

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

 



On Friday 28 October 2011 06:01 PM, Kalle Valo wrote:
On 10/25/2011 01:37 PM, rmani@xxxxxxxxxxxxxxxx wrote:
From: Raja Mani<rmani@xxxxxxxxxxxxxxxx>

Signed-off-by: Raja Mani<rmani@xxxxxxxxxxxxxxxx>

Empty commit log.

+int ath6kl_pm_wow_suspend(struct ath6kl *ar, struct cfg80211_wowlan *wow)
+{
+	struct wmi_set_wow_mode_cmd wakeup_filter_cmd;
+	struct wmi_add_wow_pattern_cmd add_pattern_cmd;
+	struct wmi_del_wow_pattern_cmd del_pattern_cmd;
+	struct wmi_set_host_sleep_mode_cmd hsleep_cmd;
+	int i, ret, pos, left;
+	u8 mask[WOW_PATTERN_SIZE];
+
+	if (WARN_ON(!wow))
+		return -EINVAL;

Unnecessary null check, wow is non-zero here.

+		for (pos = 0; pos<  wow->patterns[i].pattern_len;
+				pos++) {
+			if (wow->patterns[i].mask[pos / 8]&  (0x1<<  (pos % 8)))
+				mask[pos] = 0xFF;

This loop needs a comment.

+	if (ar->tx_pending[ar->ctrl_ep]) {
+		left = wait_event_interruptible_timeout(ar->event_wq,
+				ar->tx_pending[ar->ctrl_ep] == 0, WMI_TIMEOUT);
+		if (left == 0) {
+			ret = -ETIMEDOUT;
+			goto wow_setup_failed;
+		} else if (left<  0) {
+			ret = left;
+			goto wow_setup_failed;

A warning message for both of these error case would be nice.

+#define WOW_HOST_REQ_DELAY	500 /* 500 ms */

No point of duplicating the value, "/* ms */" is enough.

Kalle

Will fix all your above comments in V2..
--
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