LenovoZ5s/device_lenovo_sdm710-common
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

sdm710-common: powerhint: set /dev/cpu_dma_latency value in hex

Browse source 
/dev/cpu_dma_latency accepts values in either s32 or hex format,
so convert 67 (dec) to 43 (hex).

sdm845-common comment:

I assume Google meant to set cpu dma latency to 44 in decimal
rather than hex because 44 is the lowest valid latency value for
C3 LPM state, according to qcom,latency-us in sdm845-pm.dtsi.

The pm qos request is only active when the fd is kept open, so
the dummy value of 100 does not really matter. Keep it as is.

Test: hexdump -C /dev/cpu_dma_latency

00000000  2c 00 00 00                                       |,...|
00000004

Test: kernel pr_err() in pm_qos_power_write() before
      pm_qos_update_request()

Inspired by Artem's commit "qos: Workaround for faulty libperfmgr
config".

Signed-off-by: Chenyang Zhong <zhongcy95@gmail.com>
This commit is contained in:
jjpprrrr 2021-08-19 00:45:23 +02:00 committed by Giammarco Senatore
parent b799885863
commit 13d3363ece
No known key found for this signature in database
GPG key ID: 661348FC1E144F04
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
configs/powerhint.json
View file

@ -179,7 +179,7 @@
"Name": "PMQoSCpuDmaLatency", "Name": "PMQoSCpuDmaLatency",
"Path": "/dev/cpu_dma_latency", "Path": "/dev/cpu_dma_latency",
"Values": [ "Values": [
"67", "43",
"100" "100"
], ],
"HoldFd": true "HoldFd": true
@ -322,7 +322,7 @@
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
"Node": "PMQoSCpuDmaLatency", "Node": "PMQoSCpuDmaLatency",
"Duration": 5000, "Duration": 5000,
"Value": "67" "Value": "43"
}, },
{ {
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
@ -412,7 +412,7 @@
"PowerHint": "CAMERA_LAUNCH", "PowerHint": "CAMERA_LAUNCH",
"Node": "PMQoSCpuDmaLatency", "Node": "PMQoSCpuDmaLatency",
"Duration": 1000, "Duration": 1000,
"Value": "67" "Value": "43"
}, },
{ {
"PowerHint": "CAMERA_STREAMING_MID", "PowerHint": "CAMERA_STREAMING_MID",
@ -472,7 +472,7 @@
"PowerHint": "CAMERA_SHOT", "PowerHint": "CAMERA_SHOT",
"Node": "PMQoSCpuDmaLatency", "Node": "PMQoSCpuDmaLatency",
"Duration": 1000, "Duration": 1000,
"Value": "67" "Value": "43"
}, },
{ {
"PowerHint": "AUDIO_LAUNCH", "PowerHint": "AUDIO_LAUNCH",
@ -484,7 +484,7 @@
"PowerHint": "AUDIO_LAUNCH", "PowerHint": "AUDIO_LAUNCH",
"Node": "PMQoSCpuDmaLatency", "Node": "PMQoSCpuDmaLatency",
"Duration": 2000, "Duration": 2000,
"Value": "67" "Value": "43"
}, },
{ {
"PowerHint": "AUDIO_STREAMING_LOW_LATENCY", "PowerHint": "AUDIO_STREAMING_LOW_LATENCY",
@ -496,7 +496,7 @@
"PowerHint": "AUDIO_STREAMING_LOW_LATENCY", "PowerHint": "AUDIO_STREAMING_LOW_LATENCY",
"Node": "PMQoSCpuDmaLatency", "Node": "PMQoSCpuDmaLatency",
"Duration": 0, "Duration": 0,
"Value": "67" "Value": "43"
} }
] ]
} }