From: Shuah Khan <shuahkh@osg.samsung.com>
To: Johannes Stezenbach <js@linuxtv.org>,
Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Cc: Shuah Khan <shuah.kh@samsung.com>,
linux-media@vger.kernel.org, Shuah Khan <shuahkh@osg.samsung.com>
Subject: Re: em28xx breaks after hibernate
Date: Fri, 26 Sep 2014 08:25:07 -0600 [thread overview]
Message-ID: <54257743.6050509@osg.samsung.com> (raw)
In-Reply-To: <20140926132513.GA30084@linuxtv.org>
On 09/26/2014 07:25 AM, Johannes Stezenbach wrote:
> On Fri, Sep 26, 2014 at 10:12:22AM -0300, Mauro Carvalho Chehab wrote:
>> Try to add a WARN_ON or printk at em28xx_usb_resume().
>
> It is called two times, once during hibernate and once during resume:
>
> root@debian:~# echo disk >/sys/power/state
On the upside this does look similar to what I have seen when
I was debugging suspend/resume on pctv stick that uses em28xx
and drx39xyj
One thing that helped me debug the problem is testing
hibernate in platform mode (which is default) and then
Hibernate in reboot mode.
I enabled usb debug and device debug to see what is happening
at the usb-core and ran the following cases:
I enable pm trace:
echo 1 > /sys/power/pm_trace
Hibernate in platform mode (default and recommended hibernation mode)
echo platform > /sys/power/disk
echo disk > /sys/power/state
Hibernate in reboot mode: (usb bus could go through loss of power as
platform might not maintain power to the buses). reset_resume should
recover from loss of power or have the force disconnect path handle the
case. i.e don't install reset_resume
echo reboot > /sys/power/disk
echo disk > /sys/power/state
I also simply selected suspend from the GUI, this seems to
take the usb-bus through a different path.
These behave differently when reset_resume is installed vs.
not installed. In our case, reset_resume simply points to
resume which can't handle the power loss case. It would be
good to get data on these different scenarios. I wish I have
the WinTV 930, but I don't.
If we have full debug for the above three scenarios, it would
help debug it further. I also do the following to see resume
works in a simple case: no disk involved suspend to ram
echo mem > /sys/power/state
I am looking at drxk to see if I can figure out anything.
Also I dumped em28xx eprom to see if looks ok during these
tests.
thanks,
-- Shuah
--
Shuah Khan
Sr. Linux Kernel Developer
Samsung Research America (Silicon Valley)
shuahkh@osg.samsung.com | (970) 217-8978
next prev parent reply other threads:[~2014-09-26 14:25 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-25 12:53 em28xx breaks after hibernate Johannes Stezenbach
2014-09-25 13:45 ` Shuah Khan
2014-09-25 14:10 ` Mauro Carvalho Chehab
2014-09-25 15:07 ` Shuah Khan
2014-09-25 16:01 ` Johannes Stezenbach
2014-09-25 17:36 ` Johannes Stezenbach
2014-09-25 18:01 ` Shuah Khan
2014-09-25 17:40 ` Shuah Khan
2014-09-25 18:17 ` Johannes Stezenbach
2014-09-25 18:45 ` Shuah Khan
2014-09-25 19:03 ` Shuah Khan
2014-09-26 7:41 ` Johannes Stezenbach
2014-09-26 8:00 ` Johannes Stezenbach
2014-09-26 8:08 ` Johannes Stezenbach
2014-09-26 10:14 ` Mauro Carvalho Chehab
2014-09-26 11:07 ` Johannes Stezenbach
2014-09-26 11:42 ` Mauro Carvalho Chehab
2014-09-26 12:02 ` Johannes Stezenbach
2014-09-26 12:11 ` Mauro Carvalho Chehab
2014-09-26 12:03 ` Mauro Carvalho Chehab
2014-09-26 12:27 ` Johannes Stezenbach
2014-09-26 13:12 ` Mauro Carvalho Chehab
2014-09-26 13:25 ` Johannes Stezenbach
2014-09-26 14:25 ` Shuah Khan [this message]
2014-09-26 14:25 ` Johannes Stezenbach
2014-09-26 14:30 ` Shuah Khan
2014-09-26 15:06 ` Johannes Stezenbach
2014-09-26 15:22 ` Johannes Stezenbach
2014-09-26 15:43 ` Mauro Carvalho Chehab
2014-09-28 10:55 ` Johannes Stezenbach
2014-09-28 11:12 ` Mauro Carvalho Chehab
2014-09-28 11:54 ` Johannes Stezenbach
2014-09-28 12:35 ` Mauro Carvalho Chehab
2014-09-28 12:57 ` Mauro Carvalho Chehab
2014-09-29 2:38 ` Mauro Carvalho Chehab
2014-09-26 15:22 ` Shuah Khan
2014-09-26 15:35 ` Mauro Carvalho Chehab
2014-09-26 15:40 ` Mauro Carvalho Chehab
2014-09-26 15:42 ` Johannes Stezenbach
2014-09-26 15:48 ` Mauro Carvalho Chehab
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=54257743.6050509@osg.samsung.com \
--to=shuahkh@osg.samsung.com \
--cc=js@linuxtv.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@osg.samsung.com \
--cc=shuah.kh@samsung.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: link
Be 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.