From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754775AbbKQUu6 (ORCPT ); Tue, 17 Nov 2015 15:50:58 -0500 Received: from mail.kernel.org ([198.145.29.136]:56320 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752009AbbKQUu4 (ORCPT ); Tue, 17 Nov 2015 15:50:56 -0500 Date: Tue, 17 Nov 2015 14:50:51 -0600 From: Rob Herring To: Chen-Yu Tsai Cc: Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Jean-Christophe Plagniol-Villard , Tomi Valkeinen , Hans de Goede , Mark Brown , linux-sunxi@googlegroups.com, linux-fbdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v5 1/2] dt-bindings: simplefb: Support regulator supply properties Message-ID: <20151117205051.GA6591@rob-hp-laptop> References: <1447734663-19189-1-git-send-email-wens@csie.org> <1447734663-19189-2-git-send-email-wens@csie.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1447734663-19189-2-git-send-email-wens@csie.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 17, 2015 at 12:31:02PM +0800, Chen-Yu Tsai wrote: > The physical display tied to the framebuffer may have regulators > providing power to it, such as power for LCDs or interface conversion > chips. > > The number of regulators in use may vary, but the regulator supply > binding can not be a list. Instead just support any named regulator > supply properties under the device node. These should be properly > named to match the device schematics / design. The driver should > take care to go through them all. > > Signed-off-by: Chen-Yu Tsai > Reviewed-by: Hans de Goede > Acked-by: Mark Brown Not really happy to see the continued expansion of simplefb binding, but Acked-by: Rob Herring > --- > .../devicetree/bindings/display/simple-framebuffer.txt | 13 +++++++++---- > 1 file changed, 9 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/simple-framebuffer.txt b/Documentation/devicetree/bindings/display/simple-framebuffer.txt > index 4474ef6e0b95..8c9e9f515c87 100644 > --- a/Documentation/devicetree/bindings/display/simple-framebuffer.txt > +++ b/Documentation/devicetree/bindings/display/simple-framebuffer.txt > @@ -47,10 +47,14 @@ Required properties: > - a8b8g8r8 (32-bit pixels, d[31:24]=a, d[23:16]=b, d[15:8]=g, d[7:0]=r). > > Optional properties: > -- clocks : List of clocks used by the framebuffer. Clocks listed here > - are expected to already be configured correctly. The OS must > - ensure these clocks are not modified or disabled while the > - simple framebuffer remains active. > +- clocks : List of clocks used by the framebuffer. > +- *-supply : Any number of regulators used by the framebuffer. These should > + be named according to the names in the device's design. > + > + The above resources are expected to already be configured correctly. > + The OS must ensure they are not modified or disabled while the simple > + framebuffer remains active. > + > - display : phandle pointing to the primary display hardware node > > Example: > @@ -68,6 +72,7 @@ chosen { > stride = <(1600 * 2)>; > format = "r5g6b5"; > clocks = <&ahb_gates 36>, <&ahb_gates 43>, <&ahb_gates 44>; > + lcd-supply = <®_dc1sw>; > display = <&lcdc0>; > }; > stdout-path = "display0"; > -- > 2.6.2 >