From: Felipe Balbi <felipe.balbi@linux.intel.com> To: chunfeng yun <chunfeng.yun@mediatek.com>, Mathias Nyman <mathias.nyman@intel.com> Cc: Matthias Brugger <matthias.bgg@gmail.com>, Felipe Balbi <balbi@ti.com>, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-usb@vger.kernel.org, linux-mediatek@lists.infradead.org, Daniel Kurtz <djkurtz@chromium.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org> Subject: Re: [PATCH] usb: xhci-mtk: fixup mouse wakeup failure during system suspend Date: Tue, 03 May 2016 10:51:53 +0300 [thread overview] Message-ID: <87vb2vfujq.fsf@intel.com> (raw) In-Reply-To: <1462261458.11651.7.camel@mhfsdcap03> [-- Attachment #1: Type: text/plain, Size: 2344 bytes --] Hi, chunfeng yun <chunfeng.yun@mediatek.com> writes: > On Thu, 2016-04-21 at 10:04 +0800, Chunfeng Yun wrote: >> Click mouse after xhci suspend completion but before system suspend >> completion, system will not be waken up by mouse if the duration of >> them is larger than 20ms which is the device UFP's resume signaling what is "them" here ? The duration of what is longer than 20ms ? >> lasted. Another reason is that the SPM is not enabled before system what's SPM ? >> suspend compeltion, this causes SPM also not notice the resume signal. ^^^^^^^^^^ completion >> So in order to reduce the duration less than 20ms, make use of >> syscore's suspend/resume interface. no, this is the wrong approach >> In fact it is a work around solution which only reduces the >> probability of failure, because we can't ensure that the duration from >> syscore's suspend completion to SPM working is always less than 20ms. right, which means you're not really fixing anything. Morevoer, you make it so that this won't work with multiple instances of the XHCI IP in your SoC. >> Because the syscore runs on irq disabled context, and xhci's >> suspend/resume calls some sleeping functions, enable local irq >> and then disable it during suspend/resume. This may be not a problem, >> since only boot CPU is runing. another problem :) calling local_irq_{enable,disable}() is an indication that something's wrong. >> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> >> --- >> drivers/usb/host/xhci-mtk.c | 31 ++++++++++++++++++++++++++++++- >> 1 file changed, 30 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/usb/host/xhci-mtk.c b/drivers/usb/host/xhci-mtk.c >> index 79959f1..8277f02 100644 >> --- a/drivers/usb/host/xhci-mtk.c >> +++ b/drivers/usb/host/xhci-mtk.c >> @@ -28,6 +28,7 @@ >> #include <linux/pm_runtime.h> >> #include <linux/regmap.h> >> #include <linux/regulator/consumer.h> >> +#include <linux/syscore_ops.h> >> >> #include "xhci.h" >> #include "xhci-mtk.h" >> @@ -490,6 +491,8 @@ static int xhci_mtk_setup(struct usb_hcd *hcd) >> return xhci_gen_setup(hcd, xhci_mtk_quirks); >> } >> >> +static struct device *xhci_mtk_syscore_dev; now, what happens when you have more than one XHCI instance ? -- balbi [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 818 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: felipe.balbi@linux.intel.com (Felipe Balbi) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH] usb: xhci-mtk: fixup mouse wakeup failure during system suspend Date: Tue, 03 May 2016 10:51:53 +0300 [thread overview] Message-ID: <87vb2vfujq.fsf@intel.com> (raw) In-Reply-To: <1462261458.11651.7.camel@mhfsdcap03> Hi, chunfeng yun <chunfeng.yun@mediatek.com> writes: > On Thu, 2016-04-21 at 10:04 +0800, Chunfeng Yun wrote: >> Click mouse after xhci suspend completion but before system suspend >> completion, system will not be waken up by mouse if the duration of >> them is larger than 20ms which is the device UFP's resume signaling what is "them" here ? The duration of what is longer than 20ms ? >> lasted. Another reason is that the SPM is not enabled before system what's SPM ? >> suspend compeltion, this causes SPM also not notice the resume signal. ^^^^^^^^^^ completion >> So in order to reduce the duration less than 20ms, make use of >> syscore's suspend/resume interface. no, this is the wrong approach >> In fact it is a work around solution which only reduces the >> probability of failure, because we can't ensure that the duration from >> syscore's suspend completion to SPM working is always less than 20ms. right, which means you're not really fixing anything. Morevoer, you make it so that this won't work with multiple instances of the XHCI IP in your SoC. >> Because the syscore runs on irq disabled context, and xhci's >> suspend/resume calls some sleeping functions, enable local irq >> and then disable it during suspend/resume. This may be not a problem, >> since only boot CPU is runing. another problem :) calling local_irq_{enable,disable}() is an indication that something's wrong. >> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> >> --- >> drivers/usb/host/xhci-mtk.c | 31 ++++++++++++++++++++++++++++++- >> 1 file changed, 30 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/usb/host/xhci-mtk.c b/drivers/usb/host/xhci-mtk.c >> index 79959f1..8277f02 100644 >> --- a/drivers/usb/host/xhci-mtk.c >> +++ b/drivers/usb/host/xhci-mtk.c >> @@ -28,6 +28,7 @@ >> #include <linux/pm_runtime.h> >> #include <linux/regmap.h> >> #include <linux/regulator/consumer.h> >> +#include <linux/syscore_ops.h> >> >> #include "xhci.h" >> #include "xhci-mtk.h" >> @@ -490,6 +491,8 @@ static int xhci_mtk_setup(struct usb_hcd *hcd) >> return xhci_gen_setup(hcd, xhci_mtk_quirks); >> } >> >> +static struct device *xhci_mtk_syscore_dev; now, what happens when you have more than one XHCI instance ? -- balbi -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 818 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160503/58d632ac/attachment.sig>
next prev parent reply other threads:[~2016-05-03 7:54 UTC|newest] Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-04-21 2:04 [PATCH] usb: xhci-mtk: fixup mouse wakeup failure during system suspend Chunfeng Yun 2016-04-21 2:04 ` Chunfeng Yun 2016-04-21 2:04 ` Chunfeng Yun 2016-05-03 7:44 ` chunfeng yun 2016-05-03 7:44 ` chunfeng yun 2016-05-03 7:44 ` chunfeng yun 2016-05-03 7:51 ` Felipe Balbi [this message] 2016-05-03 7:51 ` Felipe Balbi 2016-05-03 9:09 ` chunfeng yun 2016-05-03 9:09 ` chunfeng yun 2016-05-03 9:09 ` chunfeng yun 2016-05-03 9:33 ` Felipe Balbi 2016-05-03 9:33 ` Felipe Balbi 2016-05-03 9:33 ` Felipe Balbi 2016-05-04 1:21 ` chunfeng yun 2016-05-04 1:21 ` chunfeng yun 2016-05-04 1:21 ` chunfeng yun 2016-05-04 8:03 ` Felipe Balbi 2016-05-04 8:03 ` Felipe Balbi 2016-05-04 10:54 ` chunfeng yun 2016-05-04 10:54 ` chunfeng yun 2016-05-04 10:54 ` chunfeng yun 2016-05-04 11:56 ` Felipe Balbi 2016-05-04 11:56 ` Felipe Balbi 2016-05-17 2:00 ` chunfeng yun 2016-05-17 2:00 ` chunfeng yun 2016-05-17 2:00 ` chunfeng yun
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=87vb2vfujq.fsf@intel.com \ --to=felipe.balbi@linux.intel.com \ --cc=balbi@ti.com \ --cc=chunfeng.yun@mediatek.com \ --cc=djkurtz@chromium.org \ --cc=gregkh@linuxfoundation.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=linux-usb@vger.kernel.org \ --cc=mathias.nyman@intel.com \ --cc=matthias.bgg@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.