From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 097D6C432C0 for ; Sun, 24 Nov 2019 13:28:24 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id DB8D52073F for ; Sun, 24 Nov 2019 13:28:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726744AbfKXN2X (ORCPT ); Sun, 24 Nov 2019 08:28:23 -0500 Received: from bhuna.collabora.co.uk ([46.235.227.227]:40978 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726740AbfKXN2X (ORCPT ); Sun, 24 Nov 2019 08:28:23 -0500 Received: from localhost (unknown [IPv6:2a01:e0a:2c:6930:5cf4:84a1:2763:fe0d]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: bbrezillon) by bhuna.collabora.co.uk (Postfix) with ESMTPSA id E77B128FFBF; Sun, 24 Nov 2019 13:28:20 +0000 (GMT) Date: Sun, 24 Nov 2019 14:28:18 +0100 From: Boris Brezillon To: Laurent Pinchart Cc: dri-devel@lists.freedesktop.org, Lucas Stach , Chris Healy , Andrey Smirnov , Nikita Yushchenko , kernel@collabora.com, Daniel Vetter , Inki Dae , Joonyoung Shim , Seung-Woo Kim , Kyungmin Park , Thierry Reding , Sam Ravnborg , Philipp Zabel , Rob Clark , Andrzej Hajda , Neil Armstrong , Jonas Karlman , Jernej Skrabec , Rob Herring , Mark Rutland , devicetree@vger.kernel.org Subject: Re: [PATCH v3 03/21] drm/exynos: Declare the DSI encoder as a bridge element Message-ID: <20191124142818.516dfac4@collabora.com> In-Reply-To: <20191124141727.45597a6e@collabora.com> References: <20191023154512.9762-1-boris.brezillon@collabora.com> <20191023154512.9762-4-boris.brezillon@collabora.com> <20191124102433.GD4727@pendragon.ideasonboard.com> <20191124141727.45597a6e@collabora.com> Organization: Collabora X-Mailer: Claws Mail 3.17.4 (GTK+ 2.24.32; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Sun, 24 Nov 2019 14:17:27 +0100 Boris Brezillon wrote: > > > @@ -1687,16 +1705,18 @@ static int exynos_dsi_bind(struct device *dev, struct device *master, > > > drm_encoder_init(drm_dev, encoder, &exynos_dsi_encoder_funcs, > > > DRM_MODE_ENCODER_TMDS, NULL); > > > > > > - drm_encoder_helper_add(encoder, &exynos_dsi_encoder_helper_funcs); > > > - > > > ret = exynos_drm_set_possible_crtcs(encoder, EXYNOS_DISPLAY_TYPE_LCD); > > > if (ret < 0) > > > return ret; > > > > > > + /* Declare ourself as the first bridge element. */ > > > + dsi->bridge.funcs = &exynos_dsi_bridge_funcs; > > > + drm_bridge_attach(encoder, &dsi->bridge, NULL); > > > + > > > if (dsi->in_bridge_node) { > > > in_bridge = of_drm_find_bridge(dsi->in_bridge_node); > > > if (in_bridge) > > > - drm_bridge_attach(encoder, in_bridge, NULL); > > > + drm_bridge_attach(encoder, in_bridge, &dsi->bridge); > > > } > > > > Same as for patch 01/21, maybe this could be moved to this bridge's > > attach operation ? Actually, now that I've read the code, this in_bridge > > part looks weird. Why would the DSI encoder have an input bridge that is > > has to manage itself ? > > Yes, I know, it doesn't make any sense. Either we're dealing with a > bridge which can be chained to other bridges (can be placed in the > middle of a chain as well), or we're dealing with an encoder which > precedes any bridges. In the latter case (which is how exynos_dsi is > implemented) in_bridge doesn't have any meaning, and that's even worse > since we're placing the so-called input bridge (AKA previous bridge) > after our encoder (that's what drm_bridge_attach(encoder, in_bridge, > NULL) does). More on that topic: I checked the exynos dts we have in mainline and no one is making use of the ports part of the exynos_dsim bindings, so maybe we should just deprecate it. The other option would be to patch the driver to act as a real bridge, but I can't do that without someone testing my changes and I didn't get much feedback from Exynos maintainers so far... From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Subject: Re: [PATCH v3 03/21] drm/exynos: Declare the DSI encoder as a bridge element Date: Sun, 24 Nov 2019 14:28:18 +0100 Message-ID: <20191124142818.516dfac4@collabora.com> References: <20191023154512.9762-1-boris.brezillon@collabora.com> <20191023154512.9762-4-boris.brezillon@collabora.com> <20191124102433.GD4727@pendragon.ideasonboard.com> <20191124141727.45597a6e@collabora.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from bhuna.collabora.co.uk (bhuna.collabora.co.uk [46.235.227.227]) by gabe.freedesktop.org (Postfix) with ESMTPS id CF4DA89C93 for ; Sun, 24 Nov 2019 13:28:22 +0000 (UTC) In-Reply-To: <20191124141727.45597a6e@collabora.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Laurent Pinchart Cc: Mark Rutland , Neil Armstrong , dri-devel@lists.freedesktop.org, Thierry Reding , kernel@collabora.com, Sam Ravnborg , Nikita Yushchenko , Andrey Smirnov , Kyungmin Park , Chris Healy , devicetree@vger.kernel.org, Jonas Karlman , Rob Herring , Jernej Skrabec , Seung-Woo Kim List-Id: dri-devel@lists.freedesktop.org T24gU3VuLCAyNCBOb3YgMjAxOSAxNDoxNzoyNyArMDEwMApCb3JpcyBCcmV6aWxsb24gPGJvcmlz LmJyZXppbGxvbkBjb2xsYWJvcmEuY29tPiB3cm90ZToKCj4gPiA+IEBAIC0xNjg3LDE2ICsxNzA1 LDE4IEBAIHN0YXRpYyBpbnQgZXh5bm9zX2RzaV9iaW5kKHN0cnVjdCBkZXZpY2UgKmRldiwgc3Ry dWN0IGRldmljZSAqbWFzdGVyLAo+ID4gPiAgCWRybV9lbmNvZGVyX2luaXQoZHJtX2RldiwgZW5j b2RlciwgJmV4eW5vc19kc2lfZW5jb2Rlcl9mdW5jcywKPiA+ID4gIAkJCSBEUk1fTU9ERV9FTkNP REVSX1RNRFMsIE5VTEwpOwo+ID4gPiAgCj4gPiA+IC0JZHJtX2VuY29kZXJfaGVscGVyX2FkZChl bmNvZGVyLCAmZXh5bm9zX2RzaV9lbmNvZGVyX2hlbHBlcl9mdW5jcyk7Cj4gPiA+IC0KPiA+ID4g IAlyZXQgPSBleHlub3NfZHJtX3NldF9wb3NzaWJsZV9jcnRjcyhlbmNvZGVyLCBFWFlOT1NfRElT UExBWV9UWVBFX0xDRCk7Cj4gPiA+ICAJaWYgKHJldCA8IDApCj4gPiA+ICAJCXJldHVybiByZXQ7 Cj4gPiA+ICAKPiA+ID4gKwkvKiBEZWNsYXJlIG91cnNlbGYgYXMgdGhlIGZpcnN0IGJyaWRnZSBl bGVtZW50LiAqLwo+ID4gPiArCWRzaS0+YnJpZGdlLmZ1bmNzID0gJmV4eW5vc19kc2lfYnJpZGdl X2Z1bmNzOwo+ID4gPiArCWRybV9icmlkZ2VfYXR0YWNoKGVuY29kZXIsICZkc2ktPmJyaWRnZSwg TlVMTCk7Cj4gPiA+ICsKPiA+ID4gIAlpZiAoZHNpLT5pbl9icmlkZ2Vfbm9kZSkgewo+ID4gPiAg CQlpbl9icmlkZ2UgPSBvZl9kcm1fZmluZF9icmlkZ2UoZHNpLT5pbl9icmlkZ2Vfbm9kZSk7Cj4g PiA+ICAJCWlmIChpbl9icmlkZ2UpCj4gPiA+IC0JCQlkcm1fYnJpZGdlX2F0dGFjaChlbmNvZGVy LCBpbl9icmlkZ2UsIE5VTEwpOwo+ID4gPiArCQkJZHJtX2JyaWRnZV9hdHRhY2goZW5jb2Rlciwg aW5fYnJpZGdlLCAmZHNpLT5icmlkZ2UpOwo+ID4gPiAgCX0gIAo+ID4gCj4gPiBTYW1lIGFzIGZv ciBwYXRjaCAwMS8yMSwgbWF5YmUgdGhpcyBjb3VsZCBiZSBtb3ZlZCB0byB0aGlzIGJyaWRnZSdz Cj4gPiBhdHRhY2ggb3BlcmF0aW9uID8gQWN0dWFsbHksIG5vdyB0aGF0IEkndmUgcmVhZCB0aGUg Y29kZSwgdGhpcyBpbl9icmlkZ2UKPiA+IHBhcnQgbG9va3Mgd2VpcmQuIFdoeSB3b3VsZCB0aGUg RFNJIGVuY29kZXIgaGF2ZSBhbiBpbnB1dCBicmlkZ2UgdGhhdCBpcwo+ID4gaGFzIHRvIG1hbmFn ZSBpdHNlbGYgPwo+IAo+IFllcywgSSBrbm93LCBpdCBkb2Vzbid0IG1ha2UgYW55IHNlbnNlLiBF aXRoZXIgd2UncmUgZGVhbGluZyB3aXRoIGEKPiBicmlkZ2Ugd2hpY2ggY2FuIGJlIGNoYWluZWQg dG8gb3RoZXIgYnJpZGdlcyAoY2FuIGJlIHBsYWNlZCBpbiB0aGUKPiBtaWRkbGUgb2YgYSBjaGFp biBhcyB3ZWxsKSwgb3Igd2UncmUgZGVhbGluZyB3aXRoIGFuIGVuY29kZXIgd2hpY2gKPiBwcmVj ZWRlcyBhbnkgYnJpZGdlcy4gSW4gdGhlIGxhdHRlciBjYXNlICh3aGljaCBpcyBob3cgZXh5bm9z X2RzaSBpcwo+IGltcGxlbWVudGVkKSBpbl9icmlkZ2UgZG9lc24ndCBoYXZlIGFueSBtZWFuaW5n LCBhbmQgdGhhdCdzIGV2ZW4gd29yc2UKPiBzaW5jZSB3ZSdyZSBwbGFjaW5nIHRoZSBzby1jYWxs ZWQgaW5wdXQgYnJpZGdlIChBS0EgcHJldmlvdXMgYnJpZGdlKQo+IGFmdGVyIG91ciBlbmNvZGVy ICh0aGF0J3Mgd2hhdCBkcm1fYnJpZGdlX2F0dGFjaChlbmNvZGVyLCBpbl9icmlkZ2UsCj4gTlVM TCkgZG9lcykuCgpNb3JlIG9uIHRoYXQgdG9waWM6IEkgY2hlY2tlZCB0aGUgZXh5bm9zIGR0cyB3 ZSBoYXZlIGluIG1haW5saW5lIGFuZApubyBvbmUgaXMgbWFraW5nIHVzZSBvZiB0aGUgcG9ydHMg cGFydCBvZiB0aGUgZXh5bm9zX2RzaW0gYmluZGluZ3MsIHNvCm1heWJlIHdlIHNob3VsZCBqdXN0 IGRlcHJlY2F0ZSBpdC4gVGhlIG90aGVyIG9wdGlvbiB3b3VsZCBiZSB0byBwYXRjaAp0aGUgZHJp dmVyIHRvIGFjdCBhcyBhIHJlYWwgYnJpZGdlLCBidXQgSSBjYW4ndCBkbyB0aGF0IHdpdGhvdXQg c29tZW9uZQp0ZXN0aW5nIG15IGNoYW5nZXMgYW5kIEkgZGlkbid0IGdldCBtdWNoIGZlZWRiYWNr IGZyb20gRXh5bm9zCm1haW50YWluZXJzIHNvIGZhci4uLgpfX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZl bEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFp bG1hbi9saXN0aW5mby9kcmktZGV2ZWw=