Re: [PATCH 2/2] drm/exynos: decon: Add support for DECON-EXT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Mar 2, 2015 at 1:38 PM, Andrzej Hajda <a.hajda@xxxxxxxxxxx> wrote:
> On 02/26/2015 04:24 PM, Ajay Kumar wrote:
>> * Modify DECON-INT driver to support DECON-EXT.
>> * Add a table of porch values needed to set timing registers of DECON-EXT.
>> * DECON-EXT supports only H/w Triggered COMMAND mode.
>> * DECON-EXT supports only one DMA window(window 1), so modify
>>   all window management routines to support 2 windows of DECON-INT
>>   and 1 window of DECON-EXT.
>>
>> Signed-off-by: Ajay Kumar <ajaykumar.rs@xxxxxxxxxxx>
>> ---
>>  .../devicetree/bindings/video/exynos7-decon.txt    |    8 +-
>>  drivers/gpu/drm/exynos/exynos7_drm_decon.c         |  229 ++++++++++++++++++--
>>  include/video/exynos7_decon.h                      |   13 ++
>>  3 files changed, 230 insertions(+), 20 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/video/exynos7-decon.txt b/Documentation/devicetree/bindings/video/exynos7-decon.txt
>> index f5f9c8d..87350c0 100644
>> --- a/Documentation/devicetree/bindings/video/exynos7-decon.txt
>> +++ b/Documentation/devicetree/bindings/video/exynos7-decon.txt
>> @@ -2,10 +2,14 @@ Device-Tree bindings for Samsung Exynos7 SoC display controller (DECON)
>>
>>  DECON (Display and Enhancement Controller) is the Display Controller for the
>>  Exynos7 series of SoCs which transfers the image data from a video memory
>> -buffer to an external LCD interface.
>> +buffer to an external LCD/HDMI interface.
>> +
>> +Exynos7 supports DECON-INT for generating video signals for LCD.
>> +Exynos7 supports DECON-EXT for generating video signals for HDMI.
>>
>>  Required properties:
>> -- compatible: value should be "samsung,exynos7-decon";
>> +- compatible: value should be "samsung,exynos7-decon-int" for DECON-INT;
>> +           value should be "samsung,exynos7-decon-ext" for DECON-EXT;
>>
>>  - reg: physical base address and length of the DECON registers set.
>>
>> diff --git a/drivers/gpu/drm/exynos/exynos7_drm_decon.c b/drivers/gpu/drm/exynos/exynos7_drm_decon.c
>> index 63f02e2..9e2d083 100644
>> --- a/drivers/gpu/drm/exynos/exynos7_drm_decon.c
>> +++ b/drivers/gpu/drm/exynos/exynos7_drm_decon.c
>> @@ -41,6 +41,28 @@
>>
>>  #define WINDOWS_NR   2
>
> Maybe it would be better to rename it to MAX_WINDOWS_NR
Ok.
>>
>> +#define DECON_EXT_CH_MAPPING 0x432105
>
> I am not familiar with decon, could you explain what exactly
> this mapping means and why is it fixed in the driver to this value,
> not default one. By the way my specs says bits 0-3 are reserverd
> and here it seems you are using them.
I reused the value from the code which hardware team shared with me.
It tries to map a input hardware channel(IDMA or VPP channel) onto
a hardware window in DECON.
Channel_N is mapped onto window_N in case of DECON-INT.
In case of DECON-EXT, Channel 0 should be mapped to window 1 of DECON-EXT.
So, basically for the changes I have made, I should ideally set only
bits [4:6] to 0x1,
and leave other bits untouched, though modifying other bits wouldn't
affect in anyway.
>
>> +
>> +enum decon_type {
>> +     DECON_INT,
>> +     DECON_EXT,
>> +} decon_type;
>> +
>> +struct decon_driver_data {
>> +     enum decon_type                 decon_type;
>> +     unsigned int                    nr_windows;
>> +};
>> +
>> +static struct decon_driver_data exynos7_decon_int_driver_data = {
>> +     .decon_type = DECON_INT,
>
> I wonder if it wouldn't be better to use different fields to describe
> each capability/property instead of decon_type. For example
>         .display_type = EXYNOS_DISPLAY_TYPE_LCD,
>         .map_channels = 0,
Ok, let me list down the differences between INT and EXT.
1) Only h/w triggered command mode for DECON-EXT.
2) Need to feed modified porch values(decon_ext_timings)
3) Input channel to H/w window mapping(WINCHMAP)
4) default_window - 0 for DECON-INT and 1 for DECON-EXT

Out of the above differences, the first 3 can somehow be converted
to capability/property and embedded into driver_data.
But the 4th difference is bothering me.
I tried using something like start_window, end_window and tried to make
The code common for DECON-INT and DECON-EXT, and it just doesn't work.
ex:
start_window = 0, end_window = 1 for DECON-INT
start_window = 1, end_window = 1 for DECON-EXT

When win_commit gets called for window 0 from crtc_commit/plane_commit:
Configure start_window(0  for DECON-INT and 1 for DECON-EXT)

When win_commit is called from decon_apply, it is called for window 1 also.
That time win_commit can skip updating actual window 1.
How do I take care of this ambiguity? This case happens with
win_disable routine also!


> This way the code will be cleaner (less ifs).
>
>
>> +     .nr_windows = 2,
>> +};
>> +
>> +static struct decon_driver_data exynos7_decon_ext_driver_data = {
>> +     .decon_type = DECON_EXT,
>> +     .nr_windows = 1,
>> +};
>> +
>>  struct decon_win_data {
>>       unsigned int            ovl_x;
>>       unsigned int            ovl_y;
>> @@ -76,15 +98,28 @@ struct decon_context {
>>       atomic_t                        wait_vsync_event;
>>
>>       struct exynos_drm_panel_info panel;
>> +     struct decon_driver_data *driver_data;
>>       struct exynos_drm_display *display;
>>  };
>>
>>  static const struct of_device_id decon_driver_dt_match[] = {
>> -     {.compatible = "samsung,exynos7-decon"},
>> +     { .compatible = "samsung,exynos7-decon-int",
>> +       .data = &exynos7_decon_int_driver_data },
>> +     { .compatible = "samsung,exynos7-decon-ext",
>> +       .data = &exynos7_decon_ext_driver_data },
>>       {},
>>  };
>>  MODULE_DEVICE_TABLE(of, decon_driver_dt_match);
>>
>> +static inline struct decon_driver_data *drm_decon_get_driver_data(
>> +     struct platform_device *pdev)
>> +{
>> +     const struct of_device_id *of_id =
>> +                     of_match_device(decon_driver_dt_match, &pdev->dev);
>> +
>> +     return (struct decon_driver_data *)of_id->data;
>> +}
>> +
>>  static void decon_wait_for_vblank(struct exynos_drm_crtc *crtc)
>>  {
>>       struct decon_context *ctx = crtc->ctx;
>> @@ -106,13 +141,20 @@ static void decon_wait_for_vblank(struct exynos_drm_crtc *crtc)
>>
>>  static void decon_clear_channel(struct decon_context *ctx)
>>  {
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       int win, ch_enabled = 0;
>>
>>       DRM_DEBUG_KMS("%s\n", __FILE__);
>>
>>       /* Check if any channel is enabled. */
>> -     for (win = 0; win < WINDOWS_NR; win++) {
>> -             u32 val = readl(ctx->regs + WINCON(win));
>> +     for (win = 0; win < drv_data->nr_windows; win++) {
>> +             u32 val;
>> +             /* DECON EXT sw-hw window mapping */
>> +             if (drv_data->decon_type == DECON_EXT) {
>> +                     if (win == 0)
>> +                             win = 1;
>
> Changing value of for iterator inside the loop seems quite strange.
> Wouldn't be better to start loop from 1 in case of DECON_EXT.
I have explained the resulting ambiguity above.
>
>> +             }
>> +             val = readl(ctx->regs + WINCON(win));
>>
>>               if (val & WINCONx_ENWIN) {
>>                       val &= ~WINCONx_ENWIN;
>> @@ -187,10 +229,74 @@ static bool decon_mode_fixup(struct exynos_drm_crtc *crtc,
>>       return true;
>>  }
>>
>> +struct decon_ext_videomode {
>> +     u32 vfront_porch;
>> +     u32 vback_porch;
>> +     u32 hfront_porch;
>> +     u32 hback_porch;
>> +     u32 vsync_len;
>> +     u32 hsync_len;
>> +     u32 hactive;
>> +     u32 vactive;
>> +     u32 refresh_rate;
>> +};
>> +
>> +enum DECON_EXT_MODES {
>> +     MODE_720_480_60,
>> +     MODE_720_576_50,
>> +     MODE_1280_720_60,
>> +     MODE_1280_720_50,
>> +     MODE_1920_1080_60,
>> +     MODE_1920_1080_50,
>> +     MODE_1920_1080_30,
>> +     MODE_1920_1080_25,
>> +     MODE_1920_1080_24,
>> +     MODE_3840_2160_30,
>> +     MODE_3840_2160_25,
>> +     MODE_3840_2160_24,
>> +     MODE_4096_2160_24,
>> +};
>> +
>> +static struct decon_ext_videomode decon_ext_porchs[] = {
>> +     /*vfp vbp hfp hbp  vsa hsa xres yres fps */
>> +     {1, 43, 10, 92,   1, 36,  720, 480,   60},
>> +     {1, 47, 10, 92,   1, 42,  720, 576,   50},
>> +     {1, 28, 10, 192,  1, 168, 1280, 720,  60},
>> +     {1, 28, 10, 92,   1, 598, 1280, 720,  50},
>> +     {1, 43, 10, 92,   1, 178, 1920, 1080, 60},
>> +     {1, 43, 10, 92,   1, 618, 1920, 1080, 50},
>> +     {1, 43, 10, 92,   1, 178, 1920, 1080, 30},
>> +     {1, 43, 10, 92,   1, 618, 1920, 1080, 25},
>> +     {1, 43, 10, 92,   1, 728, 1920, 1080, 24},
>> +     {1, 88, 10, 458,  1, 92,  3840, 2160, 30},
>> +     {1, 88, 10, 1338, 1, 92,  3840, 2160, 25},
>> +     {1, 88, 10, 1558, 1, 92,  3840, 2160, 24},
>> +     {1, 88, 10, 1302, 1, 92,  4096, 2160, 24},
>> +};
>> +
>> +static void decon_ext_set_timing(struct drm_display_mode *mode,
>> +                             struct decon_ext_videomode *ext_mode)
>> +{
>> +     mode->crtc_hdisplay = ext_mode->hactive;
>> +     mode->crtc_hsync_start = ext_mode->hactive + ext_mode->hfront_porch;
>> +     mode->crtc_hsync_end = ext_mode->hactive + ext_mode->hfront_porch
>> +                                             + ext_mode->hsync_len;
>> +     mode->crtc_htotal = ext_mode->hactive + ext_mode->hfront_porch
>> +                     + ext_mode->hsync_len + ext_mode->hback_porch;
>> +     mode->crtc_vdisplay = ext_mode->vactive;
>> +     mode->crtc_vsync_start = ext_mode->vactive + ext_mode->vfront_porch;
>> +     mode->crtc_vsync_end = ext_mode->vactive + ext_mode->vfront_porch
>> +                                             + ext_mode->vsync_len;
>> +     mode->crtc_vtotal = ext_mode->vactive + ext_mode->vfront_porch
>> +                             + ext_mode->vsync_len + ext_mode->vback_porch;
>> +}
>> +
>>  static void decon_commit(struct exynos_drm_crtc *crtc)
>>  {
>>       struct decon_context *ctx = crtc->ctx;
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       struct drm_display_mode *mode = &crtc->base.mode;
>> +     struct decon_ext_videomode *my_mode;
>>       u32 val, clkdiv;
>>
>>       if (ctx->suspended)
>> @@ -200,6 +306,38 @@ static void decon_commit(struct exynos_drm_crtc *crtc)
>>       if (mode->htotal == 0 || mode->vtotal == 0)
>>               return;
>>
>> +     if (drv_data->decon_type == DECON_EXT) {
>> +             if ((mode->hdisplay == 720) && (mode->vdisplay == 480) &&
>> +                                             (mode->vrefresh == 60))
>> +                     my_mode = &decon_ext_porchs[MODE_720_480_60];
>> +             if ((mode->hdisplay == 720) && (mode->vdisplay == 576) &&
>> +                                             (mode->vrefresh == 50))
>> +                     my_mode = &decon_ext_porchs[MODE_720_576_50];
>> +             if ((mode->hdisplay == 1280) && (mode->vdisplay == 720) &&
>> +                                             (mode->vrefresh == 60))
>> +                     my_mode = &decon_ext_porchs[MODE_1280_720_60];
>> +             if ((mode->hdisplay == 1280) && (mode->vdisplay == 720) &&
>> +                                             (mode->vrefresh == 50))
>> +                     my_mode = &decon_ext_porchs[MODE_1280_720_50];
>> +             if ((mode->hdisplay == 1920) && (mode->vdisplay == 1080) &&
>> +                                             (mode->vrefresh == 60))
>> +                     my_mode = &decon_ext_porchs[MODE_1920_1080_60];
>> +             if ((mode->hdisplay == 1920) && (mode->vdisplay == 1080) &&
>> +                                             (mode->vrefresh == 50))
>> +                     my_mode = &decon_ext_porchs[MODE_1920_1080_50];
>> +             if ((mode->hdisplay == 1920) && (mode->vdisplay == 1080) &&
>> +                                             (mode->vrefresh == 30))
>> +                     my_mode = &decon_ext_porchs[MODE_1920_1080_30];
>> +             if ((mode->hdisplay == 1920) && (mode->vdisplay == 1080) &&
>> +                                             (mode->vrefresh == 25))
>> +                     my_mode = &decon_ext_porchs[MODE_1920_1080_25];
>> +             if ((mode->hdisplay == 1920) && (mode->vdisplay == 1080) &&
>> +                                             (mode->vrefresh == 24))
>> +                     my_mode = &decon_ext_porchs[MODE_1920_1080_24];
>> +
>> +             decon_ext_set_timing(mode, my_mode);
>> +     }
>> +
>
> I guess these ugly ifs could be replaced by simple loop:
>         for (i = 0; i < ARRAY_SIZE(decon_ext_porchs); ++i) {
>                 if (mode->hdisplay != decon_ext_porchs[i].hdisplay || ...)
>                         continue;
>                 decon_ext_set_timing(mode, &decon_ext_porchs[i]);
>                 break;
>         }
Ok, Though the number of checks is the same, your code at least looks better. :)

> Anyway I have doubts if commit should modify crtc.base.mode I guess
> better place for it can be in *_fixup callback.
Inki? What is your suggestion?

>>       if (!ctx->i80_if) {
>>               int vsync_len, vbpd, vfpd, hsync_len, hbpd, hfpd;
>>             /* setup vertical timing values. */
>> @@ -301,6 +439,7 @@ static void decon_win_mode_set(struct exynos_drm_crtc *crtc,
>>  {
>>       struct decon_context *ctx = crtc->ctx;
>>       struct decon_win_data *win_data;
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       int win, padding;
>>
>>       if (!plane) {
>> @@ -312,12 +451,18 @@ static void decon_win_mode_set(struct exynos_drm_crtc *crtc,
>>       if (win == DEFAULT_ZPOS)
>>               win = ctx->default_win;
>>
>> -     if (win < 0 || win >= WINDOWS_NR)
>> +     if (win < 0 || win >= drv_data->nr_windows)
>>               return;
>>
>>
>>       win_data = &ctx->win_data[win];
>>
>> +     /* DECON EXT sw-hw window mapping */
>> +     if (drv_data->decon_type == DECON_EXT) {
>> +             if (win == 0)
>> +                     win = 1;
>> +     }
>
> This code is repeated few times, maybe it would be good to place it into
> some helper function.
Ok.

>> +
>>       padding = (plane->pitch / (plane->bpp >> 3)) - plane->fb_width;
>>       win_data->offset_x = plane->fb_x;
>>       win_data->offset_y = plane->fb_y;
>> @@ -340,9 +485,9 @@ static void decon_win_mode_set(struct exynos_drm_crtc *crtc,
>>                       plane->fb_width, plane->crtc_width);
>>  }
>>
>> -static void decon_win_set_pixfmt(struct decon_context *ctx, unsigned int win)
>> +static void decon_win_set_pixfmt(struct decon_context *ctx, unsigned int win,
>> +                                     struct decon_win_data *win_data)
>>  {
>> -     struct decon_win_data *win_data = &ctx->win_data[win];
>>       unsigned long val;
>>
>>       val = readl(ctx->regs + WINCON(win));
>> @@ -454,6 +599,7 @@ static void decon_win_commit(struct exynos_drm_crtc *crtc, int zpos)
>>       struct decon_context *ctx = crtc->ctx;
>>       struct drm_display_mode *mode = &crtc->base.mode;
>>       struct decon_win_data *win_data;
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       int win = zpos;
>>       unsigned long val, alpha;
>>       unsigned int last_x;
>> @@ -465,11 +611,17 @@ static void decon_win_commit(struct exynos_drm_crtc *crtc, int zpos)
>>       if (win == DEFAULT_ZPOS)
>>               win = ctx->default_win;
>>
>> -     if (win < 0 || win >= WINDOWS_NR)
>> +     if (win < 0 || win >= drv_data->nr_windows)
>>               return;
>>
>>       win_data = &ctx->win_data[win];
>>
>> +     /* DECON EXT sw-hw window mapping */
>> +     if (drv_data->decon_type == DECON_EXT) {
>> +             if (win == 0)
>> +                     win = 1;
>> +     }
>> +
>>       /* If suspended, enable this on resume */
>>       if (ctx->suspended) {
>>               win_data->resume = true;
>> @@ -546,7 +698,7 @@ static void decon_win_commit(struct exynos_drm_crtc *crtc, int zpos)
>>
>>       writel(alpha, ctx->regs + VIDOSD_D(win));
>>
>> -     decon_win_set_pixfmt(ctx, win);
>> +     decon_win_set_pixfmt(ctx, win, win_data);
>>
>>       /* hardware window 0 doesn't support color key. */
>>       if (win != 0)
>> @@ -572,17 +724,24 @@ static void decon_win_disable(struct exynos_drm_crtc *crtc, int zpos)
>>  {
>>       struct decon_context *ctx = crtc->ctx;
>>       struct decon_win_data *win_data;
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       int win = zpos;
>>       u32 val;
>>
>>       if (win == DEFAULT_ZPOS)
>>               win = ctx->default_win;
>>
>> -     if (win < 0 || win >= WINDOWS_NR)
>> +     if (win < 0 || win >= drv_data->nr_windows)
>>               return;
>>
>>       win_data = &ctx->win_data[win];
>>
>> +     /* DECON EXT sw-hw window mapping */
>> +     if (drv_data->decon_type == DECON_EXT) {
>> +             if (win == 0)
>> +                     win = 1;
>> +     }
>> +
>>       if (ctx->suspended) {
>>               /* do not resume this window*/
>>               win_data->resume = false;
>> @@ -609,10 +768,11 @@ static void decon_win_disable(struct exynos_drm_crtc *crtc, int zpos)
>>
>>  static void decon_window_suspend(struct decon_context *ctx)
>>  {
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       struct decon_win_data *win_data;
>>       int i;
>>
>> -     for (i = 0; i < WINDOWS_NR; i++) {
>> +     for (i = 0; i < drv_data->nr_windows; i++) {
>>               win_data = &ctx->win_data[i];
>>               win_data->resume = win_data->enabled;
>>               if (win_data->enabled)
>> @@ -622,10 +782,11 @@ static void decon_window_suspend(struct decon_context *ctx)
>>
>>  static void decon_window_resume(struct decon_context *ctx)
>>  {
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       struct decon_win_data *win_data;
>>       int i;
>>
>> -     for (i = 0; i < WINDOWS_NR; i++) {
>> +     for (i = 0; i < drv_data->nr_windows; i++) {
>>               win_data = &ctx->win_data[i];
>>               win_data->enabled = win_data->resume;
>>               win_data->resume = false;
>> @@ -634,10 +795,11 @@ static void decon_window_resume(struct decon_context *ctx)
>>
>>  static void decon_apply(struct decon_context *ctx)
>>  {
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       struct decon_win_data *win_data;
>>       int i;
>>
>> -     for (i = 0; i < WINDOWS_NR; i++) {
>> +     for (i = 0; i < drv_data->nr_windows; i++) {
>>               win_data = &ctx->win_data[i];
>>               if (win_data->enabled)
>>                       decon_win_commit(ctx->crtc, i);
>> @@ -650,6 +812,7 @@ static void decon_apply(struct decon_context *ctx)
>>
>>  static void decon_init(struct decon_context *ctx)
>>  {
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       u32 val;
>>
>>       writel(VIDCON0_SWRESET, ctx->regs + VIDCON0);
>> @@ -657,6 +820,14 @@ static void decon_init(struct decon_context *ctx)
>>       val = VIDOUTCON0_DISP_IF_0_ON;
>>       if (!ctx->i80_if)
>>               val |= VIDOUTCON0_RGBIF;
>> +
>> +     if (drv_data->decon_type == DECON_EXT) {
>> +             writel(TRIGCON_HWTRIG_AUTO_MASK |
>> +                     TRIGCON_HWTRIGMASK_DISPIF0 |
>> +                     TRIGCON_HWTRIGEN_I80_RGB, ctx->regs + TRIGCON);
>> +             val |= VIDOUTCON0_TV_MODE | VIDOUTCON0_I80IF;
>> +     }
>> +
>>       writel(val, ctx->regs + VIDOUTCON0);
>>
>>       writel(VCLKCON0_CLKVALUP | VCLKCON0_VCLKFREE, ctx->regs + VCLKCON0);
>> @@ -667,6 +838,7 @@ static void decon_init(struct decon_context *ctx)
>>
>>  static int decon_poweron(struct decon_context *ctx)
>>  {
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       int ret;
>>
>>       if (!ctx->suspended)
>> @@ -702,6 +874,9 @@ static int decon_poweron(struct decon_context *ctx)
>>
>>       decon_init(ctx);
>>
>> +     if (drv_data->decon_type == DECON_EXT)
>> +             writel(DECON_EXT_CH_MAPPING, ctx->regs + WINCHMAP0);
>> +
>>       /* if vblank was enabled status, enable it again. */
>>       if (test_and_clear_bit(0, &ctx->irq_flags)) {
>>               ret = decon_enable_vblank(ctx->crtc);
>> @@ -817,6 +992,7 @@ out:
>>  static int decon_bind(struct device *dev, struct device *master, void *data)
>>  {
>>       struct decon_context *ctx = dev_get_drvdata(dev);
>> +     struct decon_driver_data *drv_data = ctx->driver_data;
>>       struct drm_device *drm_dev = data;
>>       int ret;
>>
>> @@ -826,16 +1002,23 @@ static int decon_bind(struct device *dev, struct device *master, void *data)
>>               return ret;
>>       }
>>
>> -     ctx->crtc = exynos_drm_crtc_create(drm_dev, ctx->pipe,
>> +     if (drv_data->decon_type == DECON_INT)
>> +             ctx->crtc = exynos_drm_crtc_create(drm_dev, ctx->pipe,
>>                                          EXYNOS_DISPLAY_TYPE_LCD,
>>                                          &decon_crtc_ops, ctx);
>> +     else
>> +             ctx->crtc = exynos_drm_crtc_create(drm_dev, ctx->pipe,
>> +                                        EXYNOS_DISPLAY_TYPE_HDMI,
>> +                                        &decon_crtc_ops, ctx);
>> +
>>       if (IS_ERR(ctx->crtc)) {
>>               decon_ctx_remove(ctx);
>>               return PTR_ERR(ctx->crtc);
>>       }
>>
>> -     if (ctx->display)
>> -             exynos_drm_create_enc_conn(drm_dev, ctx->display);
>> +     if (drv_data->decon_type == DECON_INT)
>
> This check should not be necessary, ctx->display shall be set only in
> case of presence of parallel output.
Ok, this check is not necessarily needed!

Ajay
>> +             if (ctx->display)
>> +                     exynos_drm_create_enc_conn(drm_dev, ctx->display);
>>
>>       return 0;
>>
>> @@ -848,8 +1031,9 @@ static void decon_unbind(struct device *dev, struct device *master,
>>
>>       decon_dpms(ctx->crtc, DRM_MODE_DPMS_OFF);
>>
>> -     if (ctx->display)
>> -             exynos_dpi_remove(ctx->display);
>> +     if (ctx->driver_data->decon_type == DECON_INT)
>
> ditto
>
>
> Regards
> Andrzej
>
>> +             if (ctx->display)
>> +                     exynos_dpi_remove(ctx->display);
>>
>>       decon_ctx_remove(ctx);
>>  }
>> @@ -862,11 +1046,14 @@ static const struct component_ops decon_component_ops = {
>>  static int decon_probe(struct platform_device *pdev)
>>  {
>>       struct device *dev = &pdev->dev;
>> +     struct decon_driver_data *drv_data;
>>       struct decon_context *ctx;
>>       struct device_node *i80_if_timings;
>>       struct resource *res;
>>       int ret;
>>
>> +     drv_data = drm_decon_get_driver_data(pdev);
>> +
>>       if (!dev->of_node)
>>               return -ENODEV;
>>
>> @@ -874,11 +1061,17 @@ static int decon_probe(struct platform_device *pdev)
>>       if (!ctx)
>>               return -ENOMEM;
>>
>> -     ret = exynos_drm_component_add(dev, EXYNOS_DEVICE_TYPE_CRTC,
>> +     if (drv_data->decon_type == DECON_INT)
>> +             ret = exynos_drm_component_add(dev, EXYNOS_DEVICE_TYPE_CRTC,
>>                                       EXYNOS_DISPLAY_TYPE_LCD);
>> +     else
>> +             ret = exynos_drm_component_add(dev, EXYNOS_DEVICE_TYPE_CRTC,
>> +                                     EXYNOS_DISPLAY_TYPE_HDMI);
>> +
>>       if (ret)
>>               return ret;
>>
>> +     ctx->driver_data = drv_data;
>>       ctx->dev = dev;
>>       ctx->suspended = true;
>>
>> diff --git a/include/video/exynos7_decon.h b/include/video/exynos7_decon.h
>> index a62b11b..d02320b 100644
>> --- a/include/video/exynos7_decon.h
>> +++ b/include/video/exynos7_decon.h
>> @@ -20,6 +20,7 @@
>>  /* VIDOUTCON0 */
>>  #define VIDOUTCON0                           0x4
>>
>> +#define VIDOUTCON0_TV_MODE                   (0x1 << 26)
>>  #define VIDOUTCON0_DUAL_MASK                 (0x3 << 24)
>>  #define VIDOUTCON0_DUAL_ON                   (0x3 << 24)
>>  #define VIDOUTCON0_DISP_IF_1_ON                      (0x2 << 24)
>> @@ -52,6 +53,9 @@
>>
>>  #define SHADOWCON_WINx_PROTECT(_win)         (1 << (10 + (_win)))
>>
>> +/* WINCHMAP0 */
>> +#define WINCHMAP0                            0x40
>> +
>>  /* WINCONx */
>>  #define WINCON(_win)                         (0x50 + ((_win) * 4))
>>
>> @@ -328,6 +332,15 @@
>>  /* LINECNT OP THRSHOLD*/
>>  #define LINECNT_OP_THRESHOLD                 0x630
>>
>> +/* TRIGCON */
>> +#define TRIGCON                                      0x06B0
>> +#define TRIGCON_HWTRIG_AUTO_MASK             (1 << 6)
>> +#define TRIGCON_HWTRIGMASK_DISPIF1           (1 << 5)
>> +#define TRIGCON_HWTRIGMASK_DISPIF0           (1 << 4)
>> +#define TRIGCON_HWTRIGEN_I80_RGB             (1 << 3)
>> +#define TRIGCON_SWTRIGCMD_I80_RGB            (1 << 1)
>> +#define TRIGCON_SWTRIGEN_I80_RGB             (1 << 0)
>> +
>>  /* CRCCTRL */
>>  #define CRCCTRL                                      0x6C8
>>  #define CRCCTRL_CRCCLKEN                     (0x1 << 2)
>>
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux