linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marko Ristola <marko.ristola@kolumbus.fi>
To: Antti Palosaari <crope@iki.fi>
Cc: linux-media <linux-media@vger.kernel.org>, htl10@users.sourceforge.net
Subject: Re: DVB core enhancements - comments please?
Date: Sat, 30 Jun 2012 12:44:05 +0300	[thread overview]
Message-ID: <4FEECA65.9090205@kolumbus.fi> (raw)
In-Reply-To: <4FEBA656.7060608@iki.fi>


Hi Antti.

My suspend / resume patch implemented "Kaffeine continues viewing channel after resume".
Some of the ideas could be still useful: http://www.spinics.net/lists/linux-dvb/msg19651.html

Rest of this email has a more thorough description.

Regards,
Marko Ristola

On 06/28/2012 03:33 AM, Antti Palosaari wrote:
> Here is my list of needed DVB core related changes. Feel free to comment - what are not needed or what you would like to see instead. I will try to implement what I can (and what I like most interesting :).
>
...
> suspend / resume support
> --------------------------------------------------
> * support is currently quite missing, all what is done is on interface drivers
> * needs power management
> * streaming makes it hard
> * quite a lot work to get it working in case of straming is ongoing


I've implemented Suspend/Resume for Mantis cu1216 in 2007 (PCI DVB-C device):
Kaffeine continued viewing the channel after resume.
When Tuner was idle too long, it was powered off too.

According to Manu Abraham at that time, somewhat smaller patch would have sufficed.
That patch contais nonrelated fixes too, and won't compile now.

Here is the reference (with Manu's answer):
Start of the thread: http://www.spinics.net/lists/linux-dvb/msg19532.html
The patch: http://www.spinics.net/lists/linux-dvb/msg19651.html
Manu's answer: http://www.spinics.net/lists/linux-dvb/msg19668.html

Thoughts about up-to-date implementation
- Bridge (PCI) device must implement suspend/resume callbacks.
- Frontend might need some change (power off / power on callbacks)?
- "save Tuner / DMA transfer state to memory" might be addable to dvb_core.
- Bridge device supporting suspend/resume needs to have a (non-regression)
   fallback for (frontend) devices that don't have a full tested "Kaffeine works"
   suspend/resume implementation yet.
- What changes encrypted channels need?

Marko

  parent reply	other threads:[~2012-06-30  9:44 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-28  0:33 DVB core enhancements - comments please? Antti Palosaari
2012-06-29  4:32 ` Mauro Carvalho Chehab
2012-06-29  5:03   ` Antti Palosaari
2012-06-29 11:24     ` Patrick Boettcher
2012-06-29 13:07       ` Antti Palosaari
2012-06-29 14:21       ` Patrick Boettcher
2012-06-29 14:28       ` Mauro Carvalho Chehab
2012-07-02 21:27         ` Antti Palosaari
2012-07-03 16:25           ` Mauro Carvalho Chehab
2012-07-03 16:50             ` Hin-Tak Leung
2012-07-03 19:54             ` Mauro Carvalho Chehab
2012-06-29 12:02     ` Mauro Carvalho Chehab
2012-06-29 12:55       ` Antti Palosaari
2012-06-30  9:44 ` Marko Ristola [this message]
2012-07-01 11:11   ` Antti Palosaari
2012-07-03 17:21     ` Marko Ristola
2012-07-21 16:36       ` Antti Palosaari
2012-07-25 10:38         ` 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=4FEECA65.9090205@kolumbus.fi \
    --to=marko.ristola@kolumbus.fi \
    --cc=crope@iki.fi \
    --cc=htl10@users.sourceforge.net \
    --cc=linux-media@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).