Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improving overall system configurations for thermal-engine and audio sample rate #1

Open
wants to merge 17 commits into
base: cm-14.1
Choose a base branch
from

Conversation

LuizPauloDamaceno
Copy link

This will enable HiFi audio samplerate (192kHz for speakers and headphones) and improve thermal-engine for l36 variant (i think can work without problems in another variants too).
This also add more sensors to thermal-engine improve safety of system and higher temperature ranges to improve overall system performance.

@skritchz
Copy link
Contributor

I'll cherry pick some of your commits. But I won't take the brazilian GPS configuration. More other it uses the IP address directly which isn't good.

Now regarding the audio_output_policy where did you find this file? I actually didn't have time to take a look if it changes anything on my device or not.

Could you point out directly which lines in the thermal are required for the brightness issue?

@LuizPauloDamaceno
Copy link
Author

The line for brightness issue on thermal-engine is [LCD_LED_MONITOR], replacing [LCD_SKIN_MANAGEMENT] , but another lines can give some extra performance reducing throttling. The audio_output_policy i finded some reference for this file at msm8916 audio HAL in CM source, but most phones haven't this file... This selects the "app type" on audio policy manager and sets the output sample rate according the app configuration, so you can set 8kHz to 192kHz in this phone without problems, but i having a bug at whatsapp calls, after calls the audio route bugs and i need to reboot phone to sound return... I find this when i'm searching for strange boot noise on speakers at boot on l36 variant... i think is a routing problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants