diff options
| author | Piotr Maziarz <piotr.maziarz@amd.com> | 2026-08-17 14:25:34 +0200 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2026-09-23 14:18:00 -0400 |
| commit | 184a71e58b2e98b6b273ac277687bbf186226971 (patch) | |
| tree | a5e8c61faeec2a00f93d3a2665731ffe06def7b9 /tools/perf/scripts/python/stackcollapse.py | |
| parent | 08a07e8c254c40f6c23089d259b26a3e51bfbb4f (diff) | |
| download | linux-184a71e58b2e98b6b273ac277687bbf186226971.tar.gz linux-184a71e58b2e98b6b273ac277687bbf186226971.zip | |
drm/amd/display: clamp cursor hotspot at the register write
[Why]
dcn401_build_cursor_position() clamped x_hotspot to 0xFF before
publishing *pos_out, but that field is also stored into hubp->curs_pos
and forwarded to dpp1_set_cursor_position(), which derives
src_x_offset from it. The ODM/MPC slice adjustments earlier in the same
function deliberately grow x_hotspot past 255 to keep the cursor
visible across a slice boundary, so the clamp corrupted DPP source
addressing in exactly the case it was meant to handle.
[How]
Clamp in hubp401_cursor_set_position() instead, where the 8-bit
CURSOR_HOT_SPOT_X field lives. A local is used for the register write
only; pos->x_hotspot passes through unmodified.
Reviewed-by: Alvin Lee <alvin.lee2@amd.com>
Signed-off-by: Piotr Maziarz <piotr.maziarz@amd.com>
Signed-off-by: George Zhang <george.zhang@amd.com>
Tested-by: Dan Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'tools/perf/scripts/python/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions
