github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-10-30 14:06:03 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

drm/amd/display: Test timing sync trigger

Browse Source 
[WHAT]
Add KUnit tests for amdgpu_dm_trigger_timing_sync() covering a DC without
a current state and the propagation of the force flag to every stream, and
for dm_acpi_process_phy_transition_interlock().

Assisted-by: Copilot:Claude-Opus-5 GPT-5.6-Sol
Reviewed-by: Bhawanpreet Lakha <bhawanpreet.lakha@amd.com>
Signed-off-by: Alex Hung <alex.hung@amd.com>
Tested-by: Dan Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
Alex Hung 2026-08-13 14:07:06 -06:00 committed by Alex Deucher
parent 9c17c2918f
commit 68c96a564e
2 changed files with 61 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
View File

@ -6627,6 +6627,7 @@ void amdgpu_dm_trigger_timing_sync(struct drm_device *dev)
}
mutex_unlock(&adev->dm.dc_lock);
}
EXPORT_IF_KUNIT(amdgpu_dm_trigger_timing_sync);
void dm_write_reg_func(const struct dc_context *ctx, uint32_t address,
u32 value, const char *func_name)
@ -6671,3 +6672,4 @@ void dm_acpi_process_phy_transition_interlock(
{
// Not yet implemented
}
EXPORT_IF_KUNIT(dm_acpi_process_phy_transition_interlock);

59
drivers/gpu/drm/amd/display/amdgpu_dm/tests/amdgpu_dm_test.c
View File

@ -3040,6 +3040,60 @@ static void dm_test_aquire_global_lock_waits_commit(struct kunit *test)
KUNIT_EXPECT_EQ(test, ret, 0);
}
/* Tests for amdgpu_dm_trigger_timing_sync() */
/**
* dm_test_trigger_timing_sync_no_state - Test no DC state leaves the sync untouched
* @test: The KUnit test context
*/
static void dm_test_trigger_timing_sync_no_state(struct kunit *test)
{
struct amdgpu_device *adev = dm_kunit_alloc_adev(test);
adev->dm.dc = dm_kunit_alloc_dc_with_ctx(test);
mutex_init(&adev->dm.dc_lock);
amdgpu_dm_trigger_timing_sync(&adev->ddev);
}
/**
* dm_test_trigger_timing_sync_streams - Test the force flag reaches every stream
* @test: The KUnit test context
*/
static void dm_test_trigger_timing_sync_streams(struct kunit *test)
{
struct amdgpu_device *adev = dm_kunit_alloc_adev(test);
struct dc *dc = dm_kunit_alloc_dc_with_ctx(test);
struct dc_stream_state *stream;
struct dc_state *context;
context = dm_kunit_alloc_dc_state(test);
KUNIT_ASSERT_NOT_NULL(test, context);
stream = dm_kunit_alloc_stream(test, NULL);
context->streams[0] = stream;
context->stream_count = 1;
dc->current_state = context;
adev->dm.dc = dc;
adev->dm.force_timing_sync = true;
mutex_init(&adev->dm.dc_lock);
amdgpu_dm_trigger_timing_sync(&adev->ddev);
KUNIT_EXPECT_TRUE(test, stream->triggered_crtc_reset.enabled);
}
/**
* dm_test_acpi_phy_transition_interlock - Test the PHY transition interlock stub
* @test: The KUnit test context
*/
static void dm_test_acpi_phy_transition_interlock(struct kunit *test)
{
struct dm_process_phy_transition_init_params params = { 0 };
dm_acpi_process_phy_transition_interlock(NULL, params);
}
static struct kunit_case amdgpu_dm_tests[] = {
/* Simple DM callbacks */
KUNIT_CASE(dm_test_wait_for_idle),
@ -3189,6 +3243,11 @@ static struct kunit_case amdgpu_dm_tests[] = {
KUNIT_CASE(dm_test_aquire_global_lock_no_crtc),
KUNIT_CASE(dm_test_aquire_global_lock_no_commit),
KUNIT_CASE(dm_test_aquire_global_lock_waits_commit),
/* amdgpu_dm_trigger_timing_sync */
KUNIT_CASE(dm_test_trigger_timing_sync_no_state),
KUNIT_CASE(dm_test_trigger_timing_sync_streams),
/* dm_acpi_process_phy_transition_interlock */
KUNIT_CASE(dm_test_acpi_phy_transition_interlock),
{}
};