From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?euc-kr?B?tOvAzrHi?= Subject: Re: Re: [PATCH v2 0/7] drm/exynos: more cleanup with super device support Date: Thu, 03 Apr 2014 09:05:30 +0000 (GMT) Message-ID: <22570394.148711396515929465.JavaMail.weblogic@epml12> Reply-To: inki.dae@samsung.com Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mailout1.samsung.com (mailout1.samsung.com [203.254.224.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 3A53F6E8A9 for ; Thu, 3 Apr 2014 02:05:59 -0700 (PDT) Received: from epcpsbgr1.samsung.com (u141.gpu120.samsung.co.kr [203.254.230.141]) by mailout1.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0N3G007RL6LXBHD0@mailout1.samsung.com> for dri-devel@lists.freedesktop.org; Thu, 03 Apr 2014 18:05:57 +0900 (KST) MIME-version: 1.0 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Andrzej Hajda , "airlied@linux.ie" , "dri-devel@lists.freedesktop.org" Cc: =?euc-kr?Q?=B9=DA=B0=E6=B9=CE?= List-Id: dri-devel@lists.freedesktop.org Hi Andrzej, > Hi Inki, > > On 04/01/2014 02:37 PM, Inki Dae wrote: >> This patch series cleans up exynos drm framework and kms sub drivers >> using the component framework[1]. > > I have tested it on trats platform (fimd + dsi), it works. > However it seems to be suboptimal. > In case of deferred probing of any of components (due to lack of some > resources, ex. clock, regulator, phy), all components > which were already bound are unbound and re-bound later when another > driver is added, this causes multiple unnecessary free/re-allocate cycles. The components bound and then unbound wouldn't be bound again because the components unbound are removed component_list like below, ret = try_to_bring_up_masters(component); if (ret < 0) { list_del(&component->node); kfree(component); } And I cannot see that components are freed and re-allocated repeatedly. See the below log, [ 1.425000] [drm] Initialized drm 1.1.0 20060810 [ 1.430000] exynos-drm-ipp exynos-drm-ipp: drm ipp registered successfully. [ 1.435000] exynos4-fb 11c00000.fimd: adding component (ops fimd_component_ops) [ 1.445000] exynos-dsi 11c80000.dsi: adding component (ops exynos_dsi_component_ops) [ 1.450000] [drm] Supports vblank timestamp caching Rev 2 (21.10.2013). [ 1.455000] [drm] No driver support for vblank timestamp query. [ 1.465000] exynos-drm display-subsystem.8: binding 11c00000.fimd (ops fimd_component_ops) [ 1.470000] exynos-drm display-subsystem.8: bound 11c00000.fimd (ops fimd_component_ops) [ 1.480000] exynos-drm display-subsystem.8: binding 11c80000.dsi (ops exynos_dsi_component_ops) [ 1.490000] exynos-drm display-subsystem.8: failed to bind 11c80000.dsi (ops exynos_dsi_component_ops): -12 [ 1.500000] exynos-drm display-subsystem.8: master bind failed: -12 [ 1.505000] exynos-dsi: probe of 11c80000.dsi failed with error -12 I have tested it on universal, trats, trats2 boards. And they all work fine. But there was one thing I missed. That is that dpi probe and remove shoudn't be called if fimd node has no port node. For this, I posted it again. Did you check it again? It seems your missing something. And Can you tell me more details how did you test it? If there is no any problem, I'm going to have pull request today or tomorrow. Thanks, Inki Dae > Maybe I miss something, but I guess registration list(s) + DT super-node > should perform better. > > Here is sample log from trats with mipi-phy driver delayed probe: > > binding 11c00000.fimd (ops fimd_component_ops) > bound 11c00000.fimd (ops fimd_component_ops) > binding 11c80000.dsi (ops exynos_dsi_component_ops) > failed to bind 11c80000.dsi (ops exynos_dsi_component_ops): -517 > unbinding 11c00000.fimd (ops fimd_component_ops) > master bind failed: -517 > binding 11c00000.fimd (ops fimd_component_ops) > bound 11c00000.fimd (ops fimd_component_ops) > binding 11c80000.dsi (ops exynos_dsi_component_ops) > failed to bind 11c80000.dsi (ops exynos_dsi_component_ops): -517 > unbinding 11c00000.fimd (ops fimd_component_ops) > master bind failed: -517 > binding 11c00000.fimd (ops fimd_component_ops) > bound 11c00000.fimd (ops fimd_component_ops) > binding 11c80000.dsi (ops exynos_dsi_component_ops) > failed to bind 11c80000.dsi (ops exynos_dsi_component_ops): -517 > unbinding 11c00000.fimd (ops fimd_component_ops) > master bind failed: -517 > binding 11c00000.fimd (ops fimd_component_ops) > bound 11c00000.fimd (ops fimd_component_ops) > binding 11c80000.dsi (ops exynos_dsi_component_ops) > failed to bind 11c80000.dsi (ops exynos_dsi_component_ops): -517 > unbinding 11c00000.fimd (ops fimd_component_ops) > master bind failed: -517 > binding 11c00000.fimd (ops fimd_component_ops) > bound 11c00000.fimd (ops fimd_component_ops) > binding 11c80000.dsi (ops exynos_dsi_component_ops) > bound 11c80000.dsi (ops exynos_dsi_component_ops) > > This log will grow with number of devices probed in-between. > > Regards > Andrzej > > > > > And this patch series had been posted for RFC[2]. > > > > Thanks, > > Inki Dae