From f8f72e99d932131eef0b3fe332952e3c221bf222 Mon Sep 17 00:00:00 2001 From: Peter Liu Date: Wed, 19 May 2021 15:13:06 -0700 Subject: [PATCH] camera power hint: add prefer high cap in all modes Prefer high cap will choose the mid / high cores when possible with consideration of energy efficiency. Running task on high cores with low clock is more efficient than running at little core with high clock. This hint also addresses CTS issues with keeping latency_sensitive = 1. (equivalent to prefer idle) This hint adds: 1) Enable prefer_high_cap of camera vendor group 2) Set high_capacity_start_cpu to 4 4) Set MemFreq floor 5) Change camera-daemon/cpu.uclamp.latency_sensitive owner to root (cherry-pick from device/google/gs101) Bug: 205010238 Bug: 186564130 Test: android.hardware.camera2.cts.RobustnessTest#testAeAndAfCausality (5/5) android.hardware.cts.CameraTest#testPreviewCallback (5/5) android.hardware.camera2.cts.CameraDeviceTest#testCameraDeviceRepeatingBurst (5/5) android.hardware.cts.CameraTest#testFocusDistances (3/6). (3/6) without CL. ***************************************************************** raven Camera CTS Test Results ***************************************************************** Tests Passed: 217, Failed: 0, Not found: 0, Status unknown: 0 Change-Id: Ia2abb0eb9c57fa05c4db7bf1d02053e303301781 --- conf/init.gs201.rc | 2 +- powerhint.json | 6 ------ 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/conf/init.gs201.rc b/conf/init.gs201.rc index d36fcec8..670346d0 100644 --- a/conf/init.gs201.rc +++ b/conf/init.gs201.rc @@ -127,7 +127,7 @@ on init # change permissions and default values for camera-daemon cpu controller chown system system /dev/cpuctl/camera-daemon/cpu.uclamp.min chown system system /dev/cpuctl/camera-daemon/cpu.uclamp.max - chown system system /dev/cpuctl/camera-daemon/cpu.uclamp.latency_sensitive + chown root root /dev/cpuctl/camera-daemon/cpu.uclamp.latency_sensitive chown system system /dev/cpuctl/camera-daemon/cgroup.procs write /dev/cpuctl/camera-daemon/cpu.uclamp.latency_sensitive 1 diff --git a/powerhint.json b/powerhint.json index b1795670..ac0500a7 100644 --- a/powerhint.json +++ b/powerhint.json @@ -391,12 +391,6 @@ "Duration": 0, "Value": "4" }, - { - "PowerHint": "CAMERA_STREAMING_STANDARD", - "Node": "CDPreferIdle", - "Duration": 0, - "Value": "0" - }, { "PowerHint": "CAMERA_STREAMING_STANDARD", "Node": "CPUBigClusterMaxFreq",