From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from galahad.ideasonboard.com ([185.26.127.97]:34699 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756495AbcK2U4y (ORCPT ); Tue, 29 Nov 2016 15:56:54 -0500 From: Laurent Pinchart To: Daniel Vetter Cc: Laurent Pinchart , dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org Subject: Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver Date: Tue, 29 Nov 2016 22:57:07 +0200 Message-ID: <4516476.dJ1bJMDj4s@avalon> In-Reply-To: <20161129095409.36di7nm3wnxns4qp@phenom.ffwll.local> References: <1480410283-28698-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <1480410283-28698-7-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <20161129095409.36di7nm3wnxns4qp@phenom.ffwll.local> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: Hi Daniel, On Tuesday 29 Nov 2016 10:54:09 Daniel Vetter wrote: > On Tue, Nov 29, 2016 at 11:04:36AM +0200, Laurent Pinchart wrote: > > The LVDS encoder driver is a DRM bridge driver that supports the > > parallel to LVDS encoders that don't require any configuration. The > > driver thus doesn't interact with the device, but creates an LVDS > > connector for the panel and exposes its size and timing based on > > information retrieved from DT. > > > > Signed-off-by: Laurent Pinchart > > > > Since it's 100% dummy, why put LVDS into the name? This little thing here > could be our generic "wrap drm_panel and attach it to a chain" helper. So > what about calling this _The_ drm_panel_bridge, and also linking it into > docs to feature it a bit more prominently. I'm not opposed to that, except that this driver should not be considered as just a helper to link a panel. It should only be used to support a real hardware bridge that requires no control. > I came up with this because I spotted some refactoring belows for building > this helper, until I realized that this driver _is_ the helper I think we > want ;-) Only thing missing is an exported function to instantiate a > bridge with just a drm_panel as the parameter. And putting it into the > drm_kms_helper.ko module. What would that be used for ? The bridge should be instantiated by this bridge driver, bound to a bridge device instantiated from DT (or the platform firmware of your choice). > > +static enum drm_connector_status > > +lvds_connector_detect(struct drm_connector *connector, bool force) > > +{ > > + return connector_status_connected; > > +} > > We have piles of this exact dummy callback all over, maybe make it the > default and rip them all out? Done, "[PATCH] drm: Make the connector .detect() callback optional". -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver Date: Tue, 29 Nov 2016 22:57:07 +0200 Message-ID: <4516476.dJ1bJMDj4s@avalon> References: <1480410283-28698-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <1480410283-28698-7-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <20161129095409.36di7nm3wnxns4qp@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from galahad.ideasonboard.com (galahad.ideasonboard.com [185.26.127.97]) by gabe.freedesktop.org (Postfix) with ESMTPS id 7904B6E74B for ; Tue, 29 Nov 2016 20:56:53 +0000 (UTC) In-Reply-To: <20161129095409.36di7nm3wnxns4qp@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: linux-renesas-soc@vger.kernel.org, Laurent Pinchart , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org SGkgRGFuaWVsLAoKT24gVHVlc2RheSAyOSBOb3YgMjAxNiAxMDo1NDowOSBEYW5pZWwgVmV0dGVy IHdyb3RlOgo+IE9uIFR1ZSwgTm92IDI5LCAyMDE2IGF0IDExOjA0OjM2QU0gKzAyMDAsIExhdXJl bnQgUGluY2hhcnQgd3JvdGU6Cj4gPiBUaGUgTFZEUyBlbmNvZGVyIGRyaXZlciBpcyBhIERSTSBi cmlkZ2UgZHJpdmVyIHRoYXQgc3VwcG9ydHMgdGhlCj4gPiBwYXJhbGxlbCB0byBMVkRTIGVuY29k ZXJzIHRoYXQgZG9uJ3QgcmVxdWlyZSBhbnkgY29uZmlndXJhdGlvbi4gVGhlCj4gPiBkcml2ZXIg dGh1cyBkb2Vzbid0IGludGVyYWN0IHdpdGggdGhlIGRldmljZSwgYnV0IGNyZWF0ZXMgYW4gTFZE Uwo+ID4gY29ubmVjdG9yIGZvciB0aGUgcGFuZWwgYW5kIGV4cG9zZXMgaXRzIHNpemUgYW5kIHRp bWluZyBiYXNlZCBvbgo+ID4gaW5mb3JtYXRpb24gcmV0cmlldmVkIGZyb20gRFQuCj4gPiAKPiA+ IFNpZ25lZC1vZmYtYnk6IExhdXJlbnQgUGluY2hhcnQKPiA+IDxsYXVyZW50LnBpbmNoYXJ0K3Jl bmVzYXNAaWRlYXNvbmJvYXJkLmNvbT4KPiAKPiBTaW5jZSBpdCdzIDEwMCUgZHVtbXksIHdoeSBw dXQgTFZEUyBpbnRvIHRoZSBuYW1lPyBUaGlzIGxpdHRsZSB0aGluZyBoZXJlCj4gY291bGQgYmUg b3VyIGdlbmVyaWMgIndyYXAgZHJtX3BhbmVsIGFuZCBhdHRhY2ggaXQgdG8gYSBjaGFpbiIgaGVs cGVyLiBTbwo+IHdoYXQgYWJvdXQgY2FsbGluZyB0aGlzIF9UaGVfIGRybV9wYW5lbF9icmlkZ2Us IGFuZCBhbHNvIGxpbmtpbmcgaXQgaW50bwo+IGRvY3MgdG8gZmVhdHVyZSBpdCBhIGJpdCBtb3Jl IHByb21pbmVudGx5LgoKSSdtIG5vdCBvcHBvc2VkIHRvIHRoYXQsIGV4Y2VwdCB0aGF0IHRoaXMg ZHJpdmVyIHNob3VsZCBub3QgYmUgY29uc2lkZXJlZCBhcyAKanVzdCBhIGhlbHBlciB0byBsaW5r IGEgcGFuZWwuIEl0IHNob3VsZCBvbmx5IGJlIHVzZWQgdG8gc3VwcG9ydCBhIHJlYWwgCmhhcmR3 YXJlIGJyaWRnZSB0aGF0IHJlcXVpcmVzIG5vIGNvbnRyb2wuCgo+IEkgY2FtZSB1cCB3aXRoIHRo aXMgYmVjYXVzZSBJIHNwb3R0ZWQgc29tZSByZWZhY3RvcmluZyBiZWxvd3MgZm9yIGJ1aWxkaW5n Cj4gdGhpcyBoZWxwZXIsIHVudGlsIEkgcmVhbGl6ZWQgdGhhdCB0aGlzIGRyaXZlciBfaXNfIHRo ZSBoZWxwZXIgSSB0aGluayB3ZQo+IHdhbnQgOy0pIE9ubHkgdGhpbmcgbWlzc2luZyBpcyBhbiBl eHBvcnRlZCBmdW5jdGlvbiB0byBpbnN0YW50aWF0ZSBhCj4gYnJpZGdlIHdpdGgganVzdCBhIGRy bV9wYW5lbCBhcyB0aGUgcGFyYW1ldGVyLiBBbmQgcHV0dGluZyBpdCBpbnRvIHRoZQo+IGRybV9r bXNfaGVscGVyLmtvIG1vZHVsZS4KCldoYXQgd291bGQgdGhhdCBiZSB1c2VkIGZvciA/IFRoZSBi cmlkZ2Ugc2hvdWxkIGJlIGluc3RhbnRpYXRlZCBieSB0aGlzIGJyaWRnZSAKZHJpdmVyLCBib3Vu ZCB0byBhIGJyaWRnZSBkZXZpY2UgaW5zdGFudGlhdGVkIGZyb20gRFQgKG9yIHRoZSBwbGF0Zm9y bSAKZmlybXdhcmUgb2YgeW91ciBjaG9pY2UpLgoKPiA+ICtzdGF0aWMgZW51bSBkcm1fY29ubmVj dG9yX3N0YXR1cwo+ID4gK2x2ZHNfY29ubmVjdG9yX2RldGVjdChzdHJ1Y3QgZHJtX2Nvbm5lY3Rv ciAqY29ubmVjdG9yLCBib29sIGZvcmNlKQo+ID4gK3sKPiA+ICsJcmV0dXJuIGNvbm5lY3Rvcl9z dGF0dXNfY29ubmVjdGVkOwo+ID4gK30KPiAKPiBXZSBoYXZlIHBpbGVzIG9mIHRoaXMgZXhhY3Qg ZHVtbXkgY2FsbGJhY2sgYWxsIG92ZXIsIG1heWJlIG1ha2UgaXQgdGhlCj4gZGVmYXVsdCBhbmQg cmlwIHRoZW0gYWxsIG91dD8KCkRvbmUsICJbUEFUQ0hdIGRybTogTWFrZSB0aGUgY29ubmVjdG9y IC5kZXRlY3QoKSBjYWxsYmFjayBvcHRpb25hbCIuCgotLSAKUmVnYXJkcywKCkxhdXJlbnQgUGlu Y2hhcnQKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRy aS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRw czovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=