* [PATCH 00/10] Add helper for plane reset
@ 2018-07-20 21:14 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:14 UTC (permalink / raw)
To: linux-arm-kernel
Drivers that subclass drm_plane need to copy the logic for linking the
drm_plane with its state and to initialize core properties to their
default values. E.g (alpha and rotation)
Having a helper to reset the plane_state makes sense because of multiple
reasons:
1. Eliminate code duplication.
2. Add a single place for initializing core properties to their
default values, no need for driver to do it if what the helper sets
makes sense for them.
3. No need to debug the driver when you enable a core property and
observe it doesn't have a proper default value.
Tested with mali-dp the other drivers are just built-tested.
Alexandru Gheorghe (10):
drm/atomic: Add __drm_atomic_helper_plane_reset
drm/amd/display: Use __drm_atomic_helper_plane_reset instead of
copying the logic
drm: mali-dp: Use __drm_atomic_helper_plane_reset instead of copying
the logic
drm: atmel-hlcdc: Use __drm_atomic_helper_plane_reset instead of
copying the logic
drm/exynos: Use __drm_atomic_helper_plane_reset instead of copying the
logic
drm/imx: Use __drm_atomic_helper_plane_reset instead of copying the
logic
drm: rcar-du: Use __drm_atomic_helper_plane_reset instead of copying
the logic
drm/sun4i: Use __drm_atomic_helper_plane_reset instead of copying the
logic
drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the
logic
drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the
logic
.../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 ++--
drivers/gpu/drm/arm/malidp_planes.c | 7 ++--
.../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 5 +--
drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++------
drivers/gpu/drm/exynos/exynos_drm_plane.c | 3 +-
drivers/gpu/drm/imx/ipuv3-plane.c | 8 ++---
drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 +--
drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 4 +--
drivers/gpu/drm/sun4i/sun4i_layer.c | 4 +--
drivers/gpu/drm/vc4/vc4_plane.c | 4 +--
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 4 +--
include/drm/drm_atomic_helper.h | 2 ++
12 files changed, 39 insertions(+), 45 deletions(-)
--
2.18.0
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
There are a lot of drivers that subclass drm_plane_state, all of them
duplicate the code that links toghether the plane with plane_state.
On top of that, drivers that enable core properties also have to
duplicate the code for initializing the properties to their default
values, which in all cases are the same as the defaults from core.
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
include/drm/drm_atomic_helper.h | 2 ++
2 files changed, 25 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
index 8008a7de2e10..e1c6f101652e 100644
--- a/drivers/gpu/drm/drm_atomic_helper.c
+++ b/drivers/gpu/drm/drm_atomic_helper.c
@@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
}
EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
+/**
+ * __drm_atomic_helper_plane_reset - resets planes state to default values
+ * @plane: plane object
+ * @new_state: atomic plane state
+ *
+ * Initializes plane state to default. This is useful for drivers that subclass
+ * the plane state.
+ */
+void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
+ struct drm_plane_state *state)
+{
+ if (state) {
+ state->plane = plane;
+ state->rotation = DRM_MODE_ROTATE_0;
+ /* Reset the alpha value to fully opaque if it matters */
+ if (plane->alpha_property)
+ state->alpha = plane->alpha_property->values[1];
+ }
+ plane->state = state;
+}
+EXPORT_SYMBOL(__drm_atomic_helper_plane_reset);
+
/**
* drm_atomic_helper_plane_reset - default &drm_plane_funcs.reset hook for planes
* @plane: drm plane
@@ -3521,15 +3543,7 @@ void drm_atomic_helper_plane_reset(struct drm_plane *plane)
kfree(plane->state);
plane->state = kzalloc(sizeof(*plane->state), GFP_KERNEL);
-
- if (plane->state) {
- plane->state->plane = plane;
- plane->state->rotation = DRM_MODE_ROTATE_0;
-
- /* Reset the alpha value to fully opaque if it matters */
- if (plane->alpha_property)
- plane->state->alpha = plane->alpha_property->values[1];
- }
+ __drm_atomic_helper_plane_reset(plane, plane->state);
}
EXPORT_SYMBOL(drm_atomic_helper_plane_reset);
diff --git a/include/drm/drm_atomic_helper.h b/include/drm/drm_atomic_helper.h
index 26aaba58d6ce..2dd40c761dfd 100644
--- a/include/drm/drm_atomic_helper.h
+++ b/include/drm/drm_atomic_helper.h
@@ -155,6 +155,8 @@ void __drm_atomic_helper_crtc_destroy_state(struct drm_crtc_state *state);
void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
struct drm_crtc_state *state);
+void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
+ struct drm_plane_state *state);
void drm_atomic_helper_plane_reset(struct drm_plane *plane);
void __drm_atomic_helper_plane_duplicate_state(struct drm_plane *plane,
struct drm_plane_state *state);
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread* [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
There are a lot of drivers that subclass drm_plane_state, all of them
duplicate the code that links toghether the plane with plane_state.
On top of that, drivers that enable core properties also have to
duplicate the code for initializing the properties to their default
values, which in all cases are the same as the defaults from core.
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
include/drm/drm_atomic_helper.h | 2 ++
2 files changed, 25 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
index 8008a7de2e10..e1c6f101652e 100644
--- a/drivers/gpu/drm/drm_atomic_helper.c
+++ b/drivers/gpu/drm/drm_atomic_helper.c
@@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
}
EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
+/**
+ * __drm_atomic_helper_plane_reset - resets planes state to default values
+ * @plane: plane object
+ * @new_state: atomic plane state
+ *
+ * Initializes plane state to default. This is useful for drivers that subclass
+ * the plane state.
+ */
+void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
+ struct drm_plane_state *state)
+{
+ if (state) {
+ state->plane = plane;
+ state->rotation = DRM_MODE_ROTATE_0;
+ /* Reset the alpha value to fully opaque if it matters */
+ if (plane->alpha_property)
+ state->alpha = plane->alpha_property->values[1];
+ }
+ plane->state = state;
+}
+EXPORT_SYMBOL(__drm_atomic_helper_plane_reset);
+
/**
* drm_atomic_helper_plane_reset - default &drm_plane_funcs.reset hook for planes
* @plane: drm plane
@@ -3521,15 +3543,7 @@ void drm_atomic_helper_plane_reset(struct drm_plane *plane)
kfree(plane->state);
plane->state = kzalloc(sizeof(*plane->state), GFP_KERNEL);
-
- if (plane->state) {
- plane->state->plane = plane;
- plane->state->rotation = DRM_MODE_ROTATE_0;
-
- /* Reset the alpha value to fully opaque if it matters */
- if (plane->alpha_property)
- plane->state->alpha = plane->alpha_property->values[1];
- }
+ __drm_atomic_helper_plane_reset(plane, plane->state);
}
EXPORT_SYMBOL(drm_atomic_helper_plane_reset);
diff --git a/include/drm/drm_atomic_helper.h b/include/drm/drm_atomic_helper.h
index 26aaba58d6ce..2dd40c761dfd 100644
--- a/include/drm/drm_atomic_helper.h
+++ b/include/drm/drm_atomic_helper.h
@@ -155,6 +155,8 @@ void __drm_atomic_helper_crtc_destroy_state(struct drm_crtc_state *state);
void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
struct drm_crtc_state *state);
+void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
+ struct drm_plane_state *state);
void drm_atomic_helper_plane_reset(struct drm_plane *plane);
void __drm_atomic_helper_plane_duplicate_state(struct drm_plane *plane,
struct drm_plane_state *state);
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread* Re: [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
2018-07-20 21:15 ` Alexandru Gheorghe
@ 2018-07-23 9:37 ` Ayan Halder
-1 siblings, 0 replies; 67+ messages in thread
From: Ayan Halder @ 2018-07-23 9:37 UTC (permalink / raw)
To: Alexandru Gheorghe
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, maxime.ripard, wens, kgene,
malidp, linux-graphics-maintainer, sunpeng.li, boris.brezillon,
linux-samsung-soc, nd, Tony.Cheng, linux-arm-kernel, sw0312.kim,
nicolas.ferre, shirish.s, kyungmin.park, alexander.deucher,
christian.koenig
On Fri, Jul 20, 2018 at 10:15:00PM +0100, Alexandru Gheorghe wrote:
> There are a lot of drivers that subclass drm_plane_state, all of them
> duplicate the code that links toghether the plane with plane_state.
>
> On top of that, drivers that enable core properties also have to
> duplicate the code for initializing the properties to their default
> values, which in all cases are the same as the defaults from core.
>
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> include/drm/drm_atomic_helper.h | 2 ++
> 2 files changed, 25 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> index 8008a7de2e10..e1c6f101652e 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> }
> EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
>
> +/**
> + * __drm_atomic_helper_plane_reset - resets planes state to default values
> + * @plane: plane object
> + * @new_state: atomic plane state
> + *
> + * Initializes plane state to default. This is useful for drivers that subclass
> + * the plane state.
> + */
> +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> + struct drm_plane_state *state)
> +{
> + if (state) {
> + state->plane = plane;
> + state->rotation = DRM_MODE_ROTATE_0;
> + /* Reset the alpha value to fully opaque if it matters */
> + if (plane->alpha_property)
> + state->alpha = plane->alpha_property->values[1];
> + }
> + plane->state = state;
> +}
> +EXPORT_SYMBOL(__drm_atomic_helper_plane_reset);
> +
> /**
> * drm_atomic_helper_plane_reset - default &drm_plane_funcs.reset hook for planes
> * @plane: drm plane
> @@ -3521,15 +3543,7 @@ void drm_atomic_helper_plane_reset(struct drm_plane *plane)
>
> kfree(plane->state);
> plane->state = kzalloc(sizeof(*plane->state), GFP_KERNEL);
> -
> - if (plane->state) {
> - plane->state->plane = plane;
> - plane->state->rotation = DRM_MODE_ROTATE_0;
> -
> - /* Reset the alpha value to fully opaque if it matters */
> - if (plane->alpha_property)
> - plane->state->alpha = plane->alpha_property->values[1];
> - }
> + __drm_atomic_helper_plane_reset(plane, plane->state);
> }
> EXPORT_SYMBOL(drm_atomic_helper_plane_reset);
>
> diff --git a/include/drm/drm_atomic_helper.h b/include/drm/drm_atomic_helper.h
> index 26aaba58d6ce..2dd40c761dfd 100644
> --- a/include/drm/drm_atomic_helper.h
> +++ b/include/drm/drm_atomic_helper.h
> @@ -155,6 +155,8 @@ void __drm_atomic_helper_crtc_destroy_state(struct drm_crtc_state *state);
> void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> struct drm_crtc_state *state);
>
> +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> + struct drm_plane_state *state);
> void drm_atomic_helper_plane_reset(struct drm_plane *plane);
> void __drm_atomic_helper_plane_duplicate_state(struct drm_plane *plane,
> struct drm_plane_state *state);
> --
> 2.18.0
>
With my limited knowledge on DRM, this looks OK to me:-
Reviewed-by :- Ayan Kumar halder <ayan.halder@arm.com>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
@ 2018-07-23 9:37 ` Ayan Halder
0 siblings, 0 replies; 67+ messages in thread
From: Ayan Halder @ 2018-07-23 9:37 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jul 20, 2018 at 10:15:00PM +0100, Alexandru Gheorghe wrote:
> There are a lot of drivers that subclass drm_plane_state, all of them
> duplicate the code that links toghether the plane with plane_state.
>
> On top of that, drivers that enable core properties also have to
> duplicate the code for initializing the properties to their default
> values, which in all cases are the same as the defaults from core.
>
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> include/drm/drm_atomic_helper.h | 2 ++
> 2 files changed, 25 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> index 8008a7de2e10..e1c6f101652e 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> }
> EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
>
> +/**
> + * __drm_atomic_helper_plane_reset - resets planes state to default values
> + * @plane: plane object
> + * @new_state: atomic plane state
> + *
> + * Initializes plane state to default. This is useful for drivers that subclass
> + * the plane state.
> + */
> +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> + struct drm_plane_state *state)
> +{
> + if (state) {
> + state->plane = plane;
> + state->rotation = DRM_MODE_ROTATE_0;
> + /* Reset the alpha value to fully opaque if it matters */
> + if (plane->alpha_property)
> + state->alpha = plane->alpha_property->values[1];
> + }
> + plane->state = state;
> +}
> +EXPORT_SYMBOL(__drm_atomic_helper_plane_reset);
> +
> /**
> * drm_atomic_helper_plane_reset - default &drm_plane_funcs.reset hook for planes
> * @plane: drm plane
> @@ -3521,15 +3543,7 @@ void drm_atomic_helper_plane_reset(struct drm_plane *plane)
>
> kfree(plane->state);
> plane->state = kzalloc(sizeof(*plane->state), GFP_KERNEL);
> -
> - if (plane->state) {
> - plane->state->plane = plane;
> - plane->state->rotation = DRM_MODE_ROTATE_0;
> -
> - /* Reset the alpha value to fully opaque if it matters */
> - if (plane->alpha_property)
> - plane->state->alpha = plane->alpha_property->values[1];
> - }
> + __drm_atomic_helper_plane_reset(plane, plane->state);
> }
> EXPORT_SYMBOL(drm_atomic_helper_plane_reset);
>
> diff --git a/include/drm/drm_atomic_helper.h b/include/drm/drm_atomic_helper.h
> index 26aaba58d6ce..2dd40c761dfd 100644
> --- a/include/drm/drm_atomic_helper.h
> +++ b/include/drm/drm_atomic_helper.h
> @@ -155,6 +155,8 @@ void __drm_atomic_helper_crtc_destroy_state(struct drm_crtc_state *state);
> void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> struct drm_crtc_state *state);
>
> +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> + struct drm_plane_state *state);
> void drm_atomic_helper_plane_reset(struct drm_plane *plane);
> void __drm_atomic_helper_plane_duplicate_state(struct drm_plane *plane,
> struct drm_plane_state *state);
> --
> 2.18.0
>
With my limited knowledge on DRM, this looks OK to me:-
Reviewed-by :- Ayan Kumar halder <ayan.halder@arm.com>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread
* Re: [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
2018-07-20 21:15 ` Alexandru Gheorghe
@ 2018-07-24 7:48 ` Philipp Zabel
-1 siblings, 0 replies; 67+ messages in thread
From: Philipp Zabel @ 2018-07-24 7:48 UTC (permalink / raw)
To: Alexandru Gheorghe, seanpaul, airlied, dri-devel, liviu.dudau,
brian.starkey, malidp, ville.syrjala, daniel, gustavo,
maarten.lankhorst, alexander.deucher, christian.koenig,
David1.Zhou, harry.wentland, andrey.grodzovsky, Tony.Cheng,
sunpeng.li, shirish.s, boris.brezillon, nicolas.ferre,
alexandre.belloni, inki.dae, jy0922.shim, sw0312.kim,
kyungmin.park, kgene, krzk, linux-arm-kernel, linux-samsung-soc,
laurent.pinchart, maxime.ripard, wens, eric
Cc: nd
Hi Alexandru,
On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> There are a lot of drivers that subclass drm_plane_state, all of them
> duplicate the code that links toghether the plane with plane_state.
>
> On top of that, drivers that enable core properties also have to
> duplicate the code for initializing the properties to their default
> values, which in all cases are the same as the defaults from core.
>
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> include/drm/drm_atomic_helper.h | 2 ++
> 2 files changed, 25 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> index 8008a7de2e10..e1c6f101652e 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> }
> EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
>
> +/**
> + * __drm_atomic_helper_plane_reset - resets planes state to default values
> + * @plane: plane object
> + * @new_state: atomic plane state
> + *
> + * Initializes plane state to default. This is useful for drivers that subclass
> + * the plane state.
> + */
> +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> + struct drm_plane_state *state)
> +{
> + if (state) {
> + state->plane = plane;
> + state->rotation = DRM_MODE_ROTATE_0;
> + /* Reset the alpha value to fully opaque if it matters */
> + if (plane->alpha_property)
> + state->alpha = plane->alpha_property->values[1];
> + }
Is this function supposed to be callable with state == NULL ?
> + plane->state = state;
If so, the comment could mention that this sets plane->state to NULL if
state == NULL, and a few of the call sites could be simplified.
If not, I would remove the conditional if (state) {} part and also
mention this in the comment.
regards
Philipp
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
@ 2018-07-24 7:48 ` Philipp Zabel
0 siblings, 0 replies; 67+ messages in thread
From: Philipp Zabel @ 2018-07-24 7:48 UTC (permalink / raw)
To: linux-arm-kernel
Hi Alexandru,
On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> There are a lot of drivers that subclass drm_plane_state, all of them
> duplicate the code that links toghether the plane with plane_state.
>
> On top of that, drivers that enable core properties also have to
> duplicate the code for initializing the properties to their default
> values, which in all cases are the same as the defaults from core.
>
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> include/drm/drm_atomic_helper.h | 2 ++
> 2 files changed, 25 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> index 8008a7de2e10..e1c6f101652e 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> }
> EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
>
> +/**
> + * __drm_atomic_helper_plane_reset - resets planes state to default values
> + * @plane: plane object
> + * @new_state: atomic plane state
> + *
> + * Initializes plane state to default. This is useful for drivers that subclass
> + * the plane state.
> + */
> +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> + struct drm_plane_state *state)
> +{
> + if (state) {
> + state->plane = plane;
> + state->rotation = DRM_MODE_ROTATE_0;
> + /* Reset the alpha value to fully opaque if it matters */
> + if (plane->alpha_property)
> + state->alpha = plane->alpha_property->values[1];
> + }
Is this function supposed to be callable with state == NULL ?
> + plane->state = state;
If so, the comment could mention that this sets plane->state to NULL if
state == NULL, and a few of the call sites could be simplified.
If not, I would remove the conditional if (state) {} part and also
mention this in the comment.
regards
Philipp
^ permalink raw reply [flat|nested] 67+ messages in thread* Re: [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
2018-07-24 7:48 ` Philipp Zabel
@ 2018-07-24 8:14 ` Alexandru-Cosmin Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru-Cosmin Gheorghe @ 2018-07-24 8:14 UTC (permalink / raw)
To: Philipp Zabel
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, maxime.ripard, wens, kgene,
malidp, linux-graphics-maintainer, sunpeng.li, boris.brezillon,
linux-samsung-soc, nd, Tony.Cheng, linux-arm-kernel, sw0312.kim,
nicolas.ferre, shirish.s, kyungmin.park, alexander.deucher,
christian.koenig
On Tue, Jul 24, 2018 at 09:48:53AM +0200, Philipp Zabel wrote:
> Hi Alexandru,
>
> On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> > There are a lot of drivers that subclass drm_plane_state, all of them
> > duplicate the code that links toghether the plane with plane_state.
> >
> > On top of that, drivers that enable core properties also have to
> > duplicate the code for initializing the properties to their default
> > values, which in all cases are the same as the defaults from core.
> >
> > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > ---
> > drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> > include/drm/drm_atomic_helper.h | 2 ++
> > 2 files changed, 25 insertions(+), 9 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> > index 8008a7de2e10..e1c6f101652e 100644
> > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> > }
> > EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
> >
> > +/**
> > + * __drm_atomic_helper_plane_reset - resets planes state to default values
> > + * @plane: plane object
> > + * @new_state: atomic plane state
> > + *
> > + * Initializes plane state to default. This is useful for drivers that subclass
> > + * the plane state.
> > + */
> > +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> > + struct drm_plane_state *state)
> > +{
> > + if (state) {
> > + state->plane = plane;
> > + state->rotation = DRM_MODE_ROTATE_0;
> > + /* Reset the alpha value to fully opaque if it matters */
> > + if (plane->alpha_property)
> > + state->alpha = plane->alpha_property->values[1];
> > + }
>
> Is this function supposed to be callable with state == NULL ?
>
> > + plane->state = state;
>
> If so, the comment could mention that this sets plane->state to NULL if
> state == NULL, and a few of the call sites could be simplified.
>
> If not, I would remove the conditional if (state) {} part and also
> mention this in the comment.
Yes, It's intended to be callable with null. I will update the
comment.
>
> regards
> Philipp
--
Cheers,
Alex G
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
@ 2018-07-24 8:14 ` Alexandru-Cosmin Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru-Cosmin Gheorghe @ 2018-07-24 8:14 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jul 24, 2018 at 09:48:53AM +0200, Philipp Zabel wrote:
> Hi Alexandru,
>
> On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> > There are a lot of drivers that subclass drm_plane_state, all of them
> > duplicate the code that links toghether the plane with plane_state.
> >
> > On top of that, drivers that enable core properties also have to
> > duplicate the code for initializing the properties to their default
> > values, which in all cases are the same as the defaults from core.
> >
> > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > ---
> > drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> > include/drm/drm_atomic_helper.h | 2 ++
> > 2 files changed, 25 insertions(+), 9 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> > index 8008a7de2e10..e1c6f101652e 100644
> > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> > }
> > EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
> >
> > +/**
> > + * __drm_atomic_helper_plane_reset - resets planes state to default values
> > + * @plane: plane object
> > + * @new_state: atomic plane state
> > + *
> > + * Initializes plane state to default. This is useful for drivers that subclass
> > + * the plane state.
> > + */
> > +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> > + struct drm_plane_state *state)
> > +{
> > + if (state) {
> > + state->plane = plane;
> > + state->rotation = DRM_MODE_ROTATE_0;
> > + /* Reset the alpha value to fully opaque if it matters */
> > + if (plane->alpha_property)
> > + state->alpha = plane->alpha_property->values[1];
> > + }
>
> Is this function supposed to be callable with state == NULL ?
>
> > + plane->state = state;
>
> If so, the comment could mention that this sets plane->state to NULL if
> state == NULL, and a few of the call sites could be simplified.
>
> If not, I would remove the conditional if (state) {} part and also
> mention this in the comment.
Yes, It's intended to be callable with null. I will update the
comment.
>
> regards
> Philipp
--
Cheers,
Alex G
^ permalink raw reply [flat|nested] 67+ messages in thread* Re: [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
2018-07-24 8:14 ` Alexandru-Cosmin Gheorghe
@ 2018-07-24 8:16 ` Boris Brezillon
-1 siblings, 0 replies; 67+ messages in thread
From: Boris Brezillon @ 2018-07-24 8:16 UTC (permalink / raw)
To: Alexandru-Cosmin Gheorghe
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, maxime.ripard, wens, kgene,
malidp, linux-graphics-maintainer, sunpeng.li, linux-samsung-soc,
nd, Tony.Cheng, linux-arm-kernel, sw0312.kim, nicolas.ferre,
shirish.s, kyungmin.park, alexander.deucher, christian.koenig
On Tue, 24 Jul 2018 09:14:02 +0100
Alexandru-Cosmin Gheorghe <Alexandru-Cosmin.Gheorghe@arm.com> wrote:
> On Tue, Jul 24, 2018 at 09:48:53AM +0200, Philipp Zabel wrote:
> > Hi Alexandru,
> >
> > On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> > > There are a lot of drivers that subclass drm_plane_state, all of them
> > > duplicate the code that links toghether the plane with plane_state.
> > >
> > > On top of that, drivers that enable core properties also have to
> > > duplicate the code for initializing the properties to their default
> > > values, which in all cases are the same as the defaults from core.
> > >
> > > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > > ---
> > > drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> > > include/drm/drm_atomic_helper.h | 2 ++
> > > 2 files changed, 25 insertions(+), 9 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> > > index 8008a7de2e10..e1c6f101652e 100644
> > > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > > @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> > > }
> > > EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
> > >
> > > +/**
> > > + * __drm_atomic_helper_plane_reset - resets planes state to default values
> > > + * @plane: plane object
> > > + * @new_state: atomic plane state
> > > + *
> > > + * Initializes plane state to default. This is useful for drivers that subclass
> > > + * the plane state.
> > > + */
> > > +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> > > + struct drm_plane_state *state)
> > > +{
> > > + if (state) {
> > > + state->plane = plane;
> > > + state->rotation = DRM_MODE_ROTATE_0;
> > > + /* Reset the alpha value to fully opaque if it matters */
> > > + if (plane->alpha_property)
> > > + state->alpha = plane->alpha_property->values[1];
> > > + }
> >
> > Is this function supposed to be callable with state == NULL ?
> >
> > > + plane->state = state;
> >
> > If so, the comment could mention that this sets plane->state to NULL if
> > state == NULL, and a few of the call sites could be simplified.
> >
> > If not, I would remove the conditional if (state) {} part and also
> > mention this in the comment.
>
> Yes, It's intended to be callable with null.
May I ask why? I'd assume drivers that call this function to pass a
non-NULL plane state. What's the use case for passing a NULL state here?
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
@ 2018-07-24 8:16 ` Boris Brezillon
0 siblings, 0 replies; 67+ messages in thread
From: Boris Brezillon @ 2018-07-24 8:16 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, 24 Jul 2018 09:14:02 +0100
Alexandru-Cosmin Gheorghe <Alexandru-Cosmin.Gheorghe@arm.com> wrote:
> On Tue, Jul 24, 2018 at 09:48:53AM +0200, Philipp Zabel wrote:
> > Hi Alexandru,
> >
> > On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> > > There are a lot of drivers that subclass drm_plane_state, all of them
> > > duplicate the code that links toghether the plane with plane_state.
> > >
> > > On top of that, drivers that enable core properties also have to
> > > duplicate the code for initializing the properties to their default
> > > values, which in all cases are the same as the defaults from core.
> > >
> > > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > > ---
> > > drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> > > include/drm/drm_atomic_helper.h | 2 ++
> > > 2 files changed, 25 insertions(+), 9 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> > > index 8008a7de2e10..e1c6f101652e 100644
> > > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > > @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> > > }
> > > EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
> > >
> > > +/**
> > > + * __drm_atomic_helper_plane_reset - resets planes state to default values
> > > + * @plane: plane object
> > > + * @new_state: atomic plane state
> > > + *
> > > + * Initializes plane state to default. This is useful for drivers that subclass
> > > + * the plane state.
> > > + */
> > > +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> > > + struct drm_plane_state *state)
> > > +{
> > > + if (state) {
> > > + state->plane = plane;
> > > + state->rotation = DRM_MODE_ROTATE_0;
> > > + /* Reset the alpha value to fully opaque if it matters */
> > > + if (plane->alpha_property)
> > > + state->alpha = plane->alpha_property->values[1];
> > > + }
> >
> > Is this function supposed to be callable with state == NULL ?
> >
> > > + plane->state = state;
> >
> > If so, the comment could mention that this sets plane->state to NULL if
> > state == NULL, and a few of the call sites could be simplified.
> >
> > If not, I would remove the conditional if (state) {} part and also
> > mention this in the comment.
>
> Yes, It's intended to be callable with null.
May I ask why? I'd assume drivers that call this function to pass a
non-NULL plane state. What's the use case for passing a NULL state here?
^ permalink raw reply [flat|nested] 67+ messages in thread* Re: [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
2018-07-24 8:16 ` Boris Brezillon
@ 2018-07-24 9:28 ` Alexandru-Cosmin Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru-Cosmin Gheorghe @ 2018-07-24 9:28 UTC (permalink / raw)
To: Boris Brezillon
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, maxime.ripard, wens, kgene,
malidp, linux-graphics-maintainer, sunpeng.li, linux-samsung-soc,
nd, Tony.Cheng, linux-arm-kernel, sw0312.kim, nicolas.ferre,
shirish.s, kyungmin.park, alexander.deucher, christian.koenig
On Tue, Jul 24, 2018 at 10:16:56AM +0200, Boris Brezillon wrote:
> On Tue, 24 Jul 2018 09:14:02 +0100
> Alexandru-Cosmin Gheorghe <Alexandru-Cosmin.Gheorghe@arm.com> wrote:
>
> > On Tue, Jul 24, 2018 at 09:48:53AM +0200, Philipp Zabel wrote:
> > > Hi Alexandru,
> > >
> > > On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> > > > There are a lot of drivers that subclass drm_plane_state, all of them
> > > > duplicate the code that links toghether the plane with plane_state.
> > > >
> > > > On top of that, drivers that enable core properties also have to
> > > > duplicate the code for initializing the properties to their default
> > > > values, which in all cases are the same as the defaults from core.
> > > >
> > > > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > > > ---
> > > > drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> > > > include/drm/drm_atomic_helper.h | 2 ++
> > > > 2 files changed, 25 insertions(+), 9 deletions(-)
> > > >
> > > > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> > > > index 8008a7de2e10..e1c6f101652e 100644
> > > > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > > > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > > > @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> > > > }
> > > > EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
> > > >
> > > > +/**
> > > > + * __drm_atomic_helper_plane_reset - resets planes state to default values
> > > > + * @plane: plane object
> > > > + * @new_state: atomic plane state
> > > > + *
> > > > + * Initializes plane state to default. This is useful for drivers that subclass
> > > > + * the plane state.
> > > > + */
> > > > +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> > > > + struct drm_plane_state *state)
> > > > +{
> > > > + if (state) {
> > > > + state->plane = plane;
> > > > + state->rotation = DRM_MODE_ROTATE_0;
> > > > + /* Reset the alpha value to fully opaque if it matters */
> > > > + if (plane->alpha_property)
> > > > + state->alpha = plane->alpha_property->values[1];
> > > > + }
> > >
> > > Is this function supposed to be callable with state == NULL ?
> > >
> > > > + plane->state = state;
> > >
> > > If so, the comment could mention that this sets plane->state to NULL if
> > > state == NULL, and a few of the call sites could be simplified.
> > >
> > > If not, I would remove the conditional if (state) {} part and also
> > > mention this in the comment.
> >
> > Yes, It's intended to be callable with null.
>
> May I ask why? I'd assume drivers that call this function to pass a
> non-NULL plane state. What's the use case for passing a NULL state here?
Drivers check it, drm_atomic_helper_plane_reset doesn't.
Looking at the existing __drm_atomic_helper_plane_* it seems they all
assume state not null, so I think it probably makes more sense to do
that for this function as well.
--
Cheers,
Alex G
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 01/10] drm/atomic: Add __drm_atomic_helper_plane_reset
@ 2018-07-24 9:28 ` Alexandru-Cosmin Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru-Cosmin Gheorghe @ 2018-07-24 9:28 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jul 24, 2018 at 10:16:56AM +0200, Boris Brezillon wrote:
> On Tue, 24 Jul 2018 09:14:02 +0100
> Alexandru-Cosmin Gheorghe <Alexandru-Cosmin.Gheorghe@arm.com> wrote:
>
> > On Tue, Jul 24, 2018 at 09:48:53AM +0200, Philipp Zabel wrote:
> > > Hi Alexandru,
> > >
> > > On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> > > > There are a lot of drivers that subclass drm_plane_state, all of them
> > > > duplicate the code that links toghether the plane with plane_state.
> > > >
> > > > On top of that, drivers that enable core properties also have to
> > > > duplicate the code for initializing the properties to their default
> > > > values, which in all cases are the same as the defaults from core.
> > > >
> > > > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > > > ---
> > > > drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++++++++++--------
> > > > include/drm/drm_atomic_helper.h | 2 ++
> > > > 2 files changed, 25 insertions(+), 9 deletions(-)
> > > >
> > > > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> > > > index 8008a7de2e10..e1c6f101652e 100644
> > > > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > > > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > > > @@ -3507,6 +3507,28 @@ void drm_atomic_helper_crtc_destroy_state(struct drm_crtc *crtc,
> > > > }
> > > > EXPORT_SYMBOL(drm_atomic_helper_crtc_destroy_state);
> > > >
> > > > +/**
> > > > + * __drm_atomic_helper_plane_reset - resets planes state to default values
> > > > + * @plane: plane object
> > > > + * @new_state: atomic plane state
> > > > + *
> > > > + * Initializes plane state to default. This is useful for drivers that subclass
> > > > + * the plane state.
> > > > + */
> > > > +void __drm_atomic_helper_plane_reset(struct drm_plane *plane,
> > > > + struct drm_plane_state *state)
> > > > +{
> > > > + if (state) {
> > > > + state->plane = plane;
> > > > + state->rotation = DRM_MODE_ROTATE_0;
> > > > + /* Reset the alpha value to fully opaque if it matters */
> > > > + if (plane->alpha_property)
> > > > + state->alpha = plane->alpha_property->values[1];
> > > > + }
> > >
> > > Is this function supposed to be callable with state == NULL ?
> > >
> > > > + plane->state = state;
> > >
> > > If so, the comment could mention that this sets plane->state to NULL if
> > > state == NULL, and a few of the call sites could be simplified.
> > >
> > > If not, I would remove the conditional if (state) {} part and also
> > > mention this in the comment.
> >
> > Yes, It's intended to be callable with null.
>
> May I ask why? I'd assume drivers that call this function to pass a
> non-NULL plane state. What's the use case for passing a NULL state here?
Drivers check it, drm_atomic_helper_plane_reset doesn't.
Looking at the existing __drm_atomic_helper_plane_* it seems they all
assume state not null, so I think it probably makes more sense to do
that for this function as well.
--
Cheers,
Alex G
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 02/10] drm/amd/display: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index ca017c1dd4da..c08157686782 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -3013,11 +3013,8 @@ static void dm_drm_plane_reset(struct drm_plane *plane)
amdgpu_state = kzalloc(sizeof(*amdgpu_state), GFP_KERNEL);
WARN_ON(amdgpu_state == NULL);
- if (amdgpu_state) {
- plane->state = &amdgpu_state->base;
- plane->state->plane = plane;
- plane->state->rotation = DRM_MODE_ROTATE_0;
- }
+ if (amdgpu_state)
+ __drm_atomic_helper_plane_reset(plane, &amdgpu_state->base);
}
static struct drm_plane_state *
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread* [PATCH 02/10] drm/amd/display: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index ca017c1dd4da..c08157686782 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -3013,11 +3013,8 @@ static void dm_drm_plane_reset(struct drm_plane *plane)
amdgpu_state = kzalloc(sizeof(*amdgpu_state), GFP_KERNEL);
WARN_ON(amdgpu_state == NULL);
- if (amdgpu_state) {
- plane->state = &amdgpu_state->base;
- plane->state->plane = plane;
- plane->state->rotation = DRM_MODE_ROTATE_0;
- }
+ if (amdgpu_state)
+ __drm_atomic_helper_plane_reset(plane, &amdgpu_state->base);
}
static struct drm_plane_state *
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread
* [PATCH 03/10] drm: mali-dp: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/arm/malidp_planes.c | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/arm/malidp_planes.c b/drivers/gpu/drm/arm/malidp_planes.c
index 29409a65d864..49c37f6dd63e 100644
--- a/drivers/gpu/drm/arm/malidp_planes.c
+++ b/drivers/gpu/drm/arm/malidp_planes.c
@@ -78,11 +78,8 @@ static void malidp_plane_reset(struct drm_plane *plane)
kfree(state);
plane->state = NULL;
state = kzalloc(sizeof(*state), GFP_KERNEL);
- if (state) {
- state->base.plane = plane;
- state->base.rotation = DRM_MODE_ROTATE_0;
- plane->state = &state->base;
- }
+ if (state)
+ __drm_atomic_helper_plane_reset(plane, &state->base);
}
static struct
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread* [PATCH 03/10] drm: mali-dp: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/arm/malidp_planes.c | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/arm/malidp_planes.c b/drivers/gpu/drm/arm/malidp_planes.c
index 29409a65d864..49c37f6dd63e 100644
--- a/drivers/gpu/drm/arm/malidp_planes.c
+++ b/drivers/gpu/drm/arm/malidp_planes.c
@@ -78,11 +78,8 @@ static void malidp_plane_reset(struct drm_plane *plane)
kfree(state);
plane->state = NULL;
state = kzalloc(sizeof(*state), GFP_KERNEL);
- if (state) {
- state->base.plane = plane;
- state->base.rotation = DRM_MODE_ROTATE_0;
- plane->state = &state->base;
- }
+ if (state)
+ __drm_atomic_helper_plane_reset(plane, &state->base);
}
static struct
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread* Re: [PATCH 03/10] drm: mali-dp: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:15 ` Alexandru Gheorghe
@ 2018-07-23 9:35 ` Ayan Halder
-1 siblings, 0 replies; 67+ messages in thread
From: Ayan Halder @ 2018-07-23 9:35 UTC (permalink / raw)
To: Alexandru Gheorghe
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, maxime.ripard, wens, kgene,
malidp, linux-graphics-maintainer, sunpeng.li, boris.brezillon,
linux-samsung-soc, nd, Tony.Cheng, linux-arm-kernel, sw0312.kim,
nicolas.ferre, shirish.s, kyungmin.park, alexander.deucher,
christian.koenig
On Fri, Jul 20, 2018 at 10:15:02PM +0100, Alexandru Gheorghe wrote:
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/arm/malidp_planes.c | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/arm/malidp_planes.c b/drivers/gpu/drm/arm/malidp_planes.c
> index 29409a65d864..49c37f6dd63e 100644
> --- a/drivers/gpu/drm/arm/malidp_planes.c
> +++ b/drivers/gpu/drm/arm/malidp_planes.c
> @@ -78,11 +78,8 @@ static void malidp_plane_reset(struct drm_plane *plane)
> kfree(state);
> plane->state = NULL;
> state = kzalloc(sizeof(*state), GFP_KERNEL);
> - if (state) {
> - state->base.plane = plane;
> - state->base.rotation = DRM_MODE_ROTATE_0;
> - plane->state = &state->base;
> - }
> + if (state)
> + __drm_atomic_helper_plane_reset(plane, &state->base);
> }
>
> static struct
> --
> 2.18.0
>
A commit description like the following might be useful (though the
patch is a part of series) :-
Invoke a newly added drm atomic helper function
(__drm_atomic_helper_plane_reset) to reset the 'drm_plane_state' to
its default values for rotation, alpha and plane.
Otherwise, with my limited knowledge on DRM, this looks OK to me:-
Reviewed-by :- Ayan Kumar halder <ayan.halder@arm.com>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 03/10] drm: mali-dp: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-23 9:35 ` Ayan Halder
0 siblings, 0 replies; 67+ messages in thread
From: Ayan Halder @ 2018-07-23 9:35 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jul 20, 2018 at 10:15:02PM +0100, Alexandru Gheorghe wrote:
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/arm/malidp_planes.c | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/arm/malidp_planes.c b/drivers/gpu/drm/arm/malidp_planes.c
> index 29409a65d864..49c37f6dd63e 100644
> --- a/drivers/gpu/drm/arm/malidp_planes.c
> +++ b/drivers/gpu/drm/arm/malidp_planes.c
> @@ -78,11 +78,8 @@ static void malidp_plane_reset(struct drm_plane *plane)
> kfree(state);
> plane->state = NULL;
> state = kzalloc(sizeof(*state), GFP_KERNEL);
> - if (state) {
> - state->base.plane = plane;
> - state->base.rotation = DRM_MODE_ROTATE_0;
> - plane->state = &state->base;
> - }
> + if (state)
> + __drm_atomic_helper_plane_reset(plane, &state->base);
> }
>
> static struct
> --
> 2.18.0
>
A commit description like the following might be useful (though the
patch is a part of series) :-
Invoke a newly added drm atomic helper function
(__drm_atomic_helper_plane_reset) to reset the 'drm_plane_state' to
its default values for rotation, alpha and plane.
Otherwise, with my limited knowledge on DRM, this looks OK to me:-
Reviewed-by :- Ayan Kumar halder <ayan.halder@arm.com>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread
* Re: [PATCH 03/10] drm: mali-dp: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:15 ` Alexandru Gheorghe
@ 2018-07-24 8:26 ` Liviu Dudau
-1 siblings, 0 replies; 67+ messages in thread
From: Liviu Dudau @ 2018-07-24 8:26 UTC (permalink / raw)
To: Alexandru Gheorghe
Cc: alexandre.belloni, airlied, dri-devel, laurent.pinchart,
thellstrom, krzk, maxime.ripard, wens, kgene, malidp,
linux-graphics-maintainer, sunpeng.li, boris.brezillon,
linux-samsung-soc, nd, Tony.Cheng, linux-arm-kernel, sw0312.kim,
nicolas.ferre, shirish.s, kyungmin.park, alexander.deucher,
christian.koenig
On Fri, Jul 20, 2018 at 10:15:02PM +0100, Alexandru Gheorghe wrote:
With a more detailed commit message:
Acked-by: Liviu Dudau <liviu.dudau@arm.com>
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/arm/malidp_planes.c | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/arm/malidp_planes.c b/drivers/gpu/drm/arm/malidp_planes.c
> index 29409a65d864..49c37f6dd63e 100644
> --- a/drivers/gpu/drm/arm/malidp_planes.c
> +++ b/drivers/gpu/drm/arm/malidp_planes.c
> @@ -78,11 +78,8 @@ static void malidp_plane_reset(struct drm_plane *plane)
> kfree(state);
> plane->state = NULL;
> state = kzalloc(sizeof(*state), GFP_KERNEL);
> - if (state) {
> - state->base.plane = plane;
> - state->base.rotation = DRM_MODE_ROTATE_0;
> - plane->state = &state->base;
> - }
> + if (state)
> + __drm_atomic_helper_plane_reset(plane, &state->base);
> }
>
> static struct
> --
> 2.18.0
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
¯\_(ツ)_/¯
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 03/10] drm: mali-dp: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-24 8:26 ` Liviu Dudau
0 siblings, 0 replies; 67+ messages in thread
From: Liviu Dudau @ 2018-07-24 8:26 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jul 20, 2018 at 10:15:02PM +0100, Alexandru Gheorghe wrote:
With a more detailed commit message:
Acked-by: Liviu Dudau <liviu.dudau@arm.com>
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/arm/malidp_planes.c | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/arm/malidp_planes.c b/drivers/gpu/drm/arm/malidp_planes.c
> index 29409a65d864..49c37f6dd63e 100644
> --- a/drivers/gpu/drm/arm/malidp_planes.c
> +++ b/drivers/gpu/drm/arm/malidp_planes.c
> @@ -78,11 +78,8 @@ static void malidp_plane_reset(struct drm_plane *plane)
> kfree(state);
> plane->state = NULL;
> state = kzalloc(sizeof(*state), GFP_KERNEL);
> - if (state) {
> - state->base.plane = plane;
> - state->base.rotation = DRM_MODE_ROTATE_0;
> - plane->state = &state->base;
> - }
> + if (state)
> + __drm_atomic_helper_plane_reset(plane, &state->base);
> }
>
> static struct
> --
> 2.18.0
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
?\_(?)_/?
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 04/10] drm: atmel-hlcdc: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
index 04440064b9b7..9330a076e15a 100644
--- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
+++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
@@ -942,10 +942,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
"Failed to allocate initial plane state\n");
return;
}
-
- p->state = &state->base;
- p->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
- p->state->plane = p;
+ __drm_atomic_helper_plane_reset(p, &state->base);
}
}
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread
* [PATCH 04/10] drm: atmel-hlcdc: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
index 04440064b9b7..9330a076e15a 100644
--- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
+++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
@@ -942,10 +942,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
"Failed to allocate initial plane state\n");
return;
}
-
- p->state = &state->base;
- p->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
- p->state->plane = p;
+ __drm_atomic_helper_plane_reset(p, &state->base);
}
}
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread
* Re: [PATCH 04/10] drm: atmel-hlcdc: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:15 ` Alexandru Gheorghe
@ 2018-07-21 6:12 ` Boris Brezillon
-1 siblings, 0 replies; 67+ messages in thread
From: Boris Brezillon @ 2018-07-21 6:12 UTC (permalink / raw)
To: Alexandru Gheorghe
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, maxime.ripard, wens, kgene,
malidp, linux-graphics-maintainer, sunpeng.li, linux-samsung-soc,
nd, Tony.Cheng, linux-arm-kernel, sw0312.kim, nicolas.ferre,
shirish.s, kyungmin.park, alexander.deucher, christian.koenig
Hi Alexandru,
On Fri, 20 Jul 2018 22:15:03 +0100
Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com> wrote:
Please add a commit message here (even if it's just repeating what the
subject says).
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
With this addressed:
Acked-by: Boris Brezillon <boris.brezillon@bootlin.com>
> ---
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> index 04440064b9b7..9330a076e15a 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> @@ -942,10 +942,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
> "Failed to allocate initial plane state\n");
> return;
> }
> -
> - p->state = &state->base;
> - p->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
> - p->state->plane = p;
> + __drm_atomic_helper_plane_reset(p, &state->base);
> }
> }
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 04/10] drm: atmel-hlcdc: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-21 6:12 ` Boris Brezillon
0 siblings, 0 replies; 67+ messages in thread
From: Boris Brezillon @ 2018-07-21 6:12 UTC (permalink / raw)
To: linux-arm-kernel
Hi Alexandru,
On Fri, 20 Jul 2018 22:15:03 +0100
Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com> wrote:
Please add a commit message here (even if it's just repeating what the
subject says).
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
With this addressed:
Acked-by: Boris Brezillon <boris.brezillon@bootlin.com>
> ---
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> index 04440064b9b7..9330a076e15a 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> @@ -942,10 +942,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
> "Failed to allocate initial plane state\n");
> return;
> }
> -
> - p->state = &state->base;
> - p->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
> - p->state->plane = p;
> + __drm_atomic_helper_plane_reset(p, &state->base);
> }
> }
>
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 05/10] drm/exynos: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/exynos/exynos_drm_plane.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_plane.c b/drivers/gpu/drm/exynos/exynos_drm_plane.c
index eb9915da7dec..681328fbe7de 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_plane.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_plane.c
@@ -139,8 +139,7 @@ static void exynos_drm_plane_reset(struct drm_plane *plane)
exynos_state = kzalloc(sizeof(*exynos_state), GFP_KERNEL);
if (exynos_state) {
- plane->state = &exynos_state->base;
- plane->state->plane = plane;
+ __drm_atomic_helper_plane_reset(plane, &exynos_state->base);
plane->state->zpos = exynos_plane->config->zpos;
}
}
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread* [PATCH 05/10] drm/exynos: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/exynos/exynos_drm_plane.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_plane.c b/drivers/gpu/drm/exynos/exynos_drm_plane.c
index eb9915da7dec..681328fbe7de 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_plane.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_plane.c
@@ -139,8 +139,7 @@ static void exynos_drm_plane_reset(struct drm_plane *plane)
exynos_state = kzalloc(sizeof(*exynos_state), GFP_KERNEL);
if (exynos_state) {
- plane->state = &exynos_state->base;
- plane->state->plane = plane;
+ __drm_atomic_helper_plane_reset(plane, &exynos_state->base);
plane->state->zpos = exynos_plane->config->zpos;
}
}
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread
* [PATCH 06/10] drm/imx: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/imx/ipuv3-plane.c | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c
index 203f247d4854..1bd4de03ce9e 100644
--- a/drivers/gpu/drm/imx/ipuv3-plane.c
+++ b/drivers/gpu/drm/imx/ipuv3-plane.c
@@ -281,16 +281,14 @@ static void ipu_plane_state_reset(struct drm_plane *plane)
ipu_state = to_ipu_plane_state(plane->state);
__drm_atomic_helper_plane_destroy_state(plane->state);
kfree(ipu_state);
+ plane->state = NULL;
}
ipu_state = kzalloc(sizeof(*ipu_state), GFP_KERNEL);
- if (ipu_state) {
- ipu_state->base.plane = plane;
- ipu_state->base.rotation = DRM_MODE_ROTATE_0;
- }
+ if (ipu_state)
+ __drm_atomic_helper_plane_reset(plane, &ipu_state->base);
- plane->state = &ipu_state->base;
}
static struct drm_plane_state *
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread* [PATCH 06/10] drm/imx: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/imx/ipuv3-plane.c | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c
index 203f247d4854..1bd4de03ce9e 100644
--- a/drivers/gpu/drm/imx/ipuv3-plane.c
+++ b/drivers/gpu/drm/imx/ipuv3-plane.c
@@ -281,16 +281,14 @@ static void ipu_plane_state_reset(struct drm_plane *plane)
ipu_state = to_ipu_plane_state(plane->state);
__drm_atomic_helper_plane_destroy_state(plane->state);
kfree(ipu_state);
+ plane->state = NULL;
}
ipu_state = kzalloc(sizeof(*ipu_state), GFP_KERNEL);
- if (ipu_state) {
- ipu_state->base.plane = plane;
- ipu_state->base.rotation = DRM_MODE_ROTATE_0;
- }
+ if (ipu_state)
+ __drm_atomic_helper_plane_reset(plane, &ipu_state->base);
- plane->state = &ipu_state->base;
}
static struct drm_plane_state *
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread* Re: [PATCH 06/10] drm/imx: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:15 ` Alexandru Gheorghe
@ 2018-07-24 7:49 ` Philipp Zabel
-1 siblings, 0 replies; 67+ messages in thread
From: Philipp Zabel @ 2018-07-24 7:49 UTC (permalink / raw)
To: Alexandru Gheorghe, seanpaul, airlied, dri-devel, liviu.dudau,
brian.starkey, malidp, ville.syrjala, daniel, gustavo,
maarten.lankhorst, alexander.deucher, christian.koenig,
David1.Zhou, harry.wentland, andrey.grodzovsky, Tony.Cheng,
sunpeng.li, shirish.s, boris.brezillon, nicolas.ferre,
alexandre.belloni, inki.dae, jy0922.shim, sw0312.kim,
kyungmin.park, kgene, krzk, linux-arm-kernel, linux-samsung-soc,
laurent.pinchart, maxime.ripard, wens, eric
Cc: nd
Hi Alexandru,
On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/imx/ipuv3-plane.c | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c
> index 203f247d4854..1bd4de03ce9e 100644
> --- a/drivers/gpu/drm/imx/ipuv3-plane.c
> +++ b/drivers/gpu/drm/imx/ipuv3-plane.c
> @@ -281,16 +281,14 @@ static void ipu_plane_state_reset(struct drm_plane *plane)
> ipu_state = to_ipu_plane_state(plane->state);
> __drm_atomic_helper_plane_destroy_state(plane->state);
> kfree(ipu_state);
> + plane->state = NULL;
With __drm_atomic_helper_plane_reset as-is, this could be dropped ...
> }
>
> ipu_state = kzalloc(sizeof(*ipu_state), GFP_KERNEL);
>
> - if (ipu_state) {
> - ipu_state->base.plane = plane;
> - ipu_state->base.rotation = DRM_MODE_ROTATE_0;
> - }
> + if (ipu_state)
> + __drm_atomic_helper_plane_reset(plane, &ipu_state->base);
... if this is change to just
__drm_atomic_helper_plane_reset(plane, &ipu_state->base);
>
> - plane->state = &ipu_state->base;
> }
>
> static struct drm_plane_state *
regards
Philipp
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 06/10] drm/imx: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-24 7:49 ` Philipp Zabel
0 siblings, 0 replies; 67+ messages in thread
From: Philipp Zabel @ 2018-07-24 7:49 UTC (permalink / raw)
To: linux-arm-kernel
Hi Alexandru,
On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/imx/ipuv3-plane.c | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c
> index 203f247d4854..1bd4de03ce9e 100644
> --- a/drivers/gpu/drm/imx/ipuv3-plane.c
> +++ b/drivers/gpu/drm/imx/ipuv3-plane.c
> @@ -281,16 +281,14 @@ static void ipu_plane_state_reset(struct drm_plane *plane)
> ipu_state = to_ipu_plane_state(plane->state);
> __drm_atomic_helper_plane_destroy_state(plane->state);
> kfree(ipu_state);
> + plane->state = NULL;
With __drm_atomic_helper_plane_reset as-is, this could be dropped ...
> }
>
> ipu_state = kzalloc(sizeof(*ipu_state), GFP_KERNEL);
>
> - if (ipu_state) {
> - ipu_state->base.plane = plane;
> - ipu_state->base.rotation = DRM_MODE_ROTATE_0;
> - }
> + if (ipu_state)
> + __drm_atomic_helper_plane_reset(plane, &ipu_state->base);
... if this is change to just
__drm_atomic_helper_plane_reset(plane, &ipu_state->base);
>
> - plane->state = &ipu_state->base;
> }
>
> static struct drm_plane_state *
regards
Philipp
^ permalink raw reply [flat|nested] 67+ messages in thread* Re: [PATCH 06/10] drm/imx: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-24 7:49 ` Philipp Zabel
@ 2018-07-24 7:51 ` Philipp Zabel
-1 siblings, 0 replies; 67+ messages in thread
From: Philipp Zabel @ 2018-07-24 7:51 UTC (permalink / raw)
To: Alexandru Gheorghe, seanpaul, airlied, dri-devel, liviu.dudau,
brian.starkey, malidp, ville.syrjala, daniel, gustavo,
maarten.lankhorst, alexander.deucher, christian.koenig,
David1.Zhou, harry.wentland, andrey.grodzovsky, Tony.Cheng,
sunpeng.li, shirish.s, boris.brezillon, nicolas.ferre,
alexandre.belloni, inki.dae, jy0922.shim, sw0312.kim,
kyungmin.park, kgene, krzk, linux-arm-kernel, linux-samsung-soc,
laurent.pinchart, maxime.ripard, wens, eric
Cc: nd
On Tue, 2018-07-24 at 09:49 +0200, Philipp Zabel wrote:
> Hi Alexandru,
>
> On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > ---
> > drivers/gpu/drm/imx/ipuv3-plane.c | 8 +++-----
> > 1 file changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c
> > index 203f247d4854..1bd4de03ce9e 100644
> > --- a/drivers/gpu/drm/imx/ipuv3-plane.c
> > +++ b/drivers/gpu/drm/imx/ipuv3-plane.c
> > @@ -281,16 +281,14 @@ static void ipu_plane_state_reset(struct drm_plane *plane)
> > ipu_state = to_ipu_plane_state(plane->state);
> > __drm_atomic_helper_plane_destroy_state(plane->state);
> > kfree(ipu_state);
> > + plane->state = NULL;
>
> With __drm_atomic_helper_plane_reset as-is, this could be dropped ...
>
> > }
> >
> > ipu_state = kzalloc(sizeof(*ipu_state), GFP_KERNEL);
> >
> > - if (ipu_state) {
> > - ipu_state->base.plane = plane;
> > - ipu_state->base.rotation = DRM_MODE_ROTATE_0;
> > - }
> > + if (ipu_state)
> > + __drm_atomic_helper_plane_reset(plane, &ipu_state->base);
>
> ... if this is change to just
>
> __drm_atomic_helper_plane_reset(plane, &ipu_state->base);
Euh, obviously that won't be possible, sorry for the noise.
I blame the heat.
regards
Philipp
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread* [PATCH 06/10] drm/imx: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-24 7:51 ` Philipp Zabel
0 siblings, 0 replies; 67+ messages in thread
From: Philipp Zabel @ 2018-07-24 7:51 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, 2018-07-24 at 09:49 +0200, Philipp Zabel wrote:
> Hi Alexandru,
>
> On Fri, 2018-07-20 at 22:15 +0100, Alexandru Gheorghe wrote:
> > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > ---
> > drivers/gpu/drm/imx/ipuv3-plane.c | 8 +++-----
> > 1 file changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c
> > index 203f247d4854..1bd4de03ce9e 100644
> > --- a/drivers/gpu/drm/imx/ipuv3-plane.c
> > +++ b/drivers/gpu/drm/imx/ipuv3-plane.c
> > @@ -281,16 +281,14 @@ static void ipu_plane_state_reset(struct drm_plane *plane)
> > ipu_state = to_ipu_plane_state(plane->state);
> > __drm_atomic_helper_plane_destroy_state(plane->state);
> > kfree(ipu_state);
> > + plane->state = NULL;
>
> With __drm_atomic_helper_plane_reset as-is, this could be dropped ...
>
> > }
> >
> > ipu_state = kzalloc(sizeof(*ipu_state), GFP_KERNEL);
> >
> > - if (ipu_state) {
> > - ipu_state->base.plane = plane;
> > - ipu_state->base.rotation = DRM_MODE_ROTATE_0;
> > - }
> > + if (ipu_state)
> > + __drm_atomic_helper_plane_reset(plane, &ipu_state->base);
>
> ... if this is change to just
>
> __drm_atomic_helper_plane_reset(plane, &ipu_state->base);
Euh, obviously that won't be possible, sorry for the noise.
I blame the heat.
regards
Philipp
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 07/10] drm: rcar-du: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 +---
drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 4 +---
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_plane.c b/drivers/gpu/drm/rcar-du/rcar_du_plane.c
index c20f7ed48c8d..19a9d5f6db1c 100644
--- a/drivers/gpu/drm/rcar-du/rcar_du_plane.c
+++ b/drivers/gpu/drm/rcar-du/rcar_du_plane.c
@@ -689,15 +689,13 @@ static void rcar_du_plane_reset(struct drm_plane *plane)
state = kzalloc(sizeof(*state), GFP_KERNEL);
if (state == NULL)
return;
+ __drm_atomic_helper_plane_reset(plane, &state->state);
state->hwindex = -1;
state->source = RCAR_DU_PLANE_MEMORY;
state->colorkey = RCAR_DU_COLORKEY_NONE;
state->state.zpos = plane->type == DRM_PLANE_TYPE_PRIMARY ? 0 : 1;
- plane->state = &state->state;
- plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
- plane->state->plane = plane;
}
static int rcar_du_plane_atomic_set_property(struct drm_plane *plane,
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
index 72eebeda518e..0a0aa490f805 100644
--- a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
+++ b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
@@ -346,11 +346,9 @@ static void rcar_du_vsp_plane_reset(struct drm_plane *plane)
if (state == NULL)
return;
- state->state.alpha = DRM_BLEND_ALPHA_OPAQUE;
+ __drm_atomic_helper_plane_reset(plane, &state->state);
state->state.zpos = plane->type == DRM_PLANE_TYPE_PRIMARY ? 0 : 1;
- plane->state = &state->state;
- plane->state->plane = plane;
}
static const struct drm_plane_funcs rcar_du_vsp_plane_funcs = {
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread* [PATCH 07/10] drm: rcar-du: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 +---
drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 4 +---
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_plane.c b/drivers/gpu/drm/rcar-du/rcar_du_plane.c
index c20f7ed48c8d..19a9d5f6db1c 100644
--- a/drivers/gpu/drm/rcar-du/rcar_du_plane.c
+++ b/drivers/gpu/drm/rcar-du/rcar_du_plane.c
@@ -689,15 +689,13 @@ static void rcar_du_plane_reset(struct drm_plane *plane)
state = kzalloc(sizeof(*state), GFP_KERNEL);
if (state == NULL)
return;
+ __drm_atomic_helper_plane_reset(plane, &state->state);
state->hwindex = -1;
state->source = RCAR_DU_PLANE_MEMORY;
state->colorkey = RCAR_DU_COLORKEY_NONE;
state->state.zpos = plane->type == DRM_PLANE_TYPE_PRIMARY ? 0 : 1;
- plane->state = &state->state;
- plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
- plane->state->plane = plane;
}
static int rcar_du_plane_atomic_set_property(struct drm_plane *plane,
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
index 72eebeda518e..0a0aa490f805 100644
--- a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
+++ b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
@@ -346,11 +346,9 @@ static void rcar_du_vsp_plane_reset(struct drm_plane *plane)
if (state == NULL)
return;
- state->state.alpha = DRM_BLEND_ALPHA_OPAQUE;
+ __drm_atomic_helper_plane_reset(plane, &state->state);
state->state.zpos = plane->type == DRM_PLANE_TYPE_PRIMARY ? 0 : 1;
- plane->state = &state->state;
- plane->state->plane = plane;
}
static const struct drm_plane_funcs rcar_du_vsp_plane_funcs = {
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread
* [PATCH 08/10] drm/sun4i: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/sun4i/sun4i_layer.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/sun4i/sun4i_layer.c b/drivers/gpu/drm/sun4i/sun4i_layer.c
index 750ad24de1d7..78f77af8805a 100644
--- a/drivers/gpu/drm/sun4i/sun4i_layer.c
+++ b/drivers/gpu/drm/sun4i/sun4i_layer.c
@@ -35,9 +35,7 @@ static void sun4i_backend_layer_reset(struct drm_plane *plane)
state = kzalloc(sizeof(*state), GFP_KERNEL);
if (state) {
- plane->state = &state->state;
- plane->state->plane = plane;
- plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
+ __drm_atomic_helper_plane_reset(plane, &state->state);
plane->state->zpos = layer->id;
}
}
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread* [PATCH 08/10] drm/sun4i: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/sun4i/sun4i_layer.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/sun4i/sun4i_layer.c b/drivers/gpu/drm/sun4i/sun4i_layer.c
index 750ad24de1d7..78f77af8805a 100644
--- a/drivers/gpu/drm/sun4i/sun4i_layer.c
+++ b/drivers/gpu/drm/sun4i/sun4i_layer.c
@@ -35,9 +35,7 @@ static void sun4i_backend_layer_reset(struct drm_plane *plane)
state = kzalloc(sizeof(*state), GFP_KERNEL);
if (state) {
- plane->state = &state->state;
- plane->state->plane = plane;
- plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
+ __drm_atomic_helper_plane_reset(plane, &state->state);
plane->state->zpos = layer->id;
}
}
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread
* [PATCH 09/10] drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/vc4/vc4_plane.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c
index 9d7a36f148cf..688ad9bb0f08 100644
--- a/drivers/gpu/drm/vc4/vc4_plane.c
+++ b/drivers/gpu/drm/vc4/vc4_plane.c
@@ -200,9 +200,7 @@ static void vc4_plane_reset(struct drm_plane *plane)
if (!vc4_state)
return;
- plane->state = &vc4_state->base;
- plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
- vc4_state->base.plane = plane;
+ __drm_atomic_helper_plane_reset(plane, &vc4_state->base);
}
static void vc4_dlist_write(struct vc4_plane_state *vc4_state, u32 val)
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread
* [PATCH 09/10] drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/vc4/vc4_plane.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c
index 9d7a36f148cf..688ad9bb0f08 100644
--- a/drivers/gpu/drm/vc4/vc4_plane.c
+++ b/drivers/gpu/drm/vc4/vc4_plane.c
@@ -200,9 +200,7 @@ static void vc4_plane_reset(struct drm_plane *plane)
if (!vc4_state)
return;
- plane->state = &vc4_state->base;
- plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
- vc4_state->base.plane = plane;
+ __drm_atomic_helper_plane_reset(plane, &vc4_state->base);
}
static void vc4_dlist_write(struct vc4_plane_state *vc4_state, u32 val)
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread
* Re: [PATCH 09/10] drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:15 ` Alexandru Gheorghe
@ 2018-07-24 6:57 ` Maxime Ripard
-1 siblings, 0 replies; 67+ messages in thread
From: Maxime Ripard @ 2018-07-24 6:57 UTC (permalink / raw)
To: Alexandru Gheorghe
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, wens, kgene, malidp,
linux-graphics-maintainer, sunpeng.li, boris.brezillon,
linux-samsung-soc, nd, Tony.Cheng, linux-arm-kernel, sw0312.kim,
nicolas.ferre, shirish.s, kyungmin.park, alexander.deucher,
christian.koenig
[-- Attachment #1.1: Type: text/plain, Size: 1122 bytes --]
On Fri, Jul 20, 2018 at 10:15:08PM +0100, Alexandru Gheorghe wrote:
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/vc4/vc4_plane.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c
> index 9d7a36f148cf..688ad9bb0f08 100644
> --- a/drivers/gpu/drm/vc4/vc4_plane.c
> +++ b/drivers/gpu/drm/vc4/vc4_plane.c
> @@ -200,9 +200,7 @@ static void vc4_plane_reset(struct drm_plane *plane)
> if (!vc4_state)
> return;
>
> - plane->state = &vc4_state->base;
> - plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
> - vc4_state->base.plane = plane;
> + __drm_atomic_helper_plane_reset(plane, &vc4_state->base);
For vc4, rcar-du, atmel-hlcdc and sun4i, you're changing the reset
value of alpha from DRM_BLEND_ALPHA_OPAQUE to
plane->alpha_property->values[1].
This might or might not be a good idea, but you should definitely
explain why.
Maxime
--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 09/10] drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-24 6:57 ` Maxime Ripard
0 siblings, 0 replies; 67+ messages in thread
From: Maxime Ripard @ 2018-07-24 6:57 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jul 20, 2018 at 10:15:08PM +0100, Alexandru Gheorghe wrote:
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/vc4/vc4_plane.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c
> index 9d7a36f148cf..688ad9bb0f08 100644
> --- a/drivers/gpu/drm/vc4/vc4_plane.c
> +++ b/drivers/gpu/drm/vc4/vc4_plane.c
> @@ -200,9 +200,7 @@ static void vc4_plane_reset(struct drm_plane *plane)
> if (!vc4_state)
> return;
>
> - plane->state = &vc4_state->base;
> - plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
> - vc4_state->base.plane = plane;
> + __drm_atomic_helper_plane_reset(plane, &vc4_state->base);
For vc4, rcar-du, atmel-hlcdc and sun4i, you're changing the reset
value of alpha from DRM_BLEND_ALPHA_OPAQUE to
plane->alpha_property->values[1].
This might or might not be a good idea, but you should definitely
explain why.
Maxime
--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180724/68382632/attachment.sig>
^ permalink raw reply [flat|nested] 67+ messages in thread
* Re: [PATCH 09/10] drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-24 6:57 ` Maxime Ripard
@ 2018-07-24 8:11 ` Alexandru-Cosmin Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru-Cosmin Gheorghe @ 2018-07-24 8:11 UTC (permalink / raw)
To: Maxime Ripard
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, wens, kgene, malidp,
linux-graphics-maintainer, sunpeng.li, boris.brezillon,
linux-samsung-soc, nd, Tony.Cheng, linux-arm-kernel, sw0312.kim,
nicolas.ferre, shirish.s, kyungmin.park, alexander.deucher,
christian.koenig
Hi Maxime,
On Tue, Jul 24, 2018 at 08:57:20AM +0200, Maxime Ripard wrote:
> On Fri, Jul 20, 2018 at 10:15:08PM +0100, Alexandru Gheorghe wrote:
> > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > ---
> > drivers/gpu/drm/vc4/vc4_plane.c | 4 +---
> > 1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c
> > index 9d7a36f148cf..688ad9bb0f08 100644
> > --- a/drivers/gpu/drm/vc4/vc4_plane.c
> > +++ b/drivers/gpu/drm/vc4/vc4_plane.c
> > @@ -200,9 +200,7 @@ static void vc4_plane_reset(struct drm_plane *plane)
> > if (!vc4_state)
> > return;
> >
> > - plane->state = &vc4_state->base;
> > - plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
> > - vc4_state->base.plane = plane;
> > + __drm_atomic_helper_plane_reset(plane, &vc4_state->base);
>
> For vc4, rcar-du, atmel-hlcdc and sun4i, you're changing the reset
> value of alpha from DRM_BLEND_ALPHA_OPAQUE to
> plane->alpha_property->values[1].
plane->alpha_property->values[1] holds the max value for alpha, and
it's initialized by the core to DRM_BLEND_ALPHA_OPAQUE.
>
> This might or might not be a good idea, but you should definitely
> explain why.
Would a commit message suffice? Or do you have other ideas?
>
> Maxime
>
> --
> Maxime Ripard, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
> https://bootlin.com
--
Cheers,
Alex G
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 09/10] drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-24 8:11 ` Alexandru-Cosmin Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru-Cosmin Gheorghe @ 2018-07-24 8:11 UTC (permalink / raw)
To: linux-arm-kernel
Hi Maxime,
On Tue, Jul 24, 2018 at 08:57:20AM +0200, Maxime Ripard wrote:
> On Fri, Jul 20, 2018 at 10:15:08PM +0100, Alexandru Gheorghe wrote:
> > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > ---
> > drivers/gpu/drm/vc4/vc4_plane.c | 4 +---
> > 1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c
> > index 9d7a36f148cf..688ad9bb0f08 100644
> > --- a/drivers/gpu/drm/vc4/vc4_plane.c
> > +++ b/drivers/gpu/drm/vc4/vc4_plane.c
> > @@ -200,9 +200,7 @@ static void vc4_plane_reset(struct drm_plane *plane)
> > if (!vc4_state)
> > return;
> >
> > - plane->state = &vc4_state->base;
> > - plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
> > - vc4_state->base.plane = plane;
> > + __drm_atomic_helper_plane_reset(plane, &vc4_state->base);
>
> For vc4, rcar-du, atmel-hlcdc and sun4i, you're changing the reset
> value of alpha from DRM_BLEND_ALPHA_OPAQUE to
> plane->alpha_property->values[1].
plane->alpha_property->values[1] holds the max value for alpha, and
it's initialized by the core to DRM_BLEND_ALPHA_OPAQUE.
>
> This might or might not be a good idea, but you should definitely
> explain why.
Would a commit message suffice? Or do you have other ideas?
>
> Maxime
>
> --
> Maxime Ripard, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
> https://bootlin.com
--
Cheers,
Alex G
^ permalink raw reply [flat|nested] 67+ messages in thread
* Re: [PATCH 09/10] drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-24 8:11 ` Alexandru-Cosmin Gheorghe
@ 2018-07-24 12:54 ` Maxime Ripard
-1 siblings, 0 replies; 67+ messages in thread
From: Maxime Ripard @ 2018-07-24 12:54 UTC (permalink / raw)
To: Alexandru-Cosmin Gheorghe
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, wens, kgene, malidp,
linux-graphics-maintainer, sunpeng.li, boris.brezillon,
linux-samsung-soc, nd, Tony.Cheng, linux-arm-kernel, sw0312.kim,
nicolas.ferre, shirish.s, kyungmin.park, alexander.deucher,
christian.koenig
[-- Attachment #1.1: Type: text/plain, Size: 1617 bytes --]
On Tue, Jul 24, 2018 at 09:11:21AM +0100, Alexandru-Cosmin Gheorghe wrote:
> Hi Maxime,
>
> On Tue, Jul 24, 2018 at 08:57:20AM +0200, Maxime Ripard wrote:
> > On Fri, Jul 20, 2018 at 10:15:08PM +0100, Alexandru Gheorghe wrote:
> > > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > > ---
> > > drivers/gpu/drm/vc4/vc4_plane.c | 4 +---
> > > 1 file changed, 1 insertion(+), 3 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c
> > > index 9d7a36f148cf..688ad9bb0f08 100644
> > > --- a/drivers/gpu/drm/vc4/vc4_plane.c
> > > +++ b/drivers/gpu/drm/vc4/vc4_plane.c
> > > @@ -200,9 +200,7 @@ static void vc4_plane_reset(struct drm_plane *plane)
> > > if (!vc4_state)
> > > return;
> > >
> > > - plane->state = &vc4_state->base;
> > > - plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
> > > - vc4_state->base.plane = plane;
> > > + __drm_atomic_helper_plane_reset(plane, &vc4_state->base);
> >
> > For vc4, rcar-du, atmel-hlcdc and sun4i, you're changing the reset
> > value of alpha from DRM_BLEND_ALPHA_OPAQUE to
> > plane->alpha_property->values[1].
>
> plane->alpha_property->values[1] holds the max value for alpha, and
> it's initialized by the core to DRM_BLEND_ALPHA_OPAQUE.
>
> >
> > This might or might not be a good idea, but you should definitely
> > explain why.
>
> Would a commit message suffice? Or do you have other ideas?
Yes, that would be enough.
Maxime
--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 09/10] drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-24 12:54 ` Maxime Ripard
0 siblings, 0 replies; 67+ messages in thread
From: Maxime Ripard @ 2018-07-24 12:54 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jul 24, 2018 at 09:11:21AM +0100, Alexandru-Cosmin Gheorghe wrote:
> Hi Maxime,
>
> On Tue, Jul 24, 2018 at 08:57:20AM +0200, Maxime Ripard wrote:
> > On Fri, Jul 20, 2018 at 10:15:08PM +0100, Alexandru Gheorghe wrote:
> > > Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> > > ---
> > > drivers/gpu/drm/vc4/vc4_plane.c | 4 +---
> > > 1 file changed, 1 insertion(+), 3 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c
> > > index 9d7a36f148cf..688ad9bb0f08 100644
> > > --- a/drivers/gpu/drm/vc4/vc4_plane.c
> > > +++ b/drivers/gpu/drm/vc4/vc4_plane.c
> > > @@ -200,9 +200,7 @@ static void vc4_plane_reset(struct drm_plane *plane)
> > > if (!vc4_state)
> > > return;
> > >
> > > - plane->state = &vc4_state->base;
> > > - plane->state->alpha = DRM_BLEND_ALPHA_OPAQUE;
> > > - vc4_state->base.plane = plane;
> > > + __drm_atomic_helper_plane_reset(plane, &vc4_state->base);
> >
> > For vc4, rcar-du, atmel-hlcdc and sun4i, you're changing the reset
> > value of alpha from DRM_BLEND_ALPHA_OPAQUE to
> > plane->alpha_property->values[1].
>
> plane->alpha_property->values[1] holds the max value for alpha, and
> it's initialized by the core to DRM_BLEND_ALPHA_OPAQUE.
>
> >
> > This might or might not be a good idea, but you should definitely
> > explain why.
>
> Would a commit message suffice? Or do you have other ideas?
Yes, that would be enough.
Maxime
--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180724/2848509e/attachment-0001.sig>
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 10/10] drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-20 21:15 ` Alexandru Gheorghe
-1 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: seanpaul, airlied, dri-devel, liviu.dudau, brian.starkey, malidp,
ville.syrjala, daniel, gustavo, maarten.lankhorst,
alexander.deucher, christian.koenig, David1.Zhou, harry.wentland,
andrey.grodzovsky, Tony.Cheng, sunpeng.li, shirish.s,
boris.brezillon, nicolas.ferre, alexandre.belloni, inki.dae,
jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graphics-maintainer, syeh
Cc: nd, Alexandru Gheorghe
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
index 466336b34fff..1e0fb3c79b50 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
@@ -720,9 +720,7 @@ void vmw_du_plane_reset(struct drm_plane *plane)
return;
}
- plane->state = &vps->base;
- plane->state->plane = plane;
- plane->state->rotation = DRM_MODE_ROTATE_0;
+ __drm_atomic_helper_plane_reset(plane, &vps->base);
}
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 67+ messages in thread
* [PATCH 10/10] drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-20 21:15 ` Alexandru Gheorghe
0 siblings, 0 replies; 67+ messages in thread
From: Alexandru Gheorghe @ 2018-07-20 21:15 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
---
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
index 466336b34fff..1e0fb3c79b50 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
@@ -720,9 +720,7 @@ void vmw_du_plane_reset(struct drm_plane *plane)
return;
}
- plane->state = &vps->base;
- plane->state->plane = plane;
- plane->state->rotation = DRM_MODE_ROTATE_0;
+ __drm_atomic_helper_plane_reset(plane, &vps->base);
}
--
2.18.0
^ permalink raw reply related [flat|nested] 67+ messages in thread
* RE: [PATCH 10/10] drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:15 ` Alexandru Gheorghe
@ 2018-07-23 17:16 ` Deepak Singh Rawat
-1 siblings, 0 replies; 67+ messages in thread
From: Deepak Singh Rawat @ 2018-07-23 17:16 UTC (permalink / raw)
To: Alexandru Gheorghe
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, Thomas Hellstrom, krzk, maxime.ripard, wens,
kgene, malidp, linux-graphics-maintainer, sunpeng.li,
boris.brezillon, linux-samsung-soc, nd, Tony.Cheng,
linux-arm-kernel
Hi Alexandru,
Thanks for the patch, for the vmwgfx part:
Reviewed-by: Deepak Rawat <drawat@vmware.com>
>
> Signed-off-by: Alexandru Gheorghe <alexandru-
> cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> index 466336b34fff..1e0fb3c79b50 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> @@ -720,9 +720,7 @@ void vmw_du_plane_reset(struct drm_plane *plane)
> return;
> }
>
> - plane->state = &vps->base;
> - plane->state->plane = plane;
> - plane->state->rotation = DRM_MODE_ROTATE_0;
> + __drm_atomic_helper_plane_reset(plane, &vps->base);
> }
>
>
> --
> 2.18.0
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 10/10] drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-23 17:16 ` Deepak Singh Rawat
0 siblings, 0 replies; 67+ messages in thread
From: Deepak Singh Rawat @ 2018-07-23 17:16 UTC (permalink / raw)
To: linux-arm-kernel
Hi Alexandru,
Thanks for the patch, for the vmwgfx part:
Reviewed-by: Deepak Rawat <drawat@vmware.com>
>
> Signed-off-by: Alexandru Gheorghe <alexandru-
> cosmin.gheorghe at arm.com>
> ---
> drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> index 466336b34fff..1e0fb3c79b50 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> @@ -720,9 +720,7 @@ void vmw_du_plane_reset(struct drm_plane *plane)
> return;
> }
>
> - plane->state = &vps->base;
> - plane->state->plane = plane;
> - plane->state->rotation = DRM_MODE_ROTATE_0;
> + __drm_atomic_helper_plane_reset(plane, &vps->base);
> }
>
>
> --
> 2.18.0
>
^ permalink raw reply [flat|nested] 67+ messages in thread
* Re: [PATCH 10/10] drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the logic
2018-07-20 21:15 ` Alexandru Gheorghe
@ 2018-07-23 18:02 ` Sinclair Yeh
-1 siblings, 0 replies; 67+ messages in thread
From: Sinclair Yeh @ 2018-07-23 18:02 UTC (permalink / raw)
To: Alexandru Gheorghe
Cc: alexandre.belloni, airlied, liviu.dudau, dri-devel,
laurent.pinchart, thellstrom, krzk, maxime.ripard, wens, kgene,
malidp, linux-graphics-maintainer, sunpeng.li, boris.brezillon,
linux-samsung-soc, nd, Tony.Cheng, linux-arm-kernel, sw0312.kim,
nicolas.ferre, shirish.s, kyungmin.park, alexander.deucher,
christian.koenig
Hello Alex,
other than adding a brief commit message to this patch,
Reviewed-by: Sinclair Yeh <syeh@vmware.com>
On Fri, Jul 20, 2018 at 10:15:09PM +0100, Alexandru Gheorghe wrote:
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> index 466336b34fff..1e0fb3c79b50 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> @@ -720,9 +720,7 @@ void vmw_du_plane_reset(struct drm_plane *plane)
> return;
> }
>
> - plane->state = &vps->base;
> - plane->state->plane = plane;
> - plane->state->rotation = DRM_MODE_ROTATE_0;
> + __drm_atomic_helper_plane_reset(plane, &vps->base);
> }
>
>
> --
> 2.18.0
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 10/10] drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the logic
@ 2018-07-23 18:02 ` Sinclair Yeh
0 siblings, 0 replies; 67+ messages in thread
From: Sinclair Yeh @ 2018-07-23 18:02 UTC (permalink / raw)
To: linux-arm-kernel
Hello Alex,
other than adding a brief commit message to this patch,
Reviewed-by: Sinclair Yeh <syeh@vmware.com>
On Fri, Jul 20, 2018 at 10:15:09PM +0100, Alexandru Gheorghe wrote:
> Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
> ---
> drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> index 466336b34fff..1e0fb3c79b50 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> @@ -720,9 +720,7 @@ void vmw_du_plane_reset(struct drm_plane *plane)
> return;
> }
>
> - plane->state = &vps->base;
> - plane->state->plane = plane;
> - plane->state->rotation = DRM_MODE_ROTATE_0;
> + __drm_atomic_helper_plane_reset(plane, &vps->base);
> }
>
>
> --
> 2.18.0
>
^ permalink raw reply [flat|nested] 67+ messages in thread
* Re: [PATCH 00/10] Add helper for plane reset
2018-07-20 21:14 ` Alexandru Gheorghe
@ 2018-07-24 15:31 ` Harry Wentland
-1 siblings, 0 replies; 67+ messages in thread
From: Harry Wentland @ 2018-07-24 15:31 UTC (permalink / raw)
To: Alexandru Gheorghe, seanpaul, airlied, dri-devel, liviu.dudau,
brian.starkey, malidp, ville.syrjala, daniel, gustavo,
maarten.lankhorst, alexander.deucher, christian.koenig,
David1.Zhou, andrey.grodzovsky, Tony.Cheng, sunpeng.li,
shirish.s, boris.brezillon, nicolas.ferre, alexandre.belloni,
inki.dae, jy0922.shim, sw0312.kim, kyungmin.park, kgene, krzk,
linux-arm-kernel, linux-samsung-soc, p.zabel, laurent.pinchart,
maxime.ripard, wens, eric, linux-graph
Cc: nd
On 2018-07-20 05:14 PM, Alexandru Gheorghe wrote:
> Drivers that subclass drm_plane need to copy the logic for linking the
> drm_plane with its state and to initialize core properties to their
> default values. E.g (alpha and rotation)
>
> Having a helper to reset the plane_state makes sense because of multiple
> reasons:
> 1. Eliminate code duplication.
> 2. Add a single place for initializing core properties to their
> default values, no need for driver to do it if what the helper sets
> makes sense for them.
> 3. No need to debug the driver when you enable a core property and
> observe it doesn't have a proper default value.
>
> Tested with mali-dp the other drivers are just built-tested.
>
For some reason I lost 02/10 hence I'm replying to the cover letter.
Patches 1 & 2 are
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Harry
>
> Alexandru Gheorghe (10):
> drm/atomic: Add __drm_atomic_helper_plane_reset
> drm/amd/display: Use __drm_atomic_helper_plane_reset instead of
> copying the logic
> drm: mali-dp: Use __drm_atomic_helper_plane_reset instead of copying
> the logic
> drm: atmel-hlcdc: Use __drm_atomic_helper_plane_reset instead of
> copying the logic
> drm/exynos: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
> drm/imx: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
> drm: rcar-du: Use __drm_atomic_helper_plane_reset instead of copying
> the logic
> drm/sun4i: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
> drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
> drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
>
> .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 ++--
> drivers/gpu/drm/arm/malidp_planes.c | 7 ++--
> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 5 +--
> drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++------
> drivers/gpu/drm/exynos/exynos_drm_plane.c | 3 +-
> drivers/gpu/drm/imx/ipuv3-plane.c | 8 ++---
> drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 +--
> drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 4 +--
> drivers/gpu/drm/sun4i/sun4i_layer.c | 4 +--
> drivers/gpu/drm/vc4/vc4_plane.c | 4 +--
> drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 4 +--
> include/drm/drm_atomic_helper.h | 2 ++
> 12 files changed, 39 insertions(+), 45 deletions(-)
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 67+ messages in thread
* [PATCH 00/10] Add helper for plane reset
@ 2018-07-24 15:31 ` Harry Wentland
0 siblings, 0 replies; 67+ messages in thread
From: Harry Wentland @ 2018-07-24 15:31 UTC (permalink / raw)
To: linux-arm-kernel
On 2018-07-20 05:14 PM, Alexandru Gheorghe wrote:
> Drivers that subclass drm_plane need to copy the logic for linking the
> drm_plane with its state and to initialize core properties to their
> default values. E.g (alpha and rotation)
>
> Having a helper to reset the plane_state makes sense because of multiple
> reasons:
> 1. Eliminate code duplication.
> 2. Add a single place for initializing core properties to their
> default values, no need for driver to do it if what the helper sets
> makes sense for them.
> 3. No need to debug the driver when you enable a core property and
> observe it doesn't have a proper default value.
>
> Tested with mali-dp the other drivers are just built-tested.
>
For some reason I lost 02/10 hence I'm replying to the cover letter.
Patches 1 & 2 are
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Harry
>
> Alexandru Gheorghe (10):
> drm/atomic: Add __drm_atomic_helper_plane_reset
> drm/amd/display: Use __drm_atomic_helper_plane_reset instead of
> copying the logic
> drm: mali-dp: Use __drm_atomic_helper_plane_reset instead of copying
> the logic
> drm: atmel-hlcdc: Use __drm_atomic_helper_plane_reset instead of
> copying the logic
> drm/exynos: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
> drm/imx: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
> drm: rcar-du: Use __drm_atomic_helper_plane_reset instead of copying
> the logic
> drm/sun4i: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
> drm/vc4: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
> drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the
> logic
>
> .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 ++--
> drivers/gpu/drm/arm/malidp_planes.c | 7 ++--
> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 5 +--
> drivers/gpu/drm/drm_atomic_helper.c | 32 +++++++++++++------
> drivers/gpu/drm/exynos/exynos_drm_plane.c | 3 +-
> drivers/gpu/drm/imx/ipuv3-plane.c | 8 ++---
> drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 +--
> drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 4 +--
> drivers/gpu/drm/sun4i/sun4i_layer.c | 4 +--
> drivers/gpu/drm/vc4/vc4_plane.c | 4 +--
> drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 4 +--
> include/drm/drm_atomic_helper.h | 2 ++
> 12 files changed, 39 insertions(+), 45 deletions(-)
>
^ permalink raw reply [flat|nested] 67+ messages in thread