998254 - [Flame]Connect usb/charging device then reboot, the charging light indicator doesn't work

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Vendcom, defect)

Description

[Mike Lien[:mlien]]

[Device]
  Flame: PVT & DVT sample
---------------------------------------------
STR:
  1. Power-on device
  2. Connect usb/charging device
  3. Reboot device
  4. Verify charging light indicator

Expected Result:
  Charging light indicator will shine

Actual Result:
  Charging light indicator doesn't work

  ADB command is fine
  Device charging is fine

Comment 1

[viral [:viralwang]]

I think the problem is why it's not charing even we do have usb connected

root@msm8610:/ # cat /sys/class/power_supply/battery/status                    
Discharging

It needs partner's help to check it.

Comment 2

[Francis Lee [:frlee]]

hi Yanan,

please check this issue with priority. please refer to comment 1.
the reproduced rate is not 100%, please try more times

Comment 3

[Tony Zhao]

Hi Mike& Viral,

Just want to double confirm with you, the failure of "Charging light indicator doesn't work" means the battery icon is still in white, not shown as charging status(in green and with lighting sign inside battery icon).
Or means the LED light beside front camera not turn?
Thanks

Comment 4

[Mike Lien[:mlien]]

(In reply to Tony Zhao from comment #3)
> Hi Mike& Viral,
> 
> Just want to double confirm with you, the failure of "Charging light
> indicator doesn't work" means the battery icon is still in white, not shown
> as charging status(in green and with lighting sign inside battery icon).
> Or means the LED light beside front camera not turn?
> Thanks

My device only get LED light not turn problem
But Viral's device might get different result

Comment 5

[viral [:viralwang]]

Attachment: fail symptom

please refer the attachment to understand the failed symptom, adb works but still discharging.
I also try to reduce power consumption by make backlight really dim, but still can not charge.

Comment 6

[Tony Zhao]

I have tried more than 10 times on my side but not able to reproduce this issue. 
Only find that the LED light some time delay around 30s~50s to turn on after device power up.
My steps are:
1.Connect flame and Laptop with USB cable
2.Short press power key then select Restart

BTW, from Viral's screenshot, the battery seems already charging full. 
So I'm wondering if high battery level is the precondition when issue found on your side, or you can also reproduce it when battery level not higher than 90%

Comment 7

[viral [:viralwang]]

Hi Tony,

Yes, it looks like this issue can only found when battery level is nearly full.
But here's another quesion, if the battery is full and connect USB, the status should "Full" not "Discharging"
It's still an issue to me.

Comment 8

[Tony Zhao]

Hi Viral,

As the current design, device will detect battery level when power on, if it's higher than 99%. Charger will not work so the device is in "discharging" status. If the battery level lower than 99%, device should on "Charging" status and shown "Charged" when battery level up to 100%. 
My device is working in this way as I checked.
Is that the case on your side? Or any abnormal behavior you have seem?
Thanks

Comment 9

[viral [:viralwang]]

The symptom is same in my side.
However, I expect the status is "Full" when battery level is over 99% with cable in.
showing "Discharging" will make me feel I didn't connect cable successfully.

Comment 10

[viral [:viralwang]]

(In reply to viral [:viralwang] from comment #9)
> The symptom is same in my side.
> However, I expect the status is "Full" when battery level is over 99% with
> cable in.
> showing "Discharging" will make me feel I didn't connect cable successfully.

Hi Yanan,

Any feedback on this?

Comment 11

[Tony Zhao]

Hi Viral,

We can fix it by this way: while device power up with charger or USB, if battery level not reach 100%, device will continue charging. 
But this solution will also impact normal charging procedure, that means, when device connected with charger or USB and already full charged, once battery level lower than 100%(make call, watching video or something else may speed battery power consume), it will start charging again. This will make battery charged more times than current SW situation(may impact battery life cycle).

Comment 12

[viral [:viralwang]]

Hi Yanan,

Thanks for your feedback.
Few questions inline need your help.

(In reply to Tony Zhao from comment #11)
> Hi Viral,
> 
> We can fix it by this way: while device power up with charger or USB, if
> battery level not reach 100%, device will continue charging.
What's the behavior in our code now?
Take image 10E for example, if we have 99% battery with cable, will it charge or not?
(in this period battery status shows Discharging)

> But this solution will also impact normal charging procedure, that means,
> when device connected with charger or USB and already full charged, once
> battery level lower than 100%(make call, watching video or something else
> may speed battery power consume), it will start charging again. This will
> make battery charged more times than current SW situation(may impact battery
> life cycle).
It looks like a re-charging process.
I expect the battery status shows "Full" in whole period (100% -> 99% -> 100%), is that same as your proposal?

Comment 13

[Tony Zhao]

(In reply to viral [:viralwang] from comment #12)
> Hi Yanan,
> 
> Thanks for your feedback.
> Few questions inline need your help.
> 
> (In reply to Tony Zhao from comment #11)
> > Hi Viral,
> > 
> > We can fix it by this way: while device power up with charger or USB, if
> > battery level not reach 100%, device will continue charging.
> What's the behavior in our code now?
> Take image 10E for example, if we have 99% battery with cable, will it
> charge or not?
> (in this period battery status shows Discharging)
For current code, if device have higher than 99% battery when boot up, it won't charge even with USB/Charger plugged in. And will auto start charge when battery level lower than 99% again.

> > But this solution will also impact normal charging procedure, that means,
> > when device connected with charger or USB and already full charged, once
> > battery level lower than 100%(make call, watching video or something else
> > may speed battery power consume), it will start charging again. This will
> > make battery charged more times than current SW situation(may impact battery
> > life cycle).
> It looks like a re-charging process.
> I expect the battery status shows "Full" in whole period (100% -> 99% ->
> 100%), is that same as your proposal?
Yes, we can modify SW to shown "Full" status in settings as your expect. But as we know the battery icon status shown is control by Mozilla code, so that we need modify it on both side. Otherwise, user will see battery "Full" in settings but on status bar, the battery icon still not shown as full charged.

Thanks

Comment 14

[viral [:viralwang]]

(In reply to Tony Zhao from comment #13)
> (In reply to viral [:viralwang] from comment #12)
> > Hi Yanan,
> > 
> > Thanks for your feedback.
> > Few questions inline need your help.
> > 
> > (In reply to Tony Zhao from comment #11)
> > > Hi Viral,
> > > 
> > > We can fix it by this way: while device power up with charger or USB, if
> > > battery level not reach 100%, device will continue charging.
> > What's the behavior in our code now?
> > Take image 10E for example, if we have 99% battery with cable, will it
> > charge or not?
> > (in this period battery status shows Discharging)
> For current code, if device have higher than 99% battery when boot up, it
> won't charge even with USB/Charger plugged in. And will auto start charge
> when battery level lower than 99% again.
It looks good to me and I think we can keep the same behavior not only when boot up.
It should be same after times plug-in/plug-out cable

> > > But this solution will also impact normal charging procedure, that means,
> > > when device connected with charger or USB and already full charged, once
> > > battery level lower than 100%(make call, watching video or something else
> > > may speed battery power consume), it will start charging again. This will
> > > make battery charged more times than current SW situation(may impact battery
> > > life cycle).
> > It looks like a re-charging process.
> > I expect the battery status shows "Full" in whole period (100% -> 99% ->
> > 100%), is that same as your proposal?
> Yes, we can modify SW to shown "Full" status in settings as your expect. But
> as we know the battery icon status shown is control by Mozilla code, so that
> we need modify it on both side. Otherwise, user will see battery "Full" in
> settings but on status bar, the battery icon still not shown as full charged.
> 
> Thanks

I think we already used the status "Full" to show different UI on status bar here.
Shall we have a test build for this first? Thanks :)

Comment 15

[Tony Zhao]

Attachment: boot.img for fix charge issue

Hi Viral,

Please help to verify this issue with attached boot image, if it works on your side we will take this solution on next version release. Thanks

Comment 16

[Andrew Halberstadt [:ahal]]

We are seeing behaviour that may or may not be a side effect of this. Basically we're attempting to restart the Flame using a custom battery harness we can turn power on/off for (the equivalent of pulling the battery). In 1.3 this worked fine, but in 2.0 instead of powering up after turning the battery back on, the phone goes into a charging mode (black screen with big white battery indicator in the middle).

Does this seem like it could be related?

Comment 17

[Mike Lien[:mlien]]

If Flame is power off, connect usb or ac will go into charging mode.
It's current behavior and related to bug 997087

Comment 18

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

Hi Viral -

Per comment#15, could you help to verify if the attached boot.img solves the problem?

Thanks for your help

Vance

Comment 19

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

ni Viral per Comment#18

Comment 20

[viral [:viralwang]]

Yes, it works in my side.

root@msm8610:/ # cat /sys/class/power_supply/battery/status       
Full

Comment 21

[Tony Zhao]

Attachment: boot-charger-full.img new solution

Hi Vance & Viral,

We found the previous solution have some risk that make have side effect when normal charge, there is a new solution added in the attached boot image, please help to verify on your side, thanks

Comment 22

[BMO Automation]

Firefox OS is not being worked on