From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751485AbeDGJ4z (ORCPT ); Sat, 7 Apr 2018 05:56:55 -0400 Received: from galahad.ideasonboard.com ([185.26.127.97]:40791 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751093AbeDGJ4u (ORCPT ); Sat, 7 Apr 2018 05:56:50 -0400 From: Laurent Pinchart To: jacopo mondi Cc: Jacopo Mondi , architt@codeaurora.org, a.hajda@samsung.com, airlied@linux.ie, vladimir_zapolskiy@mentor.com, horms@verge.net.au, magnus.damm@gmail.com, geert@linux-m68k.org, niklas.soderlund@ragnatech.se, sergei.shtylyov@cogentembedded.com, robh+dt@kernel.org, mark.rutland@arm.com, dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Mark Brown Subject: Re: [PATCH v7 1/2] dt-bindings: display: bridge: Document THC63LVD1024 LVDS decoder Date: Sat, 07 Apr 2018 12:56:48 +0300 Message-ID: <2813569.TyIUb2mkfU@avalon> Organization: Ideas on Board Oy In-Reply-To: <20180407094556.GQ20945@w540> References: <1523018517-24121-1-git-send-email-jacopo+renesas@jmondi.org> <2441543.5xTHaLXzZ0@avalon> <20180407094556.GQ20945@w540> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jacopo, On Saturday, 7 April 2018 12:45:56 EEST jacopo mondi wrote: > On Fri, Apr 06, 2018 at 06:40:14PM +0300, Laurent Pinchart wrote: > > Hi Jacopo, > > > > (CC'ing Mark Brown) > > Hi Mark > > [snip] > > >> Anyway, we spent enough time on naming issues, starting from my first > >> stupid 'pdwn' permutations then on this semi-standard names. > >> > >> I'll send next version with 'powerdown-gpios' and 'oe-gpios' > >> properties hoping that would be finally accepted by everyone. > > > > I certainly won't complain (as long as you write pwdn instead of pdwn in > > the driver :-)). > > Oh, so you won't complain if I address your comments? Thank you! :D > By the way, the dumb pdwn name comes, again, from the chip name. I can > change it to a saner name for sure... And I've just realized that, I thought it was a typo :-/ If it comes with the datasheet I'm fine with either. > >> Same on the mandatory/optional VCC supply thing. Let's try to make > >> next version the final one. If the optional property with the dummy > >> regulator doesn't satisfy you and it is preferred to have a > >> fixed-regulator anyhow in DT I'll do in next version, othewise let's try > >> not to change it again. I'll just remark here that in the current Eagle > >> design vcc is connected to a power rail with no regulator at all :) > > > > I don't like the dummy regulator much, as it generates a dev_warn(), which > > makes me believe that it's a hack rather than a proper solution. You might > > want to ask Mark Brown for his opinion. > > Sure: Hi Mark, a bit of context here to save you a long(er) reading. > > Unsurprisingly, the chip for which I'm writing this small driver needs > a power supply to be properly functional :) In the development board > it is installed on, the power supply is connected to a power rail, > with no regulator. At the same time, some other designs may instead > include a regulator. To be precise, with an always-on regulator that can't be software-controlled. > I assume that's a very common situation. I started by describing the > regulator as optional in DT bindings, and use regulator_get_optional(). If > that function returns PTR_ERR, the regulator is then ignored in driver's > power management routines. > > In this last version I thought I was acting smart and copied what happens > in other DRM bridges like adv7511, which use 'regulator_get()' and work > with a dummy if no regulator is provided in DT. Laurent has the above > doubts on using a dummy, and I actually share some of his concerns > (that dev_warn() is how I found out adv7511 was using a dummy, actually). > > To sum up: when a regulator is described as optional in DT, do you > suggest to work with a dummy if it's not there, or use the _optional() > version of regulator_get() and manually set it to NULL and ignore it > in the enable/disable driver's routines? > > Bonus question: Laurent likes to have the regulator described as required, > and thus require it to be described in DT also in cases where it is not > actually there using a 'fixed-regulator' and refuse to probe if the > regulator is not available. Do you encourage this approach, or prefer to > have it optional and handle it in the driver in one of the above described > ways? -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH v7 1/2] dt-bindings: display: bridge: Document THC63LVD1024 LVDS decoder Date: Sat, 07 Apr 2018 12:56:48 +0300 Message-ID: <2813569.TyIUb2mkfU@avalon> References: <1523018517-24121-1-git-send-email-jacopo+renesas@jmondi.org> <2441543.5xTHaLXzZ0@avalon> <20180407094556.GQ20945@w540> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20180407094556.GQ20945@w540> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: jacopo mondi Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, sergei.shtylyov@cogentembedded.com, airlied@linux.ie, Mark Brown , magnus.damm@gmail.com, linux-kernel@vger.kernel.org, robh+dt@kernel.org, linux-renesas-soc@vger.kernel.org, horms@verge.net.au, Jacopo Mondi , dri-devel@lists.freedesktop.org, niklas.soderlund@ragnatech.se, geert@linux-m68k.org, vladimir_zapolskiy@mentor.com List-Id: devicetree@vger.kernel.org SGkgSmFjb3BvLAoKT24gU2F0dXJkYXksIDcgQXByaWwgMjAxOCAxMjo0NTo1NiBFRVNUIGphY29w byBtb25kaSB3cm90ZToKPiBPbiBGcmksIEFwciAwNiwgMjAxOCBhdCAwNjo0MDoxNFBNICswMzAw LCBMYXVyZW50IFBpbmNoYXJ0IHdyb3RlOgo+ID4gSGkgSmFjb3BvLAo+ID4gCj4gPiAoQ0MnaW5n IE1hcmsgQnJvd24pCj4gCj4gSGkgTWFyawo+IAo+IFtzbmlwXQo+IAo+ID4+IEFueXdheSwgd2Ug c3BlbnQgZW5vdWdoIHRpbWUgb24gbmFtaW5nIGlzc3Vlcywgc3RhcnRpbmcgZnJvbSBteSBmaXJz dAo+ID4+IHN0dXBpZCAncGR3bicgcGVybXV0YXRpb25zIHRoZW4gb24gdGhpcyBzZW1pLXN0YW5k YXJkIG5hbWVzLgo+ID4+IAo+ID4+IEknbGwgc2VuZCBuZXh0IHZlcnNpb24gd2l0aCAncG93ZXJk b3duLWdwaW9zJyBhbmQgJ29lLWdwaW9zJwo+ID4+IHByb3BlcnRpZXMgaG9waW5nIHRoYXQgd291 bGQgYmUgZmluYWxseSBhY2NlcHRlZCBieSBldmVyeW9uZS4KPiA+IAo+ID4gSSBjZXJ0YWlubHkg d29uJ3QgY29tcGxhaW4gKGFzIGxvbmcgYXMgeW91IHdyaXRlIHB3ZG4gaW5zdGVhZCBvZiBwZHdu IGluCj4gPiB0aGUgZHJpdmVyIDotKSkuCj4gCj4gT2gsIHNvIHlvdSB3b24ndCBjb21wbGFpbiBp ZiBJIGFkZHJlc3MgeW91ciBjb21tZW50cz8gVGhhbmsgeW91ISA6RAo+IEJ5IHRoZSB3YXksIHRo ZSBkdW1iIHBkd24gbmFtZSBjb21lcywgYWdhaW4sIGZyb20gdGhlIGNoaXAgbmFtZS4gSSBjYW4K PiBjaGFuZ2UgaXQgdG8gYSBzYW5lciBuYW1lIGZvciBzdXJlLi4uCgpBbmQgSSd2ZSBqdXN0IHJl YWxpemVkIHRoYXQsIEkgdGhvdWdodCBpdCB3YXMgYSB0eXBvIDotLyBJZiBpdCBjb21lcyB3aXRo IHRoZSAKZGF0YXNoZWV0IEknbSBmaW5lIHdpdGggZWl0aGVyLgoKPiA+PiBTYW1lIG9uIHRoZSBt YW5kYXRvcnkvb3B0aW9uYWwgVkNDIHN1cHBseSB0aGluZy4gTGV0J3MgdHJ5IHRvIG1ha2UKPiA+ PiBuZXh0IHZlcnNpb24gdGhlIGZpbmFsIG9uZS4gSWYgdGhlIG9wdGlvbmFsIHByb3BlcnR5IHdp dGggdGhlIGR1bW15Cj4gPj4gcmVndWxhdG9yIGRvZXNuJ3Qgc2F0aXNmeSB5b3UgYW5kIGl0IGlz IHByZWZlcnJlZCB0byBoYXZlIGEKPiA+PiBmaXhlZC1yZWd1bGF0b3IgYW55aG93IGluIERUIEkn bGwgZG8gaW4gbmV4dCB2ZXJzaW9uLCBvdGhld2lzZSBsZXQncyB0cnkKPiA+PiBub3QgdG8gY2hh bmdlIGl0IGFnYWluLiBJJ2xsIGp1c3QgcmVtYXJrIGhlcmUgdGhhdCBpbiB0aGUgY3VycmVudCBF YWdsZQo+ID4+IGRlc2lnbiB2Y2MgaXMgY29ubmVjdGVkIHRvIGEgcG93ZXIgcmFpbCB3aXRoIG5v IHJlZ3VsYXRvciBhdCBhbGwgOikKPiA+IAo+ID4gSSBkb24ndCBsaWtlIHRoZSBkdW1teSByZWd1 bGF0b3IgbXVjaCwgYXMgaXQgZ2VuZXJhdGVzIGEgZGV2X3dhcm4oKSwgd2hpY2gKPiA+IG1ha2Vz IG1lIGJlbGlldmUgdGhhdCBpdCdzIGEgaGFjayByYXRoZXIgdGhhbiBhIHByb3BlciBzb2x1dGlv bi4gWW91IG1pZ2h0Cj4gPiB3YW50IHRvIGFzayBNYXJrIEJyb3duIGZvciBoaXMgb3Bpbmlvbi4K PiAKPiBTdXJlOiBIaSBNYXJrLCBhIGJpdCBvZiBjb250ZXh0IGhlcmUgdG8gc2F2ZSB5b3UgYSBs b25nKGVyKSByZWFkaW5nLgo+IAo+IFVuc3VycHJpc2luZ2x5LCB0aGUgY2hpcCBmb3Igd2hpY2gg SSdtIHdyaXRpbmcgdGhpcyBzbWFsbCBkcml2ZXIgbmVlZHMKPiBhIHBvd2VyIHN1cHBseSB0byBi ZSBwcm9wZXJseSBmdW5jdGlvbmFsIDopIEluIHRoZSBkZXZlbG9wbWVudCBib2FyZAo+IGl0IGlz IGluc3RhbGxlZCBvbiwgdGhlIHBvd2VyIHN1cHBseSBpcyBjb25uZWN0ZWQgdG8gYSBwb3dlciBy YWlsLAo+IHdpdGggbm8gcmVndWxhdG9yLiBBdCB0aGUgc2FtZSB0aW1lLCBzb21lIG90aGVyIGRl c2lnbnMgbWF5IGluc3RlYWQKPiBpbmNsdWRlIGEgcmVndWxhdG9yLgoKVG8gYmUgcHJlY2lzZSwg d2l0aCBhbiBhbHdheXMtb24gcmVndWxhdG9yIHRoYXQgY2FuJ3QgYmUgc29mdHdhcmUtY29udHJv bGxlZC4KCj4gSSBhc3N1bWUgdGhhdCdzIGEgdmVyeSBjb21tb24gc2l0dWF0aW9uLiBJIHN0YXJ0 ZWQgYnkgZGVzY3JpYmluZyB0aGUKPiByZWd1bGF0b3IgYXMgb3B0aW9uYWwgaW4gRFQgYmluZGlu Z3MsIGFuZCB1c2UgcmVndWxhdG9yX2dldF9vcHRpb25hbCgpLiBJZgo+IHRoYXQgZnVuY3Rpb24g cmV0dXJucyBQVFJfRVJSLCB0aGUgcmVndWxhdG9yIGlzIHRoZW4gaWdub3JlZCBpbiBkcml2ZXIn cwo+IHBvd2VyIG1hbmFnZW1lbnQgcm91dGluZXMuCj4gCj4gSW4gdGhpcyBsYXN0IHZlcnNpb24g SSB0aG91Z2h0IEkgd2FzIGFjdGluZyBzbWFydCBhbmQgY29waWVkIHdoYXQgaGFwcGVucwo+IGlu IG90aGVyIERSTSBicmlkZ2VzIGxpa2UgYWR2NzUxMSwgd2hpY2ggdXNlICdyZWd1bGF0b3JfZ2V0 KCknIGFuZCB3b3JrCj4gd2l0aCBhIGR1bW15IGlmIG5vIHJlZ3VsYXRvciBpcyBwcm92aWRlZCBp biBEVC4gTGF1cmVudCBoYXMgdGhlIGFib3ZlCj4gZG91YnRzIG9uIHVzaW5nIGEgZHVtbXksIGFu ZCBJIGFjdHVhbGx5IHNoYXJlIHNvbWUgb2YgaGlzIGNvbmNlcm5zCj4gKHRoYXQgZGV2X3dhcm4o KSBpcyBob3cgSSBmb3VuZCBvdXQgYWR2NzUxMSB3YXMgdXNpbmcgYSBkdW1teSwgYWN0dWFsbHkp Lgo+IAo+IFRvIHN1bSB1cDogd2hlbiBhIHJlZ3VsYXRvciBpcyBkZXNjcmliZWQgYXMgb3B0aW9u YWwgaW4gRFQsIGRvIHlvdQo+IHN1Z2dlc3QgdG8gd29yayB3aXRoIGEgZHVtbXkgaWYgaXQncyBu b3QgdGhlcmUsIG9yIHVzZSB0aGUgX29wdGlvbmFsKCkKPiB2ZXJzaW9uIG9mIHJlZ3VsYXRvcl9n ZXQoKSBhbmQgbWFudWFsbHkgc2V0IGl0IHRvIE5VTEwgYW5kIGlnbm9yZSBpdAo+IGluIHRoZSBl bmFibGUvZGlzYWJsZSBkcml2ZXIncyByb3V0aW5lcz8KPiAKPiBCb251cyBxdWVzdGlvbjogTGF1 cmVudCBsaWtlcyB0byBoYXZlIHRoZSByZWd1bGF0b3IgZGVzY3JpYmVkIGFzIHJlcXVpcmVkLAo+ IGFuZCB0aHVzIHJlcXVpcmUgaXQgdG8gYmUgZGVzY3JpYmVkIGluIERUIGFsc28gaW4gY2FzZXMg d2hlcmUgaXQgaXMgbm90Cj4gYWN0dWFsbHkgdGhlcmUgdXNpbmcgYSAnZml4ZWQtcmVndWxhdG9y JyBhbmQgcmVmdXNlIHRvIHByb2JlIGlmIHRoZQo+IHJlZ3VsYXRvciBpcyBub3QgYXZhaWxhYmxl LiBEbyB5b3UgZW5jb3VyYWdlIHRoaXMgYXBwcm9hY2gsIG9yIHByZWZlciB0bwo+IGhhdmUgaXQg b3B0aW9uYWwgYW5kIGhhbmRsZSBpdCBpbiB0aGUgZHJpdmVyIGluIG9uZSBvZiB0aGUgYWJvdmUg ZGVzY3JpYmVkCj4gd2F5cz8KCi0tIApSZWdhcmRzLAoKTGF1cmVudCBQaW5jaGFydAoKCgpfX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFp bGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5m cmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK