summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBhawanpreet Lakha <bhawanpreet.lakha@amd.com>2026-06-24 14:12:05 -0400
committerAlex Deucher <alexander.deucher@amd.com>2026-07-15 09:15:40 -0400
commit0eec0ebf820db3c1a160ce41c29a29f224e4ec75 (patch)
treeb633a9d5006bf01c4ab355a35e4951ee34a5149c
parent1803ad56ab8f1055eabfa584f1d4a8d1dcc57dce (diff)
downloadlinux-0eec0ebf820db3c1a160ce41c29a29f224e4ec75.tar.gz
linux-0eec0ebf820db3c1a160ce41c29a29f224e4ec75.zip
drm/amd/display: Add CACP caps tests for connector
Add KUnit coverage for amdgpu_dm_update_cacp_caps(): eDP and LVDS supported, old/3.1.6 IP versions unsupported, non-eDP/LVDS and LCD panels unsupported. Assisted-by: Copilot:Claude-Opus-4.8 Reviewed-by: Alex Hung <alex.hung@amd.com> Signed-off-by: Bhawanpreet Lakha <bhawanpreet.lakha@amd.com> Signed-off-by: George Zhang <george.zhang@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
-rw-r--r--drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_connector.h1
-rw-r--r--drivers/gpu/drm/amd/display/amdgpu_dm/tests/amdgpu_dm_connector_test.c149
2 files changed, 150 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_connector.h b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_connector.h
index 01f35bb55c93..c5f0bda5f16e 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_connector.h
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_connector.h
@@ -150,6 +150,7 @@ enum drm_mode_subconnector get_subconnector_type(struct dc_link *link);
void update_subconnector_property(struct amdgpu_dm_connector *aconnector);
void amdgpu_dm_fbc_init(struct drm_connector *connector);
void amdgpu_dm_set_panel_type(struct amdgpu_dm_connector *aconnector);
+void amdgpu_dm_update_cacp_caps(struct amdgpu_dm_connector *aconnector);
enum display_content_type
get_output_content_type(const struct drm_connector_state *connector_state);
bool adjust_colour_depth_from_display_info(struct dc_crtc_timing *timing_out,
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/tests/amdgpu_dm_connector_test.c b/drivers/gpu/drm/amd/display/amdgpu_dm/tests/amdgpu_dm_connector_test.c
index cc41efaf5298..69949f76d64f 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/tests/amdgpu_dm_connector_test.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/tests/amdgpu_dm_connector_test.c
@@ -3199,6 +3199,148 @@ static void dm_test_set_panel_type_default_lcd(struct kunit *test)
(uint64_t)DRM_MODE_PANEL_TYPE_LCD);
}
+/* Tests for amdgpu_dm_update_cacp_caps() */
+
+/*
+ * Build an amdgpu_dm_connector wired to a real kunit drm_device embedded in an
+ * amdgpu_device, so drm_to_adev() resolves and drm_dbg_kms() has a valid
+ * device. Defaults are seeded to the fully-supported configuration so each
+ * test only flips a single knob.
+ */
+struct dm_test_cacp_ctx {
+ struct amdgpu_device *adev;
+ struct amdgpu_dm_connector *aconnector;
+ struct dc_link *link;
+};
+
+static struct dm_test_cacp_ctx *dm_test_cacp_ctx_alloc(struct kunit *test)
+{
+ struct dm_test_cacp_ctx *ctx;
+ struct drm_device *drm;
+ struct device *dev;
+
+ ctx = kunit_kzalloc(test, sizeof(*ctx), GFP_KERNEL);
+ KUNIT_ASSERT_NOT_NULL(test, ctx);
+
+ dev = drm_kunit_helper_alloc_device(test);
+ KUNIT_ASSERT_NOT_ERR_OR_NULL(test, dev);
+
+ drm = __drm_kunit_helper_alloc_drm_device(test, dev,
+ sizeof(*ctx->adev),
+ offsetof(struct amdgpu_device, ddev),
+ DRIVER_MODESET);
+ KUNIT_ASSERT_NOT_ERR_OR_NULL(test, drm);
+ ctx->adev = drm_to_adev(drm);
+
+ ctx->aconnector = kunit_kzalloc(test, sizeof(*ctx->aconnector), GFP_KERNEL);
+ KUNIT_ASSERT_NOT_NULL(test, ctx->aconnector);
+ ctx->aconnector->base.dev = drm;
+
+ ctx->link = kunit_kzalloc(test, sizeof(*ctx->link), GFP_KERNEL);
+ KUNIT_ASSERT_NOT_NULL(test, ctx->link);
+ ctx->aconnector->dc_link = ctx->link;
+
+ /* Fully-supported defaults: new enough DCE, eDP, non-LCD panel. */
+ ctx->adev->ip_versions[DCE_HWIP][0] = IP_VERSION(3, 1, 4);
+ ctx->link->connector_signal = SIGNAL_TYPE_EDP;
+ ctx->link->panel_type = PANEL_TYPE_OLED;
+
+ return ctx;
+}
+
+/**
+ * dm_test_cacp_caps_edp_supported - Test CACP supported on a new eDP panel
+ * @test: The KUnit test context
+ */
+static void dm_test_cacp_caps_edp_supported(struct kunit *test)
+{
+ struct dm_test_cacp_ctx *ctx = dm_test_cacp_ctx_alloc(test);
+
+ amdgpu_dm_update_cacp_caps(ctx->aconnector);
+
+ KUNIT_EXPECT_TRUE(test, ctx->link->panel_config.cacp.cacp_supported);
+}
+
+/**
+ * dm_test_cacp_caps_lvds_supported - Test CACP supported on an LVDS panel
+ * @test: The KUnit test context
+ */
+static void dm_test_cacp_caps_lvds_supported(struct kunit *test)
+{
+ struct dm_test_cacp_ctx *ctx = dm_test_cacp_ctx_alloc(test);
+
+ ctx->link->connector_signal = SIGNAL_TYPE_LVDS;
+
+ amdgpu_dm_update_cacp_caps(ctx->aconnector);
+
+ KUNIT_EXPECT_TRUE(test, ctx->link->panel_config.cacp.cacp_supported);
+}
+
+/**
+ * dm_test_cacp_caps_old_ip_unsupported - Test CACP unsupported on old DCE
+ * @test: The KUnit test context
+ *
+ * DCE versions older than 3.1.4 do not support CACP.
+ */
+static void dm_test_cacp_caps_old_ip_unsupported(struct kunit *test)
+{
+ struct dm_test_cacp_ctx *ctx = dm_test_cacp_ctx_alloc(test);
+
+ ctx->adev->ip_versions[DCE_HWIP][0] = IP_VERSION(3, 1, 3);
+
+ amdgpu_dm_update_cacp_caps(ctx->aconnector);
+
+ KUNIT_EXPECT_FALSE(test, ctx->link->panel_config.cacp.cacp_supported);
+}
+
+/**
+ * dm_test_cacp_caps_ip_3_1_6_unsupported - Test CACP unsupported on DCE 3.1.6
+ * @test: The KUnit test context
+ *
+ * DCE 3.1.6 is explicitly excluded even though it is newer than 3.1.4.
+ */
+static void dm_test_cacp_caps_ip_3_1_6_unsupported(struct kunit *test)
+{
+ struct dm_test_cacp_ctx *ctx = dm_test_cacp_ctx_alloc(test);
+
+ ctx->adev->ip_versions[DCE_HWIP][0] = IP_VERSION(3, 1, 6);
+
+ amdgpu_dm_update_cacp_caps(ctx->aconnector);
+
+ KUNIT_EXPECT_FALSE(test, ctx->link->panel_config.cacp.cacp_supported);
+}
+
+/**
+ * dm_test_cacp_caps_non_edp_lvds_unsupported - Test CACP unsupported on a
+ * non-eDP/LVDS signal
+ * @test: The KUnit test context
+ */
+static void dm_test_cacp_caps_non_edp_lvds_unsupported(struct kunit *test)
+{
+ struct dm_test_cacp_ctx *ctx = dm_test_cacp_ctx_alloc(test);
+
+ ctx->link->connector_signal = SIGNAL_TYPE_DISPLAY_PORT;
+
+ amdgpu_dm_update_cacp_caps(ctx->aconnector);
+
+ KUNIT_EXPECT_FALSE(test, ctx->link->panel_config.cacp.cacp_supported);
+}
+
+/**
+ * dm_test_cacp_caps_lcd_unsupported - Test CACP unsupported on an LCD panel
+ * @test: The KUnit test context
+ */
+static void dm_test_cacp_caps_lcd_unsupported(struct kunit *test)
+{
+ struct dm_test_cacp_ctx *ctx = dm_test_cacp_ctx_alloc(test);
+
+ ctx->link->panel_type = PANEL_TYPE_LCD;
+
+ amdgpu_dm_update_cacp_caps(ctx->aconnector);
+
+ KUNIT_EXPECT_FALSE(test, ctx->link->panel_config.cacp.cacp_supported);
+}
+
static struct kunit_case amdgpu_dm_connector_tests[] = {
/* get_subconnector_type */
KUNIT_CASE(dm_test_subconnector_type_none),
@@ -3376,6 +3518,13 @@ static struct kunit_case amdgpu_dm_connector_tests[] = {
KUNIT_CASE(dm_test_set_panel_type_samsung_miniled),
KUNIT_CASE(dm_test_set_panel_type_samsung_below_threshold),
KUNIT_CASE(dm_test_set_panel_type_default_lcd),
+ /* amdgpu_dm_update_cacp_caps */
+ KUNIT_CASE(dm_test_cacp_caps_edp_supported),
+ KUNIT_CASE(dm_test_cacp_caps_lvds_supported),
+ KUNIT_CASE(dm_test_cacp_caps_old_ip_unsupported),
+ KUNIT_CASE(dm_test_cacp_caps_ip_3_1_6_unsupported),
+ KUNIT_CASE(dm_test_cacp_caps_non_edp_lvds_unsupported),
+ KUNIT_CASE(dm_test_cacp_caps_lcd_unsupported),
{}
};