From 66128d2b643a841507b27c0be121afe5ac51c39c Mon Sep 17 00:00:00 2001 From: Wayne Lin Date: Sat, 17 Feb 2024 11:08:58 +0800 Subject: [PATCH 1/4] gps: maintain one solution Bug: 315915958 Test: build pass and GPS function works Change-Id: I3bd3b09edd4e1ca1ecfa578095800c9052db5689 --- factory_tegu.mk | 4 ---- location/device-gnss.mk | 1 - 2 files changed, 5 deletions(-) diff --git a/factory_tegu.mk b/factory_tegu.mk index 993a60c..a64ef18 100644 --- a/factory_tegu.mk +++ b/factory_tegu.mk @@ -33,10 +33,6 @@ PRODUCT_MANUFACTURER := Google PRODUCT_PROPERTY_OVERRIDES += \ ro.vendor.bluetooth.evb_bdaddr="22:22:22:33:44:55" -# Override to factory SDK -$(call soong_config_set, gpssdk, sdkv1, True) -$(call soong_config_set, gpssdk, gpsmcuversion, gpsv1_$(TARGET_BUILD_VARIANT)) - # Factory binaries of camera PRODUCT_PACKAGES += fatp_tg4_wide_hat_tool diff --git a/location/device-gnss.mk b/location/device-gnss.mk index b4fb011..5083178 100644 --- a/location/device-gnss.mk +++ b/location/device-gnss.mk @@ -20,4 +20,3 @@ endif # sepolicy BOARD_VENDOR_SEPOLICY_DIRS += device/google/gs-common/gps/lsi/sepolicy -BOARD_VENDOR_SEPOLICY_DIRS += device/google/gs-common/gps/brcm/sepolicy From 3eba1a8f630cad1c9d903dd8695d58baf006a0fc Mon Sep 17 00:00:00 2001 From: Paulsu Su Date: Tue, 4 Jun 2024 00:36:26 +0000 Subject: [PATCH 2/4] powerhint: Set big core to cpu skip mask by default sync from ag/27232593 Bug: 343134499 Test: boot to home Change-Id: I4255b5e05697ee4181f8b5c68b82ea6080fb733e --- perf/powerhint-tegu.json | 37 ++++++++++++++++++++++++++++++++++++- 1 file changed, 36 insertions(+), 1 deletion(-) diff --git a/perf/powerhint-tegu.json b/perf/powerhint-tegu.json index 669b81f..8f0ee83 100644 --- a/perf/powerhint-tegu.json +++ b/perf/powerhint-tegu.json @@ -265,7 +265,6 @@ "0x0", "0x80" ], - "DefaultIndex": 0, "ResetOnInit": true }, { @@ -834,6 +833,12 @@ "Duration": 1000, "Value": "0-3" }, + { + "PowerHint": "LAUNCH", + "Node": "CPUSkipMask", + "Duration": 5000, + "Value": "0x0" + }, { "PowerHint": "LAUNCH", "Node": "ReducePreferIdle", @@ -975,6 +980,12 @@ "Duration": 5000, "Value": "1" }, + { + "PowerHint": "LAUNCH_EXTEND", + "Node": "CPUSkipMask", + "Duration": 2000, + "Value": "0x0" + }, { "PowerHint": "LAUNCH_EXTEND", "Node": "CPUBigClusterMaxFreq", @@ -1231,6 +1242,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_LAUNCH", + "Node": "CPUSkipMask", + "Duration": 1000, + "Value": "0x0" + }, { "PowerHint": "CAMERA_LAUNCH", "Node": "CPU_BIG_TSKIN_BYPASS", @@ -1807,6 +1824,12 @@ "Duration": 0, "Value": "camera" }, + { + "PowerHint": "CAMERA_STREAMING_EXTREME", + "Node": "CPUSkipMask", + "Duration": 0, + "Value": "0x0" + }, { "PowerHint": "CAMERA_STREAMING_HIGH", "Node": "PowerHALCameraRunning", @@ -2191,6 +2214,12 @@ "Duration": 0, "Value": "4-7" }, + { + "PowerHint": "CAMERA_STREAMING_LOW", + "Node": "CPUSkipMask", + "Duration": 0, + "Value": "0x0" + }, { "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", "Node": "PowerHALCameraRunning", @@ -2329,6 +2358,12 @@ "Duration": 0, "Value": "30" }, + { + "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", + "Node": "CPUSkipMask", + "Duration": 0, + "Value": "0x0" + }, { "PowerHint": "CAMERA_VIDEO_RECORDING", "Type": "MaskHint", From e3e0bcb6c93aa74eba7df74c218a5c97b1c96bc4 Mon Sep 17 00:00:00 2001 From: Paulsu Su Date: Mon, 3 Jun 2024 01:51:02 +0000 Subject: [PATCH 3/4] powerhint: Disable auto prefer idle explictly for some conditions sync from ag/26728408 Bug: 343134499 Test: boot to home Change-Id: Ic4940816966ab342c46ade27b3f381f9ad47f5ba --- perf/powerhint-tegu.json | 82 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 82 insertions(+) diff --git a/perf/powerhint-tegu.json b/perf/powerhint-tegu.json index 669b81f..430e610 100644 --- a/perf/powerhint-tegu.json +++ b/perf/powerhint-tegu.json @@ -215,6 +215,16 @@ ], "ResetOnInit": true }, + { + "Name": "AutoPreferIdle", + "Path": "/proc/vendor_sched/auto_prefer_idle", + "Values": [ + "0", + "1" + ], + "DefaultIndex": 0, + "ResetOnInit": true + }, { "Name": "CPUUtilThreshold", "Path": "/proc/vendor_sched/util_threshold", @@ -840,6 +850,12 @@ "Duration": 5000, "Value": "0" }, + { + "PowerHint": "LAUNCH", + "Node": "AutoPreferIdle", + "Duration": 5000, + "Value": "0" + }, { "PowerHint": "LAUNCH_GPU", "Node": "GPUDvfsPeriod", @@ -1053,6 +1069,12 @@ "Duration": 2000, "Value": "1" }, + { + "PowerHint": "LAUNCH_EXTEND", + "Node": "AutoPreferIdle", + "Duration": 2000, + "Value": "0" + }, { "PowerHint": "LAUNCH_PMU", "Node": "PMU_POLL", @@ -1065,6 +1087,12 @@ "Duration": 5000, "Value": "0" }, + { + "PowerHint": "DISPLAY_CHANGE", + "Node": "AutoPreferIdle", + "Duration": 5000, + "Value": "0" + }, { "PowerHint": "DISPLAY_CHANGE_GPU", "Node": "GPUDvfsPeriod", @@ -1225,6 +1253,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "DISPLAY_INACTIVE", + "Node": "AutoPreferIdle", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "DISPLAY_INACTIVE", "Node": "FGPreferIdle", @@ -1315,6 +1349,12 @@ "Duration": 1000, "Value": "1" }, + { + "PowerHint": "CAMERA_LAUNCH", + "Node": "AutoPreferIdle", + "Duration": 1000, + "Value": "0" + }, { "PowerHint": "CAMERA_LAUNCH", "Node": "FGCpuset", @@ -1477,6 +1517,12 @@ "Duration": 100, "Value": "1" }, + { + "PowerHint": "CAMERA_ZOOMING_BOOST", + "Node": "AutoPreferIdle", + "Duration": 100, + "Value": "0" + }, { "PowerHint": "CAMERA_ZOOMING_BOOST", "Node": "PMU_POLL", @@ -1771,6 +1817,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_STREAMING_EXTREME", + "Node": "AutoPreferIdle", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_EXTREME", "Node": "CDHighCpusetCpus", @@ -1921,6 +1973,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_STREAMING_HIGH", + "Node": "AutoPreferIdle", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_HIGH", "Node": "CDHighCpusetCpus", @@ -1969,6 +2027,12 @@ "Duration": 0, "Value": "0x80" }, + { + "PowerHint": "CAMERA_STREAMING_STANDARD", + "Node": "AutoPreferIdle", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_STANDARD", "Node": "PowerHALCameraRunning", @@ -2107,6 +2171,12 @@ "Duration": 0, "Value": "0x80" }, + { + "PowerHint": "CAMERA_STREAMING_LOW", + "Node": "AutoPreferIdle", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_LOW", "Node": "CPUDVFSHeadroom", @@ -2221,6 +2291,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", + "Node": "AutoPreferIdle", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", "Node": "CDPreferIdle", @@ -2678,6 +2754,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "GAME", + "Node": "AutoPreferIdle", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "GAME", "Node": "FGPreferIdle", From c9a2bf6db9429c63ebc47c1e96f01d56133d68de Mon Sep 17 00:00:00 2001 From: Paulsu Su Date: Tue, 4 Jun 2024 01:39:22 +0000 Subject: [PATCH 4/4] powerhint: Set big core to group_cfs_skip_mask for sys group sync from ag/27280353 Set big core to group_cfs_skip_mask for sys group when screen is off, so that sys group tasks will run less in big core. Bug: 342968374 Test: boot to home Change-Id: I4456641e8e99c83573887c4a3b56ba484b490773 --- perf/powerhint-tegu.json | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/perf/powerhint-tegu.json b/perf/powerhint-tegu.json index 897ef1e..7e7b1a2 100644 --- a/perf/powerhint-tegu.json +++ b/perf/powerhint-tegu.json @@ -277,6 +277,16 @@ ], "ResetOnInit": true }, + { + "Name": "SysGroupCfsSkipMask", + "Path": "/proc/vendor_sched/groups/sys/group_cfs_skip_mask", + "Values": [ + "0x0", + "0x80" + ], + "DefaultIndex": 0, + "ResetOnInit": true + }, { "Name": "BGUClampMaxBoost", "Path": "/proc/vendor_sched/groups/bg/uclamp_max", @@ -1276,6 +1286,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "DISPLAY_INACTIVE", + "Node": "SysGroupCfsSkipMask", + "Duration": 0, + "Value": "0x80" + }, { "PowerHint": "CAMERA_LAUNCH", "Node": "CPUSkipMask",