public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Marko Ristola <marko.ristola@kolumbus.fi>
To: Devin Heitmueller <dheitmueller@kernellabs.com>
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>,
	Hans Verkuil <hverkuil@xs4all.nl>,
	Andy Walls <awalls@md.metrocast.net>,
	linux-media@vger.kernel.org
Subject: Re: [RFCv2 PATCH 0/5] tuner-core: fix s_std and s_tuner
Date: Wed, 06 Jul 2011 22:59:15 +0300	[thread overview]
Message-ID: <4E14BE93.2030205@kolumbus.fi> (raw)
In-Reply-To: <CAGoCfiwjXYBR8FBYMS8BsBM20mCQLvWQbyhLh-psA_HX73SGjw@mail.gmail.com>

06.07.2011 21:17, Devin Heitmueller kirjoitti:
> On Wed, Jul 6, 2011 at 1:53 PM, Marko Ristola <marko.ristola@kolumbus.fi> wrote:
>>
> 
> All that said, I believe that you are correct in that the business
> logic needs to ultimately be decided by the bridge driver, rather than
> having the dvb/tuner core blindly calling the sleep routines against
> the tuner and demod drivers without a full understanding of what
> impact it has on the board as a whole.

You wrote it nicely and compactly.

What do you think about tracking coarse last busy time rather than figuring out accurate idle time?

dvb_frontend.c and V4L side would just poll the device:
"bridge->wake()". wake() will just store current "busy" timestamp to the bridge device
with coarse accuracy, if subdevices are already at active state.
If subdevices are powered off, it will first power them on and resume them, and then store "busy" timestamp.

Bridge device would have a "delayed task": "Check after 3 minutes: If I haven't been busy
for three minutes, I'll go to sleep. I'll suspend the subdevices and power them off."

The "delayed task" would refresh itself: check again after last awake time + 3 minutes.

"Delayed task" could be further developed to support multiple suspend states.

> 
> Cheers,
> 
> Devin
> 


Marko

  reply	other threads:[~2011-07-06 19:59 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-11 15:05 [RFCv2 PATCH 0/5] tuner-core: fix s_std and s_tuner Hans Verkuil
2011-06-11 15:05 ` [RFCv2 PATCH 1/5] tuner-core: rename check_mode to supported_mode Hans Verkuil
2011-06-11 15:05   ` [RFCv2 PATCH 2/5] tuner-core: change return type of set_mode_freq to bool Hans Verkuil
2011-06-11 15:05   ` [RFCv2 PATCH 3/5] tuner-core: simplify the standard fixup Hans Verkuil
2011-06-11 15:05   ` [RFCv2 PATCH 4/5] tuner-core: fix s_std and s_tuner Hans Verkuil
2011-06-11 17:23     ` Hans Verkuil
2011-06-11 15:05   ` [RFCv2 PATCH 5/5] tuner-core: fix tuner_resume: use t->mode instead of t->type Hans Verkuil
2011-06-11 15:32 ` [RFCv2 PATCH 0/5] tuner-core: fix s_std and s_tuner Devin Heitmueller
2011-06-11 15:53   ` Hans Verkuil
2011-06-11 16:06     ` Andy Walls
2011-06-11 16:57       ` Devin Heitmueller
2011-06-11 17:02         ` Hans Verkuil
2011-06-11 17:08           ` Devin Heitmueller
2011-06-15 19:55             ` Hans Verkuil
2011-06-15 20:09               ` Devin Heitmueller
2011-06-15 20:37                 ` Hans Verkuil
2011-06-15 20:49                   ` Devin Heitmueller
2011-06-16  6:21                     ` Hans Verkuil
2011-06-16 11:14                       ` Mauro Carvalho Chehab
2011-06-16 12:51                         ` Devin Heitmueller
2011-07-06 17:53                           ` Marko Ristola
2011-07-06 18:17                             ` Devin Heitmueller
2011-07-06 19:59                               ` Marko Ristola [this message]
2011-07-07 15:14                                 ` Mauro Carvalho Chehab
2011-07-07 18:36                                   ` Marko Ristola
2011-06-11 17:00     ` Devin Heitmueller
2011-06-11 16:04   ` Andy Walls

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=4E14BE93.2030205@kolumbus.fi \
    --to=marko.ristola@kolumbus.fi \
    --cc=awalls@md.metrocast.net \
    --cc=dheitmueller@kernellabs.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox