AndreiLux / Perseus-UNIVERSAL5410

This is a clean kernel glued together upon Samsung's last public branch of android-exynos-3.4 with sources provided by the OSRC releases. The OSRC release has been stripped clean of all non-i9500 related code and sourced, as much as possible through the original patches.
https://github.com/AndreiLux/Perseus-UNIVERSAL5410/wiki
Other
30 stars 19 forks source link

Charge current not applied if reboot with USB cable plugged. #23

Closed sorgelig closed 10 years ago

sorgelig commented 11 years ago

It seams, SDP settings (may be other too) are not applied if reboot with USB cable plugged. After reboot max/avg/now values are all 460ma, although sdp_input_curr and sdp_chrg_curr are 900ma. After replugging the USB cable values jump to STweak defined values (in my case 900ma).

AndreiLux commented 11 years ago

The charger chip gets programmed in the boot loader and also before init at boot.

They also don't get applied when you change them live, and have to reconnect the cable.

I'm not sure I want to fix this since it gets convoluted in terms of code (telling the driver to reattach itself). I'll see what I can do if it can be managed in a clean way.

sorgelig commented 11 years ago

I think, only apply at boot is important to fix. Changing them live in STweaks is not important because usually you change it once and forget. But while developing i often need to reboot the phone. So it would be good to avoid replugging the cable after each reboot.

AndreiLux commented 10 years ago

I decided to forego this, won't be fixed or altered due to the nature of how the charger code works.

sorgelig commented 10 years ago

I've patched original table to put 900ma/1000ma as initial value for SDP. Since circuit has automatic current adjustment, it shouldn't be a problem.