q_data w/h must be set when encoding.

Fixes: 1c84e7f9d5dc596be (media: s5p-jpeg: Add support for resolution change 
event)
Signed-off-by: Andrzej Pietrasiewicz <andrze...@samsung.com>
---
 drivers/media/platform/s5p-jpeg/jpeg-core.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/drivers/media/platform/s5p-jpeg/jpeg-core.c 
b/drivers/media/platform/s5p-jpeg/jpeg-core.c
index c00e3a1..e1babb8 100644
--- a/drivers/media/platform/s5p-jpeg/jpeg-core.c
+++ b/drivers/media/platform/s5p-jpeg/jpeg-core.c
@@ -1634,6 +1634,12 @@ static int s5p_jpeg_s_fmt(struct s5p_jpeg_ctx *ct, 
struct v4l2_format *f)
                        FMT_TYPE_OUTPUT : FMT_TYPE_CAPTURE;
 
        q_data->fmt = s5p_jpeg_find_format(ct, pix->pixelformat, f_type);
+       if (ct->mode == S5P_JPEG_ENCODE ||
+               (ct->mode == S5P_JPEG_DECODE &&
+               q_data->fmt->fourcc != V4L2_PIX_FMT_JPEG)) {
+               q_data->w = pix->width;
+               q_data->h = pix->height;
+       }
        if (q_data->fmt->fourcc != V4L2_PIX_FMT_JPEG) {
                /*
                 * During encoding Exynos4x12 SoCs access wider memory area
@@ -1641,8 +1647,6 @@ static int s5p_jpeg_s_fmt(struct s5p_jpeg_ctx *ct, struct 
v4l2_format *f)
                 * the JPEG_IMAGE_SIZE register. In order to avoid sysmmu
                 * page fault calculate proper buffer size in such a case.
                 */
-               q_data->w = pix->width;
-               q_data->h = pix->height;
                if (ct->jpeg->variant->hw_ex4_compat &&
                    f_type == FMT_TYPE_OUTPUT && ct->mode == S5P_JPEG_ENCODE)
                        q_data->size = exynos4_jpeg_get_output_buffer_size(ct,
-- 
1.9.1

Reply via email to