sonyxperiadev / bug_tracker

Empty repository that is used as a bugtracker for Open Devices project
52 stars 13 forks source link

[Loire] 3 amp chargers doesn't work #279

Closed maluus closed 5 years ago

maluus commented 5 years ago

Platform: Loire Device: Kugo Kernel version: 4.9 Android version: 9.0

Description: Xperia X Compact doesn't charge with 3.0 amp fast chargers. 2.0, 2.1 amp chargers works. 1 amp chargers works till certain point and after that constantly cuts power after 44-45%. See attached log which includes both 3 amp and 1 amp (after 45%) charging attempts.

Symptoms: When charger connected at first begins to charge like 1 sec. then charging light goes off. Battery stat shows no charging but battery icon shows like charging.

How to reproduce: Connect 3 amp charger. Connect 1 amp charging until 44-45% logcat_12-16-2018_15-41-17.txt

charging_dmesg.log

lubik1 commented 5 years ago

can confirm the same problem on Suzu (loire)

kholk commented 5 years ago

The Loire platform does not support the new QuickCharge standard by hardware, so that's totally normal.

maluus commented 5 years ago

Which standards and what changed? It was working until the time when beta v3 binaries published.


From: AngeloGioacchino Del Regno notifications@github.com Sent: Saturday, December 22, 2018 8:48:24 PM To: sonyxperiadev/bug_tracker Cc: maluus; Author Subject: Re: [sonyxperiadev/bug_tracker] [Loire] 3 amp chargers doesn't work (#279)

The Loire platform does not support the new QuickCharge standard by hardware, so that's totally normal.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/sonyxperiadev/bug_tracker/issues/279#issuecomment-449587145, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFEchhnIT6WeWymsltzfzH_C4S-WZxJeks5u7nBogaJpZM4Zdnu3.

kholk commented 5 years ago

Perhaps I don't understand the issue. Please post a dmesg. The logcat is pretty useless here :P

I can guess, around the v3 release the kernel was updated, maybe something went wrong.

maluus commented 5 years ago

OP updated. You can find dmesg in the first post.

jerpelea commented 5 years ago

this platform should not support the feature!

harryharryharry commented 5 years ago

@jerpelea According to Sony's own website Sony X Compact does support QC3.0: https://support.sonymobile.com/us/xperiaxcompact/faq/battery,-power-&-charging/02310188014550f700148fcce243e002e57/

maluus commented 5 years ago

Sure it supports because my phone came with a qc3 charger out of the box. But I think it's to do with 4.9 kernel. Because it was working with 4.4 kernel even with pie.


From: harryharryharry notifications@github.com Sent: Saturday, January 12, 2019 5:44:13 PM To: sonyxperiadev/bug_tracker Cc: maluus; Author Subject: Re: [sonyxperiadev/bug_tracker] [Loire] 3 amp chargers doesn't work (#279)

@jerpeleahttps://github.com/jerpelea According to Sony's own website Sony X Compact does support QC3.0: https://support.sonymobile.com/us/xperiaxcompact/faq/battery,-power-&-charging/02310188014550f700148fcce243e002e57/

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/sonyxperiadev/bug_tracker/issues/279#issuecomment-453752907, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFEchkwTQRPMwxxWA1nzu009MntfzaPgks5vCfS9gaJpZM4Zdnu3.

Gray47Maxx commented 5 years ago

Iirc even suzu supports it. At least, in stock 6.0 fw... And in 2016 white papers

kholk commented 5 years ago

Wait a minute. I think we have to reopen this issue.

The QC3.0 - supported or not - is an evolution of QC2.0, so on an unsupported platform it should anyway charge the device, even if it falls back to a lower amperage... but the device is not charging at all.

This is a bug.

I'll look at it whenever I'll get some time to.

harryharryharry commented 5 years ago

Thanks. You're right, on kugo stock and even omnirom 8.1 do indeed work, although at a lower amperage (~1 to 1.5 as I recall). But omnirom 9: crickets chirping (only the led indicator goes on for about 2 seconds)

maluus commented 5 years ago

As I stated in my previous posts if its not something that 4.9 kernel restricts somehow then it should normally fully benefit from qc3. I had just thought over jerpeleas statement that for some reason 4.9 kernel restricts qc3 working on loire. Anyway I had given up builds and tests after jerpeleas comment but although the problem exists for all the loire users, if needed I can start to build, test and provide necessary logs.


From: harryharryharry notifications@github.com Sent: Saturday, January 12, 2019 7:21:09 PM To: sonyxperiadev/bug_tracker Cc: maluus; Author Subject: Re: [sonyxperiadev/bug_tracker] [Loire] 3 amp chargers doesn't work (#279)

Thanks. You're right, stock and even omnirom 8.1 do indeed work, although at a lower amperage (~1 to 1.5 as I recall). But omnirom 9: crickets chirping (only the led indicator goes on for about 2 seconds)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/sonyxperiadev/bug_tracker/issues/279#issuecomment-453760214, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFEchqCRI5ta5J7qqX5JFVFJ6GuTl4cBks5vCgt1gaJpZM4Zdnu3.

jerpelea commented 5 years ago

please provide logs from 8.1 and 9.0 to speedup the investigation

harryharryharry commented 5 years ago

I've saved dmesg-snippets during cable attachment to a qc3 charger from stock and omni8.

dmesg-snippet from stock (F5321 34.4.A.2.118) with genesis kernel (to get root): https://del.dog/ucikonogim

dmesg-snippet from omni8: https://del.dog/uqalaqojus

And this is a full dmesg I just took from omni9: https://del.dog/yagiqonovo

maluus commented 5 years ago

9.0 logs are in first post. You can find attached 8.0 stock rom dmesg log. I plugged in and out a few times before taking the log.


From: Alin Jerpelea notifications@github.com Sent: Sunday, January 13, 2019 8:17:29 AM To: sonyxperiadev/bug_tracker Cc: maluus; Author Subject: Re: [sonyxperiadev/bug_tracker] [Loire] 3 amp chargers doesn't work (#279)

please provide logs from 8.1 and 9.0 to speedup the investigation

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/sonyxperiadev/bug_tracker/issues/279#issuecomment-453802979, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFEchsETCfB2uftmeMTnxfG3lMphvwTAks5vCsFpgaJpZM4Zdnu3.

sjllls commented 5 years ago

This issue still exists on Suzu, phone can't charge with a QC3.0 charger at all. AOSP build on 2019.2.8 with OEM V5. @jerpelea

maluus commented 5 years ago

Lots of commits and merges lately. Is there a possibility that some of them somehow affects this issue? [Late edit] @kholk

sjllls commented 5 years ago

Lots of commits and merges lately. Is there a possibility that some of them somehow affects this issue? [Late edit] @kholk

Issue still exists on Android 9, OEM V7, built on 2019.3.23. My charger is UCH10, only support QC2.0. SuzuCharge.log

Log uploaded.

maluus commented 5 years ago

I think devs couldn't have time to look at the problem yet. Fingers crossed.

harryharryharry commented 5 years ago

@maluus, Pavel recently made a commit that fixes this issue: https://github.com/bartcubbins/kernel_loire/commit/2b5fdc84600620e8ae146a7c888f7c9ebfbe397d

My kugo now charges at 2A with a QC3 charger. Thanks @bartcubbins !

maluus commented 5 years ago

Oh at long last. I can start building again.


From: harryharryharry notifications@github.com Sent: Sunday, May 5, 2019 12:57:39 AM To: sonyxperiadev/bug_tracker Cc: maluus; Mention Subject: Re: [sonyxperiadev/bug_tracker] [Loire] 3 amp chargers doesn't work (#279)

@maluushttps://github.com/maluus, Pavel recently made a commit that fixes this issue: bartcubbins/kernel_loire@2b5fdc8https://github.com/bartcubbins/kernel_loire/commit/2b5fdc84600620e8ae146a7c888f7c9ebfbe397d

My kugo now charges at 2A with a QC3 charger. Thanks @bartcubbinshttps://github.com/bartcubbins !

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/sonyxperiadev/bug_tracker/issues/279#issuecomment-489368766, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABIRZBUBHHR4HSVS7R3FZSDPTYBFHANCNFSM4GLWPO3Q.

bartcubbins commented 5 years ago

@jerpelea fixed by https://github.com/sonyxperiadev/kernel/pull/1972

maluus commented 5 years ago

@harryharryharry, Today I synced the source and built but problem persist on my side. Did you cherry picked the commit or compiled directly from the source? I'll check it again soon.

harryharryharry commented 5 years ago

Since it was such a small modification, I dumbly added the line in the commit to the appropriate place in msm8956-loire-common.dtsi in my sources. Only syncing your sources will not be enough, since this is still just a commit from a repo from Pavel's github. So either manually add it or cherry-pick the commit from Pavel's repo.

maluus commented 5 years ago

I checked that the code is there and rebuilt but problem persists for me. Tried with both proprietary and genetic QC3 chargers. @bartcubbins, @jerpelea, @harryharryharry,

bartcubbins commented 5 years ago

just tried and can't reproduce the issue anymore

bartcubbins commented 5 years ago

please, provide me dmesg and I will take a look as soon as possible

maluus commented 5 years ago

Dmesg attached. I hope it helps. I plugged it in different chargers and pc before taking the log. (proprietary qc3 charger, other generic qc3 charger, and ipad charger and pc.)

dmesg.txt

bartcubbins commented 5 years ago

@maluus please update your kernel. you have outdated kernel. (HEAD 96e6d64d2d0c) https://github.com/sonyxperiadev/kernel/commit/96e6d64d2d0cda43e77ef5274bd1118362c1d0c2 [ 0.000000] Linux version 4.9.170-g96e6d64d2d0c-dirty

bartcubbins commented 5 years ago

@maluus looks like you use prebuilt kernel. change this to true and try once again https://github.com/sonyxperiadev/vendor-sony-kernel/blob/aosp/LE.UM.2.3.2.r1.4/KernelConfig.mk#L15

maluus commented 5 years ago

It's ok now. Problem seems solved. But I have a question. What's preventing it benefiting full QC3 charging capability?

bartcubbins commented 5 years ago

@maluus due to SoC HW limitations. @jerpelea I think this issue can be closed after updating prebuilt kernel

kholk commented 5 years ago

@bartcubbins that was great man, thanks :D

bartcubbins commented 5 years ago

@kholk 😉

maluus commented 5 years ago

I forgot to thank for all your valuable efforts @kholk @jerpelea @bartcubbins @harryharryharry