Thank's your comments.

I answer your comments. please check that.

Thank's

BR
Eunchul Kim

On 12/11/2012 02:16 PM, Inki Dae wrote:
>
>
>> -----Original Message-----
>> From: Eunchul Kim [mailto:chulspro.kim at samsung.com]
>> Sent: Monday, December 10, 2012 10:46 PM
>> To: dri-devel at lists.freedesktop.org; inki.dae at samsung.com
>> Cc: jy0.jeon at samsung.com; yj44.cho at samsung.com; jmock.shin at 
>> samsung.com;
>> jaejoon.seo at samsung.com; kyungmin.park at samsung.com;
>> chulspro.kim at samsung.com
>> Subject: [RFC v2 3/5] drm/exynos: add fimc ipp driver
>>
>> FIMC is stand for Fully Interfactive Mobile Camera and
>> supports image scaler/rotator/crop/flip/csc and input/output DMA
>> operations.
>> input DMA reads image data from the memory.
>> output DMA writes image data to memory.
>> FIMC supports image rotation and image effect functions.
>> also supports writeback and display output operations.
>>
>> Signed-off-by: Eunchul Kim <chulspro.kim at samsung.com>
>> Signed-off-by: Jinyoung Jeon <jy0.jeon at samsung.com>
>> ---
>>   drivers/gpu/drm/exynos/Kconfig           |    6 +
>>   drivers/gpu/drm/exynos/Makefile          |    1 +
>>   drivers/gpu/drm/exynos/exynos_drm_drv.c  |   15 +
>>   drivers/gpu/drm/exynos/exynos_drm_drv.h  |    1 +
>>   drivers/gpu/drm/exynos/exynos_drm_fimc.c | 2041
>> ++++++++++++++++++++++++++++++
>>   drivers/gpu/drm/exynos/exynos_drm_fimc.h |   35 +
>>   drivers/gpu/drm/exynos/regs-fimc.h       |  669 ++++++++++
>>   include/drm/exynos_drm.h                 |   34 +
>>   8 files changed, 2802 insertions(+), 0 deletions(-)
>>   create mode 100644 drivers/gpu/drm/exynos/exynos_drm_fimc.c
>>   create mode 100644 drivers/gpu/drm/exynos/exynos_drm_fimc.h
>>   create mode 100644 drivers/gpu/drm/exynos/regs-fimc.h
>>
>> diff --git a/drivers/gpu/drm/exynos/Kconfig
>> b/drivers/gpu/drm/exynos/Kconfig
>> index bcf1c9d..4915ab6 100644
>> --- a/drivers/gpu/drm/exynos/Kconfig
>> +++ b/drivers/gpu/drm/exynos/Kconfig
>> @@ -51,3 +51,9 @@ config DRM_EXYNOS_IPP
>>      depends on DRM_EXYNOS
>>      help
>>        Choose this option if you want to use IPP feature for DRM.
>> +
>> +config DRM_EXYNOS_FIMC
>> +    bool "Exynos DRM FIMC"
>> +    depends on DRM_EXYNOS_IPP
>> +    help
>> +      Choose this option if you want to use Exynos FIMC for DRM.
>> diff --git a/drivers/gpu/drm/exynos/Makefile
>> b/drivers/gpu/drm/exynos/Makefile
>> index 6c536ce..9710024 100644
>> --- a/drivers/gpu/drm/exynos/Makefile
>> +++ b/drivers/gpu/drm/exynos/Makefile
>> @@ -17,5 +17,6 @@ exynosdrm-$(CONFIG_DRM_EXYNOS_HDMI)        +=
>> exynos_hdmi.o exynos_mixer.o \
>>   exynosdrm-$(CONFIG_DRM_EXYNOS_VIDI)        += exynos_drm_vidi.o
>>   exynosdrm-$(CONFIG_DRM_EXYNOS_G2D) += exynos_drm_g2d.o
>>   exynosdrm-$(CONFIG_DRM_EXYNOS_IPP) += exynos_drm_ipp.o
>> +exynosdrm-$(CONFIG_DRM_EXYNOS_FIMC) += exynos_drm_fimc.o
>>
>>   obj-$(CONFIG_DRM_EXYNOS)           += exynosdrm.o
>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.c
>> b/drivers/gpu/drm/exynos/exynos_drm_drv.c
>> index 0eb8a97..73f02ac 100644
>> --- a/drivers/gpu/drm/exynos/exynos_drm_drv.c
>> +++ b/drivers/gpu/drm/exynos/exynos_drm_drv.c
>> @@ -372,6 +372,12 @@ static int __init exynos_drm_init(void)
>>              goto out_g2d;
>>   #endif
>>
>> +#ifdef CONFIG_DRM_EXYNOS_FIMC
>> +    ret = platform_driver_register(&fimc_driver);
>> +    if (ret < 0)
>> +            goto out_fimc;
>> +#endif
>> +
>>   #ifdef CONFIG_DRM_EXYNOS_IPP
>>      ret = platform_driver_register(&ipp_driver);
>>      if (ret < 0)
>> @@ -400,6 +406,11 @@ out_drm:
>>   out_ipp:
>>   #endif
>>
>> +#ifdef CONFIG_DRM_EXYNOS_FIMC
>> +    platform_driver_unregister(&fimc_driver);
>> +out_fimc:
>> +#endif
>> +
>>   #ifdef CONFIG_DRM_EXYNOS_G2D
>>      platform_driver_unregister(&g2d_driver);
>>   out_g2d:
>> @@ -440,6 +451,10 @@ static void __exit exynos_drm_exit(void)
>>      platform_driver_unregister(&ipp_driver);
>>   #endif
>>
>> +#ifdef CONFIG_DRM_EXYNOS_FIMC
>> +    platform_driver_unregister(&fimc_driver);
>> +#endif
>> +
>>   #ifdef CONFIG_DRM_EXYNOS_G2D
>>      platform_driver_unregister(&g2d_driver);
>>   #endif
>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.h
>> b/drivers/gpu/drm/exynos/exynos_drm_drv.h
>> index a365788..14f9490 100644
>> --- a/drivers/gpu/drm/exynos/exynos_drm_drv.h
>> +++ b/drivers/gpu/drm/exynos/exynos_drm_drv.h
>> @@ -352,5 +352,6 @@ extern struct platform_driver mixer_driver;
>>   extern struct platform_driver exynos_drm_common_hdmi_driver;
>>   extern struct platform_driver vidi_driver;
>>   extern struct platform_driver g2d_driver;
>> +extern struct platform_driver fimc_driver;
>>   extern struct platform_driver ipp_driver;
>>   #endif
>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimc.c
>> b/drivers/gpu/drm/exynos/exynos_drm_fimc.c
>> new file mode 100644
>> index 0000000..3adb452
>> --- /dev/null
>> +++ b/drivers/gpu/drm/exynos/exynos_drm_fimc.c
>> @@ -0,0 +1,2041 @@
>> +/*
>> + * Copyright (C) 2012 Samsung Electronics Co.Ltd
>> + * Authors:
>> + *  Eunchul Kim <chulspro.kim at samsung.com>
>> + *  Jinyoung Jeon <jy0.jeon at samsung.com>
>> + *  Sangmin Lee <lsmin.lee at samsung.com>
>> + *
>> + * This program is free software; you can redistribute  it and/or modify
>> it
>> + * under  the terms of  the GNU General  Public License as published by
>> the
>> + * Free Software Foundation;  either version 2 of the  License, or (at
>> your
>> + * option) any later version.
>> + *
>> + */
>> +#include <linux/kernel.h>
>> +#include <linux/module.h>
>> +#include <linux/platform_device.h>
>> +#include <linux/clk.h>
>> +#include <linux/pm_runtime.h>
>> +#include <plat/map-base.h>
>> +
>> +#include <drm/drmP.h>
>> +#include <drm/exynos_drm.h>
>> +#include "regs-fimc.h"
>> +#include "exynos_drm_drv.h"
>> +#include "exynos_drm_gem.h"
>> +#include "exynos_drm_ipp.h"
>> +#include "exynos_drm_fimc.h"
>> +
>> +/*
>> + * FIMC is stand for Fully Interactive Mobile Camera and
>> + * supports image scaler/rotator and input/output DMA operations.
>> + * input DMA reads image data from the memory.
>> + * output DMA writes image data to memory.
>> + * FIMC supports image rotation and image effect functions.
>> + */
>> +
>> +#define FIMC_MAX_DEVS       4
>> +#define FIMC_MAX_SRC        2
>> +#define FIMC_MAX_DST        32
>> +#define FIMC_CLK_RATE               166750000
>> +#define FIMC_CLK_RATE_R2            180000000
>
> FIMC should use fixed clock rate? I think clock rate and source or parent
> clocks should be decided by machine code.
>
>> +#define FIMC_BUF_STOP       1
>> +#define FIMC_BUF_START      2
>> +#define FIMC_REG_SZ         32
>> +#define FIMC_WIDTH_ITU_709  1280
>> +
>> +#define get_fimc_context(dev)
>>      platform_get_drvdata(to_platform_device(dev))
>> +#define get_ctx_from_ippdrv(ippdrv) container_of(ippdrv,\
>> +                                    struct fimc_context, ippdrv);
>> +#define fimc_read(offset)           readl(ctx->regs + (offset));
>> +#define fimc_write(cfg, offset)     writel(cfg, ctx->regs + (offset));
>> +
>> +enum fimc_wb {
>> +    FIMC_WB_NONE,
>> +    FIMC_WB_A,
>> +    FIMC_WB_B,
>> +};
>> +
>> +/*
>> + * A structure of scaler.
>> + *
>> + * @range: narrow, wide.
>> + * @bypass: unused scaler path.
>> + * @up_h: horizontal scale up.
>> + * @up_v: vertical scale up.
>> + * @hratio: horizontal ratio.
>> + * @vratio: vertical ratio.
>> + */
>> +struct fimc_scaler {
>> +    bool    range;
>> +    bool bypass;
>> +    bool up_h;
>> +    bool up_v;
>> +    u32 hratio;
>> +    u32 vratio;
>> +};
>> +
>> +/*
>> + * A structure of scaler capability.
>> + *
>> + * find user manual table 43-1.
>> + * @in_hori: scaler input horizontal size.
>> + * @bypass: scaler bypass mode.
>> + * @dst_h_wo_rot: target horizontal size without output rotation.
>> + * @dst_h_rot: target horizontal size with output rotation.
>> + * @rl_w_wo_rot: real width without input rotation.
>> + * @rl_h_rot: real height without output rotation.
>> + */
>> +struct fimc_capability {
>> +    /* scaler */
>> +    u32     in_hori;
>> +    u32     bypass;
>> +    /* output rotator */
>> +    u32     dst_h_wo_rot;
>> +    u32     dst_h_rot;
>> +    /* input rotator */
>> +    u32     rl_w_wo_rot;
>> +    u32     rl_h_rot;
>> +};
>> +
>> +/*
>> + * A structure of fimc context.
>> + *
>> + * @ippdrv: prepare initialization using ippdrv.
>> + * @regs_res: register resources.
>> + * @regs: memory mapped io registers.
>> + * @lock: locking of operations.
>> + * @sclk_fimc_clk: fimc source clock.
>> + * @fimc_clk: fimc clock.
>> + * @wb_clk: writeback a clock.
>> + * @wb_b_clk: writeback b clock.
>> + * @sc: scaler infomations.
>> + * @capa: scaler capability.
>> + * @odr: ordering of YUV.
>> + * @ver: fimc version.
>> + * @pol: porarity of writeback.
>> + * @id: fimc id.
>> + * @irq: irq number.
>> + * @suspended: qos operations.
>> + */
>> +struct fimc_context {
>> +    struct exynos_drm_ippdrv        ippdrv;
>> +    struct resource *regs_res;
>> +    void __iomem    *regs;
>> +    spinlock_t      lock;
>> +    struct clk      *sclk_fimc_clk;
>> +    struct clk      *fimc_clk;
>> +    struct clk      *wb_clk;
>> +    struct clk      *wb_b_clk;
>> +    struct fimc_scaler      sc;
>> +    struct fimc_capability  *capa;
>> +    enum exynos_drm_fimc_ver        ver;
>> +    struct exynos_drm_fimc_pol      pol;
>> +    int     id;
>> +    int     irq;
>> +    bool    suspended;
>> +};
>> +
>> +struct fimc_capability fimc51_capa[FIMC_MAX_DEVS] = {
>> +    {
>> +            .in_hori = 4224,
>> +            .bypass = 8192,
>> +            .dst_h_wo_rot = 4224,
>> +            .dst_h_rot = 1920,
>> +            .rl_w_wo_rot = 8192,
>> +            .rl_h_rot = 1920,
>> +    }, {
>> +            .in_hori = 4224,
>> +            .bypass = 8192,
>> +            .dst_h_wo_rot = 4224,
>> +            .dst_h_rot = 1920,
>> +            .rl_w_wo_rot = 8192,
>> +            .rl_h_rot = 1920,
>> +    }, {
>> +            .in_hori = 4224,
>> +            .bypass = 8192,
>> +            .dst_h_wo_rot = 4224,
>> +            .dst_h_rot = 1920,
>> +            .rl_w_wo_rot = 8192,
>> +            .rl_h_rot = 1920,
>> +    }, {
>> +            .in_hori = 1920,
>> +            .bypass = 8192,
>> +            .dst_h_wo_rot = 1920,
>> +            .dst_h_rot = 1366,
>> +            .rl_w_wo_rot = 8192,
>> +            .rl_h_rot = 1366,
>> +    },
>> +};
>> +
>> +static void fimc_sw_reset(struct fimc_context *ctx, bool pattern)
>> +{
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:pattern[%d]\n", __func__, pattern);
>> +
>> +    cfg = fimc_read(EXYNOS_CISRCFMT);
>> +    cfg |= EXYNOS_CISRCFMT_ITU601_8BIT;
>> +    if (pattern)
>> +            cfg |= EXYNOS_CIGCTRL_TESTPATTERN_COLOR_BAR;
>> +
>> +    fimc_write(cfg, EXYNOS_CISRCFMT);
>> +
>> +    /* s/w reset */
>> +    cfg = fimc_read(EXYNOS_CIGCTRL);
>> +    cfg |= (EXYNOS_CIGCTRL_SWRST);
>> +    fimc_write(cfg, EXYNOS_CIGCTRL);
>> +
>> +    /* s/w reset complete */
>> +    cfg = fimc_read(EXYNOS_CIGCTRL);
>> +    cfg &= ~EXYNOS_CIGCTRL_SWRST;
>> +    fimc_write(cfg, EXYNOS_CIGCTRL);
>> +
>> +    /* reset sequence */
>> +    fimc_write(0x0, EXYNOS_CIFCNTSEQ);
>> +}
>> +
>> +static void fimc_set_camblk_fimd0_wb(struct fimc_context *ctx)
>> +{
>> +    u32 camblk_cfg;
>> +
>> +    DRM_DEBUG_KMS("%s\n", __func__);
>> +
>> +    camblk_cfg = readl(SYSREG_CAMERA_BLK);
>> +    camblk_cfg &= ~(SYSREG_FIMD0WB_DEST_MASK);
>> +    camblk_cfg |= ctx->id << (SYSREG_FIMD0WB_DEST_SHIFT);
>> +
>> +    writel(camblk_cfg, SYSREG_CAMERA_BLK);
>> +}
>> +
>> +static void fimc_set_type_ctrl(struct fimc_context *ctx, enum fimc_wb wb)
>> +{
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:wb[%d]\n", __func__, wb);
>> +
>> +    cfg = fimc_read(EXYNOS_CIGCTRL);
>> +    cfg &= ~(EXYNOS_CIGCTRL_TESTPATTERN_MASK |
>> +            EXYNOS_CIGCTRL_SELCAM_ITU_MASK |
>> +            EXYNOS_CIGCTRL_SELCAM_MIPI_MASK |
>> +            EXYNOS_CIGCTRL_SELCAM_FIMC_MASK |
>> +            EXYNOS_CIGCTRL_SELWB_CAMIF_MASK |
>> +            EXYNOS_CIGCTRL_SELWRITEBACK_MASK);
>> +
>> +    switch (wb) {
>> +    case FIMC_WB_A:
>> +            cfg |= (EXYNOS_CIGCTRL_SELWRITEBACK_A |
>> +                    EXYNOS_CIGCTRL_SELWB_CAMIF_WRITEBACK);
>> +            break;
>> +    case FIMC_WB_B:
>> +            cfg |= (EXYNOS_CIGCTRL_SELWRITEBACK_B |
>> +                    EXYNOS_CIGCTRL_SELWB_CAMIF_WRITEBACK);
>> +            break;
>> +    case FIMC_WB_NONE:
>> +    default:
>> +            cfg |= (EXYNOS_CIGCTRL_SELCAM_ITU_A |
>> +                    EXYNOS_CIGCTRL_SELWRITEBACK_A |
>> +                    EXYNOS_CIGCTRL_SELCAM_MIPI_A |
>> +                    EXYNOS_CIGCTRL_SELCAM_FIMC_ITU);
>> +            break;
>> +    }
>> +
>> +    fimc_write(cfg, EXYNOS_CIGCTRL);
>> +}
>> +
>> +static void fimc_set_polarity(struct fimc_context *ctx,
>> +    struct exynos_drm_fimc_pol *pol)
>> +{
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:inv_pclk[%d]inv_vsync[%d]\n",
>> +            __func__, pol->inv_pclk, pol->inv_vsync);
>> +    DRM_DEBUG_KMS("%s:inv_href[%d]inv_hsync[%d]\n",
>> +            __func__, pol->inv_href, pol->inv_hsync);
>> +
>> +    cfg = fimc_read(EXYNOS_CIGCTRL);
>> +    cfg &= ~(EXYNOS_CIGCTRL_INVPOLPCLK | EXYNOS_CIGCTRL_INVPOLVSYNC |
>> +             EXYNOS_CIGCTRL_INVPOLHREF | EXYNOS_CIGCTRL_INVPOLHSYNC);
>> +
>> +    if (pol->inv_pclk)
>> +            cfg |= EXYNOS_CIGCTRL_INVPOLPCLK;
>> +    if (pol->inv_vsync)
>> +            cfg |= EXYNOS_CIGCTRL_INVPOLVSYNC;
>> +    if (pol->inv_href)
>> +            cfg |= EXYNOS_CIGCTRL_INVPOLHREF;
>> +    if (pol->inv_hsync)
>> +            cfg |= EXYNOS_CIGCTRL_INVPOLHSYNC;
>> +
>> +    fimc_write(cfg, EXYNOS_CIGCTRL);
>> +}
>> +
>> +static void fimc_handle_jpeg(struct fimc_context *ctx, bool enable)
>> +{
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:enable[%d]\n", __func__, enable);
>> +
>> +    cfg = fimc_read(EXYNOS_CIGCTRL);
>> +    if (enable)
>> +            cfg |= EXYNOS_CIGCTRL_CAM_JPEG;
>> +    else
>> +            cfg &= ~EXYNOS_CIGCTRL_CAM_JPEG;
>> +
>> +    fimc_write(cfg, EXYNOS_CIGCTRL);
>> +}
>> +
>> +static void fimc_handle_irq(struct fimc_context *ctx, bool enable,
>> +    bool overflow, bool level)
>> +{
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:enable[%d]overflow[%d]level[%d]\n", __func__,
>> +                    enable, overflow, level);
>> +
>> +    cfg = fimc_read(EXYNOS_CIGCTRL);
>> +    if (enable) {
>> +            cfg &= ~(EXYNOS_CIGCTRL_IRQ_OVFEN |
>> EXYNOS_CIGCTRL_IRQ_LEVEL);
>> +            cfg |= EXYNOS_CIGCTRL_IRQ_ENABLE;
>> +            if (overflow)
>> +                    cfg |= EXYNOS_CIGCTRL_IRQ_OVFEN;
>> +            if (level)
>> +                    cfg |= EXYNOS_CIGCTRL_IRQ_LEVEL;
>> +    } else
>> +            cfg &= ~(EXYNOS_CIGCTRL_IRQ_OVFEN |
>> EXYNOS_CIGCTRL_IRQ_ENABLE);
>> +
>> +    fimc_write(cfg, EXYNOS_CIGCTRL);
>> +}
>> +
>> +static void fimc_clear_irq(struct fimc_context *ctx)
>> +{
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s\n", __func__);
>> +
>> +    cfg = fimc_read(EXYNOS_CIGCTRL);
>> +    cfg |= EXYNOS_CIGCTRL_IRQ_CLR;
>> +    fimc_write(cfg, EXYNOS_CIGCTRL);
>> +}
>> +
>> +static bool fimc_check_ovf(struct fimc_context *ctx)
>> +{
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    u32 cfg, status, flag;
>> +
>> +    status = fimc_read(EXYNOS_CISTATUS);
>> +    flag = EXYNOS_CISTATUS_OVFIY | EXYNOS_CISTATUS_OVFICB |
>> +            EXYNOS_CISTATUS_OVFICR;
>> +
>> +    DRM_DEBUG_KMS("%s:flag[0x%x]\n", __func__, flag);
>> +
>> +    if (status & flag) {
>> +            cfg = fimc_read(EXYNOS_CIWDOFST);
>> +            cfg |= (EXYNOS_CIWDOFST_CLROVFIY | EXYNOS_CIWDOFST_CLROVFICB
>> |
>> +                    EXYNOS_CIWDOFST_CLROVFICR);
>> +
>> +            fimc_write(cfg, EXYNOS_CIWDOFST);
>> +
>> +            cfg = fimc_read(EXYNOS_CIWDOFST);
>> +            cfg &= ~(EXYNOS_CIWDOFST_CLROVFIY |
>> EXYNOS_CIWDOFST_CLROVFICB |
>> +                    EXYNOS_CIWDOFST_CLROVFICR);
>> +
>> +            fimc_write(cfg, EXYNOS_CIWDOFST);
>> +
>> +            dev_err(ippdrv->dev, "occured overflow at %d, status
>> 0x%x.\n",
>> +                    ctx->id, status);
>> +            return true;
>> +    }
>> +
>> +    return false;
>> +}
>> +
>> +static bool fimc_check_frame_end(struct fimc_context *ctx)
>> +{
>> +    u32 cfg;
>> +
>> +    cfg = fimc_read(EXYNOS_CISTATUS);
>> +
>> +    DRM_DEBUG_KMS("%s:cfg[0x%x]\n", __func__, cfg);
>> +
>> +    if (!(cfg & EXYNOS_CISTATUS_FRAMEEND))
>> +            return false;
>> +
>> +    cfg &= ~(EXYNOS_CISTATUS_FRAMEEND);
>> +    fimc_write(cfg, EXYNOS_CISTATUS);
>> +
>> +    return true;
>> +}
>> +
>> +static int fimc_get_buf_id(struct fimc_context *ctx)
>> +{
>> +    u32 cfg;
>> +    int frame_cnt, buf_id;
>> +
>> +    DRM_DEBUG_KMS("%s\n", __func__);
>> +
>> +    cfg = fimc_read(EXYNOS_CISTATUS2);
>> +    frame_cnt = EXYNOS_CISTATUS2_GET_FRAMECOUNT_BEFORE(cfg);
>> +
>> +    if (frame_cnt == 0)
>> +            frame_cnt = EXYNOS_CISTATUS2_GET_FRAMECOUNT_PRESENT(cfg);
>> +
>> +    DRM_DEBUG_KMS("%s:present[%d]before[%d]\n", __func__,
>> +            EXYNOS_CISTATUS2_GET_FRAMECOUNT_PRESENT(cfg),
>> +            EXYNOS_CISTATUS2_GET_FRAMECOUNT_BEFORE(cfg));
>> +
>> +    if (frame_cnt == 0) {
>> +            DRM_ERROR("failed to get frame count.\n");
>> +            return -EIO;
>> +    }
>> +
>> +    buf_id = frame_cnt - 1;
>> +    DRM_DEBUG_KMS("%s:buf_id[%d]\n", __func__, buf_id);
>> +
>> +    return buf_id;
>> +}
>> +
>> +static void fimc_handle_lastend(struct fimc_context *ctx, bool enable)
>> +{
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:enable[%d]\n", __func__, enable);
>> +
>> +    cfg = fimc_read(EXYNOS_CIOCTRL);
>> +    if (enable)
>> +            cfg |= EXYNOS_CIOCTRL_LASTENDEN;
>> +    else
>> +            cfg &= ~EXYNOS_CIOCTRL_LASTENDEN;
>> +
>> +    fimc_write(cfg, EXYNOS_CIOCTRL);
>> +}
>> +
>> +
>> +static int fimc_src_set_fmt_order(struct fimc_context *ctx, u32 fmt)
>> +{
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:fmt[0x%x]\n", __func__, fmt);
>> +
>> +    /* RGB */
>> +    cfg = fimc_read(EXYNOS_CISCCTRL);
>> +    cfg &= ~EXYNOS_CISCCTRL_INRGB_FMT_RGB_MASK;
>> +
>> +    switch (fmt) {
>> +    case DRM_FORMAT_RGB565:
>> +            cfg |= EXYNOS_CISCCTRL_INRGB_FMT_RGB565;
>> +            fimc_write(cfg, EXYNOS_CISCCTRL);
>> +            return 0;
>> +    case DRM_FORMAT_RGB888:
>> +    case DRM_FORMAT_XRGB8888:
>> +            cfg |= EXYNOS_CISCCTRL_INRGB_FMT_RGB888;
>> +            fimc_write(cfg, EXYNOS_CISCCTRL);
>> +            return 0;
>> +    default:
>> +            /* bypass */
>> +            break;
>> +    }
>> +
>> +    /* YUV */
>> +    cfg = fimc_read(EXYNOS_MSCTRL);
>> +    cfg &= ~(EXYNOS_MSCTRL_ORDER2P_SHIFT_MASK |
>> +            EXYNOS_MSCTRL_C_INT_IN_2PLANE |
>> +            EXYNOS_MSCTRL_ORDER422_YCBYCR);
>> +
>> +    switch (fmt) {
>> +    case DRM_FORMAT_YUYV:
>> +            cfg |= EXYNOS_MSCTRL_ORDER422_YCBYCR;
>> +            break;
>> +    case DRM_FORMAT_YVYU:
>> +            cfg |= EXYNOS_MSCTRL_ORDER422_YCRYCB;
>> +            break;
>> +    case DRM_FORMAT_UYVY:
>> +            cfg |= EXYNOS_MSCTRL_ORDER422_CBYCRY;
>> +            break;
>> +    case DRM_FORMAT_VYUY:
>> +    case DRM_FORMAT_YUV444:
>> +            cfg |= EXYNOS_MSCTRL_ORDER422_CRYCBY;
>> +            break;
>> +    case DRM_FORMAT_NV21:
>> +    case DRM_FORMAT_NV61:
>> +            cfg |= (EXYNOS_MSCTRL_ORDER2P_LSB_CRCB |
>> +                    EXYNOS_MSCTRL_C_INT_IN_2PLANE);
>> +            break;
>> +    case DRM_FORMAT_YUV422:
>> +    case DRM_FORMAT_YUV420:
>> +    case DRM_FORMAT_YVU420:
>> +            cfg |= EXYNOS_MSCTRL_C_INT_IN_3PLANE;
>> +            break;
>> +    case DRM_FORMAT_NV12:
>> +    case DRM_FORMAT_NV12MT:
>> +    case DRM_FORMAT_NV16:
>> +            cfg |= (EXYNOS_MSCTRL_ORDER2P_LSB_CBCR |
>> +                    EXYNOS_MSCTRL_C_INT_IN_2PLANE);
>> +            break;
>> +    default:
>> +            dev_err(ippdrv->dev, "inavlid source yuv order 0x%x.\n",
>> fmt);
>> +            return -EINVAL;
>> +    }
>> +
>> +    fimc_write(cfg, EXYNOS_MSCTRL);
>> +
>> +    return 0;
>> +}
>> +
>> +static int fimc_src_set_fmt(struct device *dev, u32 fmt)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:fmt[0x%x]\n", __func__, fmt);
>> +
>> +    cfg = fimc_read(EXYNOS_MSCTRL);
>> +    cfg &= ~EXYNOS_MSCTRL_INFORMAT_RGB;
>> +
>> +    switch (fmt) {
>> +    case DRM_FORMAT_RGB565:
>> +    case DRM_FORMAT_RGB888:
>> +    case DRM_FORMAT_XRGB8888:
>> +            cfg |= EXYNOS_MSCTRL_INFORMAT_RGB;
>> +            break;
>> +    case DRM_FORMAT_YUV444:
>> +            cfg |= EXYNOS_MSCTRL_INFORMAT_YCBCR420;
>> +            break;
>> +    case DRM_FORMAT_YUYV:
>> +    case DRM_FORMAT_YVYU:
>> +    case DRM_FORMAT_UYVY:
>> +    case DRM_FORMAT_VYUY:
>> +            cfg |= EXYNOS_MSCTRL_INFORMAT_YCBCR422_1PLANE;
>> +            break;
>> +    case DRM_FORMAT_NV16:
>> +    case DRM_FORMAT_NV61:
>> +    case DRM_FORMAT_YUV422:
>> +            cfg |= EXYNOS_MSCTRL_INFORMAT_YCBCR422;
>> +            break;
>> +    case DRM_FORMAT_YUV420:
>> +    case DRM_FORMAT_YVU420:
>> +    case DRM_FORMAT_NV12:
>> +    case DRM_FORMAT_NV21:
>> +    case DRM_FORMAT_NV12MT:
>> +            cfg |= EXYNOS_MSCTRL_INFORMAT_YCBCR420;
>> +            break;
>> +    default:
>> +            dev_err(ippdrv->dev, "inavlid source format 0x%x.\n", fmt);
>> +            return -EINVAL;
>> +    }
>> +
>> +    fimc_write(cfg, EXYNOS_MSCTRL);
>> +
>> +    cfg = fimc_read(EXYNOS_CIDMAPARAM);
>> +    cfg &= ~EXYNOS_CIDMAPARAM_R_MODE_MASK;
>> +
>> +    if (fmt == DRM_FORMAT_NV12MT)
>> +            cfg |= EXYNOS_CIDMAPARAM_R_MODE_64X32;
>> +    else
>> +            cfg |= EXYNOS_CIDMAPARAM_R_MODE_LINEAR;
>> +
>> +    fimc_write(cfg, EXYNOS_CIDMAPARAM);
>> +
>> +    return fimc_src_set_fmt_order(ctx, fmt);
>> +}
>> +
>> +static int fimc_src_set_transf(struct device *dev,
>> +            enum drm_exynos_degree degree,
>> +            enum drm_exynos_flip flip)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    u32 cfg1, cfg2;
>> +
>> +    DRM_DEBUG_KMS("%s:degree[%d]flip[0x%x]\n", __func__,
>> +            degree, flip);
>> +
>> +    cfg1 = fimc_read(EXYNOS_MSCTRL);
>> +    cfg1 &= ~(EXYNOS_MSCTRL_FLIP_X_MIRROR |
>> +            EXYNOS_MSCTRL_FLIP_Y_MIRROR);
>> +
>> +    cfg2 = fimc_read(EXYNOS_CITRGFMT);
>> +    cfg2 &= ~EXYNOS_CITRGFMT_INROT90_CLOCKWISE;
>> +
>> +    switch (degree) {
>> +    case EXYNOS_DRM_DEGREE_0:
>> +            if (flip & EXYNOS_DRM_FLIP_HORIZONTAL)
>> +                    cfg1 |= EXYNOS_MSCTRL_FLIP_X_MIRROR;
>> +            if (flip & EXYNOS_DRM_FLIP_VERTICAL)
>> +                    cfg1 |= EXYNOS_MSCTRL_FLIP_Y_MIRROR;
>> +            break;
>> +    case EXYNOS_DRM_DEGREE_90:
>> +            cfg2 |= EXYNOS_CITRGFMT_INROT90_CLOCKWISE;
>> +            if (flip & EXYNOS_DRM_FLIP_HORIZONTAL)
>> +                    cfg1 |= EXYNOS_MSCTRL_FLIP_X_MIRROR;
>> +            if (flip & EXYNOS_DRM_FLIP_VERTICAL)
>> +                    cfg1 |= EXYNOS_MSCTRL_FLIP_Y_MIRROR;
>> +            break;
>> +    case EXYNOS_DRM_DEGREE_180:
>> +            cfg1 |= (EXYNOS_MSCTRL_FLIP_X_MIRROR |
>> +                    EXYNOS_MSCTRL_FLIP_Y_MIRROR);
>> +            if (flip & EXYNOS_DRM_FLIP_HORIZONTAL)
>> +                    cfg1 &= ~EXYNOS_MSCTRL_FLIP_X_MIRROR;
>> +            if (flip & EXYNOS_DRM_FLIP_VERTICAL)
>> +                    cfg1 &= ~EXYNOS_MSCTRL_FLIP_Y_MIRROR;
>> +            break;
>> +    case EXYNOS_DRM_DEGREE_270:
>> +            cfg1 |= (EXYNOS_MSCTRL_FLIP_X_MIRROR |
>> +                    EXYNOS_MSCTRL_FLIP_Y_MIRROR);
>> +            cfg2 |= EXYNOS_CITRGFMT_INROT90_CLOCKWISE;
>> +            if (flip & EXYNOS_DRM_FLIP_HORIZONTAL)
>> +                    cfg1 &= ~EXYNOS_MSCTRL_FLIP_X_MIRROR;
>> +            if (flip & EXYNOS_DRM_FLIP_VERTICAL)
>> +                    cfg1 &= ~EXYNOS_MSCTRL_FLIP_Y_MIRROR;
>> +            break;
>> +    default:
>> +            dev_err(ippdrv->dev, "inavlid degree value %d.\n", degree);
>> +            return -EINVAL;
>> +    }
>> +
>> +    fimc_write(cfg1, EXYNOS_MSCTRL);
>> +    fimc_write(cfg2, EXYNOS_CITRGFMT);
>> +
>> +    return (cfg2 & EXYNOS_CITRGFMT_INROT90_CLOCKWISE) ? 1 : 0;
>> +}
>> +
>> +static int fimc_set_window(struct fimc_context *ctx,
>> +    struct drm_exynos_pos *pos, struct drm_exynos_sz *sz)
>> +{
>> +    u32 cfg, h1, h2, v1, v2;
>> +
>> +    /* cropped image */
>> +    h1 = pos->x;
>> +    h2 = sz->hsize - pos->w - pos->x;
>> +    v1 = pos->y;
>> +    v2 = sz->vsize - pos->h - pos->y;
>> +
>> +    DRM_DEBUG_KMS("%s:x[%d]y[%d]w[%d]h[%d]hsize[%d]vsize[%d]\n",
>> +    __func__, pos->x, pos->y, pos->w, pos->h, sz->hsize, sz->vsize);
>> +    DRM_DEBUG_KMS("%s:h1[%d]h2[%d]v1[%d]v2[%d]\n", __func__,
>> +            h1, h2, v1, v2);
>> +
>> +    /*
>> +     * set window offset 1, 2 size
>> +     * check figure 43-21 in user manual
>> +     */
>> +    cfg = fimc_read(EXYNOS_CIWDOFST);
>> +    cfg &= ~(EXYNOS_CIWDOFST_WINHOROFST_MASK |
>> +            EXYNOS_CIWDOFST_WINVEROFST_MASK);
>> +    cfg |= (EXYNOS_CIWDOFST_WINHOROFST(h1) |
>> +            EXYNOS_CIWDOFST_WINVEROFST(v1));
>> +    cfg |= EXYNOS_CIWDOFST_WINOFSEN;
>> +    fimc_write(cfg, EXYNOS_CIWDOFST);
>> +
>> +    cfg = (EXYNOS_CIWDOFST2_WINHOROFST2(h2) |
>> +            EXYNOS_CIWDOFST2_WINVEROFST2(v2));
>> +    fimc_write(cfg, EXYNOS_CIWDOFST2);
>> +
>> +    return 0;
>> +}
>> +
>> +static int fimc_src_set_size(struct device *dev, int swap,
>> +    struct drm_exynos_pos *pos, struct drm_exynos_sz *sz)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct drm_exynos_pos img_pos = *pos;
>> +    struct drm_exynos_sz img_sz = *sz;
>> +    u32 cfg;
>> +
>> +    /* ToDo: check width and height */
>> +
>> +    DRM_DEBUG_KMS("%s:swap[%d]hsize[%d]vsize[%d]\n",
>> +            __func__, swap, sz->hsize, sz->vsize);
>> +
>> +    /* original size */
>> +    cfg = (EXYNOS_ORGISIZE_HORIZONTAL(img_sz.hsize) |
>> +            EXYNOS_ORGISIZE_VERTICAL(img_sz.vsize));
>> +
>> +    fimc_write(cfg, EXYNOS_ORGISIZE);
>> +
>> +    DRM_DEBUG_KMS("%s:x[%d]y[%d]w[%d]h[%d]\n", __func__,
>> +            pos->x, pos->y, pos->w, pos->h);
>> +
>> +    if (swap) {
>> +            img_pos.w = pos->h;
>> +            img_pos.h = pos->w;
>> +            img_sz.hsize = sz->vsize;
>> +            img_sz.vsize = sz->hsize;
>> +    }
>> +
>> +    /* set input DMA image size */
>> +    cfg = fimc_read(EXYNOS_CIREAL_ISIZE);
>> +    cfg &= ~(EXYNOS_CIREAL_ISIZE_HEIGHT_MASK |
>> +            EXYNOS_CIREAL_ISIZE_WIDTH_MASK);
>> +    cfg |= (EXYNOS_CIREAL_ISIZE_WIDTH(img_pos.w) |
>> +            EXYNOS_CIREAL_ISIZE_HEIGHT(img_pos.h));
>> +    fimc_write(cfg, EXYNOS_CIREAL_ISIZE);
>> +
>> +    /*
>> +     * set input FIFO image size
>> +     * for now, we support only ITU601 8 bit mode
>> +     */
>> +    cfg = (EXYNOS_CISRCFMT_ITU601_8BIT |
>> +            EXYNOS_CISRCFMT_SOURCEHSIZE(img_sz.hsize) |
>> +            EXYNOS_CISRCFMT_SOURCEVSIZE(img_sz.vsize));
>> +    fimc_write(cfg, EXYNOS_CISRCFMT);
>> +
>> +    /* offset Y(RGB), Cb, Cr */
>> +    cfg = (EXYNOS_CIIYOFF_HORIZONTAL(img_pos.x) |
>> +            EXYNOS_CIIYOFF_VERTICAL(img_pos.y));
>> +    fimc_write(cfg, EXYNOS_CIIYOFF);
>> +    cfg = (EXYNOS_CIICBOFF_HORIZONTAL(img_pos.x) |
>> +            EXYNOS_CIICBOFF_VERTICAL(img_pos.y));
>> +    fimc_write(cfg, EXYNOS_CIICBOFF);
>> +    cfg = (EXYNOS_CIICROFF_HORIZONTAL(img_pos.x) |
>> +            EXYNOS_CIICROFF_VERTICAL(img_pos.y));
>> +    fimc_write(cfg, EXYNOS_CIICROFF);
>> +
>> +    return fimc_set_window(ctx, &img_pos, &img_sz);
>> +}
>> +
>> +static int fimc_src_set_addr(struct device *dev,
>> +                    struct drm_exynos_ipp_buf_info *buf_info, u32
> buf_id,
>> +                    enum drm_exynos_ipp_buf_type buf_type)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    struct drm_exynos_ipp_cmd_node *c_node = ippdrv->cmd;
>> +    struct drm_exynos_ipp_property *property;
>> +    struct drm_exynos_ipp_config *config;
>> +
>> +    if (!c_node) {
>> +            DRM_ERROR("failed to get c_node.\n");
>> +            return -EINVAL;
>> +    }
>> +
>> +    property = &c_node->property;
>> +    if (!property) {
>> +            DRM_ERROR("failed to get property.\n");
>> +            return -EINVAL;
>> +    }
>> +
>> +    DRM_DEBUG_KMS("%s:prop_id[%d]buf_id[%d]buf_type[%d]\n", __func__,
>> +            property->prop_id, buf_id, buf_type);
>> +
>> +    if (buf_id > FIMC_MAX_SRC) {
>> +            dev_info(ippdrv->dev, "inavlid buf_id %d.\n", buf_id);
>> +            return -ENOMEM;
>> +    }
>> +
>> +    /* address register set */
>> +    switch (buf_type) {
>> +    case IPP_BUF_ENQUEUE:
>> +            config = &property->config[EXYNOS_DRM_OPS_SRC];
>> +            fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_Y],
>> +                    EXYNOS_CIIYSA(buf_id));
>
> YUV format should always be used as Source buffer?

