drivers/gpu/drm/tests/drm_bridge_test.c

// SPDX-License-Identifier: GPL-2.0
/*
 * Kunit test for drm_bridge functions
 */
#include <linux/cleanup.h>
#include <linux/media-bus-format.h>

#include <drm/drm_atomic_helper.h>
#include <drm/drm_atomic_state_helper.h>
#include <drm/drm_atomic_uapi.h>
#include <drm/drm_bridge.h>
#include <drm/drm_bridge_connector.h>
#include <drm/drm_bridge_helper.h>
#include <drm/drm_edid.h>
#include <drm/drm_kunit_helpers.h>
#include <drm/drm_managed.h>

#include <kunit/device.h>
#include <kunit/test.h>

#include "drm_kunit_edid.h"

/*
 * Mimick the typical "private" struct defined by a bridge driver, which
 * embeds a bridge plus other fields.
 *
 * Having at least one member before @bridge ensures we test non-zero
 * @bridge offset.
 */
struct drm_bridge_priv {
	unsigned int enable_count;
	unsigned int disable_count;
	struct drm_bridge bridge;
	void *data;
};

struct drm_bridge_init_priv {
	struct drm_device drm;
	/** @dev: device, only for tests not needing a whole drm_device */
	struct device *dev;
	struct drm_plane *plane;
	struct drm_crtc *crtc;
	struct drm_encoder encoder;
	struct drm_bridge_priv *test_bridge;
	struct drm_connector *connector;
	bool destroyed;
};

struct drm_bridge_chain_priv {
	struct drm_device drm;
	struct drm_encoder encoder;
	struct drm_plane *plane;
	struct drm_crtc *crtc;
	struct drm_connector *connector;
	unsigned int num_bridges;

	/**
	 * @test_bridges: array of pointers to &struct drm_bridge_priv entries
	 *                of which the first @num_bridges entries are valid.
	 */
	struct drm_bridge_priv **test_bridges;
};

static struct drm_bridge_priv *bridge_to_priv(struct drm_bridge *bridge)
{
	return container_of(bridge, struct drm_bridge_priv, bridge);
}

static void drm_test_bridge_priv_destroy(struct drm_bridge *bridge)
{
	struct drm_bridge_priv *bridge_priv = bridge_to_priv(bridge);
	struct drm_bridge_init_priv *priv = (struct drm_bridge_init_priv *)bridge_priv->data;

	priv->destroyed = true;
}

static void drm_test_bridge_atomic_enable(struct drm_bridge *bridge,
					  struct drm_atomic_commit *state)
{
	struct drm_bridge_priv *priv = bridge_to_priv(bridge);

	priv->enable_count++;
}

static void drm_test_bridge_atomic_disable(struct drm_bridge *bridge,
					   struct drm_atomic_commit *state)
{
	struct drm_bridge_priv *priv = bridge_to_priv(bridge);

	priv->disable_count++;
}

static const struct drm_bridge_funcs drm_test_bridge_atomic_funcs = {
	.destroy		= drm_test_bridge_priv_destroy,
	.atomic_enable		= drm_test_bridge_atomic_enable,
	.atomic_disable		= drm_test_bridge_atomic_disable,
	.atomic_destroy_state	= drm_atomic_helper_bridge_destroy_state,
	.atomic_duplicate_state	= drm_atomic_helper_bridge_duplicate_state,
	.atomic_create_state	= drm_atomic_helper_bridge_create_state,
};

static int dummy_clear_infoframe(struct drm_bridge *bridge)
{
	return 0;
}

static int dummy_write_infoframe(struct drm_bridge *bridge, const u8 *buffer,
				 size_t len)
{
	return 0;
}

static const struct drm_bridge_funcs drm_test_bridge_bus_fmts_funcs = {
	.atomic_get_output_bus_fmts	= drm_atomic_helper_bridge_get_hdmi_output_bus_fmts,
	.atomic_destroy_state		= drm_atomic_helper_bridge_destroy_state,
	.atomic_duplicate_state		= drm_atomic_helper_bridge_duplicate_state,
	.atomic_create_state		= drm_atomic_helper_bridge_create_state,
	.hdmi_write_avi_infoframe	= dummy_write_infoframe,
	.hdmi_write_hdmi_infoframe	= dummy_write_infoframe,
	.hdmi_clear_avi_infoframe	= dummy_clear_infoframe,
	.hdmi_clear_hdmi_infoframe	= dummy_clear_infoframe,
};

/**
 * struct fmt_tuple - a tuple of input/output MEDIA_BUS_FMT_*
 */
struct fmt_tuple {
	u32 in_fmt;
	u32 out_fmt;
};

/*
 * Format mapping that only accepts RGB888, and outputs only RGB888
 */
static const struct fmt_tuple rgb8_passthrough[] = {
	{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_RGB888_1X24 },
};

/*
 * Format mapping that only accepts YUV444, and outputs only YUV444
 */
static const struct fmt_tuple yuv8_passthrough[] = {
	{ MEDIA_BUS_FMT_YUV8_1X24,   MEDIA_BUS_FMT_YUV8_1X24 },
};

/*
 * Format mapping where 8bpc RGB -> 8bpc YUV444, or ID(RGB) or ID(YUV444)
 */
static const struct fmt_tuple rgb8_to_yuv8_or_id[] = {
	{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_RGB888_1X24 },
	{ MEDIA_BUS_FMT_YUV8_1X24,   MEDIA_BUS_FMT_YUV8_1X24 },
	{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_YUV8_1X24 },
};

