From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies Date: Thu, 19 Apr 2018 11:11:26 +0300 Message-ID: <20599695.GSkeUvMnvx@avalon> References: <20180410051927.8268-1-philippe.cornu@st.com> <20180410051927.8268-2-philippe.cornu@st.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20180410051927.8268-2-philippe.cornu@st.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Philippe Cornu Cc: Mark Rutland , devicetree@vger.kernel.org, Alexandre Torgue , David Airlie , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Yannick Fertre , Rob Herring List-Id: devicetree@vger.kernel.org SGkgUGhpbGlwcGUsCgpUaGFuayB5b3UgZm9yIHRoZSBwYXRjaC4KCk9uIFR1ZXNkYXksIDEwIEFw cmlsIDIwMTggMDg6MTk6MjYgRUVTVCBQaGlsaXBwZSBDb3JudSB3cm90ZToKPiBBZGQgdGhlIDMg b3B0aW9uYWwgcG93ZXIgc3VwcGxpZXMgdXNpbmcgdGhlIGV4YWN0IGRlc2NyaXB0aW9uCj4gZm91 bmQgaW4gdGhlIGRvY3VtZW50IG5hbWVkCj4gIlNpSTkwMjJBL1NpSTkwMjRBIEhETUkgVHJhbnNt aXR0ZXIgRGF0YSBTaGVldCAoQXVndXN0IDIwMTYpIi4KPiAKPiBTaWduZWQtb2ZmLWJ5OiBQaGls aXBwZSBDb3JudSA8cGhpbGlwcGUuY29ybnVAc3QuY29tPgo+IC0tLQo+ICBEb2N1bWVudGF0aW9u L2RldmljZXRyZWUvYmluZGluZ3MvZGlzcGxheS9icmlkZ2Uvc2lpOTAyeC50eHQgfCAzICsrKwo+ ICAxIGZpbGUgY2hhbmdlZCwgMyBpbnNlcnRpb25zKCspCj4gCj4gZGlmZiAtLWdpdCBhL0RvY3Vt ZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9kaXNwbGF5L2JyaWRnZS9zaWk5MDJ4LnR4dAo+ IGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvYnJpZGdlL3NpaTkw MngudHh0IGluZGV4Cj4gNTZhM2U2OGNjYjgwLi5jZjUzNjc4ZmU1NzQgMTAwNjQ0Cj4gLS0tIGEv RG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvYnJpZGdlL3NpaTkwMngu dHh0Cj4gKysrIGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvYnJp ZGdlL3NpaTkwMngudHh0Cj4gQEAgLTgsNiArOCw5IEBAIE9wdGlvbmFsIHByb3BlcnRpZXM6Cj4g IAktIGludGVycnVwdHMtZXh0ZW5kZWQgb3IgaW50ZXJydXB0LXBhcmVudCArIGludGVycnVwdHM6 IGRlc2NyaWJlCj4gIAkgIHRoZSBpbnRlcnJ1cHQgbGluZSB1c2VkIHRvIGluZm9ybSB0aGUgaG9z dCBhYm91dCBob3RwbHVnIGV2ZW50cy4KPiAgCS0gcmVzZXQtZ3Bpb3M6IE9GIGRldmljZS10cmVl IGdwaW8gc3BlY2lmaWNhdGlvbiBmb3IgUlNUX04gcGluLgo+ICsJLSBpb3ZjYy1zdXBwbHk6IEkv TyBzdXBwbHkgdm9sdGFnZSAoMS44ViBvciAzLjNWLCBob3N0LWRlcGVuZGVudCkuCj4gKwktIGF2 Y2MxMi1zdXBwbHk6IFRNRFMgYW5hbG9nIHN1cHBseSB2b2x0YWdlICgxLjJWKS4KPiArCS0gY3Zj YzEyLXN1cHBseTogRGlnaXRhbCBjb3JlIHN1cHBseSB2b2x0YWdlICgxLjJWKS4KCkl0IHNlZW1z IHRoYXQgdGhlIEFWQ0MxMiBhbmQgQ1ZDQzEyIHBvd2VyIHN1cHBsaWVzIGFyZSB1c3VhbGx5IGRl cml2ZWQgZnJvbSAKdGhlIHNhbWUgc291cmNlLiBIb3cgYWJvdXQgc3RhcnRpbmcgd2l0aCBvbmUg RFQgcHJvcGVydHkgZm9yIGJvdGgsIGFuZCBhZGRpbmcgCmEgc2Vjb25kIG9uZSBsYXRlciBpZiBu ZWVkZWQgPwoKPiAgT3B0aW9uYWwgc3Vibm9kZXM6Cj4gIAktIHZpZGVvIGlucHV0OiB0aGlzIHN1 Ym5vZGUgY2FuIGNvbnRhaW4gYSB2aWRlbyBpbnB1dCBwb3J0IG5vZGUKCi0tIApSZWdhcmRzLAoK TGF1cmVudCBQaW5jaGFydAoKCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVz a3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9k cmktZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752848AbeDSITt (ORCPT ); Thu, 19 Apr 2018 04:19:49 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:34270 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750972AbeDSITq (ORCPT ); Thu, 19 Apr 2018 04:19:46 -0400 From: Laurent Pinchart To: Philippe Cornu Cc: Archit Taneja , Andrzej Hajda , David Airlie , Rob Herring , Mark Rutland , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Yannick Fertre , Benjamin Gaignard , Alexandre Torgue Subject: Re: [PATCH 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies Date: Thu, 19 Apr 2018 11:11:26 +0300 Message-ID: <20599695.GSkeUvMnvx@avalon> Organization: Ideas on Board Oy In-Reply-To: <20180410051927.8268-2-philippe.cornu@st.com> References: <20180410051927.8268-1-philippe.cornu@st.com> <20180410051927.8268-2-philippe.cornu@st.com> 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 Philippe, Thank you for the patch. On Tuesday, 10 April 2018 08:19:26 EEST Philippe Cornu wrote: > Add the 3 optional power supplies using the exact description > found in the document named > "SiI9022A/SiI9024A HDMI Transmitter Data Sheet (August 2016)". > > Signed-off-by: Philippe Cornu > --- > Documentation/devicetree/bindings/display/bridge/sii902x.txt | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/bridge/sii902x.txt > b/Documentation/devicetree/bindings/display/bridge/sii902x.txt index > 56a3e68ccb80..cf53678fe574 100644 > --- a/Documentation/devicetree/bindings/display/bridge/sii902x.txt > +++ b/Documentation/devicetree/bindings/display/bridge/sii902x.txt > @@ -8,6 +8,9 @@ Optional properties: > - interrupts-extended or interrupt-parent + interrupts: describe > the interrupt line used to inform the host about hotplug events. > - reset-gpios: OF device-tree gpio specification for RST_N pin. > + - iovcc-supply: I/O supply voltage (1.8V or 3.3V, host-dependent). > + - avcc12-supply: TMDS analog supply voltage (1.2V). > + - cvcc12-supply: Digital core supply voltage (1.2V). It seems that the AVCC12 and CVCC12 power supplies are usually derived from the same source. How about starting with one DT property for both, and adding a second one later if needed ? > Optional subnodes: > - video input: this subnode can contain a video input port node -- Regards, Laurent Pinchart