- What mean that ? This api is set_src_addr handler api.
   so, we set source buffer using buf_info. src buffer support YUV, RGB 
both.

>
>> +
>> +            if (config->fmt == DRM_FORMAT_YVU420) {
>> +                    fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_CR],
>> +                            EXYNOS_CIICBSA(buf_id));
>> +                    fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_CB],
>> +                            EXYNOS_CIICRSA(buf_id));
>> +            } else {
>> +                    fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_CB],
>> +                            EXYNOS_CIICBSA(buf_id));
>> +                    fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_CR],
>> +                            EXYNOS_CIICRSA(buf_id));
>> +            }
>> +            break;
>> +    case IPP_BUF_DEQUEUE:
>> +            fimc_write(0x0, EXYNOS_CIIYSA(buf_id));
>> +            fimc_write(0x0, EXYNOS_CIICBSA(buf_id));
>> +            fimc_write(0x0, EXYNOS_CIICRSA(buf_id));
>> +            break;
>> +    default:
>> +            /* bypass */
>> +            break;
>> +    }
>> +
>> +    return 0;
>> +}
>> +
>> +static struct exynos_drm_ipp_ops fimc_src_ops = {
>> +    .set_fmt = fimc_src_set_fmt,
>> +    .set_transf = fimc_src_set_transf,
>> +    .set_size = fimc_src_set_size,
>> +    .set_addr = fimc_src_set_addr,
>> +};
>> +
>> +static int fimc_dst_set_fmt_order(struct fimc_context *ctx, u32 fmt)
>> +{
>
> Combine fimc_src/dst_set_fmt_order function. And you could use each offset
> to check if src or dst.

- I think that incleased complexity.
   src,dst format have every other value.

>
>
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:fmt[0x%x]\n", __func__, fmt);
>> +
>> +    /* RGB */
>> +    cfg = fimc_read(EXYNOS_CISCCTRL);
>> +    cfg &= ~EXYNOS_CISCCTRL_OUTRGB_FMT_RGB_MASK;
>> +
>> +    switch (fmt) {
>> +    case DRM_FORMAT_RGB565:
>> +            cfg |= EXYNOS_CISCCTRL_OUTRGB_FMT_RGB565;
>> +            fimc_write(cfg, EXYNOS_CISCCTRL);
>> +            return 0;
>> +    case DRM_FORMAT_RGB888:
>> +            cfg |= EXYNOS_CISCCTRL_OUTRGB_FMT_RGB888;
>> +            fimc_write(cfg, EXYNOS_CISCCTRL);
>> +            return 0;
>> +    case DRM_FORMAT_XRGB8888:
>> +            cfg |= (EXYNOS_CISCCTRL_OUTRGB_FMT_RGB888 |
>> +                    EXYNOS_CISCCTRL_EXTRGB_EXTENSION);
>> +            fimc_write(cfg, EXYNOS_CISCCTRL);
>> +            break;
>> +    default:
>> +            /* bypass */
>> +            break;
>> +    }
>> +
>> +    /* YUV */
>> +    cfg = fimc_read(EXYNOS_CIOCTRL);
>> +    cfg &= ~(EXYNOS_CIOCTRL_ORDER2P_MASK |
>> +            EXYNOS_CIOCTRL_ORDER422_MASK |
>> +            EXYNOS_CIOCTRL_YCBCR_PLANE_MASK);
>> +
>> +    switch (fmt) {
>> +    case DRM_FORMAT_XRGB8888:
>> +            cfg |= EXYNOS_CIOCTRL_ALPHA_OUT;
>> +            break;
>> +    case DRM_FORMAT_YUYV:
>> +            cfg |= EXYNOS_CIOCTRL_ORDER422_YCBYCR;
>> +            break;
>> +    case DRM_FORMAT_YVYU:
>> +            cfg |= EXYNOS_CIOCTRL_ORDER422_YCRYCB;
>> +            break;
>> +    case DRM_FORMAT_UYVY:
>> +            cfg |= EXYNOS_CIOCTRL_ORDER422_CBYCRY;
>> +            break;
>> +    case DRM_FORMAT_VYUY:
>> +            cfg |= EXYNOS_CIOCTRL_ORDER422_CRYCBY;
>> +            break;
>> +    case DRM_FORMAT_NV21:
>> +    case DRM_FORMAT_NV61:
>> +            cfg |= EXYNOS_CIOCTRL_ORDER2P_LSB_CRCB;
>> +            cfg |= EXYNOS_CIOCTRL_YCBCR_2PLANE;
>> +            break;
>> +    case DRM_FORMAT_YUV422:
>> +    case DRM_FORMAT_YUV420:
>> +    case DRM_FORMAT_YVU420:
>> +            cfg |= EXYNOS_CIOCTRL_YCBCR_3PLANE;
>> +            break;
>> +    case DRM_FORMAT_NV12:
>> +    case DRM_FORMAT_NV12MT:
>> +    case DRM_FORMAT_NV16:
>> +            cfg |= EXYNOS_CIOCTRL_ORDER2P_LSB_CBCR;
>> +            cfg |= EXYNOS_CIOCTRL_YCBCR_2PLANE;
>> +            break;
>> +    default:
>> +            dev_err(ippdrv->dev, "inavlid target yuv order 0x%x.\n",
>> fmt);
>> +            return -EINVAL;
>> +    }
>> +
>> +    fimc_write(cfg, EXYNOS_CIOCTRL);
>> +
>> +    return 0;
>> +}
>> +
>> +static int fimc_dst_set_fmt(struct device *dev, u32 fmt)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:fmt[0x%x]\n", __func__, fmt);
>> +
>> +    cfg = fimc_read(EXYNOS_CIEXTEN);
>> +
>> +    if (fmt == DRM_FORMAT_AYUV) {
>> +            cfg |= EXYNOS_CIEXTEN_YUV444_OUT;
>> +            fimc_write(cfg, EXYNOS_CIEXTEN);
>> +    } else {
>> +            cfg &= ~EXYNOS_CIEXTEN_YUV444_OUT;
>> +            fimc_write(cfg, EXYNOS_CIEXTEN);
>> +
>> +            cfg = fimc_read(EXYNOS_CITRGFMT);
>> +            cfg &= ~EXYNOS_CITRGFMT_OUTFORMAT_MASK;
>> +
>> +            switch (fmt) {
>> +            case DRM_FORMAT_RGB565:
>> +            case DRM_FORMAT_RGB888:
>> +            case DRM_FORMAT_XRGB8888:
>> +                    cfg |= EXYNOS_CITRGFMT_OUTFORMAT_RGB;
>> +                    break;
>> +            case DRM_FORMAT_YUYV:
>> +            case DRM_FORMAT_YVYU:
>> +            case DRM_FORMAT_UYVY:
>> +            case DRM_FORMAT_VYUY:
>> +                    cfg |= EXYNOS_CITRGFMT_OUTFORMAT_YCBCR422_1PLANE;
>> +                    break;
>> +            case DRM_FORMAT_NV16:
>> +            case DRM_FORMAT_NV61:
>> +            case DRM_FORMAT_YUV422:
>> +                    cfg |= EXYNOS_CITRGFMT_OUTFORMAT_YCBCR422;
>> +                    break;
>> +            case DRM_FORMAT_YUV420:
>> +            case DRM_FORMAT_YVU420:
>> +            case DRM_FORMAT_NV12:
>> +            case DRM_FORMAT_NV12MT:
>> +            case DRM_FORMAT_NV21:
>> +                    cfg |= EXYNOS_CITRGFMT_OUTFORMAT_YCBCR420;
>> +                    break;
>> +            default:
>> +                    dev_err(ippdrv->dev, "inavlid target format
> 0x%x.\n",
>> +                            fmt);
>> +                    return -EINVAL;
>> +            }
>> +
>> +            fimc_write(cfg, EXYNOS_CITRGFMT);
>> +    }
>> +
>> +    cfg = fimc_read(EXYNOS_CIDMAPARAM);
>> +    cfg &= ~EXYNOS_CIDMAPARAM_W_MODE_MASK;
>> +
>> +    if (fmt == DRM_FORMAT_NV12MT)
>> +            cfg |= EXYNOS_CIDMAPARAM_W_MODE_64X32;
>> +    else
>> +            cfg |= EXYNOS_CIDMAPARAM_W_MODE_LINEAR;
>> +
>> +    fimc_write(cfg, EXYNOS_CIDMAPARAM);
>> +
>> +    return fimc_dst_set_fmt_order(ctx, fmt);
>> +}
>> +
>> +static int fimc_dst_set_transf(struct device *dev,
>> +            enum drm_exynos_degree degree,
>> +            enum drm_exynos_flip flip)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:degree[%d]flip[0x%x]\n", __func__,
>> +            degree, flip);
>> +
>> +    cfg = fimc_read(EXYNOS_CITRGFMT);
>> +    cfg &= ~EXYNOS_CITRGFMT_FLIP_MASK;
>> +    cfg &= ~EXYNOS_CITRGFMT_OUTROT90_CLOCKWISE;
>> +
>> +    switch (degree) {
>> +    case EXYNOS_DRM_DEGREE_0:
>> +            if (flip & EXYNOS_DRM_FLIP_HORIZONTAL)
>> +                    cfg |= EXYNOS_CITRGFMT_FLIP_X_MIRROR;
>> +            if (flip & EXYNOS_DRM_FLIP_VERTICAL)
>> +                    cfg |= EXYNOS_CITRGFMT_FLIP_Y_MIRROR;
>> +            break;
>> +    case EXYNOS_DRM_DEGREE_90:
>> +            cfg |= EXYNOS_CITRGFMT_OUTROT90_CLOCKWISE;
>> +            if (flip & EXYNOS_DRM_FLIP_HORIZONTAL)
>> +                    cfg |= EXYNOS_CITRGFMT_FLIP_X_MIRROR;
>> +            if (flip & EXYNOS_DRM_FLIP_VERTICAL)
>> +                    cfg |= EXYNOS_CITRGFMT_FLIP_Y_MIRROR;
>> +            break;
>> +    case EXYNOS_DRM_DEGREE_180:
>> +            cfg |= (EXYNOS_CITRGFMT_FLIP_X_MIRROR |
>> +                    EXYNOS_CITRGFMT_FLIP_Y_MIRROR);
>> +            if (flip & EXYNOS_DRM_FLIP_HORIZONTAL)
>> +                    cfg &= ~EXYNOS_CITRGFMT_FLIP_X_MIRROR;
>> +            if (flip & EXYNOS_DRM_FLIP_VERTICAL)
>> +                    cfg &= ~EXYNOS_CITRGFMT_FLIP_Y_MIRROR;
>> +            break;
>> +    case EXYNOS_DRM_DEGREE_270:
>> +            cfg |= (EXYNOS_CITRGFMT_OUTROT90_CLOCKWISE |
>> +                    EXYNOS_CITRGFMT_FLIP_X_MIRROR |
>> +                    EXYNOS_CITRGFMT_FLIP_Y_MIRROR);
>> +            if (flip & EXYNOS_DRM_FLIP_HORIZONTAL)
>> +                    cfg &= ~EXYNOS_CITRGFMT_FLIP_X_MIRROR;
>> +            if (flip & EXYNOS_DRM_FLIP_VERTICAL)
>> +                    cfg &= ~EXYNOS_CITRGFMT_FLIP_Y_MIRROR;
>> +            break;
>> +    default:
>> +            dev_err(ippdrv->dev, "inavlid degree value %d.\n", degree);
>> +            return -EINVAL;
>> +    }
>> +
>> +    fimc_write(cfg, EXYNOS_CITRGFMT);
>> +
>> +    return (cfg & EXYNOS_CITRGFMT_OUTROT90_CLOCKWISE) ? 1 : 0;
>> +}
>> +
>> +static int fimc_get_ratio_shift(u32 src, u32 dst, u32 *ratio, u32 *shift)
>> +{
>> +    DRM_DEBUG_KMS("%s:src[%d]dst[%d]\n", __func__, src, dst);
>> +
>> +    if (src >= dst * 64) {
>> +            DRM_ERROR("failed to make ratio and shift.\n");
>> +            return -EINVAL;
>> +    } else if (src >= dst * 32) {
>> +            *ratio = 32;
>> +            *shift = 5;
>> +    } else if (src >= dst * 16) {
>> +            *ratio = 16;
>> +            *shift = 4;
>> +    } else if (src >= dst * 8) {
>> +            *ratio = 8;
>> +            *shift = 3;
>> +    } else if (src >= dst * 4) {
>> +            *ratio = 4;
>> +            *shift = 2;
>> +    } else if (src >= dst * 2) {
>> +            *ratio = 2;
>> +            *shift = 1;
>> +    } else {
>> +            *ratio = 1;
>> +            *shift = 0;
>> +    }
>> +
>> +    return 0;
>> +}
>> +
>> +static int fimc_set_prescaler(struct fimc_context *ctx, struct
>> fimc_scaler *sc,
>> +            struct drm_exynos_pos *src, struct drm_exynos_pos *dst)
>> +{
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    u32 cfg, cfg_ext, shfactor;
>> +    u32 pre_dst_width, pre_dst_height;
>> +    u32 pre_hratio, hfactor, pre_vratio, vfactor;
>> +    int ret = 0;
>> +    u32 src_w, src_h, dst_w, dst_h;
>> +
>> +    cfg_ext = fimc_read(EXYNOS_CITRGFMT);
>> +    if (cfg_ext & EXYNOS_CITRGFMT_INROT90_CLOCKWISE) {
>> +            src_w = src->h;
>> +            src_h = src->w;
>> +    } else {
>> +            src_w = src->w;
>> +            src_h = src->h;
>> +    }
>> +
>> +    if (cfg_ext & EXYNOS_CITRGFMT_OUTROT90_CLOCKWISE) {
>> +            dst_w = dst->h;
>> +            dst_h = dst->w;
>> +    } else {
>> +            dst_w = dst->w;
>> +            dst_h = dst->h;
>> +    }
>> +
>> +    ret = fimc_get_ratio_shift(src_w, dst_w, &pre_hratio, &hfactor);
>> +    if (ret) {
>> +            dev_err(ippdrv->dev, "failed to get ratio horizontal.\n");
>> +            return ret;
>> +    }
>> +
>> +    ret = fimc_get_ratio_shift(src_h, dst_h, &pre_vratio, &vfactor);
>> +    if (ret) {
>> +            dev_err(ippdrv->dev, "failed to get ratio vertical.\n");
>> +            return ret;
>> +    }
>> +
>> +    pre_dst_width = src_w / pre_hratio;
>> +    pre_dst_height = src_h / pre_vratio;
>> +    DRM_DEBUG_KMS("%s:pre_dst_width[%d]pre_dst_height[%d]\n", __func__,
>> +            pre_dst_width, pre_dst_height);
>> +
>>      DRM_DEBUG_KMS("%s:pre_hratio[%d]hfactor[%d]pre_vratio[%d]vfactor[%d
>> ]\n",
>> +            __func__, pre_hratio, hfactor, pre_vratio, vfactor);
>> +
>> +    sc->hratio = (src_w << 14) / (dst_w << hfactor);
>> +    sc->vratio = (src_h << 14) / (dst_h << vfactor);
>> +    sc->up_h = (dst_w >= src_w) ? true : false;
>> +    sc->up_v = (dst_h >= src_h) ? true : false;
>> +    DRM_DEBUG_KMS("%s:hratio[%d]vratio[%d]up_h[%d]up_v[%d]\n",
>> +    __func__, sc->hratio, sc->vratio, sc->up_h, sc->up_v);
>> +
>> +    shfactor = 10 - (hfactor + vfactor);
>
> What is 10? use macro instead.

- done.

>
>> +    DRM_DEBUG_KMS("%s:shfactor[%d]\n", __func__, shfactor);
>> +
>> +    cfg = (EXYNOS_CISCPRERATIO_SHFACTOR(shfactor) |
>> +            EXYNOS_CISCPRERATIO_PREHORRATIO(pre_hratio) |
>> +            EXYNOS_CISCPRERATIO_PREVERRATIO(pre_vratio));
>> +    fimc_write(cfg, EXYNOS_CISCPRERATIO);
>> +
>> +    cfg = (EXYNOS_CISCPREDST_PREDSTWIDTH(pre_dst_width) |
>> +            EXYNOS_CISCPREDST_PREDSTHEIGHT(pre_dst_height));
>> +    fimc_write(cfg, EXYNOS_CISCPREDST);
>> +
>> +    return ret;
>> +}
>> +
>> +static void fimc_set_scaler(struct fimc_context *ctx, struct fimc_scaler
>> *sc)
>> +{
>> +    u32 cfg, cfg_ext;
>> +
>> +    DRM_DEBUG_KMS("%s:range[%d]bypass[%d]up_h[%d]up_v[%d]\n",
>> +            __func__, sc->range, sc->bypass, sc->up_h, sc->up_v);
>> +    DRM_DEBUG_KMS("%s:hratio[%d]vratio[%d]\n",
>> +            __func__, sc->hratio, sc->vratio);
>> +
>> +    cfg = fimc_read(EXYNOS_CISCCTRL);
>> +    cfg &= ~(EXYNOS_CISCCTRL_SCALERBYPASS |
>> +            EXYNOS_CISCCTRL_SCALEUP_H | EXYNOS_CISCCTRL_SCALEUP_V |
>> +            EXYNOS_CISCCTRL_MAIN_V_RATIO_MASK |
>> +            EXYNOS_CISCCTRL_MAIN_H_RATIO_MASK |
>> +            EXYNOS_CISCCTRL_CSCR2Y_WIDE |
>> +            EXYNOS_CISCCTRL_CSCY2R_WIDE);
>> +
>> +    if (sc->range)
>> +            cfg |= (EXYNOS_CISCCTRL_CSCR2Y_WIDE |
>> +                    EXYNOS_CISCCTRL_CSCY2R_WIDE);
>> +    if (sc->bypass)
>> +            cfg |= EXYNOS_CISCCTRL_SCALERBYPASS;
>> +    if (sc->up_h)
>> +            cfg |= EXYNOS_CISCCTRL_SCALEUP_H;
>> +    if (sc->up_v)
>> +            cfg |= EXYNOS_CISCCTRL_SCALEUP_V;
>> +
>> +    cfg |= (EXYNOS_CISCCTRL_MAINHORRATIO((sc->hratio >> 6)) |
>> +            EXYNOS_CISCCTRL_MAINVERRATIO((sc->vratio >> 6)));
>> +    fimc_write(cfg, EXYNOS_CISCCTRL);
>> +
>> +    cfg_ext = fimc_read(EXYNOS_CIEXTEN);
>> +    cfg_ext &= ~EXYNOS_CIEXTEN_MAINHORRATIO_EXT_MASK;
>> +    cfg_ext &= ~EXYNOS_CIEXTEN_MAINVERRATIO_EXT_MASK;
>> +    cfg_ext |= (EXYNOS_CIEXTEN_MAINHORRATIO_EXT(sc->hratio) |
>> +            EXYNOS_CIEXTEN_MAINVERRATIO_EXT(sc->vratio));
>> +    fimc_write(cfg_ext, EXYNOS_CIEXTEN);
>> +}
>> +
>> +static int fimc_dst_set_size(struct device *dev, int swap,
>> +    struct drm_exynos_pos *pos, struct drm_exynos_sz *sz)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct drm_exynos_pos img_pos = *pos;
>> +    struct drm_exynos_sz img_sz = *sz;
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:swap[%d]hsize[%d]vsize[%d]\n",
>> +            __func__, swap, sz->hsize, sz->vsize);
>> +
>> +    /* original size */
>> +    cfg = (EXYNOS_ORGOSIZE_HORIZONTAL(img_sz.hsize) |
>> +            EXYNOS_ORGOSIZE_VERTICAL(img_sz.vsize));
>> +
>> +    fimc_write(cfg, EXYNOS_ORGOSIZE);
>> +
>> +    DRM_DEBUG_KMS("%s:x[%d]y[%d]w[%d]h[%d]\n",
>> +            __func__, pos->x, pos->y, pos->w, pos->h);
>> +
>> +    /* CSC ITU */
>> +    cfg = fimc_read(EXYNOS_CIGCTRL);
>> +    cfg &= ~EXYNOS_CIGCTRL_CSC_MASK;
>> +
>> +    if (sz->hsize >= FIMC_WIDTH_ITU_709)
>> +            cfg |= EXYNOS_CIGCTRL_CSC_ITU709;
>> +    else
>> +            cfg |= EXYNOS_CIGCTRL_CSC_ITU601;
>> +
>> +    fimc_write(cfg, EXYNOS_CIGCTRL);
>> +
>> +    if (swap) {
>> +            img_pos.w = pos->h;
>> +            img_pos.h = pos->w;
>> +            img_sz.hsize = sz->vsize;
>> +            img_sz.vsize = sz->hsize;
>> +    }
>> +
>> +    /* target image size */
>> +    cfg = fimc_read(EXYNOS_CITRGFMT);
>> +    cfg &= ~(EXYNOS_CITRGFMT_TARGETH_MASK |
>> +            EXYNOS_CITRGFMT_TARGETV_MASK);
>> +    cfg |= (EXYNOS_CITRGFMT_TARGETHSIZE(img_pos.w) |
>> +            EXYNOS_CITRGFMT_TARGETVSIZE(img_pos.h));
>> +    fimc_write(cfg, EXYNOS_CITRGFMT);
>> +
>> +    /* target area */
>> +    cfg = EXYNOS_CITAREA_TARGET_AREA(img_pos.w * img_pos.h);
>> +    fimc_write(cfg, EXYNOS_CITAREA);
>> +
>> +    /* ToDo: Move Scaler in this line and YUV */
>> +
>> +    /* offset Y(RGB), Cb, Cr */
>> +    cfg = (EXYNOS_CIOYOFF_HORIZONTAL(img_pos.x) |
>> +            EXYNOS_CIOYOFF_VERTICAL(img_pos.y));
>> +    fimc_write(cfg, EXYNOS_CIOYOFF);
>> +    cfg = (EXYNOS_CIOCBOFF_HORIZONTAL(img_pos.x) |
>> +            EXYNOS_CIOCBOFF_VERTICAL(img_pos.y));
>> +    fimc_write(cfg, EXYNOS_CIOCBOFF);
>> +    cfg = (EXYNOS_CIOCROFF_HORIZONTAL(img_pos.x) |
>> +            EXYNOS_CIOCROFF_VERTICAL(img_pos.y));
>> +    fimc_write(cfg, EXYNOS_CIOCROFF);
>> +
>> +    return 0;
>> +}
>> +
>> +static int fimc_dst_get_buf_seq(struct fimc_context *ctx)
>> +{
>> +    u32 cfg, i, buf_num = 0;
>> +    u32 mask = 0x00000001;
>> +
>> +    cfg = fimc_read(EXYNOS_CIFCNTSEQ);
>> +
>> +    for (i = 0; i < FIMC_REG_SZ; i++)
>> +            if (cfg & (mask << i))
>> +                    buf_num++;
>> +
>> +    DRM_DEBUG_KMS("%s:buf_num[%d]\n", __func__, buf_num);
>> +
>> +    return buf_num;
>> +}
>> +
>> +static int fimc_dst_set_buf_seq(struct fimc_context *ctx, u32 buf_id,
>> +    enum drm_exynos_ipp_buf_type buf_type)
>> +{
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    bool enable;
>> +    u32 cfg;
>> +    u32 mask = 0x00000001 << buf_id;
>> +    unsigned long flags;
>> +    int ret = 0;
>> +
>> +    DRM_DEBUG_KMS("%s:buf_id[%d]buf_type[%d]\n", __func__,
>> +            buf_id, buf_type);
>> +
>> +    spin_lock_irqsave(&ctx->lock, flags);
>
> Why do you use spin_lock_irqsave?

- First time we used request_irq intead of request_threaded_irq.
   so, remain this code.
   change to mutex.

>
>> +
>> +    /* mask register set */
>> +    cfg = fimc_read(EXYNOS_CIFCNTSEQ);
>> +
>> +    switch (buf_type) {
>> +    case IPP_BUF_ENQUEUE:
>> +            enable = true;
>> +            break;
>> +    case IPP_BUF_DEQUEUE:
>> +            enable = false;
>> +            break;
>> +    default:
>> +            dev_err(ippdrv->dev, "invalid buf ctrl parameter.\n");
>> +            ret =  -EINVAL;
>> +            goto err_unlock;
>> +    }
>> +
>> +    /* sequence id */
>> +    cfg &= (~mask);
>> +    cfg |= (enable << buf_id);
>> +    fimc_write(cfg, EXYNOS_CIFCNTSEQ);
>> +
>> +    /* interrupt enable */
>> +    if (buf_type == IPP_BUF_ENQUEUE &&
>> +        fimc_dst_get_buf_seq(ctx) >= FIMC_BUF_START)
>> +            fimc_handle_irq(ctx, true, false, true);
>> +
>> +    /* interrupt disable */
>> +    if (buf_type == IPP_BUF_DEQUEUE &&
>> +        fimc_dst_get_buf_seq(ctx) <= FIMC_BUF_STOP)
>> +            fimc_handle_irq(ctx, false, false, true);
>> +
>> +err_unlock:
>> +    spin_unlock_irqrestore(&ctx->lock, flags);
>> +    return ret;
>> +}
>> +
>> +static int fimc_dst_set_addr(struct device *dev,
>> +                    struct drm_exynos_ipp_buf_info *buf_info, u32
> buf_id,
>> +                    enum drm_exynos_ipp_buf_type buf_type)
>
> Combine fimc_src/dst_set_addr function. Only register offsets are different.

- I think ops don't need seperation. sometimes we need to insert condition
   about src, dst seperation. because buffer list is different.
   so, I want to remain this function.

>
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    struct drm_exynos_ipp_cmd_node *c_node = ippdrv->cmd;
>> +    struct drm_exynos_ipp_property *property;
>> +    struct drm_exynos_ipp_config *config;
>> +
>> +    if (!c_node) {
>> +            DRM_ERROR("failed to get c_node.\n");
>> +            return -EINVAL;
>> +    }
>> +
>> +    property = &c_node->property;
>> +    if (!property) {
>> +            DRM_ERROR("failed to get property.\n");
>> +            return -EINVAL;
>> +    }
>> +
>> +    DRM_DEBUG_KMS("%s:prop_id[%d]buf_id[%d]buf_type[%d]\n", __func__,
>> +            property->prop_id, buf_id, buf_type);
>> +
>> +    if (buf_id > FIMC_MAX_DST) {
>> +            dev_info(ippdrv->dev, "inavlid buf_id %d.\n", buf_id);
>> +            return -ENOMEM;
>> +    }
>> +
>> +    /* address register set */
>> +    switch (buf_type) {
>> +    case IPP_BUF_ENQUEUE:
>> +            config = &property->config[EXYNOS_DRM_OPS_DST];
>> +
>> +            fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_Y],
>> +                    EXYNOS_CIOYSA(buf_id));
>> +
>> +            if (config->fmt == DRM_FORMAT_YVU420) {
>> +                    fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_CR],
>> +                            EXYNOS_CIOCBSA(buf_id));
>> +                    fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_CB],
>> +                            EXYNOS_CIOCRSA(buf_id));
>> +            } else {
>> +                    fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_CB],
>> +                            EXYNOS_CIOCBSA(buf_id));
>> +                    fimc_write(buf_info->base[EXYNOS_DRM_PLANAR_CR],
>> +                            EXYNOS_CIOCRSA(buf_id));
>> +            }
>> +            break;
>> +    case IPP_BUF_DEQUEUE:
>> +            fimc_write(0x0, EXYNOS_CIOYSA(buf_id));
>> +            fimc_write(0x0, EXYNOS_CIOCBSA(buf_id));
>> +            fimc_write(0x0, EXYNOS_CIOCRSA(buf_id));
>> +            break;
>> +    default:
>> +            /* bypass */
>> +            break;
>> +    }
>> +
>> +    return fimc_dst_set_buf_seq(ctx, buf_id, buf_type);
>> +}
>> +
>> +static struct exynos_drm_ipp_ops fimc_dst_ops = {
>> +    .set_fmt = fimc_dst_set_fmt,
>> +    .set_transf = fimc_dst_set_transf,
>> +    .set_size = fimc_dst_set_size,
>> +    .set_addr = fimc_dst_set_addr,
>> +};
>> +
>> +static int fimc_power_on(struct fimc_context *ctx, bool enable)
>> +{
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +
>> +    DRM_DEBUG_KMS("%s:enable[%d]\n", __func__, enable);
>> +
>> +    if (enable) {
>> +            if (ctx->ver == FIMC_EXYNOS_4210) {
>> +                    ctx->sclk_fimc_clk = clk_get(ippdrv->dev,
> "sclk_fimc");
>> +                    if (IS_ERR(ctx->sclk_fimc_clk)) {
>> +                            dev_err(ippdrv->dev, "failed to get src fimc
>> clock.\n");
>> +                            return -EINVAL;
>> +                    }
>> +                    clk_enable(ctx->sclk_fimc_clk);
>> +            } else
>> +                    clk_enable(ctx->sclk_fimc_clk);
>> +
>> +            clk_enable(ctx->fimc_clk);
>> +            clk_enable(ctx->wb_clk);
>> +            /* ToDo : wb_b_clk */
>> +            ctx->suspended = false;
>> +    } else {
>> +            clk_disable(ctx->sclk_fimc_clk);
>> +            clk_disable(ctx->fimc_clk);
>> +            clk_disable(ctx->wb_clk);
>> +            /* ToDo : wb_b_clk */
>> +            ctx->suspended = true;
>> +    }
>> +
>> +    return 0;
>> +}
>> +
>> +static irqreturn_t fimc_irq_handler(int irq, void *dev_id)
>> +{
>> +    struct fimc_context *ctx = dev_id;
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    struct drm_exynos_ipp_cmd_node *c_node = ippdrv->cmd;
>> +    struct drm_exynos_ipp_event_work *event_work =
>> +            c_node->event_work;
>> +    int buf_id;
>> +
>> +    DRM_DEBUG_KMS("%s:fimc id[%d]\n", __func__, ctx->id);
>> +
>> +    fimc_clear_irq(ctx);
>> +    if (fimc_check_ovf(ctx))
>> +            return IRQ_NONE;
>> +
>> +    if (!fimc_check_frame_end(ctx))
>> +            return IRQ_NONE;
>> +
>> +    buf_id = fimc_get_buf_id(ctx);
>> +    if (buf_id < 0)
>> +            return IRQ_HANDLED;
>> +
>> +    DRM_DEBUG_KMS("%s:buf_id[%d]\n", __func__, buf_id);
>> +
>> +    if (fimc_dst_set_buf_seq(ctx, buf_id, IPP_BUF_DEQUEUE) < 0) {
>> +            DRM_ERROR("failed to dequeue.\n");
>> +            return IRQ_HANDLED;
>> +    }
>> +
>> +    event_work->ippdrv = ippdrv;
>> +    event_work->buf_id[EXYNOS_DRM_OPS_DST] = buf_id;
>> +    queue_work(ippdrv->event_workq, (struct work_struct *)event_work);
>> +
>> +    return IRQ_HANDLED;
>> +}
>> +
>> +static int fimc_init_prop_list(struct drm_exynos_ipp_prop_list
>> **prop_list)
>> +{
>> +    DRM_DEBUG_KMS("%s\n", __func__);
>> +
>> +    if (!prop_list) {
>> +            DRM_ERROR("empty prop_list\n");
>> +            return -EINVAL;
>> +    }
>> +
>> +    *prop_list = kzalloc(sizeof(**prop_list), GFP_KERNEL);
>> +    if (!*prop_list) {
>> +            DRM_ERROR("failed to alloc property list.\n");
>> +            return -ENOMEM;
>> +    }
>> +    /*ToDo : fix supported function list*/
>> +    (*prop_list)->version = 1;
>> +    (*prop_list)->writeback = 1;
>> +    (*prop_list)->refresh_min = 12;
>> +    (*prop_list)->refresh_max = 60;
>
> Use macro instead.

- done.

>
>> +    (*prop_list)->flip = (1 << EXYNOS_DRM_FLIP_NONE) |
>> +                            (1 << EXYNOS_DRM_FLIP_VERTICAL) |
>> +                            (1 << EXYNOS_DRM_FLIP_HORIZONTAL);
>> +    (*prop_list)->degree = (1 << EXYNOS_DRM_DEGREE_0) |
>> +                            (1 << EXYNOS_DRM_DEGREE_90) |
>> +                            (1 << EXYNOS_DRM_DEGREE_180) |
>> +                            (1 << EXYNOS_DRM_DEGREE_270);
>> +    (*prop_list)->csc = 1;
>> +    (*prop_list)->crop = 1;
>> +    (*prop_list)->crop_max.hsize = 8192;
>> +    (*prop_list)->crop_max.vsize = 8192;
>> +    (*prop_list)->crop_min.hsize = 32;
>> +    (*prop_list)->crop_min.vsize = 32;
>
> Ditto.

- done.

>
>> +    (*prop_list)->scale = 1;
>> +    (*prop_list)->scale_max.hsize = 4224;
>> +    (*prop_list)->scale_max.vsize = 4224;
>> +    (*prop_list)->scale_min.hsize = 32;
>> +    (*prop_list)->scale_min.vsize = 32;
>
> Ditto.

- done.

>
>> +
>> +    return 0;
>> +}
>> +
>> +static int fimc_ippdrv_check_property(struct device *dev,
>> +                            struct drm_exynos_ipp_property *property)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    struct drm_exynos_ipp_prop_list *pp = ippdrv->prop_list;
>> +    struct drm_exynos_ipp_config *config;
>> +    struct drm_exynos_pos *pos;
>> +    struct drm_exynos_sz *sz;
>> +    bool swap;
>> +    int i;
>> +
>> +    DRM_DEBUG_KMS("%s\n", __func__);
>> +
>> +    for_each_ipp_ops(i) {
>> +            if ((i == EXYNOS_DRM_OPS_SRC) &&
>> +                    (property->cmd == IPP_CMD_WB))
>> +                    continue;
>> +
>> +            config = &property->config[i];
>> +            pos = &config->pos;
>> +            sz = &config->sz;
>> +
>> +            /* check for flip */
>> +            switch (config->flip) {
>> +            case EXYNOS_DRM_FLIP_NONE:
>> +            case EXYNOS_DRM_FLIP_VERTICAL:
>> +            case EXYNOS_DRM_FLIP_HORIZONTAL:
>> +                    /* No problem */
>> +                    break;
>> +            default:
>> +                    DRM_ERROR("invalid flip.\n");
>> +                    goto err_property;
>> +            }
>
> Use if-else instead of switch-case. That could be checked using if-else
> simply.

- I prefer switch case in enumeration case.
   If we add another flip operation at the future,
   then we make some confusion about if-else.

>
>> +
>> +            /* check for degree */
>> +            switch (config->degree) {
>> +            case EXYNOS_DRM_DEGREE_90:
>> +            case EXYNOS_DRM_DEGREE_270:
>> +                    swap = true;
>> +                    break;
>> +            case EXYNOS_DRM_DEGREE_0:
>> +            case EXYNOS_DRM_DEGREE_180:
>> +                    swap = false;
>> +                    break;
>> +            default:
>> +                    DRM_ERROR("invalid degree.\n");
>> +                    goto err_property;
>> +            }
>> +
>> +            /* check for buffer bound */
>> +            if ((pos->x + pos->w > sz->hsize) ||
>> +                    (pos->y + pos->h > sz->vsize)) {
>> +                    DRM_ERROR("out of buf bound.\n");
>> +                    goto err_property;
>> +            }
>> +
>> +            /* check for crop */
>> +            if ((i == EXYNOS_DRM_OPS_SRC) && (pp->crop)) {
>> +                    if (swap) {
>> +                            if ((pos->h < pp->crop_min.hsize) ||
>> +                                    (sz->vsize > pp->crop_max.hsize) ||
>> +                                    (pos->w < pp->crop_min.vsize) ||
>> +                                    (sz->hsize > pp->crop_max.vsize)) {
>> +                                    DRM_ERROR("out of crop size.\n");
>> +                                    goto err_property;
>> +                            }
>> +                    } else {
>> +                            if ((pos->w < pp->crop_min.hsize) ||
>> +                                    (sz->hsize > pp->crop_max.hsize) ||
>> +                                    (pos->h < pp->crop_min.vsize) ||
>> +                                    (sz->vsize > pp->crop_max.vsize)) {
>> +                                    DRM_ERROR("out of crop size.\n");
>> +                                    goto err_property;
>> +                            }
>> +                    }
>> +            }
>> +
>> +            /* check for scale */
>> +            if ((i == EXYNOS_DRM_OPS_DST) && (pp->scale)) {
>> +                    if (swap) {
>> +                            if ((pos->h < pp->scale_min.hsize) ||
>> +                                    (sz->vsize > pp->scale_max.hsize) ||
>> +                                    (pos->w < pp->scale_min.vsize) ||
>> +                                    (sz->hsize > pp->scale_max.vsize)) {
>> +                                    DRM_ERROR("out of scale size.\n");
>> +                                    goto err_property;
>> +                            }
>> +                    } else {
>> +                            if ((pos->w < pp->scale_min.hsize) ||
>> +                                    (sz->hsize > pp->scale_max.hsize) ||
>> +                                    (pos->h < pp->scale_min.vsize) ||
>> +                                    (sz->vsize > pp->scale_max.vsize)) {
>> +                                    DRM_ERROR("out of scale size.\n");
>> +                                    goto err_property;
>> +                            }
>> +                    }
>> +            }
>> +    }
>> +
>> +    return 0;
>> +
>> +err_property:
>> +    for_each_ipp_ops(i) {
>> +            if ((i == EXYNOS_DRM_OPS_SRC) &&
>> +                    (property->cmd == IPP_CMD_WB))
>> +                    continue;
>> +
>> +            config = &property->config[i];
>> +            pos = &config->pos;
>> +            sz = &config->sz;
>> +
>> +            DRM_ERROR("[%s]f[%d]r[%d]pos[%d %d %d %d]sz[%d %d]\n",
>> +                    i ? "dst" : "src", config->flip, config->degree,
>> +                    pos->x, pos->y, pos->w, pos->h,
>> +                    sz->hsize, sz->vsize);
>> +    }
>> +
>> +    return -EINVAL;
>> +}
>> +
>> +static int fimc_ippdrv_reset(struct device *dev)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +
>> +    DRM_DEBUG_KMS("%s\n", __func__);
>> +
>> +    /* reset h/w block */
>> +    fimc_sw_reset(ctx, false);
>> +
>> +    /* reset scaler capability */
>> +    memset(&ctx->sc, 0x0, sizeof(ctx->sc));
>> +
>> +    return 0;
>> +}
>> +
>> +static int fimc_check_prepare(struct fimc_context *ctx)
>> +{
>> +    /* ToDo: check prepare using read register */
>> +    DRM_DEBUG_KMS("%s\n", __func__);
>> +
>> +    return 0;
>> +}
>> +
>> +static int fimc_ippdrv_start(struct device *dev, enum drm_exynos_ipp_cmd
>> cmd)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +    struct drm_exynos_ipp_cmd_node *c_node = ippdrv->cmd;
>> +    struct drm_exynos_ipp_property *property;
>> +    struct drm_exynos_ipp_config *config;
>> +    struct drm_exynos_pos   img_pos[EXYNOS_DRM_OPS_MAX];
>> +    struct drm_exynos_ipp_set_wb set_wb;
>> +    int ret, i;
>> +    u32 cfg0, cfg1;
>> +
>> +    DRM_DEBUG_KMS("%s:cmd[%d]\n", __func__, cmd);
>> +
>> +    if (!c_node) {
>> +            DRM_ERROR("failed to get c_node.\n");
>> +            return -EINVAL;
>> +    }
>> +
>> +    property = &c_node->property;
>> +    if (!property) {
>> +            DRM_ERROR("failed to get property.\n");
>> +            return -EINVAL;
>> +    }
>> +
>> +    ret = fimc_check_prepare(ctx);
>> +    if (ret) {
>> +            dev_err(dev, "failed to check prepare.\n");
>> +            return ret;
>> +    }
>> +
>> +    fimc_handle_irq(ctx, true, false, true);
>> +
>> +    /* ToDo: window size, prescaler config */
>> +    for_each_ipp_ops(i) {
>> +            config = &property->config[i];
>> +            img_pos[i] = config->pos;
>> +    }
>> +
>> +    ret = fimc_set_prescaler(ctx, &ctx->sc,
>> +            &img_pos[EXYNOS_DRM_OPS_SRC],
>> +            &img_pos[EXYNOS_DRM_OPS_DST]);
>> +    if (ret) {
>> +            dev_err(dev, "failed to set precalser.\n");
>> +            return ret;
>> +    }
>> +
>> +    /* If set ture, we can save jpeg about screen */
>> +    fimc_handle_jpeg(ctx, false);
>> +    fimc_set_scaler(ctx, &ctx->sc);
>> +    fimc_set_polarity(ctx, &ctx->pol);
>> +
>> +    switch (cmd) {
>> +    case IPP_CMD_M2M:
>> +            fimc_set_type_ctrl(ctx, FIMC_WB_NONE);
>> +            fimc_handle_lastend(ctx, false);
>> +
>> +            /* setup dma */
>> +            cfg0 = fimc_read(EXYNOS_MSCTRL);
>> +            cfg0 &= ~EXYNOS_MSCTRL_INPUT_MASK;
>> +            cfg0 |= EXYNOS_MSCTRL_INPUT_MEMORY;
>> +            fimc_write(cfg0, EXYNOS_MSCTRL);
>> +            break;
>> +    case IPP_CMD_WB:
>> +            fimc_set_type_ctrl(ctx, FIMC_WB_A);
>> +            fimc_handle_lastend(ctx, true);
>> +
>> +            /* setup FIMD */
>> +            fimc_set_camblk_fimd0_wb(ctx);
>> +
>> +            /* ToDo: need to replace the property structure. */
>> +            set_wb.enable = 1;
>> +            set_wb.refresh = property->reserved;
>> +            exynos_drm_ippnb_send_event(IPP_SET_WRITEBACK, (void
>> *)&set_wb);
>> +            break;
>> +    case IPP_CMD_OUTPUT:
>> +    default:
>> +            ret = -EINVAL;
>> +            dev_err(dev, "invalid operations.\n");
>> +            return ret;
>> +    }
>> +
>> +    /* Reset status */
>> +    fimc_write(0x0, EXYNOS_CISTATUS);
>> +
>> +    cfg0 = fimc_read(EXYNOS_CIIMGCPT);
>> +    cfg0 &= ~EXYNOS_CIIMGCPT_IMGCPTEN_SC;
>> +    cfg0 |= EXYNOS_CIIMGCPT_IMGCPTEN_SC;
>> +
>> +    /* Scaler */
>> +    cfg1 = fimc_read(EXYNOS_CISCCTRL);
>> +    cfg1 &= ~EXYNOS_CISCCTRL_SCAN_MASK;
>> +    cfg1 |= (EXYNOS_CISCCTRL_PROGRESSIVE |
>> +            EXYNOS_CISCCTRL_SCALERSTART);
>> +
>> +    fimc_write(cfg1, EXYNOS_CISCCTRL);
>> +
>> +    /* Enable image capture*/
>> +    cfg0 |= EXYNOS_CIIMGCPT_IMGCPTEN;
>> +    fimc_write(cfg0, EXYNOS_CIIMGCPT);
>> +
>> +    /* Disable frame end irq */
>> +    cfg0 = fimc_read(EXYNOS_CIGCTRL);
>> +    cfg0 &= ~EXYNOS_CIGCTRL_IRQ_END_DISABLE;
>> +    fimc_write(cfg0, EXYNOS_CIGCTRL);
>> +
>> +    cfg0 = fimc_read(EXYNOS_CIOCTRL);
>> +    cfg0 &= ~EXYNOS_CIOCTRL_WEAVE_MASK;
>> +    fimc_write(cfg0, EXYNOS_CIOCTRL);
>> +
>> +    /* ToDo: m2m start errata - refer fimd */
>> +    if (cmd == IPP_CMD_M2M) {
>> +            cfg0 = fimc_read(EXYNOS_MSCTRL);
>> +            cfg0 |= EXYNOS_MSCTRL_ENVID;
>> +            fimc_write(cfg0, EXYNOS_MSCTRL);
>> +
>> +            cfg0 = fimc_read(EXYNOS_MSCTRL);
>> +            cfg0 |= EXYNOS_MSCTRL_ENVID;
>> +            fimc_write(cfg0, EXYNOS_MSCTRL);
>> +    }
>> +
>> +    return 0;
>> +}
>> +
>> +static void fimc_ippdrv_stop(struct device *dev, enum drm_exynos_ipp_cmd
>> cmd)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct drm_exynos_ipp_set_wb set_wb = {0, 0};
>> +    u32 cfg;
>> +
>> +    DRM_DEBUG_KMS("%s:cmd[%d]\n", __func__, cmd);
>> +
>> +    switch (cmd) {
>> +    case IPP_CMD_M2M:
>> +            /* Source clear */
>> +            cfg = fimc_read(EXYNOS_MSCTRL);
>> +            cfg &= ~EXYNOS_MSCTRL_INPUT_MASK;
>> +            cfg &= ~EXYNOS_MSCTRL_ENVID;
>> +            fimc_write(cfg, EXYNOS_MSCTRL);
>> +            break;
>> +    case IPP_CMD_WB:
>> +            exynos_drm_ippnb_send_event(IPP_SET_WRITEBACK, (void
>> *)&set_wb);
>> +            break;
>> +    case IPP_CMD_OUTPUT:
>> +    default:
>> +            dev_err(dev, "invalid operations.\n");
>> +            break;
>> +    }
>> +
>> +    fimc_handle_irq(ctx, false, false, true);
>> +
>> +    /* reset sequence */
>> +    fimc_write(0x0, EXYNOS_CIFCNTSEQ);
>> +
>> +    /* Scaler disable */
>> +    cfg = fimc_read(EXYNOS_CISCCTRL);
>> +    cfg &= ~EXYNOS_CISCCTRL_SCALERSTART;
>> +    fimc_write(cfg, EXYNOS_CISCCTRL);
>> +
>> +    /* Disable image capture */
>> +    cfg = fimc_read(EXYNOS_CIIMGCPT);
>> +    cfg &= ~(EXYNOS_CIIMGCPT_IMGCPTEN_SC | EXYNOS_CIIMGCPT_IMGCPTEN);
>> +    fimc_write(cfg, EXYNOS_CIIMGCPT);
>> +
>> +    /* Enable frame end irq */
>> +    cfg = fimc_read(EXYNOS_CIGCTRL);
>> +    cfg |= EXYNOS_CIGCTRL_IRQ_END_DISABLE;
>> +    fimc_write(cfg, EXYNOS_CIGCTRL);
>> +}
>> +
>> +static struct fimc_capability *fimc_get_capability(
>> +    enum exynos_drm_fimc_ver ver)
>> +{
>> +    struct fimc_capability *capa;
>> +
>> +    DRM_DEBUG_KMS("%s:ver[0x%x]\n", __func__, ver);
>> +
>> +    /* ToDo: version check */
>> +    switch (ver) {
>> +    case FIMC_EXYNOS_4412:
>> +    case FIMC_EXYNOS_4412_R2:
>> +    default:
>> +            capa = fimc51_capa;
>> +            break;
>> +    }
>> +
>> +    return capa;
>> +}
>> +
>> +static int fimc_get_clk_rate(enum exynos_drm_fimc_ver ver)
>> +{
>> +    int clk_rate;
>> +
>> +    DRM_DEBUG_KMS("%s:ver[0x%x]\n", __func__, ver);
>> +
>> +    switch (ver) {
>> +    case FIMC_EXYNOS_4412_R2:
>> +            clk_rate = FIMC_CLK_RATE_R2;
>> +            break;
>> +    case FIMC_EXYNOS_4412:
>> +    default:
>> +            clk_rate = FIMC_CLK_RATE;
>> +            break;
>> +    }
>> +
>> +    return clk_rate;
>> +}
>> +
>> +static int __devinit fimc_probe(struct platform_device *pdev)
>> +{
>> +    struct device *dev = &pdev->dev;
>> +    struct fimc_context *ctx;
>> +    struct clk      *parent_clk;
>> +    struct resource *res;
>> +    struct exynos_drm_ippdrv *ippdrv;
>> +    struct exynos_drm_fimc_pdata *pdata;
>> +    int ret = -EINVAL;
>
> Just use int ret;

- done.

>
>> +
>> +    pdata = pdev->dev.platform_data;
>> +    if (!pdata) {
>> +            dev_err(dev, "no platform data specified.\n");
>> +            return -EINVAL;
>> +    }
>> +
>> +    ctx = kzalloc(sizeof(*ctx), GFP_KERNEL);
>> +    if (!ctx)
>> +            return -ENOMEM;
>> +
>> +    /* clock control */
>> +    ctx->sclk_fimc_clk = clk_get(dev, "sclk_fimc");
>> +    if (IS_ERR(ctx->sclk_fimc_clk)) {
>> +            dev_err(dev, "failed to get src fimc clock.\n");
>> +            ret = PTR_ERR(ctx->sclk_fimc_clk);
>> +            goto err_ctx;
>> +    }
>> +    clk_enable(ctx->sclk_fimc_clk);
>> +
>> +    ctx->fimc_clk = clk_get(dev, "fimc");
>> +    if (IS_ERR(ctx->fimc_clk)) {
>> +            dev_err(dev, "failed to get fimc clock.\n");
>> +            ret = PTR_ERR(ctx->fimc_clk);
>
> clk_disable(ctx->sclk_fimc_clk);

- done.

>
>> +            clk_put(ctx->sclk_fimc_clk);
>> +            goto err_ctx;
>> +    }
>> +
>> +    ctx->wb_clk = clk_get(dev, "pxl_async0");
>> +    if (IS_ERR(ctx->wb_clk)) {
>> +            dev_err(dev, "failed to get writeback a clock.\n");
>> +            ret = PTR_ERR(ctx->wb_clk);
>
> clk_disable(ctx->sclk_fimc_clk);

- done.

>
>> +            clk_put(ctx->sclk_fimc_clk);
>> +            clk_put(ctx->fimc_clk);
>> +            goto err_ctx;
>> +    }
>> +
>> +    ctx->wb_b_clk = clk_get(dev, "pxl_async1");
>> +    if (IS_ERR(ctx->wb_b_clk)) {
>> +            dev_err(dev, "failed to get writeback b clock.\n");
>
> clk_disable(ctx->sclk_fimc_clk);

- done.

>
>> +            ret = PTR_ERR(ctx->wb_b_clk);
>
>> +            clk_put(ctx->sclk_fimc_clk);
>> +            clk_put(ctx->fimc_clk);
>> +            clk_put(ctx->wb_clk);
>> +            goto err_ctx;
>> +    }
>> +
>> +    if (pdata->ver == FIMC_EXYNOS_4210)
>> +            parent_clk = clk_get(dev, "mout_mpll");
>> +    else
>> +            parent_clk = clk_get(dev, "mout_mpll_user");
>> +
>> +    if (IS_ERR(parent_clk)) {
>> +            dev_err(dev, "failed to get parent clock.\n");
>> +            ret = PTR_ERR(parent_clk);
>
> clk_disable(ctx->sclk_fimc_clk);

- done.

>
>> +            clk_put(ctx->sclk_fimc_clk);
>> +            clk_put(ctx->fimc_clk);
>> +            clk_put(ctx->wb_clk);
>> +            clk_put(ctx->wb_b_clk);
>> +            goto err_ctx;
>> +    }
>> +
>> +    if (clk_set_parent(ctx->sclk_fimc_clk, parent_clk)) {
>> +            dev_err(dev, "failed to set parent.\n");
>
> clk_disable(ctx->sclk_fimc_clk);

- done.

>
>> +            clk_put(parent_clk);
>> +            clk_put(ctx->sclk_fimc_clk);
>> +            clk_put(ctx->fimc_clk);
>> +            clk_put(ctx->wb_clk);
>> +            clk_put(ctx->wb_b_clk);
>> +            goto err_ctx;
>> +    }
>> +    clk_put(parent_clk);
>> +    clk_set_rate(ctx->sclk_fimc_clk, fimc_get_clk_rate(pdata->ver));
>> +    if (pdata->ver == FIMC_EXYNOS_4210)
>> +            clk_put(ctx->sclk_fimc_clk);
>> +    else
>> +            clk_disable(ctx->sclk_fimc_clk);
>> +
>> +    /* resource memory */
>> +    res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
>> +    if (!res) {
>> +            dev_err(dev, "failed to find registers.\n");
>> +            ret = -ENOENT;
>> +            goto err_clk;
>> +    }
>> +
>> +    ctx->regs_res = request_mem_region(res->start, resource_size(res),
>> +                                       dev_name(dev));
>> +    if (!ctx->regs_res) {
>> +            dev_err(dev, "failed to claim register region.\n");
>> +            ret = -ENOENT;
>> +            goto err_clk;
>> +    }
>> +
>> +    ctx->regs = ioremap(res->start, resource_size(res));
>> +    if (!ctx->regs) {
>> +            dev_err(dev, "failed to map registers.\n");
>> +            ret = -ENXIO;
>> +            goto err_req_region;
>> +    }
>> +
>> +    /* resource irq */
>> +    res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
>> +    if (!res) {
>> +            dev_err(dev, "failed to request irq resource.\n");
>> +            goto err_get_regs;
>> +    }
>> +
>> +    ctx->irq = res->start;
>> +    ret = request_threaded_irq(ctx->irq, NULL, fimc_irq_handler,
>> +            IRQF_ONESHOT, "drm_fimc", ctx);
>> +    if (ret < 0) {
>> +            dev_err(dev, "failed to request irq.\n");
>> +            goto err_get_regs;
>> +    }
>> +
>> +    /* context initailization */
>> +    ctx->ver = pdata->ver;
>> +    ctx->id = pdev->id;
>> +    ctx->capa = fimc_get_capability(ctx->ver);
>> +    if (!ctx->capa) {
>> +            dev_err(dev, "failed to get capability.\n");
>> +            goto err_get_irq;
>> +    }
>> +    ctx->pol = pdata->pol;
>> +
>> +    ippdrv = &ctx->ippdrv;
>> +    ippdrv->dev = dev;
>> +    ippdrv->ops[EXYNOS_DRM_OPS_SRC] = &fimc_src_ops;
>> +    ippdrv->ops[EXYNOS_DRM_OPS_DST] = &fimc_dst_ops;
>> +    ippdrv->check_property = fimc_ippdrv_check_property;
>> +    ippdrv->reset = fimc_ippdrv_reset;
>> +    ippdrv->start = fimc_ippdrv_start;
>> +    ippdrv->stop = fimc_ippdrv_stop;
>> +    ret = fimc_init_prop_list(&ippdrv->prop_list);
>> +    if (ret < 0) {
>> +            dev_err(dev, "failed to init property list.\n");
>> +            goto err_get_irq;
>> +    }
>> +
>> +    DRM_DEBUG_KMS("%s:id[%d]ippdrv[0x%x]\n", __func__, ctx->id,
>> +            (int)ippdrv);
>> +
>> +    spin_lock_init(&ctx->lock);
>> +    platform_set_drvdata(pdev, ctx);
>> +
>> +    pm_runtime_set_active(dev);
>> +    pm_runtime_enable(dev);
>> +
>> +    ret = exynos_drm_ippdrv_register(ippdrv);
>> +    if (ret < 0) {
>> +            dev_err(dev, "failed to register drm fimc device.\n");
>> +            goto err_ippdrv_register;
>> +    }
>> +
>> +    dev_info(&pdev->dev, "drm fimc registered successfully.\n");
>> +
>> +    return 0;
>> +
>> +err_ippdrv_register:
>> +    kfree(ippdrv->prop_list);
>> +    pm_runtime_disable(dev);
>> +    free_irq(ctx->irq, ctx);
>> +err_get_irq:
>> +    free_irq(ctx->irq, ctx);
>> +err_get_regs:
>> +    iounmap(ctx->regs);
>> +err_req_region:
>> +    release_resource(ctx->regs_res);
>> +    kfree(ctx->regs_res);
>> +err_clk:
>> +    clk_put(ctx->sclk_fimc_clk);
>> +    clk_put(ctx->fimc_clk);
>> +    clk_put(ctx->wb_clk);
>> +    clk_put(ctx->wb_b_clk);
>> +err_ctx:
>> +    kfree(ctx);
>> +    return ret;
>> +}
>> +
>> +static int __devexit fimc_remove(struct platform_device *pdev)
>> +{
>> +    struct device *dev = &pdev->dev;
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +    struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
>> +
>> +    kfree(ippdrv->prop_list);
>> +    exynos_drm_ippdrv_unregister(ippdrv);
>> +
>> +    pm_runtime_set_suspended(dev);
>> +    pm_runtime_disable(dev);
>> +
>> +    free_irq(ctx->irq, ctx);
>> +    iounmap(ctx->regs);
>> +    release_resource(ctx->regs_res);
>> +    kfree(ctx->regs_res);
>> +
>> +    clk_put(ctx->sclk_fimc_clk);
>> +    clk_put(ctx->fimc_clk);
>> +    clk_put(ctx->wb_clk);
>> +    clk_put(ctx->wb_b_clk);
>> +
>> +    kfree(ctx);
>> +
>> +    return 0;
>> +}
>> +
>> +#ifdef CONFIG_PM_SLEEP
>> +static int fimc_suspend(struct device *dev)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +
>> +    DRM_DEBUG_KMS("%s:id[%d]\n", __func__, ctx->id);
>> +    if (pm_runtime_suspended(dev))
>> +            return 0;
>> +    /* ToDo */
>> +    return fimc_power_on(ctx, false);
>> +}
>> +
>> +static int fimc_resume(struct device *dev)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +
>> +    DRM_DEBUG_KMS("%s:id[%d]\n", __func__, ctx->id);
>> +    if (!pm_runtime_suspended(dev))
>> +            return fimc_power_on(ctx, true);
>> +    /* ToDo */
>> +    return 0;
>> +}
>> +#endif
>> +
>> +#ifdef CONFIG_PM_RUNTIME
>> +static int fimc_runtime_suspend(struct device *dev)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +
>> +    DRM_DEBUG_KMS("%s:id[%d]\n", __func__, ctx->id);
>> +    /* ToDo */
>> +    return  fimc_power_on(ctx, false);
>> +}
>> +
>> +static int fimc_runtime_resume(struct device *dev)
>> +{
>> +    struct fimc_context *ctx = get_fimc_context(dev);
>> +
>> +    DRM_DEBUG_KMS("%s:id[%d]\n", __func__, ctx->id);
>> +    /* ToDo */
>> +    return  fimc_power_on(ctx, true);
>> +}
>> +#endif
>> +
>> +static const struct dev_pm_ops fimc_pm_ops = {
>> +    SET_SYSTEM_SLEEP_PM_OPS(fimc_suspend, fimc_resume)
>> +    SET_RUNTIME_PM_OPS(fimc_runtime_suspend, fimc_runtime_resume, NULL)
>> +};
>> +
>> +/* ToDo: need to check use case platform_device_id */
>> +struct platform_driver fimc_driver = {
>> +    .probe          = fimc_probe,
>> +    .remove         = __devexit_p(fimc_remove),
>> +    .driver         = {
>> +            .name   = "exynos-drm-fimc",
>> +            .owner  = THIS_MODULE,
>> +            .pm     = &fimc_pm_ops,
>> +    },
>> +};
>> +
>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimc.h
>> b/drivers/gpu/drm/exynos/exynos_drm_fimc.h
>> new file mode 100644
>> index 0000000..b80e773
>> --- /dev/null
>> +++ b/drivers/gpu/drm/exynos/exynos_drm_fimc.h
>> @@ -0,0 +1,35 @@
>> +/*
>> + * Copyright (c) 2012 Samsung Electronics Co., Ltd.
>> + *
>> + * Authors:
>> + *  Eunchul Kim <chulspro.kim at samsung.com>
>> + *  Jinyoung Jeon <jy0.jeon at samsung.com>
>> + *  Sangmin Lee <lsmin.lee at samsung.com>
>> + *
>> + * Permission is hereby granted, free of charge, to any person obtaining
>> a
>> + * copy of this software and associated documentation files (the
>> "Software"),
>> + * to deal in the Software without restriction, including without
>> limitation
>> + * the rights to use, copy, modify, merge, publish, distribute,
>> sublicense,
>> + * and/or sell copies of the Software, and to permit persons to whom the
>> + * Software is furnished to do so, subject to the following conditions:
>> + *
>> + * The above copyright notice and this permission notice (including the
>> next
>> + * paragraph) shall be included in all copies or substantial portions of
>> the
>> + * Software.
>> + *
>> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>> EXPRESS OR
>> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>> MERCHANTABILITY,
>> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
>> SHALL
>> + * VA LINUX SYSTEMS AND/OR ITS SUPPLIERS BE LIABLE FOR ANY CLAIM, DAMAGES
>> OR
>> + * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>> + * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
>> + * OTHER DEALINGS IN THE SOFTWARE.
>> + */
>> +
>> +#ifndef _EXYNOS_DRM_FIMC_H_
>> +#define _EXYNOS_DRM_FIMC_H_
>> +
>> +/* ToDo */
>> +
>> +#endif /* _EXYNOS_DRM_FIMC_H_ */
>> +
>> diff --git a/drivers/gpu/drm/exynos/regs-fimc.h
>> b/drivers/gpu/drm/exynos/regs-fimc.h
>> new file mode 100644
>> index 0000000..b4f9ca1
>> --- /dev/null
>> +++ b/drivers/gpu/drm/exynos/regs-fimc.h
>> @@ -0,0 +1,669 @@
>> +/* drivers/gpu/drm/exynos/regs-fimc.h
>> + *
>> + * Copyright (c) 2012 Samsung Electronics Co., Ltd.
>> + *          http://www.samsung.com/
>> + *
>> + * Register definition file for Samsung Camera Interface (FIMC) driver
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License version 2 as
>> + * published by the Free Software Foundation.
>> +*/
>> +
>> +#ifndef EXYNOS_REGS_FIMC_H
>> +#define EXYNOS_REGS_FIMC_H
>> +
>> +/*
>> + * Register part
>> +*/
>> +/* Input source format */
>> +#define EXYNOS_CISRCFMT             (0x00)
>> +/* Window offset */
>> +#define EXYNOS_CIWDOFST             (0x04)
>> +/* Global control */
>> +#define EXYNOS_CIGCTRL              (0x08)
>> +/* Window offset 2 */
>> +#define EXYNOS_CIWDOFST2    (0x14)
>> +/* Y 1st frame start address for output DMA */
>> +#define EXYNOS_CIOYSA1              (0x18)
>> +/* Y 2nd frame start address for output DMA */
>> +#define EXYNOS_CIOYSA2              (0x1c)
>> +/* Y 3rd frame start address for output DMA */
>> +#define EXYNOS_CIOYSA3              (0x20)
>> +/* Y 4th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA4              (0x24)
>> +/* Cb 1st frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA1             (0x28)
>> +/* Cb 2nd frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA2             (0x2c)
>> +/* Cb 3rd frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA3             (0x30)
>> +/* Cb 4th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA4             (0x34)
>> +/* Cr 1st frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA1             (0x38)
>> +/* Cr 2nd frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA2             (0x3c)
>> +/* Cr 3rd frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA3             (0x40)
>> +/* Cr 4th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA4             (0x44)
>> +/* Target image format */
>> +#define EXYNOS_CITRGFMT             (0x48)
>> +/* Output DMA control */
>> +#define EXYNOS_CIOCTRL              (0x4c)
>> +/* Pre-scaler control 1 */
>> +#define EXYNOS_CISCPRERATIO (0x50)
>> +/* Pre-scaler control 2 */
>> +#define EXYNOS_CISCPREDST           (0x54)
>> +/* Main scaler control */
>> +#define EXYNOS_CISCCTRL             (0x58)
>> +/* Target area */
>> +#define EXYNOS_CITAREA              (0x5c)
>> +/* Status */
>> +#define EXYNOS_CISTATUS             (0x64)
>> +/* Status2 */
>> +#define EXYNOS_CISTATUS2            (0x68)
>> +/* Image capture enable command */
>> +#define EXYNOS_CIIMGCPT             (0xc0)
>> +/* Capture sequence */
>> +#define EXYNOS_CICPTSEQ             (0xc4)
>> +/* Image effects */
>> +#define EXYNOS_CIIMGEFF             (0xd0)
>> +/* Y frame start address for input DMA */
>> +#define EXYNOS_CIIYSA0              (0xd4)
>> +/* Cb frame start address for input DMA */
>> +#define EXYNOS_CIICBSA0             (0xd8)
>> +/* Cr frame start address for input DMA */
>> +#define EXYNOS_CIICRSA0             (0xdc)
>> +/* Input DMA Y Line Skip */
>> +#define EXYNOS_CIILINESKIP_Y        (0xec)
>> +/* Input DMA Cb Line Skip */
>> +#define EXYNOS_CIILINESKIP_CB       (0xf0)
>> +/* Input DMA Cr Line Skip */
>> +#define EXYNOS_CIILINESKIP_CR       (0xf4)
>> +/* Real input DMA image size */
>> +#define EXYNOS_CIREAL_ISIZE (0xf8)
>> +/* Input DMA control */
>> +#define EXYNOS_MSCTRL               (0xfc)
>> +/* Y frame start address for input DMA */
>> +#define EXYNOS_CIIYSA1              (0x144)
>> +/* Cb frame start address for input DMA */
>> +#define EXYNOS_CIICBSA1             (0x148)
>> +/* Cr frame start address for input DMA */
>> +#define EXYNOS_CIICRSA1             (0x14c)
>> +/* Output DMA Y offset */
>> +#define EXYNOS_CIOYOFF              (0x168)
>> +/* Output DMA CB offset */
>> +#define EXYNOS_CIOCBOFF             (0x16c)
>> +/* Output DMA CR offset */
>> +#define EXYNOS_CIOCROFF             (0x170)
>> +/* Input DMA Y offset */
>> +#define EXYNOS_CIIYOFF              (0x174)
>> +/* Input DMA CB offset */
>> +#define EXYNOS_CIICBOFF             (0x178)
>> +/* Input DMA CR offset */
>> +#define EXYNOS_CIICROFF             (0x17c)
>> +/* Input DMA original image size */
>> +#define EXYNOS_ORGISIZE             (0x180)
>> +/* Output DMA original image size */
>> +#define EXYNOS_ORGOSIZE             (0x184)
>> +/* Real output DMA image size */
>> +#define EXYNOS_CIEXTEN              (0x188)
>> +/* DMA parameter */
>> +#define EXYNOS_CIDMAPARAM           (0x18c)
>> +/* MIPI CSI image format */
>> +#define EXYNOS_CSIIMGFMT            (0x194)
>> +/* FIMC Clock Source Select */
>> +#define EXYNOS_MISC_FIMC            (0x198)
>> +
>> +/* Add for FIMC v5.1 */
>> +/* Output Frame Buffer Sequence */
>> +#define EXYNOS_CIFCNTSEQ            (0x1fc)
>> +/* Y 5th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA5              (0x200)
>> +/* Y 6th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA6              (0x204)
>> +/* Y 7th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA7              (0x208)
>> +/* Y 8th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA8              (0x20c)
>> +/* Y 9th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA9              (0x210)
>> +/* Y 10th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA10             (0x214)
>> +/* Y 11th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA11             (0x218)
>> +/* Y 12th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA12             (0x21c)
>> +/* Y 13th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA13             (0x220)
>> +/* Y 14th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA14             (0x224)
>> +/* Y 15th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA15             (0x228)
>> +/* Y 16th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA16             (0x22c)
>> +/* Y 17th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA17             (0x230)
>> +/* Y 18th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA18             (0x234)
>> +/* Y 19th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA19             (0x238)
>> +/* Y 20th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA20             (0x23c)
>> +/* Y 21th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA21             (0x240)
>> +/* Y 22th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA22             (0x244)
>> +/* Y 23th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA23             (0x248)
>> +/* Y 24th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA24             (0x24c)
>> +/* Y 25th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA25             (0x250)
>> +/* Y 26th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA26             (0x254)
>> +/* Y 27th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA27             (0x258)
>> +/* Y 28th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA28             (0x25c)
>> +/* Y 29th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA29             (0x260)
>> +/* Y 30th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA30             (0x264)
>> +/* Y 31th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA31             (0x268)
>> +/* Y 32th frame start address for output DMA */
>> +#define EXYNOS_CIOYSA32             (0x26c)
>> +
>> +/* CB 5th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA5             (0x270)
>> +/* CB 6th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA6             (0x274)
>> +/* CB 7th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA7             (0x278)
>> +/* CB 8th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA8             (0x27c)
>> +/* CB 9th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA9             (0x280)
>> +/* CB 10th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA10            (0x284)
>> +/* CB 11th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA11            (0x288)
>> +/* CB 12th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA12            (0x28c)
>> +/* CB 13th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA13            (0x290)
>> +/* CB 14th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA14            (0x294)
>> +/* CB 15th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA15            (0x298)
>> +/* CB 16th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA16            (0x29c)
>> +/* CB 17th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA17            (0x2a0)
>> +/* CB 18th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA18            (0x2a4)
>> +/* CB 19th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA19            (0x2a8)
>> +/* CB 20th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA20            (0x2ac)
>> +/* CB 21th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA21            (0x2b0)
>> +/* CB 22th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA22            (0x2b4)
>> +/* CB 23th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA23            (0x2b8)
>> +/* CB 24th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA24            (0x2bc)
>> +/* CB 25th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA25            (0x2c0)
>> +/* CB 26th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA26            (0x2c4)
>> +/* CB 27th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA27            (0x2c8)
>> +/* CB 28th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA28            (0x2cc)
>> +/* CB 29th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA29            (0x2d0)
>> +/* CB 30th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA30            (0x2d4)
>> +/* CB 31th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA31            (0x2d8)
>> +/* CB 32th frame start address for output DMA */
>> +#define EXYNOS_CIOCBSA32            (0x2dc)
>> +
>> +/* CR 5th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA5             (0x2e0)
>> +/* CR 6th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA6             (0x2e4)
>> +/* CR 7th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA7             (0x2e8)
>> +/* CR 8th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA8             (0x2ec)
>> +/* CR 9th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA9             (0x2f0)
>> +/* CR 10th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA10            (0x2f4)
>> +/* CR 11th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA11            (0x2f8)
>> +/* CR 12th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA12            (0x2fc)
>> +/* CR 13th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA13            (0x300)
>> +/* CR 14th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA14            (0x304)
>> +/* CR 15th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA15            (0x308)
>> +/* CR 16th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA16            (0x30c)
>> +/* CR 17th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA17            (0x310)
>> +/* CR 18th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA18            (0x314)
>> +/* CR 19th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA19            (0x318)
>> +/* CR 20th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA20            (0x31c)
>> +/* CR 21th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA21            (0x320)
>> +/* CR 22th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA22            (0x324)
>> +/* CR 23th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA23            (0x328)
>> +/* CR 24th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA24            (0x32c)
>> +/* CR 25th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA25            (0x330)
>> +/* CR 26th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA26            (0x334)
>> +/* CR 27th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA27            (0x338)
>> +/* CR 28th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA28            (0x33c)
>> +/* CR 29th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA29            (0x340)
>> +/* CR 30th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA30            (0x344)
>> +/* CR 31th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA31            (0x348)
>> +/* CR 32th frame start address for output DMA */
>> +#define EXYNOS_CIOCRSA32            (0x34c)
>> +
>> +/*
>> + * Macro part
>> +*/
>> +/* frame start address 1 ~ 4, 5 ~ 32 */
>> +/* Number of Default PingPong Memory */
>> +#define DEF_PP              4
>> +#define EXYNOS_CIOYSA(__x)          \
>> +    (((__x) < DEF_PP) ?     \
>> +     (EXYNOS_CIOYSA1  + (__x) * 4) : \
>> +    (EXYNOS_CIOYSA5  + ((__x) - DEF_PP) * 4))
>> +#define EXYNOS_CIOCBSA(__x) \
>> +    (((__x) < DEF_PP) ?     \
>> +     (EXYNOS_CIOCBSA1 + (__x) * 4) : \
>> +    (EXYNOS_CIOCBSA5 + ((__x) - DEF_PP) * 4))
>> +#define EXYNOS_CIOCRSA(__x) \
>> +    (((__x) < DEF_PP) ?     \
>> +     (EXYNOS_CIOCRSA1 + (__x) * 4) : \
>> +    (EXYNOS_CIOCRSA5 + ((__x) - DEF_PP) * 4))
>> +/* Number of Default PingPong Memory */
>> +#define DEF_IPP             1
>> +#define EXYNOS_CIIYSA(__x)          \
>> +    (((__x) < DEF_IPP) ?    \
>> +     (EXYNOS_CIIYSA0) : (EXYNOS_CIIYSA1))
>> +#define EXYNOS_CIICBSA(__x) \
>> +    (((__x) < DEF_IPP) ?    \
>> +     (EXYNOS_CIICBSA0) : (EXYNOS_CIICBSA1))
>> +#define EXYNOS_CIICRSA(__x) \
>> +    (((__x) < DEF_IPP) ?    \
>> +     (EXYNOS_CIICRSA0) : (EXYNOS_CIICRSA1))
>> +
>> +#define EXYNOS_CISRCFMT_SOURCEHSIZE(x)              ((x) << 16)
>> +#define EXYNOS_CISRCFMT_SOURCEVSIZE(x)              ((x) << 0)
>> +
>> +#define EXYNOS_CIWDOFST_WINHOROFST(x)               ((x) << 16)
>> +#define EXYNOS_CIWDOFST_WINVEROFST(x)               ((x) << 0)
>> +
>> +#define EXYNOS_CIWDOFST2_WINHOROFST2(x)             ((x) << 16)
>> +#define EXYNOS_CIWDOFST2_WINVEROFST2(x)             ((x) << 0)
>> +
>> +#define EXYNOS_CITRGFMT_TARGETHSIZE(x)              (((x) & 0x1fff) <<
> 16)
>> +#define EXYNOS_CITRGFMT_TARGETVSIZE(x)              (((x) & 0x1fff) <<
> 0)
>> +
>> +#define EXYNOS_CISCPRERATIO_SHFACTOR(x)             ((x) << 28)
>> +#define EXYNOS_CISCPRERATIO_PREHORRATIO(x)          ((x) << 16)
>> +#define EXYNOS_CISCPRERATIO_PREVERRATIO(x)          ((x) << 0)
>> +
>> +#define EXYNOS_CISCPREDST_PREDSTWIDTH(x)            ((x) << 16)
>> +#define EXYNOS_CISCPREDST_PREDSTHEIGHT(x)           ((x) << 0)
>> +
>> +#define EXYNOS_CISCCTRL_MAINHORRATIO(x)             ((x) << 16)
>> +#define EXYNOS_CISCCTRL_MAINVERRATIO(x)             ((x) << 0)
>> +
>> +#define EXYNOS_CITAREA_TARGET_AREA(x)               ((x) << 0)
>> +
>> +#define EXYNOS_CISTATUS_GET_FRAME_COUNT(x)          (((x) >> 26) & 0x3)
>> +#define EXYNOS_CISTATUS_GET_FRAME_END(x)            (((x) >> 17) & 0x1)
>> +#define EXYNOS_CISTATUS_GET_LAST_CAPTURE_END(x)     (((x) >> 16) & 0x1)
>> +#define EXYNOS_CISTATUS_GET_LCD_STATUS(x)           (((x) >> 9) & 0x1)
>> +#define EXYNOS_CISTATUS_GET_ENVID_STATUS(x) (((x) >> 8) & 0x1)
>> +
>> +#define EXYNOS_CISTATUS2_GET_FRAMECOUNT_BEFORE(x)   (((x) >> 7) & 0x3f)
>> +#define EXYNOS_CISTATUS2_GET_FRAMECOUNT_PRESENT(x)  ((x) & 0x3f)
>> +
>> +#define EXYNOS_CIIMGEFF_FIN(x)                      ((x & 0x7) << 26)
>> +#define EXYNOS_CIIMGEFF_PAT_CB(x)                   ((x) << 13)
>> +#define EXYNOS_CIIMGEFF_PAT_CR(x)                   ((x) << 0)
>> +
>> +#define EXYNOS_CIILINESKIP(x)                       (((x) & 0xf) << 24)
>> +
>> +#define EXYNOS_CIREAL_ISIZE_HEIGHT(x)               ((x) << 16)
>> +#define EXYNOS_CIREAL_ISIZE_WIDTH(x)                ((x) << 0)
>> +
>> +#define EXYNOS_MSCTRL_SUCCESSIVE_COUNT(x)           ((x) << 24)
>> +#define EXYNOS_MSCTRL_GET_INDMA_STATUS(x)           ((x) & 0x1)
>> +
>> +#define EXYNOS_CIOYOFF_VERTICAL(x)                  ((x) << 16)
>> +#define EXYNOS_CIOYOFF_HORIZONTAL(x)                ((x) << 0)
>> +
>> +#define EXYNOS_CIOCBOFF_VERTICAL(x)         ((x) << 16)
>> +#define EXYNOS_CIOCBOFF_HORIZONTAL(x)               ((x) << 0)
>> +
>> +#define EXYNOS_CIOCROFF_VERTICAL(x)         ((x) << 16)
>> +#define EXYNOS_CIOCROFF_HORIZONTAL(x)               ((x) << 0)
>> +
>> +#define EXYNOS_CIIYOFF_VERTICAL(x)                  ((x) << 16)
>> +#define EXYNOS_CIIYOFF_HORIZONTAL(x)                ((x) << 0)
>> +
>> +#define EXYNOS_CIICBOFF_VERTICAL(x)         ((x) << 16)
>> +#define EXYNOS_CIICBOFF_HORIZONTAL(x)               ((x) << 0)
>> +
>> +#define EXYNOS_CIICROFF_VERTICAL(x)         ((x) << 16)
>> +#define EXYNOS_CIICROFF_HORIZONTAL(x)               ((x) << 0)
>> +
>> +#define EXYNOS_ORGISIZE_VERTICAL(x)         ((x) << 16)
>> +#define EXYNOS_ORGISIZE_HORIZONTAL(x)               ((x) << 0)
>> +
>> +#define EXYNOS_ORGOSIZE_VERTICAL(x)         ((x) << 16)
>> +#define EXYNOS_ORGOSIZE_HORIZONTAL(x)               ((x) << 0)
>> +
>> +#define EXYNOS_CIEXTEN_TARGETH_EXT(x)               ((((x) & 0x2000) >>
> 13) <<
>> 26)
>> +#define EXYNOS_CIEXTEN_TARGETV_EXT(x)               ((((x) & 0x2000) >>
> 13) <<
>> 24)
>> +#define EXYNOS_CIEXTEN_MAINHORRATIO_EXT(x)          (((x) & 0x3F) << 10)
>> +#define EXYNOS_CIEXTEN_MAINVERRATIO_EXT(x)          ((x) & 0x3F)
>> +
>> +/*
>> + * Bit definition part
>> +*/
>> +/* Source format register */
>> +#define EXYNOS_CISRCFMT_ITU601_8BIT         (1 << 31)
>> +#define EXYNOS_CISRCFMT_ITU656_8BIT         (0 << 31)
>> +#define EXYNOS_CISRCFMT_ITU601_16BIT                (1 << 29)
>> +#define EXYNOS_CISRCFMT_ORDER422_YCBYCR             (0 << 14)
>> +#define EXYNOS_CISRCFMT_ORDER422_YCRYCB             (1 << 14)
>> +#define EXYNOS_CISRCFMT_ORDER422_CBYCRY             (2 << 14)
>> +#define EXYNOS_CISRCFMT_ORDER422_CRYCBY             (3 << 14)
>> +/* ITU601 16bit only */
>> +#define EXYNOS_CISRCFMT_ORDER422_Y4CBCRCBCR (0 << 14)
>> +/* ITU601 16bit only */
>> +#define EXYNOS_CISRCFMT_ORDER422_Y4CRCBCRCB (1 << 14)
>> +
>> +/* Window offset register */
>> +#define EXYNOS_CIWDOFST_WINOFSEN                    (1 << 31)
>> +#define EXYNOS_CIWDOFST_CLROVFIY                    (1 << 30)
>> +#define EXYNOS_CIWDOFST_CLROVRLB                    (1 << 29)
>> +#define EXYNOS_CIWDOFST_WINHOROFST_MASK             (0x7ff << 16)
>> +#define EXYNOS_CIWDOFST_CLROVFICB                   (1 << 15)
>> +#define EXYNOS_CIWDOFST_CLROVFICR                   (1 << 14)
>> +#define EXYNOS_CIWDOFST_WINVEROFST_MASK             (0xfff << 0)
>> +
>> +/* Global control register */
>> +#define EXYNOS_CIGCTRL_SWRST                        (1 << 31)
>> +#define EXYNOS_CIGCTRL_CAMRST_A                     (1 << 30)
>> +#define EXYNOS_CIGCTRL_SELCAM_ITU_B         (0 << 29)
>> +#define EXYNOS_CIGCTRL_SELCAM_ITU_A         (1 << 29)
>> +#define EXYNOS_CIGCTRL_SELCAM_ITU_MASK              (1 << 29)
>> +#define EXYNOS_CIGCTRL_TESTPATTERN_NORMAL           (0 << 27)
>> +#define EXYNOS_CIGCTRL_TESTPATTERN_COLOR_BAR        (1 << 27)
>> +#define EXYNOS_CIGCTRL_TESTPATTERN_HOR_INC          (2 << 27)
>> +#define EXYNOS_CIGCTRL_TESTPATTERN_VER_INC          (3 << 27)
>> +#define EXYNOS_CIGCTRL_TESTPATTERN_MASK             (3 << 27)
>> +#define EXYNOS_CIGCTRL_TESTPATTERN_SHIFT            (27)
>> +#define EXYNOS_CIGCTRL_INVPOLPCLK                   (1 << 26)
>> +#define EXYNOS_CIGCTRL_INVPOLVSYNC                  (1 << 25)
>> +#define EXYNOS_CIGCTRL_INVPOLHREF                   (1 << 24)
>> +#define EXYNOS_CIGCTRL_IRQ_OVFEN                    (1 << 22)
>> +#define EXYNOS_CIGCTRL_HREF_MASK                    (1 << 21)
>> +#define EXYNOS_CIGCTRL_IRQ_EDGE                     (0 << 20)
>> +#define EXYNOS_CIGCTRL_IRQ_LEVEL                    (1 << 20)
>> +#define EXYNOS_CIGCTRL_IRQ_CLR                      (1 << 19)
>> +#define EXYNOS_CIGCTRL_IRQ_END_DISABLE              (1 << 18)
>> +#define EXYNOS_CIGCTRL_IRQ_DISABLE                  (0 << 16)
>> +#define EXYNOS_CIGCTRL_IRQ_ENABLE                   (1 << 16)
>> +#define EXYNOS_CIGCTRL_SHADOW_DISABLE               (1 << 12)
>> +#define EXYNOS_CIGCTRL_CAM_JPEG                     (1 << 8)
>> +#define EXYNOS_CIGCTRL_SELCAM_MIPI_B                (0 << 7)
>> +#define EXYNOS_CIGCTRL_SELCAM_MIPI_A                (1 << 7)
>> +#define EXYNOS_CIGCTRL_SELCAM_MIPI_MASK             (1 << 7)
>> +#define EXYNOS_CIGCTRL_SELWB_CAMIF_CAMERA   (0 << 6)
>> +#define EXYNOS_CIGCTRL_SELWB_CAMIF_WRITEBACK        (1 << 6)
>> +#define EXYNOS_CIGCTRL_SELWRITEBACK_MASK            (1 << 10)
>> +#define EXYNOS_CIGCTRL_SELWRITEBACK_A               (1 << 10)
>> +#define EXYNOS_CIGCTRL_SELWRITEBACK_B               (0 << 10)
>> +#define EXYNOS_CIGCTRL_SELWB_CAMIF_MASK             (1 << 6)
>> +#define EXYNOS_CIGCTRL_CSC_ITU601                   (0 << 5)
>> +#define EXYNOS_CIGCTRL_CSC_ITU709                   (1 << 5)
>> +#define EXYNOS_CIGCTRL_CSC_MASK                     (1 << 5)
>> +#define EXYNOS_CIGCTRL_INVPOLHSYNC                  (1 << 4)
>> +#define EXYNOS_CIGCTRL_SELCAM_FIMC_ITU              (0 << 3)
>> +#define EXYNOS_CIGCTRL_SELCAM_FIMC_MIPI             (1 << 3)
>> +#define EXYNOS_CIGCTRL_SELCAM_FIMC_MASK             (1 << 3)
>> +#define EXYNOS_CIGCTRL_PROGRESSIVE                  (0 << 0)
>> +#define EXYNOS_CIGCTRL_INTERLACE                    (1 << 0)
>> +
>> +/* Window offset2 register */
>> +#define EXYNOS_CIWDOFST_WINHOROFST2_MASK            (0xfff << 16)
>> +#define EXYNOS_CIWDOFST_WINVEROFST2_MASK            (0xfff << 16)
>> +
>> +/* Target format register */
>> +#define EXYNOS_CITRGFMT_INROT90_CLOCKWISE           (1 << 31)
>> +#define EXYNOS_CITRGFMT_OUTFORMAT_YCBCR420          (0 << 29)
>> +#define EXYNOS_CITRGFMT_OUTFORMAT_YCBCR422          (1 << 29)
>> +#define EXYNOS_CITRGFMT_OUTFORMAT_YCBCR422_1PLANE   (2 << 29)
>> +#define EXYNOS_CITRGFMT_OUTFORMAT_RGB               (3 << 29)
>> +#define EXYNOS_CITRGFMT_OUTFORMAT_MASK              (3 << 29)
>> +#define EXYNOS_CITRGFMT_FLIP_SHIFT                  (14)
>> +#define EXYNOS_CITRGFMT_FLIP_NORMAL         (0 << 14)
>> +#define EXYNOS_CITRGFMT_FLIP_X_MIRROR               (1 << 14)
>> +#define EXYNOS_CITRGFMT_FLIP_Y_MIRROR               (2 << 14)
>> +#define EXYNOS_CITRGFMT_FLIP_180                    (3 << 14)
>> +#define EXYNOS_CITRGFMT_FLIP_MASK                   (3 << 14)
>> +#define EXYNOS_CITRGFMT_OUTROT90_CLOCKWISE          (1 << 13)
>> +#define EXYNOS_CITRGFMT_TARGETV_MASK                (0x1fff << 0)
>> +#define EXYNOS_CITRGFMT_TARGETH_MASK                (0x1fff << 16)
>> +
>> +/* Output DMA control register */
>> +#define EXYNOS_CIOCTRL_WEAVE_OUT                    (1 << 31)
>> +#define EXYNOS_CIOCTRL_WEAVE_MASK                   (1 << 31)
>> +#define EXYNOS_CIOCTRL_LASTENDEN                    (1 << 30)
>> +#define EXYNOS_CIOCTRL_ORDER2P_LSB_CBCR             (0 << 24)
>> +#define EXYNOS_CIOCTRL_ORDER2P_LSB_CRCB             (1 << 24)
>> +#define EXYNOS_CIOCTRL_ORDER2P_MSB_CRCB             (2 << 24)
>> +#define EXYNOS_CIOCTRL_ORDER2P_MSB_CBCR             (3 << 24)
>> +#define EXYNOS_CIOCTRL_ORDER2P_SHIFT                (24)
>> +#define EXYNOS_CIOCTRL_ORDER2P_MASK         (3 << 24)
>> +#define EXYNOS_CIOCTRL_YCBCR_3PLANE         (0 << 3)
>> +#define EXYNOS_CIOCTRL_YCBCR_2PLANE         (1 << 3)
>> +#define EXYNOS_CIOCTRL_YCBCR_PLANE_MASK             (1 << 3)
>> +#define EXYNOS_CIOCTRL_LASTIRQ_ENABLE               (1 << 2)
>> +#define EXYNOS_CIOCTRL_ALPHA_OUT                    (0xff << 4)
>> +#define EXYNOS_CIOCTRL_ORDER422_YCBYCR              (0 << 0)
>> +#define EXYNOS_CIOCTRL_ORDER422_YCRYCB              (1 << 0)
>> +#define EXYNOS_CIOCTRL_ORDER422_CBYCRY              (2 << 0)
>> +#define EXYNOS_CIOCTRL_ORDER422_CRYCBY              (3 << 0)
>> +#define EXYNOS_CIOCTRL_ORDER422_MASK                (3 << 0)
>> +
>> +/* Main scaler control register */
>> +#define EXYNOS_CISCCTRL_SCALERBYPASS                (1 << 31)
>> +#define EXYNOS_CISCCTRL_SCALEUP_H                   (1 << 30)
>> +#define EXYNOS_CISCCTRL_SCALEUP_V                   (1 << 29)
>> +#define EXYNOS_CISCCTRL_CSCR2Y_NARROW               (0 << 28)
>> +#define EXYNOS_CISCCTRL_CSCR2Y_WIDE         (1 << 28)
>> +#define EXYNOS_CISCCTRL_CSCY2R_NARROW               (0 << 27)
>> +#define EXYNOS_CISCCTRL_CSCY2R_WIDE         (1 << 27)
>> +#define EXYNOS_CISCCTRL_LCDPATHEN_FIFO              (1 << 26)
>> +#define EXYNOS_CISCCTRL_PROGRESSIVE         (0 << 25)
>> +#define EXYNOS_CISCCTRL_INTERLACE                   (1 << 25)
>> +#define EXYNOS_CISCCTRL_SCAN_MASK                   (1 << 25)
>> +#define EXYNOS_CISCCTRL_SCALERSTART         (1 << 15)
>> +#define EXYNOS_CISCCTRL_INRGB_FMT_RGB565            (0 << 13)
>> +#define EXYNOS_CISCCTRL_INRGB_FMT_RGB666            (1 << 13)
>> +#define EXYNOS_CISCCTRL_INRGB_FMT_RGB888            (2 << 13)
>> +#define EXYNOS_CISCCTRL_INRGB_FMT_RGB_MASK          (3 << 13)
>> +#define EXYNOS_CISCCTRL_OUTRGB_FMT_RGB565           (0 << 11)
>> +#define EXYNOS_CISCCTRL_OUTRGB_FMT_RGB666           (1 << 11)
>> +#define EXYNOS_CISCCTRL_OUTRGB_FMT_RGB888           (2 << 11)
>> +#define EXYNOS_CISCCTRL_OUTRGB_FMT_RGB_MASK (3 << 11)
>> +#define EXYNOS_CISCCTRL_EXTRGB_NORMAL               (0 << 10)
>> +#define EXYNOS_CISCCTRL_EXTRGB_EXTENSION            (1 << 10)
>> +#define EXYNOS_CISCCTRL_ONE2ONE                     (1 << 9)
>> +#define EXYNOS_CISCCTRL_MAIN_V_RATIO_MASK           (0x1ff << 0)
>> +#define EXYNOS_CISCCTRL_MAIN_H_RATIO_MASK           (0x1ff << 16)
>> +
>> +/* Status register */
>> +#define EXYNOS_CISTATUS_OVFIY                       (1 << 31)
>> +#define EXYNOS_CISTATUS_OVFICB                      (1 << 30)
>> +#define EXYNOS_CISTATUS_OVFICR                      (1 << 29)
>> +#define EXYNOS_CISTATUS_VSYNC                       (1 << 28)
>> +#define EXYNOS_CISTATUS_SCALERSTART         (1 << 26)
>> +#define EXYNOS_CISTATUS_WINOFSTEN                   (1 << 25)
>> +#define EXYNOS_CISTATUS_IMGCPTEN                    (1 << 22)
>> +#define EXYNOS_CISTATUS_IMGCPTENSC                  (1 << 21)
>> +#define EXYNOS_CISTATUS_VSYNC_A                     (1 << 20)
>> +#define EXYNOS_CISTATUS_VSYNC_B                     (1 << 19)
>> +#define EXYNOS_CISTATUS_OVRLB                       (1 << 18)
>> +#define EXYNOS_CISTATUS_FRAMEEND                    (1 << 17)
>> +#define EXYNOS_CISTATUS_LASTCAPTUREEND              (1 << 16)
>> +#define EXYNOS_CISTATUS_VVALID_A                    (1 << 15)
>> +#define EXYNOS_CISTATUS_VVALID_B                    (1 << 14)
>> +
>> +/* Image capture enable register */
>> +#define EXYNOS_CIIMGCPT_IMGCPTEN                    (1 << 31)
>> +#define EXYNOS_CIIMGCPT_IMGCPTEN_SC         (1 << 30)
>> +#define EXYNOS_CIIMGCPT_CPT_FREN_ENABLE             (1 << 25)
>> +#define EXYNOS_CIIMGCPT_CPT_FRMOD_EN                (0 << 18)
>> +#define EXYNOS_CIIMGCPT_CPT_FRMOD_CNT               (1 << 18)
>> +
>> +/* Image effects register */
>> +#define EXYNOS_CIIMGEFF_IE_DISABLE                  (0 << 30)
>> +#define EXYNOS_CIIMGEFF_IE_ENABLE                   (1 << 30)
>> +#define EXYNOS_CIIMGEFF_IE_SC_BEFORE                (0 << 29)
>> +#define EXYNOS_CIIMGEFF_IE_SC_AFTER         (1 << 29)
>> +#define EXYNOS_CIIMGEFF_FIN_BYPASS                  (0 << 26)
>> +#define EXYNOS_CIIMGEFF_FIN_ARBITRARY               (1 << 26)
>> +#define EXYNOS_CIIMGEFF_FIN_NEGATIVE                (2 << 26)
>> +#define EXYNOS_CIIMGEFF_FIN_ARTFREEZE               (3 << 26)
>> +#define EXYNOS_CIIMGEFF_FIN_EMBOSSING               (4 << 26)
>> +#define EXYNOS_CIIMGEFF_FIN_SILHOUETTE              (5 << 26)
>> +#define EXYNOS_CIIMGEFF_FIN_MASK                    (7 << 26)
>> +#define EXYNOS_CIIMGEFF_PAT_CBCR_MASK               ((0xff < 13) | (0xff
> < 0))
>> +
>> +/* Real input DMA size register */
>> +#define EXYNOS_CIREAL_ISIZE_AUTOLOAD_ENABLE (1 << 31)
>> +#define EXYNOS_CIREAL_ISIZE_ADDR_CH_DISABLE (1 << 30)
>> +#define EXYNOS_CIREAL_ISIZE_HEIGHT_MASK             (0x3FFF << 16)
>> +#define EXYNOS_CIREAL_ISIZE_WIDTH_MASK              (0x3FFF << 0)
>> +
>> +/* Input DMA control register */
>> +#define EXYNOS_MSCTRL_FIELD_MASK                    (1 << 31)
>> +#define EXYNOS_MSCTRL_FIELD_WEAVE                   (1 << 31)
>> +#define EXYNOS_MSCTRL_FIELD_NORMAL                  (0 << 31)
>> +#define EXYNOS_MSCTRL_BURST_CNT                     (24)
>> +#define EXYNOS_MSCTRL_BURST_CNT_MASK                (0xf << 24)
>> +#define EXYNOS_MSCTRL_ORDER2P_LSB_CBCR              (0 << 16)
>> +#define EXYNOS_MSCTRL_ORDER2P_LSB_CRCB              (1 << 16)
>> +#define EXYNOS_MSCTRL_ORDER2P_MSB_CRCB              (2 << 16)
>> +#define EXYNOS_MSCTRL_ORDER2P_MSB_CBCR              (3 << 16)
>> +#define EXYNOS_MSCTRL_ORDER2P_SHIFT         (16)
>> +#define EXYNOS_MSCTRL_ORDER2P_SHIFT_MASK            (0x3 << 16)
>> +#define EXYNOS_MSCTRL_C_INT_IN_3PLANE               (0 << 15)
>> +#define EXYNOS_MSCTRL_C_INT_IN_2PLANE               (1 << 15)
>> +#define EXYNOS_MSCTRL_FLIP_SHIFT                    (13)
>> +#define EXYNOS_MSCTRL_FLIP_NORMAL                   (0 << 13)
>> +#define EXYNOS_MSCTRL_FLIP_X_MIRROR         (1 << 13)
>> +#define EXYNOS_MSCTRL_FLIP_Y_MIRROR         (2 << 13)
>> +#define EXYNOS_MSCTRL_FLIP_180                      (3 << 13)
>> +#define EXYNOS_MSCTRL_FLIP_MASK                     (3 << 13)
>> +#define EXYNOS_MSCTRL_ORDER422_CRYCBY               (0 << 4)
>> +#define EXYNOS_MSCTRL_ORDER422_YCRYCB               (1 << 4)
>> +#define EXYNOS_MSCTRL_ORDER422_CBYCRY               (2 << 4)
>> +#define EXYNOS_MSCTRL_ORDER422_YCBYCR               (3 << 4)
>> +#define EXYNOS_MSCTRL_INPUT_EXTCAM                  (0 << 3)
>> +#define EXYNOS_MSCTRL_INPUT_MEMORY                  (1 << 3)
>> +#define EXYNOS_MSCTRL_INPUT_MASK                    (1 << 3)
>> +#define EXYNOS_MSCTRL_INFORMAT_YCBCR420             (0 << 1)
>> +#define EXYNOS_MSCTRL_INFORMAT_YCBCR422             (1 << 1)
>> +#define EXYNOS_MSCTRL_INFORMAT_YCBCR422_1PLANE      (2 << 1)
>> +#define EXYNOS_MSCTRL_INFORMAT_RGB                  (3 << 1)
>> +#define EXYNOS_MSCTRL_ENVID                 (1 << 0)
>> +
>> +/* DMA parameter register */
>> +#define EXYNOS_CIDMAPARAM_R_MODE_LINEAR             (0 << 29)
>> +#define EXYNOS_CIDMAPARAM_R_MODE_CONFTILE           (1 << 29)
>> +#define EXYNOS_CIDMAPARAM_R_MODE_16X16              (2 << 29)
>> +#define EXYNOS_CIDMAPARAM_R_MODE_64X32              (3 << 29)
>> +#define EXYNOS_CIDMAPARAM_R_MODE_MASK               (3 << 29)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_HSIZE_64           (0 << 24)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_HSIZE_128          (1 << 24)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_HSIZE_256          (2 << 24)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_HSIZE_512          (3 << 24)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_HSIZE_1024 (4 << 24)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_HSIZE_2048 (5 << 24)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_HSIZE_4096 (6 << 24)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_VSIZE_1            (0 << 20)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_VSIZE_2            (1 << 20)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_VSIZE_4            (2 << 20)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_VSIZE_8            (3 << 20)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_VSIZE_16           (4 << 20)
>> +#define EXYNOS_CIDMAPARAM_R_TILE_VSIZE_32           (5 << 20)
>> +#define EXYNOS_CIDMAPARAM_W_MODE_LINEAR             (0 << 13)
>> +#define EXYNOS_CIDMAPARAM_W_MODE_CONFTILE           (1 << 13)
>> +#define EXYNOS_CIDMAPARAM_W_MODE_16X16              (2 << 13)
>> +#define EXYNOS_CIDMAPARAM_W_MODE_64X32              (3 << 13)
>> +#define EXYNOS_CIDMAPARAM_W_MODE_MASK               (3 << 13)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_HSIZE_64           (0 << 8)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_HSIZE_128          (1 << 8)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_HSIZE_256          (2 << 8)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_HSIZE_512          (3 << 8)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_HSIZE_1024 (4 << 8)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_HSIZE_2048 (5 << 8)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_HSIZE_4096 (6 << 8)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_VSIZE_1            (0 << 4)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_VSIZE_2            (1 << 4)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_VSIZE_4            (2 << 4)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_VSIZE_8            (3 << 4)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_VSIZE_16           (4 << 4)
>> +#define EXYNOS_CIDMAPARAM_W_TILE_VSIZE_32           (5 << 4)
>> +
>> +/* Gathering Extension register */
>> +#define EXYNOS_CIEXTEN_TARGETH_EXT_MASK             (1 << 26)
>> +#define EXYNOS_CIEXTEN_TARGETV_EXT_MASK             (1 << 24)
>> +#define EXYNOS_CIEXTEN_MAINHORRATIO_EXT_MASK        (0x3F << 10)
>> +#define EXYNOS_CIEXTEN_MAINVERRATIO_EXT_MASK        (0x3F)
>> +#define EXYNOS_CIEXTEN_YUV444_OUT                   (1 << 22)
>> +
>> +/* FIMC Clock Source Select register */
>> +#define EXYNOS_CLKSRC_HCLK                          (0 << 1)
>> +#define EXYNOS_CLKSRC_HCLK_MASK                     (1 << 1)
>> +#define EXYNOS_CLKSRC_SCLK                          (1 << 1)
>> +
>> +/* SYSREG for FIMC writeback */
>> +#define SYSREG_CAMERA_BLK                   (S3C_VA_SYS + 0x0218)
>> +#define SYSREG_ISP_BLK                              (S3C_VA_SYS +
> 0x020c)
>> +#define SYSREG_FIMD0WB_DEST_MASK    (0x3 << 23)
>> +#define SYSREG_FIMD0WB_DEST_SHIFT   23
>> +
>> +#endif /* EXYNOS_REGS_FIMC_H */
>> diff --git a/include/drm/exynos_drm.h b/include/drm/exynos_drm.h
>> index 3c13a3a..4d55b1f 100644
>> --- a/include/drm/exynos_drm.h
>> +++ b/include/drm/exynos_drm.h
>> @@ -85,4 +85,38 @@ struct exynos_drm_hdmi_pdata {
>>      int (*get_hpd)(void);
>>   };
>>
>> +/**
>> + * Platform Specific Structure for DRM based FIMC.
>> + *
>> + * @inv_pclk: if set 1. invert pixel clock
>> + * @inv_vsync: if set 1. invert vsync signal for wb
>> + * @inv_href: if set 1. invert href signal
>> + * @inv_hsync: if set 1. invert hsync signal for wb
>> + */
>> +struct exynos_drm_fimc_pol {
>> +    unsigned int inv_pclk;
>> +    unsigned int inv_vsync;
>> +    unsigned int inv_href;
>> +    unsigned int inv_hsync;
>> +};
>> +
>> +/* definition of chipset version */
>> +enum exynos_drm_fimc_ver {
>> +    FIMC_EXYNOS_4210,
>> +    FIMC_EXYNOS_4212,
>> +    FIMC_EXYNOS_4412,
>> +    FIMC_EXYNOS_4412_R2,
>> +};
>> +
>> +/**
>> + * Platform Specific Structure for DRM based FIMC.
>> + *
>> + * @pol: current hardware block polarity settings.
>> + * @ver: current hardware block version.
>> + */
>> +struct exynos_drm_fimc_pdata {
>> +    struct exynos_drm_fimc_pol pol;
>> +    enum exynos_drm_fimc_ver ver;
>> +};
>> +
>>   #endif     /* _EXYNOS_DRM_H_ */
>> --
>> 1.7.0.4
>
>

Reply via email to