static const struct fmt_tuple rgb8_to_id_yuv8_or_yuv8_to_yuv422_yuv420[] = {
	{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_RGB888_1X24 },
	{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_YUV8_1X24 },
	{ MEDIA_BUS_FMT_YUV8_1X24,   MEDIA_BUS_FMT_UYVY8_1X16 },
	{ MEDIA_BUS_FMT_YUV8_1X24,   MEDIA_BUS_FMT_UYYVYY8_0_5X24 },
};

/*
 * Format mapping where 8bpc YUV444 -> 8bpc RGB, or ID(YUV444)
 */
static const struct fmt_tuple yuv8_to_rgb8_or_id[] = {
	{ MEDIA_BUS_FMT_YUV8_1X24, MEDIA_BUS_FMT_YUV8_1X24 },
	{ MEDIA_BUS_FMT_YUV8_1X24, MEDIA_BUS_FMT_RGB888_1X24 },
};

/*
 * A format mapping that acts like a video processor that generates an RGB signal
 */
static const struct fmt_tuple rgb_producer[] = {
	{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_RGB888_1X24 },
	{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_RGB101010_1X30 },
	{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_RGB121212_1X36 },
};

/*
 * A format mapping that acts like a video processor that generates an 8-bit RGB,
 * YUV444 or YUV420 signal
 */
static const struct fmt_tuple rgb_yuv444_yuv420_producer[] = {
	{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_RGB888_1X24 },
	{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_YUV8_1X24 },
	{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_UYYVYY8_0_5X24 },
};

static const struct fmt_tuple rgb8_yuv444_yuv422_passthrough[] = {
	{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_RGB888_1X24 },
	{ MEDIA_BUS_FMT_YUV8_1X24,   MEDIA_BUS_FMT_YUV8_1X24 },
	{ MEDIA_BUS_FMT_UYVY8_1X16,  MEDIA_BUS_FMT_UYVY8_1X16 },
};

static const struct fmt_tuple yuv444_yuv422_rgb8_passthrough[] = {
	{ MEDIA_BUS_FMT_YUV8_1X24,   MEDIA_BUS_FMT_YUV8_1X24 },
	{ MEDIA_BUS_FMT_UYVY8_1X16,  MEDIA_BUS_FMT_UYVY8_1X16 },
	{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_RGB888_1X24 },
};

static bool fmt_in_list(const u32 fmt, const u32 *out_fmts, const size_t num_fmts)
{
	size_t i;

	for (i = 0; i < num_fmts; i++)
		if (out_fmts[i] == fmt)
			return true;

	return false;
}

/**
 * get_tuples_out_fmts - Get unique output formats of a &struct fmt_tuple list
 * @fmt_tuples: array of &struct fmt_tuple
 * @num_fmt_tuples: number of entries in @fmt_tuples
 * @out_fmts: target array to store the unique output bus formats
 *
 * Returns the number of unique output formats, i.e. the number of entries in
 * @out_fmts that were populated with sensible values.
 */
static size_t get_tuples_out_fmts(const struct fmt_tuple *fmt_tuples,
				  const size_t num_fmt_tuples, u32 *out_fmts)
{
	size_t num_unique = 0;
	size_t i;

	for (i = 0; i < num_fmt_tuples; i++)
		if (!fmt_in_list(fmt_tuples[i].out_fmt, out_fmts, num_unique))
			out_fmts[num_unique++] = fmt_tuples[i].out_fmt;

	return num_unique;
}

#define DEFINE_FMT_FUNCS_FROM_TUPLES(name) \
static u32 *drm_test_bridge_ ## name ## _out_fmts(struct drm_bridge *bridge,			\
						  struct drm_bridge_state *bridge_state,	\
						  struct drm_crtc_state *crtc_state,		\
						  struct drm_connector_state *conn_state,	\
						  unsigned int *num_output_fmts)		\
{												\
	u32 *out_fmts = kcalloc(ARRAY_SIZE((name)), sizeof(u32), GFP_KERNEL);			\
												\
	if (out_fmts)										\
		*num_output_fmts = get_tuples_out_fmts((name), ARRAY_SIZE((name)), out_fmts);	\
	else											\
		*num_output_fmts = 0;								\
												\
	return out_fmts;									\
}												\
												\
static u32 *drm_test_bridge_ ## name ## _in_fmts(struct drm_bridge *bridge,			\
						 struct drm_bridge_state *bridge_state,		\
						 struct drm_crtc_state *crtc_state,		\
						 struct drm_connector_state *conn_state,	\
						 u32 output_fmt,				\
						 unsigned int *num_input_fmts)			\
{												\
	u32 *in_fmts = kcalloc(ARRAY_SIZE((name)), sizeof(u32), GFP_KERNEL);			\
	unsigned int num_fmts = 0;								\
	size_t i;										\
												\
	if (!in_fmts) {										\
		*num_input_fmts = 0;								\
		return NULL;									\
	}											\
												\
	for (i = 0; i < ARRAY_SIZE((name)); i++)						\
		if ((name)[i].out_fmt == output_fmt)						\
			in_fmts[num_fmts++] = (name)[i].in_fmt;					\
												\
	*num_input_fmts = num_fmts;								\
												\
	return in_fmts;										\
}

#define DRM_BRIDGE_ATOMIC_WITH_BUS_FMT_HDMI_FUNC(ident, input_fmts_func, output_fmts_func,	\
						 hdmi_write_infoframe_func,			\
						 hdmi_clear_infoframe_func)			\
