1021486 - [tarako] power consumption is high when play music with FM is background

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::FMRadio, defect, P1)

Description

[Peipei Cheng (needinfo if you need my action)]

Attachment: bug319139_fm_should_be_power_off_if_audio_muted.patch

Steps to reproduce
---------------------------------------------------------------------------
1. Launch FM and play
2. Send FM to background
3. Launch Music and play

during spreadtrum's power consumption test, they found that the power consumption of this case consume 50mA more power than only playing music.

After developper's analysis, they found that, it's because FM trout was not closed when music starts.

attached is [e-mail]'s patch.

Comment 1

[Peipei Cheng (needinfo if you need my action)]

Hi james/Jesse, my concern is whether this change will affect the case that:
1. resume FM from phone call
2. resume FM from music.

Comment 2

[James Zhang (Spreadtrum)]

Our QA's concern is that power consumption is very high if don't close FM when music playback.

Comment 3

[Jesse]

(In reply to pcheng from comment #1)
> Hi james/Jesse, my concern is whether this change will affect the case that:
> 1. resume FM from phone call
> 2. resume FM from music.

Well, we actually could not restore FM playing automatically if power off FM.

Comment 4

[James Zhang (Spreadtrum)]

Let xinhe implement FM mute interface and see power consumption.

Comment 5

[Xinhe Yan]

I need assess all the solution(mute\just disable fm\disable audio connection).

Comment 6

[Xinhe Yan]

When enter music, fm is running background. 
But voice not came back when close music. 

I think we can add a notification to alert end user. Like music. 
This will decrease power consume.
brcm fm current is 27ma. trout chip is around 46ma.

Comment 7

[Joe Cheng [:jcheng] (please needinfo)]

1.3T+
it seems like you are working on this so i assign this bug to you xinhe
can we not change the behavior the end users are seeing? not sure why would end users need to see notifications to resolve such bug. thanks

Comment 8

[Xinhe Yan]

(In reply to Joe Cheng [:jcheng] from comment #7)
> 1.3T+
> it seems like you are working on this so i assign this bug to you xinhe
> can we not change the behavior the end users are seeing? not sure why would
> end users need to see notifications to resolve such bug. thanks
This bug can not fix in sprd. All the control logic is in gecko.

I just checked samsung glaxy ace nxd.
1 Enter FM
2 Enter Music.  I think FM is disabled to save power.(Not sure, I can not get samsung code.)
3 Pause Music.  No FM voice. There were a notification to indicate FM is in background. Use can turn on FM  in notification. Our android will resume FM voice.

For Tarako,
1 Enter FM
2 Enter Music.  FM background and not disabled. This will consume power.
3 Pause Music.  No FM voice and no FM item in notification.

Since ffos not resume FM voice, can we just disable FM in step 2? 
Or give a end user a alert in notification. When end user drop notification, they will see FM is running on background. 
Anyway, this depend on how ffos design.

And all the control logic is in gecko. sprd only supply interface for gecko.
This can not be fixed in sprd.

Comment 9

[Xinhe Yan]

(In reply to Xinhe Yan from comment #8)
> Since ffos not resume FM voice, can we just disable FM in step 2? 
> Or give a end user a alert in notification. When end user drop notification,
> they will see FM is running on background. 
> Anyway, this depend on how ffos design.
This is not a critical bug, at least for now.

At step 3, our PM checked other reference phones like XiaoMi.
Some resume FM, some not. And most cases are not resume FM voice.

Comment 10

[Mike Lee [:mlee]]

Joe,
Per Xinhe's comment 9, should this bug be removed from the blockers list?

(In reply to Xinhe Yan from comment #9)
> ...
> This is not a critical bug, at least for now.
> 
> At step 3, our PM checked other reference phones like XiaoMi.
> Some resume FM, some not. And most cases are not resume FM voice.

Comment 11

[Joe Cheng [:jcheng] (please needinfo)]

let's take this off the blockers list

Comment 12

[BMO Automation]

Firefox OS is not being worked on