mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-12-23 02:54:32 +08:00
37418bf14c
Many drivers reference the plane->state pointer in order to get the
current plane state in their atomic_update or atomic_disable hooks,
which would be the new plane state in the global atomic state since
_swap_state happened when those hooks are run.
Use the drm_atomic_get_new_plane_state helper to get that state to make it
more obvious.
This was made using the coccinelle script below:
@ plane_atomic_func @
identifier helpers;
identifier func;
@@
(
static const struct drm_plane_helper_funcs helpers = {
...,
.atomic_disable = func,
...,
};
|
static const struct drm_plane_helper_funcs helpers = {
...,
.atomic_update = func,
...,
};
)
@ adds_new_state @
identifier plane_atomic_func.func;
identifier plane, state;
identifier new_state;
@@
func(struct drm_plane *plane, struct drm_atomic_state *state)
{
...
- struct drm_plane_state *new_state = plane->state;
+ struct drm_plane_state *new_state = drm_atomic_get_new_plane_state(state, plane);
...
}
@ include depends on adds_new_state @
@@
#include <drm/drm_atomic.h>
@ no_include depends on !include && adds_new_state @
@@
+ #include <drm/drm_atomic.h>
#include <drm/...>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Link: https://lore.kernel.org/r/20210219120032.260676-1-maxime@cerno.tech
|
||
|---|---|---|
| .. | ||
| Kconfig | ||
| Makefile | ||
| tidss_crtc.c | ||
| tidss_crtc.h | ||
| tidss_dispc_regs.h | ||
| tidss_dispc.c | ||
| tidss_dispc.h | ||
| tidss_drv.c | ||
| tidss_drv.h | ||
| tidss_encoder.c | ||
| tidss_encoder.h | ||
| tidss_irq.c | ||
| tidss_irq.h | ||
| tidss_kms.c | ||
| tidss_kms.h | ||
| tidss_plane.c | ||
| tidss_plane.h | ||
| tidss_scale_coefs.c | ||
| tidss_scale_coefs.h | ||