Moikka Olli
I ran some PM tests for that patch set, using PCTV 292e, which is em28xx
+ Si2168 B40 + Si2157 A30. Here is results:
current impementation
-------------------------------------
cold plugin 40 mA
streaming 235 mA
sleeping 42 mA
si2157: change command for sleep
-------------------------------------
cold plugin 40 mA
streaming 235 mA
sleeping 60 mA
So it increases sleep power usage surprisingly much, almost 20mA from
the USB, nominal 5V.
It is also funny that you will not lose firmware for Si2168 when sleep
with command 13, but that Si2157 tuner behaves differently.
I think I will still apply that, it is just firmware download time vs.
current use in sleep.
Antti
On 08/25/2014 09:07 PM, Olli Salonen wrote:
Instead of sending command 13 to the tuner, send command 16 when sleeping. This
behaviour is observed when using manufacturer provided binary-only Linux driver
for TechnoTrend CT2-4400 (Windows driver does not do power management).
The issue with command 13 is that firmware loading is necessary after that.
This is not an issue with tuners that do not require firmware, but resuming
from sleep on an Si2158 takes noticeable time as firmware is loaded on resume.
Signed-off-by: Olli Salonen <olli.salonen@xxxxxx>
---
drivers/media/tuners/si2157.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/media/tuners/si2157.c b/drivers/media/tuners/si2157.c
index efb5cce..c84f7b8 100644
--- a/drivers/media/tuners/si2157.c
+++ b/drivers/media/tuners/si2157.c
@@ -197,9 +197,10 @@ static int si2157_sleep(struct dvb_frontend *fe)
s->active = false;
- memcpy(cmd.args, "\x13", 1);
- cmd.wlen = 1;
- cmd.rlen = 0;
+ /* standby */
+ memcpy(cmd.args, "\x16\x00", 2);
+ cmd.wlen = 2;
+ cmd.rlen = 1;
ret = si2157_cmd_execute(s, &cmd);
if (ret)
goto err;
--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
