@@ -57,7 +57,8 @@ static int exynos_plane_get_size(int start, unsigned length, unsigned last)
return size;
}
-static void exynos_plane_mode_set(struct exynos_drm_plane_state *exynos_state)
+static void exynos_plane_mode_set(struct exynos_drm_plane_state *exynos_state,
+ const struct exynos_drm_plane_config *config)
{
struct drm_plane_state *state = &exynos_state->base;
@@ -86,7 +87,8 @@ static void exynos_plane_mode_set(struct exynos_drm_plane_state *exynos_state)
src_w = state->src_w >> 16;
src_h = state->src_h >> 16;
- exynos_plane_ipp_setup(exynos_state, &src_x, &src_y, &src_w, &src_h);
+ exynos_plane_ipp_setup(exynos_state, config, &src_x, &src_y, &src_w,
+ &src_h, &crtc_w, &crtc_h);
/* set ratio */
exynos_state->h_ratio = (src_w << 16) / crtc_w;
@@ -245,7 +247,7 @@ static int exynos_plane_atomic_check(struct drm_plane *plane,
return 0;
/* translate state into exynos_state */
- exynos_plane_mode_set(exynos_state);
+ exynos_plane_mode_set(exynos_state, exynos_plane->config);
ret = exynos_drm_plane_check_format(exynos_plane->config, exynos_state);
if (ret)
@@ -121,15 +121,24 @@ static int exynos_plane_ipp_transform(struct exynos_drm_plane_state *state)
}
void exynos_plane_ipp_setup(struct exynos_drm_plane_state *state,
+ const struct exynos_drm_plane_config *config,
unsigned int *src_x, unsigned int *src_y,
- unsigned int *src_w, unsigned int *src_h)
+ unsigned int *src_w, unsigned int *src_h,
+ unsigned int *crtc_w, unsigned int *crtc_h)
{
int rotation = state->base.rotation;
int pre_x, pre_y, post_x, post_y;
state->rotation = rotation;
- if (rotation == 0 || rotation == BIT(DRM_ROTATE_0))
+ /* check if ipp is really needed */
+ if (rotation == BIT(DRM_ROTATE_0) &&
+ (*src_w == *crtc_w ||
+ ((config->capabilities & EXYNOS_DRM_PLANE_CAP_DOUBLE_X) &&
+ *src_w * 2 == *crtc_w)) &&
+ (*src_h == *crtc_h ||
+ ((config->capabilities & EXYNOS_DRM_PLANE_CAP_DOUBLE_Y) &&
+ *src_h * 2 == *crtc_h)))
return;
state->ipp_needed = true;
@@ -183,6 +192,15 @@ void exynos_plane_ipp_setup(struct exynos_drm_plane_state *state,
swap(state->ipp_dst.w, state->ipp_dst.h);
break;
}
+
+ /* apply scalling */
+ state->ipp_dst.w = state->ipp_dst.w * *crtc_w / *src_w;
+ state->ipp_dst.h = state->ipp_dst.h * *crtc_h / *src_h;
+
+ *src_x = *src_x * *crtc_w / *src_w;
+ *src_y = *src_y * *crtc_h / *src_h;
+ *src_w = *crtc_w;
+ *src_h = *crtc_h;
}
int exynos_plane_ipp_check(struct exynos_drm_plane *plane,
@@ -15,8 +15,10 @@
#ifdef CONFIG_DRM_EXYNOS_PLANE_IPP
void exynos_plane_ipp_setup(struct exynos_drm_plane_state *state,
+ const struct exynos_drm_plane_config *config,
unsigned int *src_x, unsigned int *src_y,
- unsigned int *src_w, unsigned int *src_h);
+ unsigned int *src_w, unsigned int *src_h,
+ unsigned int *crtc_w, unsigned int *crtc_h);
int exynos_plane_ipp_check(struct exynos_drm_plane *plane,
struct exynos_drm_plane_state *state);
@@ -34,8 +36,10 @@ int exynos_plane_ipp_attach_properties(struct drm_device *dev,
static inline
void exynos_plane_ipp_setup(struct exynos_drm_plane_state *state,
+ const struct exynos_drm_plane_config *config,
unsigned int *src_x, unsigned int *src_y,
- unsigned int *src_w, unsigned int *src_h)
+ unsigned int *src_w, unsigned int *src_h,
+ unsigned int *crtc_w, unsigned int *crtc_h)
{
}
This patch adds support for plane scaling. Minor changes were needed to use existing Exynos IPP integration code for enabling scaling feature. Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> --- drivers/gpu/drm/exynos/exynos_drm_plane.c | 8 +++++--- drivers/gpu/drm/exynos/exynos_drm_plane_ipp.c | 22 ++++++++++++++++++++-- drivers/gpu/drm/exynos/exynos_drm_plane_ipp.h | 8 ++++++-- 3 files changed, 31 insertions(+), 7 deletions(-) -- 1.9.2 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html