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

Up to date as of Sept 9, 2016 #2

Open
wants to merge 28 commits into
base: mm6.0
Choose a base branch
from
Open

Up to date as of Sept 9, 2016 #2

wants to merge 28 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Sep 9, 2016

Just an update of commits from Slim

Steve Kondik and others added 28 commits September 9, 2016 16:18
Change-Id: I4131ea36afb8d7f9ca9376fa564b9f961083ecaf
 * Change migration settings to reduce use of big cores when screen
   is off.

Change-Id: I9ff2404518d0a47e0f7957ea1a1b71f140160bb4
 * Dynamically load the vendor library to kick off prefetching.

Change-Id: Ia51144c06e6976fddaa8aebbfe457b077a48c5e6
This difference has been here since the initial commit of this
code, so the side effects of it are unknown or not observable.

Change-Id: I4f1ef2a27face628550fa8498242a002846a7896
The array has 7 elements, but is specified as 5.  There's no need
to specify a statically sized array like this, so remove the
size from the declaration.

Change-Id: I4909526738f8a23167f98a888e7e32bd07133df4
This helps with code clarity.

Change-Id: I15ddf0b2e22cf88cb0eb4e088f5416df31fb0aef
 * This provides no improvement, unless one considers high
   power consumption to be an improvement :)

Change-Id: Ic992a5b1c5a1cd0730130b94305171ed1787d4d0
* Don't attempt to return a value here.

Change-Id: I06fbcdf4258fe525e0f955ca2b6262d9e09e8ecc
 * This change is based on the following commits:

From 2d717c8 Mon Sep 17 00:00:00 2001
From: Steve Kondik <[email protected]>
Date: Thu, 5 May 2016 15:24:08 -0700
Subject: [PATCH] power-8992: Fix boost and screen off params

 * Change migration settings to reduce use of big cores when screen
   is off.

Change-Id: I9ff2404518d0a47e0f7957ea1a1b71f140160bb4

From 571464a Mon Sep 17 00:00:00 2001
From: Steve Kondik <[email protected]>
Date: Fri, 6 May 2016 04:33:11 -0700
Subject: [PATCH] power-8992: Add prefetcher support

 * Dynamically load the vendor library to kick off prefetching.

Change-Id: Ia51144c06e6976fddaa8aebbfe457b077a48c5e6

From 97aa119 Mon Sep 17 00:00:00 2001
From: Steve Kondik <[email protected]>
Date: Thu, 9 Jun 2016 03:01:07 -0700
Subject: [PATCH] power-8992: Remove extraneous boosting

 * This provides no improvement, unless one considers high
   power consumption to be an improvement :)

Change-Id: Ic992a5b1c5a1cd0730130b94305171ed1787d4d0

Change-Id: Id9ed359207a1e7e1d01dab4c66f076b4eb7031fa
As previous_boost_time was not static, its value gets resetted to 0
everytime power_hint_override() is called. As a result, the following
statement will always result in elapsed_time == cur_boost_time:

	elapsed_time = (double)(cur_boost_time - previous_boost_time);

This in turn causes the following check to always return false:

	if (elapsed_time < 250000 && duration <= 750)


Change-Id: Idcc22a6f8f10c989ca74a385d3d240c49ef94c2c
Change-Id: I010d4d1d3fb495e523b065c11ca95c4305530f9f
 * This breaks our internal builds because of duplicate opcodes.
   Ifdef them for 8996.

Change-Id: Ia3aaab5c24e36e8b3abafc97c75efc1e04c65345
powerhal is enabled for Feero

Change-Id: Ice606eb34ec175f17c69d86961889cad96c53a5f
 * The underlying implementation assumes that the perfd client
   is running in-process, but in our architecture we are not.
   This results in system_server being colocated, which is not
   what we want.
 * Disable this feature until we move interaction boosts to
   app space.

Change-Id: I0f899c17e0a16b3e3491d7b8314cfdf042628abb
The common macro was introduced in the commit:
	8fe35cc

So use it.

Change-Id: Ie47d0e6b91b7bba378b0fdf53a345a021b0c2d48
This opcode is obsolete as perflock doesn't actually handle it.
It was originally meant for 8974, but was replaced later on.

The exact same opcode was removed for the interactive governor
a while back, so remove it for the ondemand governor as well.

Change-Id: I7bc5d6f28a827d3e8c83acde1319577db3d06ea6
* There is no need to build it in CM

Change-Id: I42266f5e114031e8f0be1be455c146250b8de3c2
 * Bring in Qualcomm's upstream tools for image creation.
 * These are needed to properly write out firmware images on
   many new devices, particularly devices which support
   backup/failsafe partitions.
 * This also supports encrypted files, and TZ-side key
   descruction when wiping.

Change-Id: I4e4eaccf1f647bbd316f6bb1fa34425b4c3e411f
Change Ia3aaab5c24e36e8b3abafc97c75efc1e04c65345 limited this version
to just 8996. Expand it.

Change-Id: I1af9241fbefbc25472874619d5cd3378053b02d3
37 is all MPCTL V3. Also, add the efficiency and quick profiles.

Change-Id: I8b955f6076ac2a0898e1638857610eaba451e57d
Change-Id: I56ed97396b2e6f952780591d8409dbda15b7fbe8
When acquiring the perflock, we were only passing the first pair of
arguments out of every profile. Because...
sizeof(*)/sizeof(int) is always 2 on a 64bit arch. There's no
need to put the profile into a pointer anyway, just pass it directly to
ARRAY_SIZE (and the lock)

Change-Id: I25f2a9b059290c3a0b36fc8dc29c711da938430e
Change-Id: Icaabb51d3d72ede9a77f96bbc0fe493aa979e483
 - Get soc_id in a common util function
 - Return boolean values for the target-specific soc_id checks

Change-Id: I038c435d28855859f36566de7acf881037d070f2
otherwise you get trailing garbage when the file does already exist.

Change-Id: I020120d0742413f36ea63f37bf78e3e8dc2b8bb3
Check for whether the display hint has been sent at the start
of the set_interactive() function in the common powerHAL.

This gets rid of the need to replicate the same variable in
every target-specific powerHAL that implements the
set_interactive_override() function.

Also remove the unnecessary saved_interactive_mode variable
used by the msm-dcvs governor; its function is superseded by
the variable display_hint_sent.

Change-Id: If7dd11fcb578211f4f83847f9257232c4138ce53
Change-Id: Ide973529817767d707a47722e83d3ef40922f528
Signed-off-by: Paul Keith <[email protected]>
Since is_target_8916 doesn't return "8916" anymore every device
would be stuck at 960MHz as minimum frequency without this.

Change-Id: I3c33798f03e2c2dec11e5283e09b6c0cae8af67d
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.

9 participants