From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v2 1/3] devicetree/bindings: add reset-gpios and vcc-supply for panel-dpi Date: Wed, 30 Dec 2015 08:44:27 -0600 Message-ID: References: <1450610002-10531-1-git-send-email-u.kleine-koenig@pengutronix.de> <1450610002-10531-2-git-send-email-u.kleine-koenig@pengutronix.de> <20151229205458.GA31241@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Geert Uytterhoeven Cc: =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= , Tomi Valkeinen , Jean-Christophe Plagniol-Villard , Linux Fbdev development list , "linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Sascha Hauer , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org On Wed, Dec 30, 2015 at 4:29 AM, Geert Uytterhoeven wrote: > On Tue, Dec 29, 2015 at 9:54 PM, Rob Herring wrote: >> On Sun, Dec 20, 2015 at 12:13:20PM +0100, Uwe Kleine-K=C3=B6nig wrot= e: >>> Some displays have a reset input and/or need a regulator to functio= n >>> properly. Allow to specify them for panel-dpi devices. >>> >>> Signed-off-by: Uwe Kleine-K=C3=B6nig >>> --- >>> Documentation/devicetree/bindings/display/panel/panel-dpi.txt | 2 = ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/display/panel/panel-= dpi.txt b/Documentation/devicetree/bindings/display/panel/panel-dpi.txt >>> index 216c894d4f99..b52ac52757df 100644 >>> --- a/Documentation/devicetree/bindings/display/panel/panel-dpi.txt >>> +++ b/Documentation/devicetree/bindings/display/panel/panel-dpi.txt >>> @@ -7,6 +7,8 @@ Required properties: >>> Optional properties: >>> - label: a symbolic name for the panel >>> - enable-gpios: panel enable gpio >>> +- reset-gpios: GPIO to control the RESET pin >> >> The problem with this in a generic binding is what if the panel has >> ordering requirements like enable gpio has to be inactive when reset >> is deasserted? >> >>> +- vcc-supply: phandle of regulator that will be used to enable pow= er to the display >> >> What if there are 2 supplies? > > So it should be "vcc-supplies". Most properties containing phandles u= se plural > naming, even if Linux supports a single phandle only (e.g. "power-dom= ains"). > You also may want to add an optional "vcc-supply-names" property to > differentiate. True, but the regulator binding is an exception and we don't do that with it. The prefix is the name and other supplies are not going to be called Vcc. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html