From: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
To: Johannes Stezenbach <js@linuxtv.org>
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: Tue, 30 Sep 2014 06:14:18 -0300 [thread overview]
Message-ID: <20140930061418.101be1ff@concha.lan> (raw)
In-Reply-To: <20140930073810.GA9128@linuxtv.org>
Em Tue, 30 Sep 2014 09:38:10 +0200
Johannes Stezenbach <js@linuxtv.org> escreveu:
> On Mon, Sep 29, 2014 at 08:44:28PM +0200, Johannes Stezenbach wrote:
> > 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.
>
> I test again in qemu, but this time rmmod and blacklist em28xx
> on the host, and unplug HVR-930C during hibernate. As you
> said, it breaks. Log fter resume:
>
> [ 83.308267] usb 1-1: reset high-speed USB device number 2 using ehci-pci
> [ 83.598182] em2884 #0: Resuming extensions
> [ 83.599187] em2884 #0: Resuming video extensionem2884 #0: Resuming DVB extension
> [ 83.604115] xc5000: I2C read failed
> [ 83.607091] xc5000: I2C write failed (len=3)
> [ 83.607985] xc5000: firmware upload failed...
> [ 83.608766] - too many retries. Giving up
> [ 83.609553] em2884 #0: fe0 resume -22
> [ 83.615533] PM: restore of devices complete after 937.567 msecs
> [ 83.617278] PM: Image restored successfully.
> [ 83.618262] PM: Basic memory bitmaps freed
> [ 83.619097] Restarting tasks ... done.
> [ 83.622320] xc5000: I2C read failed
> [ 83.623197] xc5000: I2C write failed (len=3)
> [ 83.623198] xc5000: firmware upload failed...
> [ 83.623198] - too many retries. Giving up
> [ 83.624071] drxk: i2c read error at addr 0x29
> [ 83.624072] drxk: Error -6 on mpegts_stop
> [ 83.624073] drxk: Error -6 on start
> [ 84.621531] drxk: i2c read error at addr 0x29
> [ 84.623426] drxk: Error -6 on get_dvbt_lock_status
> [ 84.625477] drxk: Error -6 on get_lock_status
Yeah, this is what I was expecting. I have already a patch that would fix
the issue with xc5000 (the issue is actually at em28xx, that needs to do
a full initialization of the device), but something else is needed to put
drxk into a reliable state.
I am traveling during this week (without the HVR-930C or RF generators).
So, I can't touch on it. I'll seek for some time to try to fix this issue.
Regards,
Mauro
next prev parent reply other threads:[~2014-09-30 9:14 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
2014-09-30 7:38 ` Johannes Stezenbach
2014-09-30 9:14 ` Mauro Carvalho Chehab [this message]
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=20140930061418.101be1ff@concha.lan \
--to=mchehab@osg.samsung.com \
--cc=js@linuxtv.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@infradead.org \
/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.