All of lore.kernel.org
 help / color / mirror / Atom feed
From: Johannes Stezenbach <js@linuxtv.org>
To: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>,
	Mauro Carvalho Chehab <mchehab@infradead.org>
Subject: Re: [PATCH 0/6] some fixes and cleanups for the em28xx-based HVR-930C
Date: Mon, 29 Sep 2014 20:44:28 +0200	[thread overview]
Message-ID: <20140929184428.GA447@linuxtv.org> (raw)
In-Reply-To: <20140929153018.2f701689@recife.lan>

On Mon, Sep 29, 2014 at 03:30:18PM -0300, Mauro Carvalho Chehab wrote:
> Em Mon, 29 Sep 2014 19:44:30 +0200
> Johannes Stezenbach <js@linuxtv.org> escreveu:
> 
> > Disregarding your mails from the "em28xx breaks after hibernate"
> > that hibernate doesn't work for you, I decided to give these
> > changes a try on top of today's media_tree.git
> > (cf3167c -> 3.17.0-rc5-00741-g9a3fbd8), still inside qemu
> > (can't upgrade/reboot my main machine right now).
> 
> Well, I think I was not clear: It doesn't work for me when 
> I power down the USB or the machine after suspended. If I keep
> the device energized, it works ;)

Ah, OK.  I'll try to test with power removed tomorrow.

> > Works!  For hibernate, using "echo reboot >/sys/power/disk", so
> > the host driver cannot interfere with the qemu driver during hibernate.
> > Qemu causes several USB resets to the device during
> > hibernate -> resume, but the USB power is not cut.
> > It works even while running dvbv5-zap and streaming to mplayer.
> 
> Thanks for testing it! it is great to have a separate test for
> the patches.
> 
> Could I add a tested-by tag on those patches?

sure

> > I tried both suspend-to-ram and hibernate a couple of times,
> > at least in Qemu it all works.
> > 
> > There are a lot of drxk debug prints now enabled by default,
> > not sure if that was intentional.
> 
> You're probably probing the device with debug=1, right?

right, forgot I had added this to the kernel command line
in my qemu start script


Thanks,
Johannes

  reply	other threads:[~2014-09-29 18:44 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-29  2:23 [PATCH 0/6] some fixes and cleanups for the em28xx-based HVR-930C Mauro Carvalho Chehab
2014-09-29  2:23 ` [PATCH 1/6] [media] em28xx: remove firmware before releasing xc5000 priv state Mauro Carvalho Chehab
2014-09-29  2:23 ` [PATCH 2/6] [media] drxk: Fix debug printks Mauro Carvalho Chehab
2014-09-29  2:23 ` [PATCH 3/6] [media] em28xx-dvb: remove unused mfe_sharing Mauro Carvalho Chehab
2014-09-29  2:23 ` [PATCH 4/6] [media] em28xx-dvb: handle to stop/start streaming at PM Mauro Carvalho Chehab
2014-09-29  2:23 ` [PATCH 5/6] [media] em28xx: move board-specific init code Mauro Carvalho Chehab
2014-09-29  2:23 ` [PATCH 6/6] [media] drxk: move device init code to .init callback Mauro Carvalho Chehab
2014-09-29 17:44 ` [PATCH 0/6] some fixes and cleanups for the em28xx-based HVR-930C Johannes Stezenbach
2014-09-29 18:30   ` Mauro Carvalho Chehab
2014-09-29 18:44     ` Johannes Stezenbach [this message]
2014-09-30  7:38       ` Johannes Stezenbach
2014-09-30  9:14         ` Mauro Carvalho Chehab
2014-09-30 14:43           ` Antti Palosaari

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=20140929184428.GA447@linuxtv.org \
    --to=js@linuxtv.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@infradead.org \
    --cc=mchehab@osg.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.