drm/vc4: tests: Use custom plane state for mock

The current mock planes were just using the regular drm_plane_state,
while the driver expect struct vc4_plane_state that subclasses
drm_plane_state.

Hook the proper implementations of reset, duplicate_state, destroy and
atomic_check to create vc4_plane_state.

Signed-off-by: Maxime Ripard <maxime@cerno.tech>
This commit is contained in:
Maxime Ripard
2023-04-14 11:24:37 +02:00
committed by Dom Cobley
parent 3e2550d7c9
commit ad21b8f36a
3 changed files with 26 additions and 6 deletions

View File

@@ -7,6 +7,20 @@
#include "vc4_mock.h" #include "vc4_mock.h"
static const struct drm_plane_helper_funcs vc4_dummy_plane_helper_funcs = {
.atomic_check = vc4_plane_atomic_check,
};
static const struct drm_plane_funcs vc4_dummy_plane_funcs = {
.atomic_destroy_state = vc4_plane_destroy_state,
.atomic_duplicate_state = vc4_plane_duplicate_state,
.reset = vc4_plane_reset,
};
static const uint32_t vc4_dummy_plane_formats[] = {
DRM_FORMAT_XRGB8888,
};
struct drm_plane *vc4_dummy_plane(struct kunit *test, struct drm_device *drm, struct drm_plane *vc4_dummy_plane(struct kunit *test, struct drm_device *drm,
enum drm_plane_type type) enum drm_plane_type type)
{ {

View File

@@ -1076,6 +1076,12 @@ int vc4_kms_load(struct drm_device *dev);
struct drm_plane *vc4_plane_init(struct drm_device *dev, struct drm_plane *vc4_plane_init(struct drm_device *dev,
enum drm_plane_type type, enum drm_plane_type type,
uint32_t possible_crtcs); uint32_t possible_crtcs);
void vc4_plane_reset(struct drm_plane *plane);
void vc4_plane_destroy_state(struct drm_plane *plane,
struct drm_plane_state *state);
struct drm_plane_state *vc4_plane_duplicate_state(struct drm_plane *plane);
int vc4_plane_atomic_check(struct drm_plane *plane,
struct drm_atomic_state *state);
int vc4_plane_create_additional_planes(struct drm_device *dev); int vc4_plane_create_additional_planes(struct drm_device *dev);
u32 vc4_plane_write_dlist(struct drm_plane *plane, u32 __iomem *dlist); u32 vc4_plane_write_dlist(struct drm_plane *plane, u32 __iomem *dlist);
u32 vc4_plane_dlist_size(const struct drm_plane_state *state); u32 vc4_plane_dlist_size(const struct drm_plane_state *state);

View File

@@ -280,7 +280,7 @@ static bool plane_enabled(struct drm_plane_state *state)
return state->fb && !WARN_ON(!state->crtc); return state->fb && !WARN_ON(!state->crtc);
} }
static struct drm_plane_state *vc4_plane_duplicate_state(struct drm_plane *plane) struct drm_plane_state *vc4_plane_duplicate_state(struct drm_plane *plane)
{ {
struct vc4_dev *vc4 = to_vc4_dev(plane->dev); struct vc4_dev *vc4 = to_vc4_dev(plane->dev);
struct vc4_hvs *hvs = vc4->hvs; struct vc4_hvs *hvs = vc4->hvs;
@@ -334,7 +334,7 @@ static void vc4_plane_release_upm_ida(struct vc4_hvs *hvs, unsigned int upm_hand
ida_free(&hvs->upm_handles, upm_handle); ida_free(&hvs->upm_handles, upm_handle);
} }
static void vc4_plane_destroy_state(struct drm_plane *plane, void vc4_plane_destroy_state(struct drm_plane *plane,
struct drm_plane_state *state) struct drm_plane_state *state)
{ {
struct vc4_dev *vc4 = to_vc4_dev(plane->dev); struct vc4_dev *vc4 = to_vc4_dev(plane->dev);
@@ -368,7 +368,7 @@ static void vc4_plane_destroy_state(struct drm_plane *plane,
} }
/* Called during init to allocate the plane's atomic state. */ /* Called during init to allocate the plane's atomic state. */
static void vc4_plane_reset(struct drm_plane *plane) void vc4_plane_reset(struct drm_plane *plane)
{ {
struct vc4_plane_state *vc4_state; struct vc4_plane_state *vc4_state;
@@ -2162,7 +2162,7 @@ static int vc6_plane_mode_set(struct drm_plane *plane,
* compute the dlist here and have all active plane dlists get updated * compute the dlist here and have all active plane dlists get updated
* in the CRTC's flush. * in the CRTC's flush.
*/ */
static int vc4_plane_atomic_check(struct drm_plane *plane, int vc4_plane_atomic_check(struct drm_plane *plane,
struct drm_atomic_state *state) struct drm_atomic_state *state)
{ {
struct vc4_dev *vc4 = to_vc4_dev(plane->dev); struct vc4_dev *vc4 = to_vc4_dev(plane->dev);