static const struct drm_bridge_funcs (ident) = {						\
	.atomic_enable			= drm_test_bridge_atomic_enable,			\
	.atomic_disable			= drm_test_bridge_atomic_disable,			\
	.atomic_destroy_state		= drm_atomic_helper_bridge_destroy_state,		\
	.atomic_duplicate_state		= drm_atomic_helper_bridge_duplicate_state,		\
	.atomic_create_state		= drm_atomic_helper_bridge_create_state,		\
	.atomic_get_input_bus_fmts	= (input_fmts_func),					\
	.atomic_get_output_bus_fmts	= (output_fmts_func),					\
	.hdmi_write_avi_infoframe	= (hdmi_write_infoframe_func),				\
	.hdmi_clear_avi_infoframe	= (hdmi_clear_infoframe_func),				\
	.hdmi_write_hdmi_infoframe	= (hdmi_write_infoframe_func),				\
	.hdmi_clear_hdmi_infoframe	= (hdmi_clear_infoframe_func),				\
}

#define DRM_BRIDGE_ATOMIC_WITH_BUS_FMT_FUNC(ident, input_fmts_func, output_fmts_func)		\
	DRM_BRIDGE_ATOMIC_WITH_BUS_FMT_HDMI_FUNC(ident, input_fmts_func, output_fmts_func,	\
						 NULL, NULL)

#define DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(_name)						\
	DRM_BRIDGE_ATOMIC_WITH_BUS_FMT_FUNC(_name ## _funcs,				\
					    drm_test_bridge_ ## _name ## _in_fmts,	\
					    drm_test_bridge_ ## _name ## _out_fmts)

static int drm_test_bridge_write_infoframe_stub(struct drm_bridge *bridge,
						const u8 *buffer, size_t len)
{
	return 0;
}

static int drm_test_bridge_clear_infoframe_stub(struct drm_bridge *bridge)
{
	return 0;
}

#define DRM_BRIDGE_ATOMIC_WITH_BUS_FMT_HDMI(_name)						\
	DRM_BRIDGE_ATOMIC_WITH_BUS_FMT_HDMI_FUNC(_name ## _hdmi ## _funcs,			\
						 drm_test_bridge_ ## _name ## _in_fmts,		\
						 drm_test_bridge_ ## _name ## _out_fmts,	\
						 drm_test_bridge_write_infoframe_stub,		\
						 drm_test_bridge_clear_infoframe_stub)
DEFINE_FMT_FUNCS_FROM_TUPLES(rgb8_passthrough)
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(rgb8_passthrough);

DEFINE_FMT_FUNCS_FROM_TUPLES(yuv8_passthrough)
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(yuv8_passthrough);

DEFINE_FMT_FUNCS_FROM_TUPLES(rgb8_to_yuv8_or_id)
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(rgb8_to_yuv8_or_id);

DEFINE_FMT_FUNCS_FROM_TUPLES(yuv8_to_rgb8_or_id)
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(yuv8_to_rgb8_or_id);

DEFINE_FMT_FUNCS_FROM_TUPLES(rgb_producer)
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(rgb_producer);

DEFINE_FMT_FUNCS_FROM_TUPLES(rgb_yuv444_yuv420_producer)
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(rgb_yuv444_yuv420_producer);

DEFINE_FMT_FUNCS_FROM_TUPLES(rgb8_to_id_yuv8_or_yuv8_to_yuv422_yuv420)
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(rgb8_to_id_yuv8_or_yuv8_to_yuv422_yuv420);

DEFINE_FMT_FUNCS_FROM_TUPLES(rgb8_yuv444_yuv422_passthrough)
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(rgb8_yuv444_yuv422_passthrough);

DEFINE_FMT_FUNCS_FROM_TUPLES(yuv444_yuv422_rgb8_passthrough)
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT(yuv444_yuv422_rgb8_passthrough);
DRM_BRIDGE_ATOMIC_WITH_BUS_FMT_HDMI(yuv444_yuv422_rgb8_passthrough);

KUNIT_DEFINE_ACTION_WRAPPER(drm_bridge_remove_wrapper,
			    drm_bridge_remove,
			    struct drm_bridge *);

static int drm_kunit_bridge_add(struct kunit *test,
				struct drm_bridge *bridge)
{
	drm_bridge_add(bridge);

	return kunit_add_action_or_reset(test,
					 drm_bridge_remove_wrapper,
					 bridge);
}

static struct drm_bridge_init_priv *
drm_test_bridge_init(struct kunit *test, const struct drm_bridge_funcs *funcs)
{
	struct drm_bridge_init_priv *priv;
	struct drm_encoder *enc;
	struct drm_bridge *bridge;
	struct drm_device *drm;
	struct device *dev;
	int ret;

	dev = drm_kunit_helper_alloc_device(test);
	if (IS_ERR(dev))
		return ERR_CAST(dev);

	priv = 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 = 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 = priv;

	drm = &priv->drm;
	priv->plane = 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 = drm_kunit_helper_create_crtc(test, drm,
						  priv->plane, NULL,
						  NULL,
						  NULL);
	if (IS_ERR(priv->crtc))
		return ERR_CAST(priv->crtc);

	enc = &priv->encoder;
	ret = drmm_encoder_init(drm, enc, NULL, DRM_MODE_ENCODER_TMDS, NULL);
	if (ret)
		return ERR_PTR(ret);

	enc->possible_crtcs = drm_crtc_mask(priv->crtc);

	bridge = &priv->test_bridge->bridge;
	bridge->type = DRM_MODE_CONNECTOR_VIRTUAL;

	ret = drm_kunit_bridge_add(test, bridge);
	if (ret)
		return ERR_PTR(ret);

	ret = drm_bridge_attach(enc, bridge, NULL, 0);
	if (ret)
		return ERR_PTR(ret);

	priv->connector = drm_bridge_connector_init(drm, enc);
	if (IS_ERR(priv->connector))
		return ERR_CAST(priv->connector);

	drm_mode_config_reset(drm);

	return priv;
}

static struct drm_bridge_chain_priv *
drm_test_bridge_chain_init(struct kunit *test, unsigned int num_bridges,
			   const struct drm_bridge_funcs **funcs)
{
	struct drm_bridge_chain_priv *priv;
	const struct drm_edid *edid;
	struct drm_bridge *prev;
	struct drm_encoder *enc;
	struct drm_bridge *bridge;
	struct drm_device *drm;
	bool has_hdmi = false;
	struct device *dev;
	unsigned int i;
	int ret;

	dev = drm_kunit_helper_alloc_device(test);
	if (IS_ERR(dev))
		return ERR_CAST(dev);

	priv = drm_kunit_helper_alloc_drm_device(test, dev, struct drm_bridge_chain_priv,
						 drm, DRIVER_MODESET | DRIVER_ATOMIC);
	if (IS_ERR(priv))
		return ERR_CAST(priv);

	drm = &priv->drm;

	priv->test_bridges = drmm_kmalloc_array(drm, num_bridges, sizeof(*priv->test_bridges),
						GFP_KERNEL);
	if (!priv->test_bridges)
		return ERR_PTR(-ENOMEM);

	priv->num_bridges = num_bridges;

	for (i = 0; i < num_bridges; i++) {
		priv->test_bridges[i] = devm_drm_bridge_alloc(dev, struct drm_bridge_priv,
							      bridge, funcs[i]);
		if (IS_ERR(priv->test_bridges[i]))
			return ERR_CAST(priv->test_bridges[i]);

		priv->test_bridges[i]->data = priv;
	}

	priv->plane = 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 = drm_kunit_helper_create_crtc(test, drm, priv->plane, NULL,
						  NULL, NULL);
	if (IS_ERR(priv->crtc))
		return ERR_CAST(priv->crtc);

	enc = &priv->encoder;
	ret = drmm_encoder_init(drm, enc, NULL, DRM_MODE_ENCODER_TMDS, NULL);
	if (ret)
		return ERR_PTR(ret);

	enc->possible_crtcs = drm_crtc_mask(priv->crtc);

	prev = NULL;
	for (i = 0; i < num_bridges; i++) {
		bridge = &priv->test_bridges[i]->bridge;
		bridge->type = DRM_MODE_CONNECTOR_VIRTUAL;

		if (bridge->funcs->hdmi_write_hdmi_infoframe) {
			has_hdmi = true;
			bridge->ops |= DRM_BRIDGE_OP_HDMI;
			bridge->type = DRM_MODE_CONNECTOR_HDMIA;
			bridge->vendor = "LNX";
			bridge->product = "KUnit";
			bridge->supported_formats = (BIT(DRM_OUTPUT_COLOR_FORMAT_RGB444) |
						     BIT(DRM_OUTPUT_COLOR_FORMAT_YCBCR444) |
						     BIT(DRM_OUTPUT_COLOR_FORMAT_YCBCR422) |
						     BIT(DRM_OUTPUT_COLOR_FORMAT_YCBCR420));
		}

		ret = drm_kunit_bridge_add(test, bridge);
		if (ret)
			return ERR_PTR(ret);

		ret = drm_bridge_attach(enc, bridge, prev, 0);
		if (ret)
			return ERR_PTR(ret);

		prev = bridge;
	}

	priv->connector = 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);

	if (!has_hdmi)
		return priv;

	scoped_guard(mutex, &drm->mode_config.mutex) {
		edid = drm_edid_alloc(test_edid_hdmi_1080p_rgb_yuv_4k_yuv420_dc_max_200mhz,
				ARRAY_SIZE(test_edid_hdmi_1080p_rgb_yuv_4k_yuv420_dc_max_200mhz));
		if (!edid)
			return ERR_PTR(-EINVAL);

		drm_edid_connector_update(priv->connector, edid);
		KUNIT_ASSERT_GT(test, drm_edid_connector_add_modes(priv->connector), 0);

		ret = priv->connector->funcs->fill_modes(priv->connector, 4096, 4096);
	}

	return priv;
}

static struct drm_bridge_init_priv *
drm_test_bridge_hdmi_init(struct kunit *test, const struct drm_bridge_funcs *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 = drm_kunit_helper_alloc_device(test);
	if (IS_ERR(dev))
		return ERR_CAST(dev);

	priv = 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 = 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 = priv;

	drm = &priv->drm;
	priv->plane = 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 = drm_kunit_helper_create_crtc(test, drm,
						  priv->plane, NULL,
						  NULL,
						  NULL);
	if (IS_ERR(priv->crtc))
		return ERR_CAST(priv->crtc);

	enc = &priv->encoder;
	ret = drmm_encoder_init(drm, enc, NULL, DRM_MODE_ENCODER_TMDS, NULL);
	if (ret)
		return ERR_PTR(ret);

	enc->possible_crtcs = drm_crtc_mask(priv->crtc);

	bridge = &priv->test_bridge->bridge;
	bridge->type = DRM_MODE_CONNECTOR_HDMIA;
	bridge->supported_formats = supported_formats;
	bridge->max_bpc = max_bpc;
	bridge->ops |= DRM_BRIDGE_OP_HDMI;
	bridge->vendor = "LNX";
	bridge->product = "KUnit";

	ret = drm_kunit_bridge_add(test, bridge);
	if (ret)
		return ERR_PTR(ret);

	ret = drm_bridge_attach(enc, bridge, NULL, 0);
	if (ret)
		return ERR_PTR(ret);

	priv->connector = 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.
 */
static void drm_test_drm_bridge_get_current_state_atomic(struct kunit *test)
{
	struct drm_modeset_acquire_ctx ctx;
	struct drm_bridge_init_priv *priv;
	struct drm_bridge_state *curr_bridge_state;
	struct drm_bridge_state *bridge_state;
	struct drm_atomic_commit *state;
	struct drm_bridge *bridge;
	struct drm_device *drm;
	int ret;

	priv = drm_test_bridge_init(test, &drm_test_bridge_atomic_funcs);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv);

	drm_modeset_acquire_init(&ctx, 0);

	drm = &priv->drm;
	state = drm_kunit_helper_atomic_state_alloc(test, drm, &ctx);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state);

