All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Timothy D. Lenz" <tlenz@vorgon.com>
To: linux-media@vger.kernel.org
Subject: Re: cx5000 default auto sleep mode
Date: Mon, 19 Apr 2010 17:14:50 -0700	[thread overview]
Message-ID: <4BCCF1FA.6070209@vorgon.com> (raw)
In-Reply-To: <k2h829197381004141040n4aa69e06x7a10c7ea70be3dcf@mail.gmail.com>



On 4/14/2010 10:40 AM, Devin Heitmueller wrote:
> On Wed, Apr 14, 2010 at 1:29 PM, Timothy D. Lenz<tlenz@vorgon.com>  wrote:
>> Thanks to Andy Walls, found out why I kept loosing 1 tuner on a FusionHD7
>> Dual express. Didn't know linux supported an auto sleep mode on the tuner
>> chips and that it defaulted to on. Seems like it would be better to default
>> to off. If someone wants an auto power down/sleep mode and their software
>> supports it, then let the program activate it. Seems people are more likely
>> to want the tuners to stay on then keep shutting down.
>>
>> Spent over a year trying to figure out why vdr would loose control of 1 of
>> the dual tuners when the atscepg pluging was used thinking it was a problem
>> with the plugin.
>
> The xc5000 power management changes I made were actually pretty
> thoroughly tested with that card (between myself and Michael Krufky,
> we tested it with just about every card that uses the tuner).  In
> fact, we uncovered several power management bugs in other drivers as a
> result of that effort.  It was a grueling effort that I spent almost
> three months working on.
>
> Generally I agree with the premise that functionality like this should
> only be enabled for boards it was tested with.  However, in this case
> it actually was pretty extensively tested with all the cards in
> question (including this one), and thus it was deemed safe to enable
> by default.  We've had cases in the past where developers exercised
> poor judgement and blindly turned on power management to make it work
> with one card, disregarding the possible breakage that could occur
> with other cards that use the same driver -- this was *not* one of
> those cases.
>
> If there is a bug, it should be pretty straightforward to fix provided
> it can be reproduced.
>
> Regarding the general assertion that the power management should be
> disabled by default, I disagree.  The power savings is considerable,
> the time to bring the tuner out of sleep is negligible, and it's
> generally good policy.
>
> Andy, do you have any actual details regarding the nature of the problem?
>
> Devin
>

I turned debug back on yesterday and it's been running since. No tuners 
have gone down yet, but here are some logs:
http://24.255.17.209:2400/vdr/logs/

When/if a tuner goes down again I'll put fresh logs up


  parent reply	other threads:[~2010-04-20  0:14 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-14 17:29 cx5000 default auto sleep mode Timothy D. Lenz
2010-04-14 17:40 ` Devin Heitmueller
2010-04-15  3:44   ` Andy Walls
2010-04-15  4:39     ` Devin Heitmueller
2010-04-21 22:26       ` Timothy D. Lenz
2010-05-11  1:45       ` Timothy D. Lenz
2010-09-27 18:37         ` Timothy D. Lenz
2010-06-20 21:04       ` Timothy D. Lenz
2010-04-18 19:19     ` Timothy D. Lenz
2010-04-20  0:14   ` Timothy D. Lenz [this message]
2010-04-20 17:17   ` Timothy D. Lenz
2010-04-21  2:10   ` Timothy D. Lenz

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=4BCCF1FA.6070209@vorgon.com \
    --to=tlenz@vorgon.com \
    --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 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.