[PATCH 8/9] drm/exynos/fimd: use polarization flags provided by drm_display_mode

Andrzej Hajda a.hajda at samsung.com
Thu Mar 20 00:01:41 PDT 2014


On 03/20/2014 07:03 AM, Inki Dae wrote:
> Thanks for your contributions,
> 
> 
> 2014-03-17 19:27 GMT+09:00 Andrzej Hajda <a.hajda at samsung.com>:
>> The patch replaces fimd private bindings for signal polarization by
>> polarization flags provided by drm_display_mode.
>>
> 
> This patch needs below patch not merged yet,
>      drm: drm_display_mode: add signal polarity flags
> 
> So let me pick up all of your path series except for patch 7 and 8.
> And for these two patches, later.
> 
> Your opinion?

Yes, please do it.

Regards
Andrzej

> 
> Thanks,
> Inki Dae
> 
>> Signed-off-by: Andrzej Hajda <a.hajda at samsung.com>
>> ---
>>  drivers/gpu/drm/exynos/exynos_drm_fimd.c | 12 +++++-------
>>  1 file changed, 5 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
>> index 15d6b37..dbfad4e 100644
>> --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
>> +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
>> @@ -115,7 +115,6 @@ struct fimd_context {
>>         unsigned int                    default_win;
>>         unsigned long                   irq_flags;
>>         u32                             vidcon0;
>> -       u32                             vidcon1;
>>         bool                            suspended;
>>         int                             pipe;
>>         wait_queue_head_t               wait_vsync_queue;
>> @@ -232,7 +231,11 @@ static void fimd_commit(struct exynos_drm_manager *mgr)
>>                 return;
>>
>>         /* setup polarity values */
>> -       vidcon1 = ctx->vidcon1;
>> +       vidcon1 = 0;
>> +       if (mode->pol_flags & DRM_MODE_FLAG_POL_DE_NEGEDGE)
>> +               vidcon1 |= VIDCON1_INV_VDEN;
>> +       if (mode->pol_flags & DRM_MODE_FLAG_POL_PIXDATA_NEGEDGE)
>> +               vidcon1 |= VIDCON1_INV_VCLK;
>>         if (mode->flags & DRM_MODE_FLAG_NVSYNC)
>>                 vidcon1 |= VIDCON1_INV_VSYNC;
>>         if (mode->flags & DRM_MODE_FLAG_NHSYNC)
>> @@ -875,11 +878,6 @@ static int fimd_probe(struct platform_device *pdev)
>>         ctx->dev = dev;
>>         ctx->suspended = true;
>>
>> -       if (of_property_read_bool(dev->of_node, "samsung,invert-vden"))
>> -               ctx->vidcon1 |= VIDCON1_INV_VDEN;
>> -       if (of_property_read_bool(dev->of_node, "samsung,invert-vclk"))
>> -               ctx->vidcon1 |= VIDCON1_INV_VCLK;
>> -
>>         ctx->bus_clk = devm_clk_get(dev, "fimd");
>>         if (IS_ERR(ctx->bus_clk)) {
>>                 dev_err(dev, "failed to get bus clock\n");
>> --
>> 1.8.3.2
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel



More information about the dri-devel mailing list