retry_commit:
	bridge = &priv->test_bridge->bridge;
	bridge_state = drm_atomic_get_bridge_state(state, bridge);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, bridge_state);

	ret = drm_atomic_commit(state);
	if (ret == -EDEADLK) {
		drm_atomic_commit_clear(state);
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);

	drm_modeset_acquire_init(&ctx, 0);

retry_state:
	ret = drm_modeset_lock(&bridge->base.lock, &ctx);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_state;
	}

	curr_bridge_state = drm_bridge_get_current_state(bridge);
	KUNIT_EXPECT_PTR_EQ(test, curr_bridge_state, bridge_state);

	drm_modeset_unlock(&bridge->base.lock);

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);
}

static struct kunit_case drm_bridge_get_current_state_tests[] = {
	KUNIT_CASE(drm_test_drm_bridge_get_current_state_atomic),
	{ }
};


static struct kunit_suite drm_bridge_get_current_state_test_suite = {
	.name = "drm_test_bridge_get_current_state",
	.test_cases = drm_bridge_get_current_state_tests,
};

/*
 * Test that an atomic bridge is properly power-cycled when calling
 * drm_bridge_helper_reset_crtc().
 */
static void drm_test_drm_bridge_helper_reset_crtc_atomic(struct kunit *test)
{
	struct drm_modeset_acquire_ctx ctx;
	struct drm_bridge_init_priv *priv;
	struct drm_display_mode *mode;
	struct drm_bridge_priv *bridge_priv;
	int ret;

	priv = drm_test_bridge_init(test, &drm_test_bridge_atomic_funcs);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv);

	mode = drm_kunit_display_mode_from_cea_vic(test, &priv->drm, 16);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, mode);

	drm_modeset_acquire_init(&ctx, 0);

retry_commit:
	ret = drm_kunit_helper_enable_crtc_connector(test,
						     &priv->drm, priv->crtc,
						     priv->connector,
						     mode,
						     &ctx);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);

	bridge_priv = priv->test_bridge;
	KUNIT_ASSERT_EQ(test, bridge_priv->enable_count, 1);
	KUNIT_ASSERT_EQ(test, bridge_priv->disable_count, 0);

	drm_modeset_acquire_init(&ctx, 0);

retry_reset:
	ret = drm_bridge_helper_reset_crtc(&bridge_priv->bridge, &ctx);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_reset;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);

	KUNIT_EXPECT_EQ(test, bridge_priv->enable_count, 2);
	KUNIT_EXPECT_EQ(test, bridge_priv->disable_count, 1);
}

/*
 * Test that calling drm_bridge_helper_reset_crtc() on a disabled atomic
 * bridge will fail and not call the enable / disable callbacks
 */
static void drm_test_drm_bridge_helper_reset_crtc_atomic_disabled(struct kunit *test)
{
	struct drm_modeset_acquire_ctx ctx;
	struct drm_bridge_init_priv *priv;
	struct drm_display_mode *mode;
	struct drm_bridge_priv *bridge_priv;
	int ret;

	priv = drm_test_bridge_init(test, &drm_test_bridge_atomic_funcs);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv);

	mode = drm_kunit_display_mode_from_cea_vic(test, &priv->drm, 16);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, mode);

	bridge_priv = priv->test_bridge;
	KUNIT_ASSERT_EQ(test, bridge_priv->enable_count, 0);
	KUNIT_ASSERT_EQ(test, bridge_priv->disable_count, 0);

	drm_modeset_acquire_init(&ctx, 0);

retry_reset:
	ret = drm_bridge_helper_reset_crtc(&bridge_priv->bridge, &ctx);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_reset;
	}
	KUNIT_EXPECT_LT(test, ret, 0);

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);

	KUNIT_EXPECT_EQ(test, bridge_priv->enable_count, 0);
	KUNIT_EXPECT_EQ(test, bridge_priv->disable_count, 0);
}

