| Message ID | 20200725040155.27648-1-penghao@uniontech.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | None | expand |
On Sat, Jul 25, 2020 at 12:01:55PM +0800, penghao wrote: > From: "penghao@deepin.com" <penghao@deepin.com> > > TI024Gen3 USB-audio is controlled by TI024Gen3,when TI024Gens > enter sleep mode, USB-audio will disconnect from USB bus port, > so disabled the /sys/bus/usb/*/power/wakeup Fixesimmediately > wakup form s3 state Why the indentation? > > Signed-off-by: penghao@uniontech.com <penghao@uniontech.com> > Signed-off-by: penghao@deepin.com <penghao@deepin.com> We need a name, not just an email address. And where are the 14215 other patches in this patch series? > --- > sound/usb/card.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/sound/usb/card.c b/sound/usb/card.c > index 2644a5ae2b75..969c3809e051 100644 > --- a/sound/usb/card.c > +++ b/sound/usb/card.c > @@ -640,6 +640,12 @@ static int usb_audio_probe(struct usb_interface *intf, > } > } > dev_set_drvdata(&dev->dev, chip); > + /* > + * ALSA: usb-audio: Add prevent wakeup from s3 state trig by lenovo > + * ThinkCentre TI024Gen3 USB-audio > + */ > + if ((usb_id->idVendor == 0x17ef) && (usb_id->idProduct == 0xa012)) > + device_set_wakeup_enable(&dev->dev, 0); Shouldn't there be a quirk for this type of device that can be used instead? thanks, greg k-h
Thank you for your guidance, I will improve ç»ä¿¡è½¯ä»¶ææ¯æéå¬å¸ UnionTech Software Technology Co., Ltd. ã å®ç½ï¼www.uniontech.comãã 座æºï¼+86-10-62669253 æ¤çµåé®ä»¶æ¶æ¯ä»ä¾é¢ææ¶ä»¶äººä½¿ç¨ï¼å¶ä¸å¯è½åå«ä¿å¯æç¹æ使ç¨ä¿¡æ¯ãå¦ææ¨ ä¸æ¯é¢ææ¶ä»¶äººï¼è¯·å¿ä½¿ç¨ãä¼ æãååæå¤å¶æ¤çµåé®ä»¶æä¿¡èµæ¤é®ä»¶éåä»»ä½è ¡å¨ãå¦ææ¨è¯¯æ¶äºæ¤é®ä»¶ï¼è¯·ç«å³åå¤é®ä»¶éç¥ç»ä¿¡è½¯ä»¶ææ¯æéå¬å¸åä»¶äººï ¼å¹¶å é¤è¯¯æ¶çµåé®ä»¶åå¶ç¸å³é件ãæè°¢éåï¼ This email message is intended only for the use of the individual or entity who/which is the intended recipient and may contain information that is privileged or confidential. If you are not the intended recipient, you are hereby notified that any use, dissemination, distribution or copying of, or taking any action in reliance on, this e-mail is strictly prohibited. If you have received this email in error, please notify UnionTech Software Technology immediately by replying to this e-mail and immediately delete and discard all copies of the e-mail and the attachment thereto (if any). Thank you. From: "Greg KH <gregkh@linuxfoundation.org>" To: "penghao <penghao@uniontech.com>" CC: "perex <perex@perex.cz>","tiwai <tiwai@suse.com>","alsa-devel <alsa-devel@alsa-project.org>","linux-kernel <linux-kernel@vger.kernel.org>","phz0008 <phz0008@163.com>","penghao@deepin.com <penghao@deepin.com>" Sent: 2020-07-25 13:48 Subject: Re: [PATCH 14216/14216] ALSA: usb-audio: This patch for prevent autowakeup from s3 trig by usb disconnect signal from Lenovo ThinkcentreTI024Gen3 USB-audio. On Sat, Jul 25, 2020 at 12:01:55PM +0800, penghao wrote: > From: "penghao@deepin.com" <penghao@deepin.com> > > TI024Gen3 USB-audio is controlled by TI024Gen3,when TI024Gens > enter sleep mode, USB-audio will disconnect from USB bus port, > so disabled the /sys/bus/usb/*/power/wakeup Fixesimmediately > wakup form s3 state Why the indentation? > > Signed-off-by: penghao@uniontech.com <penghao@uniontech.com> > Signed-off-by: penghao@deepin.com <penghao@deepin.com> We need a name, not just an email address. And where are the 14215 other patches in this patch series? > --- > sound/usb/card.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/sound/usb/card.c b/sound/usb/card.c > index 2644a5ae2b75..969c3809e051 100644 > --- a/sound/usb/card.c > +++ b/sound/usb/card.c > @@ -640,6 +640,12 @@ static int usb_audio_probe(struct usb_interface *intf, > } > } > dev_set_drvdata(&dev->dev, chip); > + /* > + * ALSA: usb-audio: Add prevent wakeup from s3 state trig by lenovo > + * ThinkCentre TI024Gen3 USB-audio > + */ > + if ((usb_id->idVendor == 0x17ef) && (usb_id->idProduct == 0xa012)) > + device_set_wakeup_enable(&dev->dev, 0); Shouldn't there be a quirk for this type of device that can be used instead? thanks, greg k-h
On Sat, Jul 25, 2020 at 02:09:14PM +0800, 彭浩 wrote: > This email message is intended only for the use of the individual or entity who > /which is the intended recipient and may contain information that is privileged > or confidential. If you are not the intended recipient, you are hereby notified > that any use, dissemination, distribution or copying of, or taking any action > in reliance on, this e-mail is strictly prohibited. If you have received this > email in error, please notify UnionTech Software Technology immediately by > replying to this e-mail and immediately delete and discard all copies of the > e-mail and the attachment thereto (if any). Thank you. Please note that this email footer is not compatible with kernel development, so I have now removed it from my system. greg k-h
diff --git a/sound/usb/card.c b/sound/usb/card.c index 2644a5ae2b75..969c3809e051 100644 --- a/sound/usb/card.c +++ b/sound/usb/card.c @@ -640,6 +640,12 @@ static int usb_audio_probe(struct usb_interface *intf, } } dev_set_drvdata(&dev->dev, chip); + /* + * ALSA: usb-audio: Add prevent wakeup from s3 state trig by lenovo + * ThinkCentre TI024Gen3 USB-audio + */ + if ((usb_id->idVendor == 0x17ef) && (usb_id->idProduct == 0xa012)) + device_set_wakeup_enable(&dev->dev, 0); /* * For devices with more than one control interface, we assume the @@ -914,3 +920,4 @@ static struct usb_driver usb_audio_driver = { }; module_usb_driver(usb_audio_driver); +