From nobody Thu Mar 19 05:39:39 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B53923A0B05; Thu, 5 Mar 2026 14:23:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772720598; cv=pass; b=jJCBRvoeyL0IRQh2fku6m+20C1KVzkPGnLwWcBEpDvQrsvp8qBRcCwZgeC7fgaD2CfyYV7r6J2ov6j3zzv9nY4knpwynHGXoZ0NkLibl+NAMZc3M94+MPuI327l9LHH0nnHqLLRNcMNFOj/eoBnY9ZOVnc24Newd6MTYd1Huwko= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772720598; c=relaxed/simple; bh=xms/moE1FMnV9/WQktJD+1w5+JbxTzyX1JkbNfQrUmQ=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=FBRxcbul8h4EQj108Xi3y/Kyi9OSQ7BdcLNfS6Vrb3hFnC0I3+BW99RqYulvgZHwdwTjBnguWBpLbi2B21P0fm7kOiEn9Q/KCORb44dwx5sqkLvqREdOLBq4DpOeqc7UE8SzzFRDxSF9xzSu45owKtGK1SNzRfqrQPtlbGs9FsA= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b=OA3/XvKO; arc=pass smtp.client-ip=136.143.188.112 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b="OA3/XvKO" ARC-Seal: i=1; a=rsa-sha256; t=1772720557; cv=none; d=zohomail.com; s=zohoarc; b=Rl/kVblmAvNf8r6BUPWmdLPhETQBogk1TVqlDsCVrOxaC1ZbUEP5nrXDLLgEELwYU1m7vC5BJ596L3Lzv2/HfTV9yD80FP+GD7oejIYELizcbFrCiy3scIHoUu+riVYXqrgTsfhBp9EaK8AP3/KbZgc64x7aK1gm+qBSRqQjNhs= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1772720557; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=fEV7xNtzJsiw1ZfX4EeEfrQwzyPEzW68d0qjHZY0TNc=; b=UtS6JQ+zSlR2s5uONMWvj6JgppgcZNv/ql3zZacgLpQhmRHIXKgxFxTNLj54dlhdqDMd5XwCfHx7q4598jV0QGshI+qUYlwHFp05yyDi/LtcpI+Vvc+MjXehto2QEYtruLaBkjqH2ooN0N9TMsHiHcm+Ssgz6Co5CX6bDyUVtYU= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=nicolas.frattaroli@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1772720557; s=zohomail; d=collabora.com; i=nicolas.frattaroli@collabora.com; h=From:From:Date:Date:Subject:Subject:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Message-Id:References:In-Reply-To:To:To:Cc:Cc:Reply-To; bh=fEV7xNtzJsiw1ZfX4EeEfrQwzyPEzW68d0qjHZY0TNc=; b=OA3/XvKO4XgFFZcrsOA3RnZ9dLio6a4m7BeyJ1F3ijQ9jZJn60Mo6lcDEEUu8W/u /OfgnxcOlygqvaAwI8a/Gwgz54giagwEW1WE8EVSoVGK11unEz9xeblAWHRuhvxve1t 0S5gMX41SCX+FeIwE16eQ8bMv6ihbmAtdFc2GZ/E= Received: by mx.zohomail.com with SMTPS id 1772720556915201.3094565400827; Thu, 5 Mar 2026 06:22:36 -0800 (PST) From: Nicolas Frattaroli Date: Thu, 05 Mar 2026 15:19:47 +0100 Subject: [PATCH v10 21/22] drm/tests: bridge: Add test for HDMI output bus formats helper Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260305-color-format-v10-21-a58c68a11868@collabora.com> References: <20260305-color-format-v10-0-a58c68a11868@collabora.com> In-Reply-To: <20260305-color-format-v10-0-a58c68a11868@collabora.com> To: Harry Wentland , Leo Li , Rodrigo Siqueira , Alex Deucher , =?utf-8?q?Christian_K=C3=B6nig?= , David Airlie , Simona Vetter , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Sandy Huang , =?utf-8?q?Heiko_St=C3=BCbner?= , Andy Yan , Jani Nikula , Rodrigo Vivi , Joonas Lahtinen , Tvrtko Ursulin , Dmitry Baryshkov , Sascha Hauer , Rob Herring , Jonathan Corbet , Shuah Khan Cc: kernel@collabora.com, amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, linux-doc@vger.kernel.org, Nicolas Frattaroli X-Mailer: b4 0.14.3 The common atomic_get_output_bus_fmts helper for HDMI bridge connectors, called drm_atomic_helper_bridge_get_hdmi_output_bus_fmts, should return an array of output bus formats depending on the supported formats of the connector, and the current output BPC. Add a test to exercise some of this helper. Signed-off-by: Nicolas Frattaroli Reviewed-by: Maxime Ripard --- drivers/gpu/drm/tests/drm_bridge_test.c | 184 ++++++++++++++++++++++++++++= ++++ 1 file changed, 184 insertions(+) diff --git a/drivers/gpu/drm/tests/drm_bridge_test.c b/drivers/gpu/drm/test= s/drm_bridge_test.c index cb821c606070..d9bd930b1197 100644 --- a/drivers/gpu/drm/tests/drm_bridge_test.c +++ b/drivers/gpu/drm/tests/drm_bridge_test.c @@ -5,6 +5,7 @@ #include #include =20 +#include #include #include #include @@ -118,6 +119,28 @@ static const struct drm_bridge_funcs drm_test_bridge_a= tomic_funcs =3D { .atomic_reset =3D drm_atomic_helper_bridge_reset, }; =20 +static int dummy_clear_infoframe(struct drm_bridge *bridge) +{ + return 0; +} + +static int dummy_write_infoframe(struct drm_bridge *bridge, const u8 *buff= er, + size_t len) +{ + return 0; +} + +static const struct drm_bridge_funcs drm_test_bridge_bus_fmts_funcs =3D { + .atomic_get_output_bus_fmts =3D drm_atomic_helper_bridge_get_hdmi_output_= bus_fmts, + .atomic_destroy_state =3D drm_atomic_helper_bridge_destroy_state, + .atomic_duplicate_state =3D drm_atomic_helper_bridge_duplicate_state, + .atomic_reset =3D drm_atomic_helper_bridge_reset, + .hdmi_write_avi_infoframe =3D dummy_write_infoframe, + .hdmi_write_hdmi_infoframe =3D dummy_write_infoframe, + .hdmi_clear_avi_infoframe =3D dummy_clear_infoframe, + .hdmi_clear_hdmi_infoframe =3D dummy_clear_infoframe, +}; + /** * struct fmt_tuple - a tuple of input/output MEDIA_BUS_FMT_* */ @@ -539,6 +562,83 @@ drm_test_bridge_chain_init(struct kunit *test, unsigne= d int num_bridges, return priv; } =20 +static struct drm_bridge_init_priv * +drm_test_bridge_hdmi_init(struct kunit *test, const struct drm_bridge_func= s *funcs, + unsigned int supported_formats, int max_bpc) +{ + struct drm_bridge_init_priv *priv; + struct drm_encoder *enc; + struct drm_bridge *bridge; + struct drm_device *drm; + struct device *dev; + int ret; + + dev =3D drm_kunit_helper_alloc_device(test); + if (IS_ERR(dev)) + return ERR_CAST(dev); + + priv =3D drm_kunit_helper_alloc_drm_device(test, dev, + struct drm_bridge_init_priv, drm, + DRIVER_MODESET | DRIVER_ATOMIC); + if (IS_ERR(priv)) + return ERR_CAST(priv); + + priv->test_bridge =3D devm_drm_bridge_alloc(dev, struct drm_bridge_priv, = bridge, funcs); + if (IS_ERR(priv->test_bridge)) + return ERR_CAST(priv->test_bridge); + + priv->test_bridge->data =3D priv; + + drm =3D &priv->drm; + priv->plane =3D drm_kunit_helper_create_primary_plane(test, drm, + NULL, + NULL, + NULL, 0, + NULL); + if (IS_ERR(priv->plane)) + return ERR_CAST(priv->plane); + + priv->crtc =3D drm_kunit_helper_create_crtc(test, drm, + priv->plane, NULL, + NULL, + NULL); + if (IS_ERR(priv->crtc)) + return ERR_CAST(priv->crtc); + + enc =3D &priv->encoder; + ret =3D drmm_encoder_init(drm, enc, NULL, DRM_MODE_ENCODER_TMDS, NULL); + if (ret) + return ERR_PTR(ret); + + enc->possible_crtcs =3D drm_crtc_mask(priv->crtc); + + bridge =3D &priv->test_bridge->bridge; + bridge->type =3D DRM_MODE_CONNECTOR_HDMIA; + bridge->supported_formats =3D supported_formats; + bridge->max_bpc =3D max_bpc; + bridge->ops |=3D DRM_BRIDGE_OP_HDMI; + bridge->vendor =3D "LNX"; + bridge->product =3D "KUnit"; + + ret =3D drm_kunit_bridge_add(test, bridge); + if (ret) + return ERR_PTR(ret); + + ret =3D drm_bridge_attach(enc, bridge, NULL, 0); + if (ret) + return ERR_PTR(ret); + + priv->connector =3D drm_bridge_connector_init(drm, enc); + if (IS_ERR(priv->connector)) + return ERR_CAST(priv->connector); + + drm_connector_attach_encoder(priv->connector, enc); + + drm_mode_config_reset(drm); + + return priv; +} + /* * Test that drm_bridge_get_current_state() returns the last committed * state for an atomic bridge. @@ -786,10 +886,94 @@ static void drm_test_drm_bridge_helper_reset_crtc_leg= acy(struct kunit *test) KUNIT_EXPECT_EQ(test, bridge_priv->disable_count, 1); } =20 +/* + * Test that a bridge using the drm_atomic_helper_bridge_get_hdmi_output_b= us_fmts() + * function for &drm_bridge_funcs.atomic_get_output_bus_fmts behaves as ex= pected + * for an HDMI connector bridge. Does so by creating an HDMI bridge connec= tor + * with RGB444, YCBCR444, and YCBCR420 (but not YCBCR422) as supported for= mats, + * sets the output depth to 8 bits per component, and then validates the r= eturned + * list of bus formats. + */ +static void drm_test_drm_bridge_helper_hdmi_output_bus_fmts(struct kunit *= test) +{ + struct drm_connector_state *conn_state; + struct drm_bridge_state *bridge_state; + struct drm_modeset_acquire_ctx ctx; + struct drm_bridge_init_priv *priv; + struct drm_crtc_state *crtc_state; + struct drm_atomic_state *state; + struct drm_display_mode *mode; + unsigned int num_output_fmts; + struct drm_bridge *bridge; + u32 *out_bus_fmts; + int ret; + + priv =3D drm_test_bridge_hdmi_init(test, &drm_test_bridge_bus_fmts_funcs, + BIT(DRM_OUTPUT_COLOR_FORMAT_RGB444) | + BIT(DRM_OUTPUT_COLOR_FORMAT_YCBCR444) | + BIT(DRM_OUTPUT_COLOR_FORMAT_YCBCR420), + 12); + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv); + + bridge =3D &priv->test_bridge->bridge; + + drm_modeset_acquire_init(&ctx, 0); + + state =3D drm_kunit_helper_atomic_state_alloc(test, &priv->drm, &ctx); + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state); + +retry_commit: + conn_state =3D drm_atomic_get_connector_state(state, priv->connector); + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, conn_state); + + conn_state->hdmi.output_bpc =3D 8; + + mode =3D drm_kunit_display_mode_from_cea_vic(test, &priv->drm, 16); + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, mode); + + ret =3D drm_atomic_set_crtc_for_connector(conn_state, priv->crtc); + if (ret =3D=3D -EDEADLK) { + drm_modeset_backoff(&ctx); + goto retry_commit; + } + KUNIT_ASSERT_EQ(test, ret, 0); + + crtc_state =3D drm_atomic_get_crtc_state(state, priv->crtc); + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, crtc_state); + + ret =3D drm_atomic_set_mode_for_crtc(crtc_state, mode); + if (ret =3D=3D -EDEADLK) { + drm_modeset_backoff(&ctx); + goto retry_commit; + } + KUNIT_ASSERT_EQ(test, ret, 0); + + crtc_state->enable =3D true; + crtc_state->active =3D true; + + bridge_state =3D drm_atomic_get_bridge_state(state, bridge); + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, bridge_state); + + out_bus_fmts =3D bridge->funcs->atomic_get_output_bus_fmts( + bridge, bridge_state, crtc_state, conn_state, &num_output_fmts); + KUNIT_EXPECT_NOT_NULL(test, out_bus_fmts); + KUNIT_EXPECT_EQ(test, num_output_fmts, 3); + + KUNIT_EXPECT_EQ(test, out_bus_fmts[0], MEDIA_BUS_FMT_RGB888_1X24); + KUNIT_EXPECT_EQ(test, out_bus_fmts[1], MEDIA_BUS_FMT_YUV8_1X24); + KUNIT_EXPECT_EQ(test, out_bus_fmts[2], MEDIA_BUS_FMT_UYYVYY8_0_5X24); + + drm_modeset_drop_locks(&ctx); + drm_modeset_acquire_fini(&ctx); + + kfree(out_bus_fmts); +} + static struct kunit_case drm_bridge_helper_reset_crtc_tests[] =3D { KUNIT_CASE(drm_test_drm_bridge_helper_reset_crtc_atomic), KUNIT_CASE(drm_test_drm_bridge_helper_reset_crtc_atomic_disabled), KUNIT_CASE(drm_test_drm_bridge_helper_reset_crtc_legacy), + KUNIT_CASE(drm_test_drm_bridge_helper_hdmi_output_bus_fmts), { } }; =20 --=20 2.53.0