/*
 * Test that a bridge using the drm_atomic_helper_bridge_get_hdmi_output_bus_fmts()
 * function for &drm_bridge_funcs.atomic_get_output_bus_fmts behaves as expected
 * for an HDMI connector bridge. Does so by creating an HDMI bridge connector
 * with RGB444, YCBCR444, and YCBCR420 (but not YCBCR422) as supported formats,
 * sets the output depth to 8 bits per component, and then validates the returned
 * 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_commit *state;
	struct drm_display_mode *mode;
	unsigned int num_output_fmts;
	struct drm_bridge *bridge;
	u32 *out_bus_fmts;
	int ret;

	priv = 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 = &priv->test_bridge->bridge;

	drm_modeset_acquire_init(&ctx, 0);

	state = drm_kunit_helper_atomic_state_alloc(test, &priv->drm, &ctx);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state);

retry_commit:
	conn_state = drm_atomic_get_connector_state(state, priv->connector);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, conn_state);

	conn_state->hdmi.output_bpc = 8;

	mode = drm_kunit_display_mode_from_cea_vic(test, &priv->drm, 16);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, mode);

	ret = drm_atomic_set_crtc_for_connector(conn_state, priv->crtc);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state = drm_atomic_get_crtc_state(state, priv->crtc);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, crtc_state);

	ret = drm_atomic_set_mode_for_crtc(crtc_state, mode);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state->enable = true;
	crtc_state->active = true;

	bridge_state = drm_atomic_get_bridge_state(state, bridge);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, bridge_state);

	out_bus_fmts = 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[] = {
	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_hdmi_output_bus_fmts),
	{ }
};

static struct kunit_suite drm_bridge_helper_reset_crtc_test_suite = {
	.name = "drm_test_bridge_helper_reset_crtc",
	.test_cases = drm_bridge_helper_reset_crtc_tests,
};

static int drm_test_bridge_alloc_init(struct kunit *test)
{
	struct drm_bridge_init_priv *priv;

	priv = kunit_kzalloc(test, sizeof(*priv), GFP_KERNEL);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv);

	priv->dev = kunit_device_register(test, "drm-bridge-dev");
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv->dev);

	test->priv = priv;

	priv->test_bridge = devm_drm_bridge_alloc(priv->dev, struct drm_bridge_priv, bridge,
						  &drm_test_bridge_atomic_funcs);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv->test_bridge);

	priv->test_bridge->data = priv;

	KUNIT_ASSERT_FALSE(test, priv->destroyed);

	return 0;
}

/*
 * Test that a bridge is freed when the device is destroyed in lack of
 * other drm_bridge_get/put() operations.
 */
static void drm_test_drm_bridge_alloc_basic(struct kunit *test)
{
	struct drm_bridge_init_priv *priv = test->priv;

	KUNIT_ASSERT_FALSE(test, priv->destroyed);

	kunit_device_unregister(test, priv->dev);
	KUNIT_EXPECT_TRUE(test, priv->destroyed);
}

/*
 * Test that a bridge is not freed when the device is destroyed when there
 * is still a reference to it, and freed when that reference is put.
 */
static void drm_test_drm_bridge_alloc_get_put(struct kunit *test)
{
	struct drm_bridge_init_priv *priv = test->priv;

	KUNIT_ASSERT_FALSE(test, priv->destroyed);

	drm_bridge_get(&priv->test_bridge->bridge);
	KUNIT_EXPECT_FALSE(test, priv->destroyed);

	kunit_device_unregister(test, priv->dev);
	KUNIT_EXPECT_FALSE(test, priv->destroyed);

	drm_bridge_put(&priv->test_bridge->bridge);
	KUNIT_EXPECT_TRUE(test, priv->destroyed);
}

static struct kunit_case drm_bridge_alloc_tests[] = {
	KUNIT_CASE(drm_test_drm_bridge_alloc_basic),
	KUNIT_CASE(drm_test_drm_bridge_alloc_get_put),
	{ }
};

static struct kunit_suite drm_bridge_alloc_test_suite = {
	.name = "drm_bridge_alloc",
	.init = drm_test_bridge_alloc_init,
	.test_cases = drm_bridge_alloc_tests,
};

/**
 * drm_test_bridge_chain_verify_fmt - Verify bridge chain format selection
 * @test: pointer to KUnit test object
 * @priv: pointer to a &struct drm_bridge_chain_priv for this chain
 * @expected: constant array of &struct fmt_tuple describing the expected
 *            input and output bus formats
 * @num_expected: number of entries in @expected
 *
 * Runs the KUNIT_EXPECT clauses to verify the bridge chain format selection
 * resulted in the expected formats. If %0 is given as a format in a
 * &struct fmt_tuple, then it is understood to mean "any".
 *
 * Must be called with the modeset lock held.
 */
static void drm_test_bridge_chain_verify_fmt(struct kunit *test,
					     struct drm_bridge_chain_priv *priv,
					     const struct fmt_tuple *const expected,
					     const unsigned int num_expected)
{
	struct drm_bridge_state *bstate;
	unsigned int i = 0;

	drm_for_each_bridge_in_chain(&priv->encoder, bridge) {
		KUNIT_ASSERT_LT(test, i, num_expected);

		bstate = drm_bridge_get_current_state(bridge);
		if (expected[i].in_fmt)
			KUNIT_EXPECT_EQ(test, bstate->input_bus_cfg.format,
					expected[i].in_fmt);
		if (expected[i].out_fmt)
			KUNIT_EXPECT_EQ(test, bstate->output_bus_cfg.format,
					expected[i].out_fmt);

		i++;
	}

	KUNIT_ASSERT_EQ_MSG(test, i, num_expected,
			    "Fewer bridges (%u) than expected (%u)\n", i, num_expected);
}

/*
 * Test that constructs a bridge chain in which an RGB888 producer is chained to
 * two bridges that will convert from RGB to YUV and from YUV to RGB respectively.
 *
 * The test requests an output color_format of RGB using the color_format property,
 * so to satisfy this request, the bridge chain must take a detour over YUV.
 */
static void drm_test_bridge_rgb_yuv_rgb(struct kunit *test)
{
	static const struct drm_bridge_funcs *funcs[] = {
		&rgb_producer_funcs,
		&rgb8_to_yuv8_or_id_funcs,
		&yuv8_to_rgb8_or_id_funcs,
	};
	static const struct fmt_tuple expected[] = {
		{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_RGB888_1X24 },
		{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_YUV8_1X24 },
		{ MEDIA_BUS_FMT_YUV8_1X24, MEDIA_BUS_FMT_RGB888_1X24 },
	};
	struct drm_connector_state *conn_state;
	struct drm_modeset_acquire_ctx ctx;
	struct drm_bridge_chain_priv *priv;
	struct drm_crtc_state *crtc_state;
	struct drm_atomic_commit *state;
	struct drm_display_mode *mode;
	int ret;

	priv = drm_test_bridge_chain_init(test, ARRAY_SIZE(funcs), funcs);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv);

	drm_modeset_acquire_init(&ctx, 0);

	state = drm_kunit_helper_atomic_state_alloc(test, &priv->drm, &ctx);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state);

