Return-path: Received: from wolverine01.qualcomm.com ([199.106.114.254]:5061 "EHLO wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750736Ab1JaNDc (ORCPT ); Mon, 31 Oct 2011 09:03:32 -0400 Message-ID: <4EAE9C9D.6080201@qca.qualcomm.com> (sfid-20111031_140336_524220_25DF6750) Date: Mon, 31 Oct 2011 15:03:25 +0200 From: Kalle Valo MIME-Version: 1.0 To: Raja Mani CC: Subject: Re: [PATCH 6/7] ath6kl: Perform WOW resume in RX path in case of SDIO IRQ wakeup References: <1319539047-8756-1-git-send-email-rmani@qca.qualcomm.com> <1319539047-8756-7-git-send-email-rmani@qca.qualcomm.com> <4EAAABB8.3030309@qca.qualcomm.com> <4EAE707D.6070806@qca.qualcomm.com> In-Reply-To: <4EAE707D.6070806@qca.qualcomm.com> Content-Type: text/plain; charset="ISO-8859-1" Sender: linux-wireless-owner@vger.kernel.org List-ID: On 10/31/2011 11:55 AM, Raja Mani wrote: > On Friday 28 October 2011 06:48 PM, Kalle Valo wrote: >> On 10/25/2011 01:37 PM, rmani@qca.qualcomm.com wrote: >>> --- a/drivers/net/wireless/ath/ath6kl/txrx.c >>> +++ b/drivers/net/wireless/ath/ath6kl/txrx.c >>> @@ -1081,6 +1081,8 @@ void ath6kl_rx(struct htc_target *target, struct htc_packet *packet) >>> return; >>> } >>> >>> + ath6kl_pm_check_wow_status(ar); >> >> Now we resume from two places, cfg80211 resume handler and here. Why do >> we need to do that? Why isn't cfg80211 resume handler enough? > > This path hits when the target wants to wake up the host. When given WOW > pattern matches, the target will pull SDIO data line to wake up the > host. During that time , the control will reach here (sdio irq handler > -> ath6kl_rx function), not to cfg80211 resume. I don't understand this. Are you saying that host resumes but cfg80211 resume handler is not called? It doesn't make sense, every suspend call should have an equivalent resume call. If that doesn't happen, something is broken somewhere. Kalle