Commit cb8a3db2 authored by Joonyoung Shim's avatar Joonyoung Shim Committed by Inki Dae

drm/exynos: use src_x and src_y instead of fb_x and fb_y

It's more reasonable to use src_x and src_y to represent source as
counterpart of destination(crtc). Already we are using src_width and
src_height for width and height of source.
Signed-off-by: default avatarJoonyoung Shim <jy0922.shim@samsung.com>
Signed-off-by: default avatarInki Dae <inki.dae@samsung.com>
parent 2611015c
...@@ -440,8 +440,8 @@ static void decon_win_commit(struct exynos_drm_crtc *crtc, unsigned int win) ...@@ -440,8 +440,8 @@ static void decon_win_commit(struct exynos_drm_crtc *crtc, unsigned int win)
writel(plane->fb_height, ctx->regs + VIDW_WHOLE_Y(win)); writel(plane->fb_height, ctx->regs + VIDW_WHOLE_Y(win));
/* offset from the start of the buffer to read */ /* offset from the start of the buffer to read */
writel(plane->fb_x, ctx->regs + VIDW_OFFSET_X(win)); writel(plane->src_x, ctx->regs + VIDW_OFFSET_X(win));
writel(plane->fb_y, ctx->regs + VIDW_OFFSET_Y(win)); writel(plane->src_y, ctx->regs + VIDW_OFFSET_Y(win));
DRM_DEBUG_KMS("start addr = 0x%lx\n", DRM_DEBUG_KMS("start addr = 0x%lx\n",
(unsigned long)val); (unsigned long)val);
......
...@@ -47,14 +47,14 @@ enum exynos_drm_output_type { ...@@ -47,14 +47,14 @@ enum exynos_drm_output_type {
* Exynos drm common overlay structure. * Exynos drm common overlay structure.
* *
* @base: plane object * @base: plane object
* @fb_x: offset x on a framebuffer to be displayed. * @src_x: offset x on a framebuffer to be displayed.
* - the unit is screen coordinates. * - the unit is screen coordinates.
* @fb_y: offset y on a framebuffer to be displayed. * @src_y: offset y on a framebuffer to be displayed.
* - the unit is screen coordinates. * - the unit is screen coordinates.
* @fb_width: width of a framebuffer.
* @fb_height: height of a framebuffer.
* @src_width: width of a partial image to be displayed from framebuffer. * @src_width: width of a partial image to be displayed from framebuffer.
* @src_height: height of a partial image to be displayed from framebuffer. * @src_height: height of a partial image to be displayed from framebuffer.
* @fb_width: width of a framebuffer.
* @fb_height: height of a framebuffer.
* @crtc_x: offset x on hardware screen. * @crtc_x: offset x on hardware screen.
* @crtc_y: offset y on hardware screen. * @crtc_y: offset y on hardware screen.
* @crtc_width: window width to be displayed (hardware screen). * @crtc_width: window width to be displayed (hardware screen).
...@@ -85,12 +85,12 @@ enum exynos_drm_output_type { ...@@ -85,12 +85,12 @@ enum exynos_drm_output_type {
struct exynos_drm_plane { struct exynos_drm_plane {
struct drm_plane base; struct drm_plane base;
unsigned int fb_x; unsigned int src_x;
unsigned int fb_y; unsigned int src_y;
unsigned int fb_width;
unsigned int fb_height;
unsigned int src_width; unsigned int src_width;
unsigned int src_height; unsigned int src_height;
unsigned int fb_width;
unsigned int fb_height;
unsigned int crtc_x; unsigned int crtc_x;
unsigned int crtc_y; unsigned int crtc_y;
unsigned int crtc_width; unsigned int crtc_width;
......
...@@ -653,8 +653,8 @@ static void fimd_win_commit(struct exynos_drm_crtc *crtc, unsigned int win) ...@@ -653,8 +653,8 @@ static void fimd_win_commit(struct exynos_drm_crtc *crtc, unsigned int win)
fimd_shadow_protect_win(ctx, win, true); fimd_shadow_protect_win(ctx, win, true);
offset = plane->fb_x * (plane->bpp >> 3); offset = plane->src_x * (plane->bpp >> 3);
offset += plane->fb_y * plane->pitch; offset += plane->src_y * plane->pitch;
/* buffer start address */ /* buffer start address */
dma_addr = plane->dma_addr[0] + offset; dma_addr = plane->dma_addr[0] + offset;
......
...@@ -111,12 +111,12 @@ void exynos_plane_mode_set(struct drm_plane *plane, struct drm_crtc *crtc, ...@@ -111,12 +111,12 @@ void exynos_plane_mode_set(struct drm_plane *plane, struct drm_crtc *crtc,
} }
/* set drm framebuffer data. */ /* set drm framebuffer data. */
exynos_plane->fb_x = src_x; exynos_plane->src_x = src_x;
exynos_plane->fb_y = src_y; exynos_plane->src_y = src_y;
exynos_plane->fb_width = fb->width;
exynos_plane->fb_height = fb->height;
exynos_plane->src_width = src_w; exynos_plane->src_width = src_w;
exynos_plane->src_height = src_h; exynos_plane->src_height = src_h;
exynos_plane->fb_width = fb->width;
exynos_plane->fb_height = fb->height;
exynos_plane->bpp = fb->bits_per_pixel; exynos_plane->bpp = fb->bits_per_pixel;
exynos_plane->pitch = fb->pitches[0]; exynos_plane->pitch = fb->pitches[0];
exynos_plane->pixel_format = fb->pixel_format; exynos_plane->pixel_format = fb->pixel_format;
......
...@@ -457,8 +457,8 @@ static void vp_video_buffer(struct mixer_context *ctx, int win) ...@@ -457,8 +457,8 @@ static void vp_video_buffer(struct mixer_context *ctx, int win)
vp_reg_write(res, VP_SRC_WIDTH, plane->src_width); vp_reg_write(res, VP_SRC_WIDTH, plane->src_width);
vp_reg_write(res, VP_SRC_HEIGHT, plane->src_height); vp_reg_write(res, VP_SRC_HEIGHT, plane->src_height);
vp_reg_write(res, VP_SRC_H_POSITION, vp_reg_write(res, VP_SRC_H_POSITION,
VP_SRC_H_POSITION_VAL(plane->fb_x)); VP_SRC_H_POSITION_VAL(plane->src_x));
vp_reg_write(res, VP_SRC_V_POSITION, plane->fb_y); vp_reg_write(res, VP_SRC_V_POSITION, plane->src_y);
vp_reg_write(res, VP_DST_WIDTH, plane->crtc_width); vp_reg_write(res, VP_DST_WIDTH, plane->crtc_width);
vp_reg_write(res, VP_DST_H_POSITION, plane->crtc_x); vp_reg_write(res, VP_DST_H_POSITION, plane->crtc_x);
...@@ -561,8 +561,8 @@ static void mixer_graph_buffer(struct mixer_context *ctx, int win) ...@@ -561,8 +561,8 @@ static void mixer_graph_buffer(struct mixer_context *ctx, int win)
/* converting dma address base and source offset */ /* converting dma address base and source offset */
dma_addr = plane->dma_addr[0] dma_addr = plane->dma_addr[0]
+ (plane->fb_x * plane->bpp >> 3) + (plane->src_x * plane->bpp >> 3)
+ (plane->fb_y * plane->pitch); + (plane->src_y * plane->pitch);
src_x_offset = 0; src_x_offset = 0;
src_y_offset = 0; src_y_offset = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment