public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Franklin Meng <fmeng2002@yahoo.com>
Cc: Devin Heitmueller <dheitmueller@kernellabs.com>,
	linux-media@vger.kernel.org
Subject: Re: Kworld 315U and SAA7113?
Date: Sun, 17 Jan 2010 10:28:01 -0200	[thread overview]
Message-ID: <4B530251.5000204@infradead.org> (raw)
In-Reply-To: <245243.78544.qm@web32707.mail.mud.yahoo.com>

Franklin Meng wrote:
> Devin, 
> 
>>> I'm actually not really concerned about it's
>> interaction
>>> with a demod.
>>>   I'm more worried about other products that have
>>> saa711[345] that use
>>> a bridge other than em28xx.  The introduction of
>> power
>>> management
>>> could always expose bugs in those bridges (I had this
>>> problem in
>>> several different cases where I had to fix problems
>> in
>>> other drivers
>>> because of the introduction of power management).
>>>
> 
> I retested my device and tried several different GPIO sequences but so far every time I change between the Analog and digital interface, the SAA7113 needs to be reinitialized.  

This happens on several designs. In general, the gpio sequence will turn off either the analog or the digital part of the device,
in order to save power and to avoid overheating the device.

> I tried leaving both the digital and analog interfaces enabled by setting the GPIO to 7c but then the LG demod does not initialize.  

Don't do that. You may burn your device.

> Either way it looks like I will have to reinitialize the device after switching between interfaces.  

The em28xx driver calls em28xx_set_mode(dev, EM28XX_ANALOG_MODE); when the device is opened in analog mode.
It seems that we'll need some code there to also call the analog demod to re-initiate the device, after sending
the gpio commands.

> 
> Other than that do you want me to remove the suspend GPIO?  Since I don't have the equipment to measure the power, 
> I don't know for a fact if the device really has been put in a suspend state or not.  

In suspend state, it will be cooler than when in normal state. It is better to keep the suspend state
to increase the lifetime of the device.
> 
> Thanks,
> Franklin Meng
> 
> 
>       
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


  reply	other threads:[~2010-01-17 12:28 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-09 19:30 Kworld 315U and SAA7113? Franklin Meng
2010-01-13 21:10 ` Mauro Carvalho Chehab
2010-01-14 18:17   ` Franklin Meng
2010-01-13 22:04 ` Devin Heitmueller
2010-01-14 18:54   ` Franklin Meng
2010-01-14 19:13     ` Devin Heitmueller
2010-01-14 22:00     ` CityK
2010-01-14 22:09       ` Devin Heitmueller
2010-01-15  3:03         ` Franklin Meng
2010-01-17  7:05           ` Franklin Meng
2010-01-17 12:28             ` Mauro Carvalho Chehab [this message]
2010-01-17 16:28             ` Devin Heitmueller
2010-01-17 19:02               ` Franklin Meng
  -- strict thread matches above, loose matches on Subject: below --
2010-01-09  8:30 Franklin Meng
2010-01-08  3:48 Franklin Meng

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=4B530251.5000204@infradead.org \
    --to=mchehab@infradead.org \
    --cc=dheitmueller@kernellabs.com \
    --cc=fmeng2002@yahoo.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox