From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joonyoung Shim Subject: Re: [PATCH] drm/exynos: fimd: Get signal polarities from device tree Date: Fri, 07 Jun 2013 09:27:06 +0900 Message-ID: <51B128DA.9030306@samsung.com> References: <1367435169-15025-1-git-send-email-tomasz.figa@gmail.com> <2077448.BOZFWf0KVq@flatron> <51B01079.6060701@samsung.com> <7834763.hzyJ5Gt3Ez@flatron> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mailout3.samsung.com ([203.254.224.33]:18282 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751848Ab3FGA1N (ORCPT ); Thu, 6 Jun 2013 20:27:13 -0400 Received: from epcpsbgr1.samsung.com (u141.gpu120.samsung.co.kr [203.254.230.141]) by mailout3.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0MNZ00AVWYLBEW20@mailout3.samsung.com> for linux-samsung-soc@vger.kernel.org; Fri, 07 Jun 2013 09:27:11 +0900 (KST) In-reply-to: <7834763.hzyJ5Gt3Ez@flatron> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Tomasz Figa Cc: Daniel Vetter , linux-samsung-soc@vger.kernel.org, dri-devel@lists.freedesktop.org, =?UTF-8?B?IifrjIDsnbjquLAvTW9iaWxlIFMvVw==?= =?UTF-8?B?IFBsYXRmb3JtIExhYi4o7Ya17Iug7JewKS9FMyjsgqzsm5ApL+yCvOyEseyghOyekA==?= =?UTF-8?B?JyI=?= , Tomasz Figa On 06/06/2013 06:47 PM, Tomasz Figa wrote: > Hi Joonyoung, > > On Thursday 06 of June 2013 13:30:49 Joonyoung Shim wrote: >> On 05/19/2013 08:32 PM, Tomasz Figa wrote: >>> Hi, >>> >>> On Wednesday 01 of May 2013 22:00:25 Daniel Vetter wrote: >>>> On Wed, May 01, 2013 at 09:06:09PM +0200, Tomasz Figa wrote: >>>>> This patch modifies the driver to perform two stage parsing of video >>>>> timings from device tree, to get timing information as struct >>>>> videomode, which contains more data than struct fb_videomode. >>>>> >>>>> Thanks to this change, information about polarity of control signals >>>>> (VSYNC, HSYNC, VDEN, VCLK) can be retrieved, in addition to standard >>>>> video timings. >>>>> >>>>> Signed-off-by: Tomasz Figa >>>> Since the drm mode struct also contains flags for sync polarity ... >>>> why >>>> is there no direct of -> drm_mode function? Going through an fb >>>> videomode in a kms drm driver looks _really_ backwards to me. >>>> >>>> Cc'in Dave for the fun of it ;-) >>> Struct fb_videomode is what exynos_drm_fimd driver uses internally. >>> Sure it should use drm_mode, but this is not really related to this >>> patch, because the code added in this patch only fills in the pdata >>> struct, which for compatibility reasons (the same structure is used >>> for both fbdev and drm drivers) contains struct fb_videomode. >>> >>> OK, now after having a bit of fun, could we merge this patch to at >>> least have usable support of parallel displays using this driver? >> I think it's better to use struct display_timings instead of struct >> fb_videomode in exynos_drm. > I agree that fb_videomode struct is a bit unfortunate here, but it seems > to be widely used in the exynos_drm_fimd driver. > > Actually, if I understood it properly, the correct struct to use in DRM > drivers is drm_display_mode, but there is no conversion function from > struct display_timing to it. It's easy to convert from struct display_timing to videomode and there is conversion function from struct videomode to drm_display_mode. Already it uses at drivers/gpu/drm/tilcdc/tilcdc_panel.c > > IMHO a separate patch introducing such conversion and then another one > which modifies the driver to use drm_display_mode everywhere would be the > best solution. OK, but the struct drm_display_mode doesn't include only timing information so we need also to use struct videomode or struct drm_timing properly.