From mboxrd@z Thu Jan 1 00:00:00 1970 From: boris.brezillon@bootlin.com (Boris Brezillon) Date: Wed, 18 Apr 2018 10:29:00 +0200 Subject: [PATCH v2 2/6] dt-bindings: display: atmel: optional video-interface of endpoints In-Reply-To: References: <20180417131052.16336-1-peda@axentia.se> <20180417131052.16336-3-peda@axentia.se> <20180418091658.690e3d5e@bbrezillon> Message-ID: <20180418102900.3d36e424@bbrezillon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, 18 Apr 2018 09:31:53 +0200 Peter Rosin wrote: > On 2018-04-18 09:16, Boris Brezillon wrote: > > Hi Peter, > > > > On Tue, 17 Apr 2018 15:10:48 +0200 > > Peter Rosin wrote: > > > >> With bus-type/bus-width properties in the endpoint nodes, the video- > >> interface of the connection can be specified for cases where the > >> heuristic fails to select the correct output mode. This can happen > >> e.g. if not all RGB pins are routed on the PCB; the driver has no > >> way of knowing this, and needs to be told explicitly. > >> > >> This is critical for the devices that have the "conflicting output > >> formats" issue (SAM9N12, SAM9X5, SAMA5D3), since the most significant > >> RGB bits move around depending on the selected output mode. For > >> devices that do not have the "conflicting output formats" issue > >> (SAMA5D2, SAMA5D4), this is completely irrelevant. > >> > >> Signed-off-by: Peter Rosin > >> --- > >> Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt | 8 ++++++++ > >> 1 file changed, 8 insertions(+) > >> > >> diff --git a/Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt b/Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt > >> index 82f2acb3d374..244b48869eb4 100644 > >> --- a/Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt > >> +++ b/Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt > >> @@ -15,6 +15,14 @@ Required children nodes: > >> to external devices using the OF graph reprensentation (see ../graph.txt). > >> At least one port node is required. > >> > >> +Optional properties in grandchild nodes: > >> + Any endpoint grandchild node may specify a desired video interface > >> + according to ../../media/video-interfaces.txt, specifically > >> + - bus-type: must be <0>. > >> + - bus-width: recognized values are <12>, <16>, <18> and <24>, and > >> + override any output mode selection hueristic, forcing "rgb444", > > heuristic, I'll fix that for v3, so please review as if it wasn't there... > > >> + "rgb565", "rgb666" and "rgb888" respectively. > >> + > > > > Can you add an example or update the existing one to show how this > > should be defined? > > For v3, I'll extend the binding with this after the preexisting example: > > ------------------8<----------------- > Example 2: With a video interface override to force rgb565, as above > but with these changes/additions: > > &hlcdc { > hlcdc-display-controller { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_lcd_base &pinctrl_lcd_rgb565>; > > port at 0 { > hlcdc_panel_output: endpoint at 0 { > bus-type = <0>; > bus-width = <16>; > }; > }; > }; > }; > ------------------8<----------------- > > Is that a good plan, or should I perhaps duplicate the whole example? Looks good to me, no need to add a new example. Thanks, Boris From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Subject: Re: [PATCH v2 2/6] dt-bindings: display: atmel: optional video-interface of endpoints Date: Wed, 18 Apr 2018 10:29:00 +0200 Message-ID: <20180418102900.3d36e424@bbrezillon> References: <20180417131052.16336-1-peda@axentia.se> <20180417131052.16336-3-peda@axentia.se> <20180418091658.690e3d5e@bbrezillon> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Peter Rosin Cc: Mark Rutland , Boris Brezillon , Alexandre Belloni , devicetree@vger.kernel.org, David Airlie , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Nicolas Ferre , Rob Herring , Laurent Pinchart , Daniel Vetter , Russell King - ARM Linux , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org T24gV2VkLCAxOCBBcHIgMjAxOCAwOTozMTo1MyArMDIwMApQZXRlciBSb3NpbiA8cGVkYUBheGVu dGlhLnNlPiB3cm90ZToKCj4gT24gMjAxOC0wNC0xOCAwOToxNiwgQm9yaXMgQnJlemlsbG9uIHdy b3RlOgo+ID4gSGkgUGV0ZXIsCj4gPiAKPiA+IE9uIFR1ZSwgMTcgQXByIDIwMTggMTU6MTA6NDgg KzAyMDAKPiA+IFBldGVyIFJvc2luIDxwZWRhQGF4ZW50aWEuc2U+IHdyb3RlOgo+ID4gICAKPiA+ PiBXaXRoIGJ1cy10eXBlL2J1cy13aWR0aCBwcm9wZXJ0aWVzIGluIHRoZSBlbmRwb2ludCBub2Rl cywgdGhlIHZpZGVvLQo+ID4+IGludGVyZmFjZSBvZiB0aGUgY29ubmVjdGlvbiBjYW4gYmUgc3Bl Y2lmaWVkIGZvciBjYXNlcyB3aGVyZSB0aGUKPiA+PiBoZXVyaXN0aWMgZmFpbHMgdG8gc2VsZWN0 IHRoZSBjb3JyZWN0IG91dHB1dCBtb2RlLiBUaGlzIGNhbiBoYXBwZW4KPiA+PiBlLmcuIGlmIG5v dCBhbGwgUkdCIHBpbnMgYXJlIHJvdXRlZCBvbiB0aGUgUENCOyB0aGUgZHJpdmVyIGhhcyBubwo+ ID4+IHdheSBvZiBrbm93aW5nIHRoaXMsIGFuZCBuZWVkcyB0byBiZSB0b2xkIGV4cGxpY2l0bHku Cj4gPj4KPiA+PiBUaGlzIGlzIGNyaXRpY2FsIGZvciB0aGUgZGV2aWNlcyB0aGF0IGhhdmUgdGhl ICJjb25mbGljdGluZyBvdXRwdXQKPiA+PiBmb3JtYXRzIiBpc3N1ZSAoU0FNOU4xMiwgU0FNOVg1 LCBTQU1BNUQzKSwgc2luY2UgdGhlIG1vc3Qgc2lnbmlmaWNhbnQKPiA+PiBSR0IgYml0cyBtb3Zl IGFyb3VuZCBkZXBlbmRpbmcgb24gdGhlIHNlbGVjdGVkIG91dHB1dCBtb2RlLiBGb3IKPiA+PiBk ZXZpY2VzIHRoYXQgZG8gbm90IGhhdmUgdGhlICJjb25mbGljdGluZyBvdXRwdXQgZm9ybWF0cyIg aXNzdWUKPiA+PiAoU0FNQTVEMiwgU0FNQTVENCksIHRoaXMgaXMgY29tcGxldGVseSBpcnJlbGV2 YW50Lgo+ID4+Cj4gPj4gU2lnbmVkLW9mZi1ieTogUGV0ZXIgUm9zaW4gPHBlZGFAYXhlbnRpYS5z ZT4KPiA+PiAtLS0KPiA+PiAgRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3Bs YXkvYXRtZWwvaGxjZGMtZGMudHh0IHwgOCArKysrKysrKwo+ID4+ICAxIGZpbGUgY2hhbmdlZCwg OCBpbnNlcnRpb25zKCspCj4gPj4KPiA+PiBkaWZmIC0tZ2l0IGEvRG9jdW1lbnRhdGlvbi9kZXZp Y2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvYXRtZWwvaGxjZGMtZGMudHh0IGIvRG9jdW1lbnRhdGlv bi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvYXRtZWwvaGxjZGMtZGMudHh0Cj4gPj4gaW5k ZXggODJmMmFjYjNkMzc0Li4yNDRiNDg4NjllYjQgMTAwNjQ0Cj4gPj4gLS0tIGEvRG9jdW1lbnRh dGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvYXRtZWwvaGxjZGMtZGMudHh0Cj4gPj4g KysrIGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvYXRtZWwvaGxj ZGMtZGMudHh0Cj4gPj4gQEAgLTE1LDYgKzE1LDE0IEBAIFJlcXVpcmVkIGNoaWxkcmVuIG5vZGVz Ogo+ID4+ICAgdG8gZXh0ZXJuYWwgZGV2aWNlcyB1c2luZyB0aGUgT0YgZ3JhcGggcmVwcmVuc2Vu dGF0aW9uIChzZWUgLi4vZ3JhcGgudHh0KS4KPiA+PiAgIEF0IGxlYXN0IG9uZSBwb3J0IG5vZGUg aXMgcmVxdWlyZWQuCj4gPj4gIAo+ID4+ICtPcHRpb25hbCBwcm9wZXJ0aWVzIGluIGdyYW5kY2hp bGQgbm9kZXM6Cj4gPj4gKyBBbnkgZW5kcG9pbnQgZ3JhbmRjaGlsZCBub2RlIG1heSBzcGVjaWZ5 IGEgZGVzaXJlZCB2aWRlbyBpbnRlcmZhY2UKPiA+PiArIGFjY29yZGluZyB0byAuLi8uLi9tZWRp YS92aWRlby1pbnRlcmZhY2VzLnR4dCwgc3BlY2lmaWNhbGx5Cj4gPj4gKyAtIGJ1cy10eXBlOiBt dXN0IGJlIDwwPi4KPiA+PiArIC0gYnVzLXdpZHRoOiByZWNvZ25pemVkIHZhbHVlcyBhcmUgPDEy PiwgPDE2PiwgPDE4PiBhbmQgPDI0PiwgYW5kCj4gPj4gKyAgIG92ZXJyaWRlIGFueSBvdXRwdXQg bW9kZSBzZWxlY3Rpb24gaHVlcmlzdGljLCBmb3JjaW5nICJyZ2I0NDQiLCAgCj4gCj4gaGV1cmlz dGljLCBJJ2xsIGZpeCB0aGF0IGZvciB2Mywgc28gcGxlYXNlIHJldmlldyBhcyBpZiBpdCB3YXNu J3QgdGhlcmUuLi4KPiAKPiA+PiArICAgInJnYjU2NSIsICJyZ2I2NjYiIGFuZCAicmdiODg4IiBy ZXNwZWN0aXZlbHkuCj4gPj4gKyAgCj4gPiAKPiA+IENhbiB5b3UgYWRkIGFuIGV4YW1wbGUgb3Ig dXBkYXRlIHRoZSBleGlzdGluZyBvbmUgdG8gc2hvdyBob3cgdGhpcwo+ID4gc2hvdWxkIGJlIGRl ZmluZWQ/ICAKPiAKPiBGb3IgdjMsIEknbGwgZXh0ZW5kIHRoZSBiaW5kaW5nIHdpdGggdGhpcyBh ZnRlciB0aGUgcHJlZXhpc3RpbmcgZXhhbXBsZToKPiAKPiAtLS0tLS0tLS0tLS0tLS0tLS04PC0t LS0tLS0tLS0tLS0tLS0tCj4gRXhhbXBsZSAyOiBXaXRoIGEgdmlkZW8gaW50ZXJmYWNlIG92ZXJy aWRlIHRvIGZvcmNlIHJnYjU2NSwgYXMgYWJvdmUKPiBidXQgd2l0aCB0aGVzZSBjaGFuZ2VzL2Fk ZGl0aW9uczoKPiAKPiAmaGxjZGMgewo+IAlobGNkYy1kaXNwbGF5LWNvbnRyb2xsZXIgewo+IAkJ cGluY3RybC1uYW1lcyA9ICJkZWZhdWx0IjsKPiAJCXBpbmN0cmwtMCA9IDwmcGluY3RybF9sY2Rf YmFzZSAmcGluY3RybF9sY2RfcmdiNTY1PjsKPiAKPiAJCXBvcnRAMCB7Cj4gCQkJaGxjZGNfcGFu ZWxfb3V0cHV0OiBlbmRwb2ludEAwIHsKPiAJCQkJYnVzLXR5cGUgPSA8MD47Cj4gCQkJCWJ1cy13 aWR0aCA9IDwxNj47Cj4gCQkJfTsKPiAJCX07Cj4gCX07Cj4gfTsKPiAtLS0tLS0tLS0tLS0tLS0t LS04PC0tLS0tLS0tLS0tLS0tLS0tCj4gCj4gSXMgdGhhdCBhIGdvb2QgcGxhbiwgb3Igc2hvdWxk IEkgcGVyaGFwcyBkdXBsaWNhdGUgdGhlIHdob2xlIGV4YW1wbGU/CgpMb29rcyBnb29kIHRvIG1l LCBubyBuZWVkIHRvIGFkZCBhIG5ldyBleGFtcGxlLgoKVGhhbmtzLAoKQm9yaXMKX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcg bGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRl c2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753469AbeDRI3R (ORCPT ); Wed, 18 Apr 2018 04:29:17 -0400 Received: from mail.bootlin.com ([62.4.15.54]:55340 "EHLO mail.bootlin.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753114AbeDRI3P (ORCPT ); Wed, 18 Apr 2018 04:29:15 -0400 Date: Wed, 18 Apr 2018 10:29:00 +0200 From: Boris Brezillon To: Peter Rosin Cc: linux-kernel@vger.kernel.org, David Airlie , Rob Herring , Mark Rutland , Nicolas Ferre , Alexandre Belloni , Boris Brezillon , Daniel Vetter , Gustavo Padovan , Sean Paul , Laurent Pinchart , Russell King - ARM Linux , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 2/6] dt-bindings: display: atmel: optional video-interface of endpoints Message-ID: <20180418102900.3d36e424@bbrezillon> In-Reply-To: References: <20180417131052.16336-1-peda@axentia.se> <20180417131052.16336-3-peda@axentia.se> <20180418091658.690e3d5e@bbrezillon> X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 18 Apr 2018 09:31:53 +0200 Peter Rosin wrote: > On 2018-04-18 09:16, Boris Brezillon wrote: > > Hi Peter, > > > > On Tue, 17 Apr 2018 15:10:48 +0200 > > Peter Rosin wrote: > > > >> With bus-type/bus-width properties in the endpoint nodes, the video- > >> interface of the connection can be specified for cases where the > >> heuristic fails to select the correct output mode. This can happen > >> e.g. if not all RGB pins are routed on the PCB; the driver has no > >> way of knowing this, and needs to be told explicitly. > >> > >> This is critical for the devices that have the "conflicting output > >> formats" issue (SAM9N12, SAM9X5, SAMA5D3), since the most significant > >> RGB bits move around depending on the selected output mode. For > >> devices that do not have the "conflicting output formats" issue > >> (SAMA5D2, SAMA5D4), this is completely irrelevant. > >> > >> Signed-off-by: Peter Rosin > >> --- > >> Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt | 8 ++++++++ > >> 1 file changed, 8 insertions(+) > >> > >> diff --git a/Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt b/Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt > >> index 82f2acb3d374..244b48869eb4 100644 > >> --- a/Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt > >> +++ b/Documentation/devicetree/bindings/display/atmel/hlcdc-dc.txt > >> @@ -15,6 +15,14 @@ Required children nodes: > >> to external devices using the OF graph reprensentation (see ../graph.txt). > >> At least one port node is required. > >> > >> +Optional properties in grandchild nodes: > >> + Any endpoint grandchild node may specify a desired video interface > >> + according to ../../media/video-interfaces.txt, specifically > >> + - bus-type: must be <0>. > >> + - bus-width: recognized values are <12>, <16>, <18> and <24>, and > >> + override any output mode selection hueristic, forcing "rgb444", > > heuristic, I'll fix that for v3, so please review as if it wasn't there... > > >> + "rgb565", "rgb666" and "rgb888" respectively. > >> + > > > > Can you add an example or update the existing one to show how this > > should be defined? > > For v3, I'll extend the binding with this after the preexisting example: > > ------------------8<----------------- > Example 2: With a video interface override to force rgb565, as above > but with these changes/additions: > > &hlcdc { > hlcdc-display-controller { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_lcd_base &pinctrl_lcd_rgb565>; > > port@0 { > hlcdc_panel_output: endpoint@0 { > bus-type = <0>; > bus-width = <16>; > }; > }; > }; > }; > ------------------8<----------------- > > Is that a good plan, or should I perhaps duplicate the whole example? Looks good to me, no need to add a new example. Thanks, Boris