On 09/18/2014 02:50 PM, Mauro Carvalho Chehab wrote:
Sorry, I misunderstood this patch.
There are actually two different things, each requiring a different PM
setting:
1) to put the tuner to sleep while it is not in usage;
2) put the machine to suspend.
This patch is for (1). That's FINE. I'll apply it.
Yet, for (2), assuming a suspend to ram, the best is to save more
power.
In the past, the DVB core didn't make any distinction between those
two, but we recently added a hook for suspend there.
So, it makes sense to keep the tuner powerdown mode for suspend.
I think old deep sleep should be used for suspend, whilst on warm state
that new is OK.
I2C driver has suspend/resume support and it is one thing I am going to
study in near future. Currently I have 128GB SSD and no swap at all, so
I cannot test it now. I have already bought 256GB SSD and I am just
waiting for Fedora 21 alpha version, which I will install with proper
swap. Unfortunately they have delayed many times already Fedora 21 alpha...
I am not even sure if DVB frontend needs any special suspend/resume
handling, isn't it possible to use standard kernel PM here? (I already
added some initial PM to DVB FE ~2 years ago, but IIRC those will only
kill threads and so which are not allowed when suspend).
So is there some reason DVB FE needs suspend/resume hooks?
regards
Antti
--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
