From patchwork Thu Jan 9 15:25:38 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shawn Guo X-Patchwork-Id: 23060 Return-Path: X-Original-To: linaro@patches.linaro.org Delivered-To: linaro@patches.linaro.org Received: from mail-pb0-f69.google.com (mail-pb0-f69.google.com [209.85.160.69]) by ip-10-151-82-157.ec2.internal (Postfix) with ESMTPS id 9FC75202E2 for ; Thu, 9 Jan 2014 15:26:30 +0000 (UTC) Received: by mail-pb0-f69.google.com with SMTP id md12sf7355522pbc.4 for ; Thu, 09 Jan 2014 07:26:29 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:delivered-to:date:from:to:subject:message-id :references:mime-version:in-reply-to:user-agent:cc:precedence :list-id:list-unsubscribe:list-archive:list-post:list-help :list-subscribe:sender:errors-to:x-original-sender :x-original-authentication-results:mailing-list:content-disposition :content-type:content-transfer-encoding; bh=BpHbKoFxyDEvmGSGksq8C0Zzf4FqpwfEPDOE6weQn1E=; b=ZltVv8lOLNK85O0rGJpW1svbWQjWXT4DNlJv4AjqMm3eMvWpq+KZ7yEoakPV4eDNgS PJ6G6cBo8Jpmv0+erFS246W1c1SSRkoxlaBVazg1eSQfJi9c+zPrFtSt6eGNyYnRF7fb 0ykExBvDqmCKllXGLthndVHGZ9/B7A+t5Nnqn/9pQ7NhhNT7cGeHGujhbdchw7czgdgs /aWcOocLXUa68ks2Tzbb2MXs0+bNGw0QX1n1z9PyYRnW8XEoAGXNAy6CIMgvNkdqPUsC zeKmPPJdluzp9U8gbC4ln8QPP9U+M3jpdJUzZANDVNiiR+o3W9VieW5okNyBAbfVTzgN SeSg== X-Gm-Message-State: ALoCoQn9tbgGR/t0lqUazKvfTYqAVCsfnH8JDs1kbl1A0hmsGUiFBlNUWCaBCBAvRGP95gHJcQ8e X-Received: by 10.66.218.70 with SMTP id pe6mr1366879pac.33.1389281189771; Thu, 09 Jan 2014 07:26:29 -0800 (PST) X-BeenThere: patchwork-forward@linaro.org Received: by 10.49.86.232 with SMTP id s8ls1112589qez.67.gmail; Thu, 09 Jan 2014 07:26:29 -0800 (PST) X-Received: by 10.52.117.176 with SMTP id kf16mr2604256vdb.6.1389281189558; Thu, 09 Jan 2014 07:26:29 -0800 (PST) Received: from mail-vb0-f50.google.com (mail-vb0-f50.google.com [209.85.212.50]) by mx.google.com with ESMTPS id sz9si2995515vdc.6.2014.01.09.07.26.29 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Thu, 09 Jan 2014 07:26:29 -0800 (PST) Received-SPF: neutral (google.com: 209.85.212.50 is neither permitted nor denied by best guess record for domain of patch+caf_=patchwork-forward=linaro.org@linaro.org) client-ip=209.85.212.50; Received: by mail-vb0-f50.google.com with SMTP id w18so2271254vbj.23 for ; Thu, 09 Jan 2014 07:26:29 -0800 (PST) X-Received: by 10.52.227.233 with SMTP id sd9mr2096210vdc.53.1389281189418; Thu, 09 Jan 2014 07:26:29 -0800 (PST) X-Forwarded-To: patchwork-forward@linaro.org X-Forwarded-For: patch@linaro.org patchwork-forward@linaro.org Delivered-To: patch@linaro.org Received: by 10.59.13.131 with SMTP id ey3csp19758ved; Thu, 9 Jan 2014 07:26:28 -0800 (PST) X-Received: by 10.180.95.162 with SMTP id dl2mr3821480wib.17.1389281188373; Thu, 09 Jan 2014 07:26:28 -0800 (PST) Received: from casper.infradead.org (casper.infradead.org. [2001:770:15f::2]) by mx.google.com with ESMTPS id cs3si1526794wjc.60.2014.01.09.07.26.28 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 09 Jan 2014 07:26:28 -0800 (PST) Received-SPF: pass (google.com: domain of linux-arm-kernel-bounces+patch=linaro.org@lists.infradead.org designates 2001:770:15f::2 as permitted sender) client-ip=2001:770:15f::2; Received: from merlin.infradead.org ([2001:4978:20e::2]) by casper.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1W1HUE-00064j-5k; Thu, 09 Jan 2014 15:25:34 +0000 Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1W1HU8-0000kk-81; Thu, 09 Jan 2014 15:25:28 +0000 Received: from va3ehsobe006.messaging.microsoft.com ([216.32.180.16] helo=va3outboundpool.messaging.microsoft.com) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1W1HTs-0000iu-SN for linux-arm-kernel@lists.infradead.org; Thu, 09 Jan 2014 15:25:14 +0000 Received: from mail64-va3-R.bigfish.com (10.7.14.253) by VA3EHSOBE008.bigfish.com (10.7.40.28) with Microsoft SMTP Server id 14.1.225.22; Thu, 9 Jan 2014 15:24:51 +0000 Received: from mail64-va3 (localhost [127.0.0.1]) by mail64-va3-R.bigfish.com (Postfix) with ESMTP id 55C5840013D; Thu, 9 Jan 2014 15:24:51 +0000 (UTC) X-Forefront-Antispam-Report: CIP:70.37.183.190; KIP:(null); UIP:(null); IPV:NLI; H:mail.freescale.net; RD:none; EFVD:NLI X-SpamScore: -1 X-BigFish: VS-1(z579eh551biz98dI1432Izz1f42h2148h208ch1ee6h1de0h1fdah2073h2146h1202h1e76h2189h1d1ah1d2ah1fc6hzz1de098h1de097hz2dh87h2a8h839h944hd25hf0ah1220h1288h12a5h12a9h12bdh137ah13b6h1441h1504h1537h153bh162dh1631h1758h18e1h1946h19b5h1ad9h1b0ah1b2fh2222h224fh1fb3h1d0ch1d2eh1d3fh1dfeh1dffh1fe8h1ff5h209eh2216h22d0h2336h2438h1151h1155h) X-FB-DOMAIN-IP-MATCH: fail Received: from mail64-va3 (localhost.localdomain [127.0.0.1]) by mail64-va3 (MessageSwitch) id 1389281089200400_14299; Thu, 9 Jan 2014 15:24:49 +0000 (UTC) Received: from VA3EHSMHS032.bigfish.com (unknown [10.7.14.238]) by mail64-va3.bigfish.com (Postfix) with ESMTP id 2091F36004B; Thu, 9 Jan 2014 15:24:49 +0000 (UTC) Received: from mail.freescale.net (70.37.183.190) by VA3EHSMHS032.bigfish.com (10.7.99.42) with Microsoft SMTP Server (TLS) id 14.16.227.3; Thu, 9 Jan 2014 15:24:46 +0000 Received: from az84smr01.freescale.net (10.64.34.197) by 039-SN1MMR1-004.039d.mgd.msft.net (10.84.1.14) with Microsoft SMTP Server (TLS) id 14.3.158.2; Thu, 9 Jan 2014 15:24:45 +0000 Received: from S2101-09.ap.freescale.net ([10.192.185.4]) by az84smr01.freescale.net (8.14.3/8.14.0) with ESMTP id s09FOWdh007198; Thu, 9 Jan 2014 08:24:37 -0700 Date: Thu, 9 Jan 2014 23:25:38 +0800 From: Shawn Guo To: Russell King - ARM Linux Subject: Re: [PATCH RFC 27/46] imx-drm: convert to componentised device support Message-ID: <20140109152536.GG21717@S2101-09.ap.freescale.net> References: <20140102212528.GD7383@n2100.arm.linux.org.uk> <20140107085930.GD12346@S2101-09.ap.freescale.net> <20140108213258.GS27432@n2100.arm.linux.org.uk> MIME-Version: 1.0 In-Reply-To: <20140108213258.GS27432@n2100.arm.linux.org.uk> User-Agent: Mutt/1.5.21 (2010-09-15) X-OriginatorOrg: sigmatel.com X-FOPE-CONNECTOR: Id%0$Dn%*$RO%0$TLS%0$FQDN%$TlsDn% X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20140109_102513_011712_0F8F8DAE X-CRM114-Status: GOOD ( 21.03 ) X-Spam-Score: -3.5 (---) X-Spam-Report: SpamAssassin version 3.3.2 on merlin.infradead.org summary: Content analysis details: (-3.5 points) pts rule name description ---- ---------------------- -------------------------------------------------- -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/, medium trust [216.32.180.16 listed in list.dnswl.org] 0.7 SPF_SOFTFAIL SPF: sender does not match SPF record (softfail) -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Cc: devel@driverdev.osuosl.org, David Airlie , Greg Kroah-Hartman , dri-devel@lists.freedesktop.org, Sascha Hauer , linux-arm-kernel@lists.infradead.org X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: , List-Help: , List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patch=linaro.org@lists.infradead.org X-Removed-Original-Auth: Dkim didn't pass. X-Original-Sender: shawn.guo@linaro.org X-Original-Authentication-Results: mx.google.com; spf=neutral (google.com: 209.85.212.50 is neither permitted nor denied by best guess record for domain of patch+caf_=patchwork-forward=linaro.org@linaro.org) smtp.mail=patch+caf_=patchwork-forward=linaro.org@linaro.org Mailing-list: list patchwork-forward@linaro.org; contact patchwork-forward+owners@linaro.org X-Google-Group-Id: 836684582541 Content-Disposition: inline On Wed, Jan 08, 2014 at 09:32:58PM +0000, Russell King - ARM Linux wrote: > On Tue, Jan 07, 2014 at 04:59:35PM +0800, Shawn Guo wrote: > > On Thu, Jan 02, 2014 at 09:28:03PM +0000, Russell King wrote: > > > diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi > > > index e75e11b36dff..0e005f21d241 100644 > > > --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi > > > +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi > > > @@ -62,6 +62,12 @@ > > > }; > > > }; > > > > > > + imx-drm { > > > + compatible = "fsl,imx-drm"; > > > + crtcs = <&ipu1 0>, <&ipu1 1>; > > > + connectors = <&ldb>; > > > + }; > > > + > > > > While the change works fine on imx6dl, it breaks LVDS support on imx6q > > right away. > > > > imx-ipuv3 2400000.ipu: IPUv3H probed > > imx-ipuv3 2800000.ipu: IPUv3H probed > > [drm] Supports vblank timestamp caching Rev 2 (21.10.2013). > > [drm] No driver support for vblank timestamp query. > > imx-drm imx-drm.16: bound imx-ipuv3-crtc.0 (ops ipu_crtc_ops) > > imx-drm imx-drm.16: bound imx-ipuv3-crtc.1 (ops ipu_crtc_ops) > > imx-drm imx-drm.16: failed to bind ldb.10 (ops imx_ldb_ops): -517 > > > > Because we have 4 crtcs for lvds-channel on imx6q while imx-drm master > > defines only 2 in there, the imx_drm_encoder_parse_of() call from > > imx_ldb_register() will always return -EPROBE_DEFER. > > > > lvds-channel@0 { > > crtcs = <&ipu1 0>, <&ipu1 1>, <&ipu2 0>, <&ipu2 1>; > > }; > > > > lvds-channel@1 { > > crtcs = <&ipu1 0>, <&ipu1 1>, <&ipu2 0>, <&ipu2 1>; > > }; > > This is why some help would be useful here - I think I got these right > but I've no way to check them. > > Can you confirm that adding all four is the right thing not only for > the imx6q but also the imx6dl sabresd please? Yea, adding all four into imx-drm crtcs works for imx6q, but it doesn't for imx6dl, because &ipu2 is unavailable for imx6dl at all. Here is how I get around it. ---8<----------- diff --git a/arch/arm/boot/dts/imx6q-sabresd.dts b/arch/arm/boot/dts/imx6q-sabresd.dts index 9cbdfe7..66f220a 100644 --- a/arch/arm/boot/dts/imx6q-sabresd.dts +++ b/arch/arm/boot/dts/imx6q-sabresd.dts @@ -20,6 +20,10 @@ compatible = "fsl,imx6q-sabresd", "fsl,imx6q"; }; +&imx_drm { + crtcs = <&ipu1 0>, <&ipu1 1>, <&ipu2 0>, <&ipu2 1>; +}; + &sata { status = "okay"; }; diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi index 0e005f2..dfca3e0 100644 --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi @@ -62,7 +62,7 @@ }; }; - imx-drm { + imx_drm: imx-drm { compatible = "fsl,imx-drm"; crtcs = <&ipu1 0>, <&ipu1 1>; connectors = <&ldb>;