retry_commit:
	conn_state = drm_atomic_get_connector_state(state, priv->connector);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, conn_state);

	mode = drm_kunit_display_mode_from_cea_vic(test, &priv->drm, 16);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, mode);

	conn_state->color_format = DRM_CONNECTOR_COLOR_FORMAT_RGB444;

	ret = drm_atomic_set_crtc_for_connector(conn_state, priv->crtc);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state = drm_atomic_get_crtc_state(state, priv->crtc);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, crtc_state);

	ret = drm_atomic_set_mode_for_crtc(crtc_state, mode);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state->enable = true;
	crtc_state->active = true;

	ret = drm_atomic_commit(state);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	drm_test_bridge_chain_verify_fmt(test, priv, expected, ARRAY_SIZE(expected));

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);
}

/*
 * Test in which a bridge capable of producing RGB, YUV444 and YUV420 has to
 * produce RGB and convert with a downstream bridge in the chain to reach the
 * requested YUV444 color format, as no direct path exists between its YUV444
 * and the last bridge.
 *
 * The rationale behind this test is to devise a scenario in which naively
 * assuming any format the video processor can output, and the connector
 * requests, is the right format to pick, does not work.
 */
static void drm_test_bridge_must_convert_to_yuv444(struct kunit *test)
{
	static const struct drm_bridge_funcs *funcs[] = {
		&rgb_yuv444_yuv420_producer_funcs,
		&rgb8_passthrough_funcs,
		&rgb8_to_id_yuv8_or_yuv8_to_yuv422_yuv420_funcs,
		&rgb8_yuv444_yuv422_passthrough_funcs,
	};
	static const struct fmt_tuple expected[] = {
		{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_RGB888_1X24 },
		{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_RGB888_1X24 },
		{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_YUV8_1X24 },
		{ MEDIA_BUS_FMT_YUV8_1X24, MEDIA_BUS_FMT_YUV8_1X24 },
	};
	struct drm_connector_state *conn_state;
	struct drm_modeset_acquire_ctx ctx;
	struct drm_bridge_chain_priv *priv;
	struct drm_crtc_state *crtc_state;
	struct drm_atomic_commit *state;
	struct drm_display_mode *mode;
	int ret;

	priv = drm_test_bridge_chain_init(test, ARRAY_SIZE(funcs), funcs);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv);

	drm_modeset_acquire_init(&ctx, 0);

	state = drm_kunit_helper_atomic_state_alloc(test, &priv->drm, &ctx);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state);

retry_commit:
	conn_state = drm_atomic_get_connector_state(state, priv->connector);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, conn_state);

	mode = drm_kunit_display_mode_from_cea_vic(test, &priv->drm, 16);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, mode);

	conn_state->color_format = DRM_CONNECTOR_COLOR_FORMAT_YCBCR444;

	ret = drm_atomic_set_crtc_for_connector(conn_state, priv->crtc);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state = drm_atomic_get_crtc_state(state, priv->crtc);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, crtc_state);

	ret = drm_atomic_set_mode_for_crtc(crtc_state, mode);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state->enable = true;
	crtc_state->active = true;

	ret = drm_atomic_commit(state);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	drm_test_bridge_chain_verify_fmt(test, priv, expected, ARRAY_SIZE(expected));

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);
}

/*
 * Test which checks that no matter the order of bus formats returned by an
 * HDMI bridge, RGB is preferred on DRM_CONNECTOR_COLOR_FORMAT_AUTO if it's
 * available.
 */
static void drm_test_bridge_hdmi_auto_rgb(struct kunit *test)
{
	static const struct drm_bridge_funcs *funcs[] = {
		&rgb_yuv444_yuv420_producer_funcs,
		&yuv444_yuv422_rgb8_passthrough_hdmi_funcs,
	};
	static const struct fmt_tuple expected[] = {
		{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_RGB888_1X24 },
		{ MEDIA_BUS_FMT_RGB888_1X24, MEDIA_BUS_FMT_RGB888_1X24 },
	};
	struct drm_connector_state *conn_state;
	struct drm_modeset_acquire_ctx ctx;
	struct drm_bridge_chain_priv *priv;
	struct drm_crtc_state *crtc_state;
	struct drm_atomic_commit *state;
	struct drm_display_mode *mode;
	int ret;

	priv = drm_test_bridge_chain_init(test, ARRAY_SIZE(funcs), funcs);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv);

	drm_modeset_acquire_init(&ctx, 0);

	state = drm_kunit_helper_atomic_state_alloc(test, &priv->drm, &ctx);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state);

retry_commit:
	conn_state = drm_atomic_get_connector_state(state, priv->connector);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, conn_state);

	mode = drm_kunit_display_mode_from_cea_vic(test, &priv->drm, 16);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, mode);

	KUNIT_ASSERT_EQ(test, conn_state->color_format, DRM_CONNECTOR_COLOR_FORMAT_AUTO);

	ret = drm_atomic_set_crtc_for_connector(conn_state, priv->crtc);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state = drm_atomic_get_crtc_state(state, priv->crtc);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, crtc_state);

	ret = drm_atomic_set_mode_for_crtc(crtc_state, mode);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state->enable = true;
	crtc_state->active = true;

	ret = drm_atomic_commit(state);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	KUNIT_EXPECT_EQ(test, conn_state->hdmi.output_format, DRM_OUTPUT_COLOR_FORMAT_RGB444);

	drm_test_bridge_chain_verify_fmt(test, priv, expected, ARRAY_SIZE(expected));

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);
}

/*
 * Test which checks that DRM_CONNECTOR_COLOR_FORMAT_AUTO on non-HDMI connectors
 * will result in the first bus format on the output to be picked.
 */
static void drm_test_bridge_auto_first(struct kunit *test)
{
	static const struct drm_bridge_funcs *funcs[] = {
		&rgb_yuv444_yuv420_producer_funcs,
		&yuv444_yuv422_rgb8_passthrough_funcs,
	};
	static const struct fmt_tuple expected[] = {
		{ MEDIA_BUS_FMT_FIXED, MEDIA_BUS_FMT_YUV8_1X24 },
		{ MEDIA_BUS_FMT_YUV8_1X24, MEDIA_BUS_FMT_YUV8_1X24 },
	};
	struct drm_connector_state *conn_state;
	struct drm_modeset_acquire_ctx ctx;
	struct drm_bridge_chain_priv *priv;
	struct drm_crtc_state *crtc_state;
	struct drm_atomic_commit *state;
	struct drm_display_mode *mode;
	int ret;

	priv = drm_test_bridge_chain_init(test, ARRAY_SIZE(funcs), funcs);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv);

	drm_modeset_acquire_init(&ctx, 0);

	state = drm_kunit_helper_atomic_state_alloc(test, &priv->drm, &ctx);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state);

retry_commit:
	conn_state = drm_atomic_get_connector_state(state, priv->connector);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, conn_state);

	mode = drm_kunit_display_mode_from_cea_vic(test, &priv->drm, 16);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, mode);

	KUNIT_ASSERT_EQ(test, conn_state->color_format, DRM_CONNECTOR_COLOR_FORMAT_AUTO);

	ret = drm_atomic_set_crtc_for_connector(conn_state, priv->crtc);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state = drm_atomic_get_crtc_state(state, priv->crtc);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, crtc_state);

	ret = drm_atomic_set_mode_for_crtc(crtc_state, mode);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state->enable = true;
	crtc_state->active = true;

	ret = drm_atomic_commit(state);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	drm_test_bridge_chain_verify_fmt(test, priv, expected, ARRAY_SIZE(expected));

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);
}

/*
 * Test which checks that in a configuration of bridge chains where an RGB
 * producer is hooked to a YUV444-only pass-through, the atomic commit fails as
 * the bridge format selection cannot find a valid sequence of bus formats.
 */
static void drm_test_bridge_rgb_yuv_no_path(struct kunit *test)
{
	static const struct drm_bridge_funcs *funcs[] = {
		&rgb_producer_funcs,
		&yuv8_passthrough_funcs,
		&rgb8_yuv444_yuv422_passthrough_funcs,
	};
	struct drm_connector_state *conn_state;
	struct drm_modeset_acquire_ctx ctx;
	struct drm_bridge_chain_priv *priv;
	struct drm_crtc_state *crtc_state;
	struct drm_atomic_commit *state;
	struct drm_display_mode *mode;
	int ret;

	priv = drm_test_bridge_chain_init(test, ARRAY_SIZE(funcs), funcs);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv);

	drm_modeset_acquire_init(&ctx, 0);

	state = drm_kunit_helper_atomic_state_alloc(test, &priv->drm, &ctx);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state);

retry_commit:
	conn_state = drm_atomic_get_connector_state(state, priv->connector);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, conn_state);

	mode = drm_kunit_display_mode_from_cea_vic(test, &priv->drm, 16);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, mode);

	ret = drm_atomic_set_crtc_for_connector(conn_state, priv->crtc);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state = drm_atomic_get_crtc_state(state, priv->crtc);
	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, crtc_state);

	ret = drm_atomic_set_mode_for_crtc(crtc_state, mode);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_ASSERT_EQ(test, ret, 0);

	crtc_state->enable = true;
	crtc_state->active = true;

	ret = drm_atomic_commit(state);
	if (ret == -EDEADLK) {
		drm_modeset_backoff(&ctx);
		goto retry_commit;
	}
	KUNIT_EXPECT_EQ(test, ret, -ENOTSUPP);

	drm_modeset_drop_locks(&ctx);
	drm_modeset_acquire_fini(&ctx);
}

static struct kunit_case drm_bridge_bus_fmt_tests[] = {
	KUNIT_CASE(drm_test_bridge_rgb_yuv_rgb),
	KUNIT_CASE(drm_test_bridge_must_convert_to_yuv444),
	KUNIT_CASE(drm_test_bridge_hdmi_auto_rgb),
	KUNIT_CASE(drm_test_bridge_auto_first),
	KUNIT_CASE(drm_test_bridge_rgb_yuv_no_path),
	{ }
};

static struct kunit_suite drm_bridge_bus_fmt_test_suite = {
	.name = "drm_bridge_bus_fmt",
	.test_cases = drm_bridge_bus_fmt_tests,
};

kunit_test_suites(
	&drm_bridge_get_current_state_test_suite,
	&drm_bridge_helper_reset_crtc_test_suite,
	&drm_bridge_alloc_test_suite,
	&drm_bridge_bus_fmt_test_suite,
);

MODULE_AUTHOR("Maxime Ripard <mripard@kernel.org>");
MODULE_AUTHOR("Luca Ceresoli <luca.ceresoli@bootlin.com>");
MODULE_AUTHOR("Nicolas Frattaroli <nicolas.frattaroli@collabora.com>");

MODULE_DESCRIPTION("Kunit test for drm_bridge functions");
MODULE_LICENSE("GPL");