Re: [PATCH 4/4] video: ARM CLCD: add support of an optional GPIO to enable panel

On 12/30/2016 10:20 AM, Linus Walleij wrote:

On Wed, Dec 21, 2016 at 4:27 AM, Vladimir Zapolskiy [off-list ref] wrote:

quoted

The change adds handling of "enable-gpios" property of panel-dpi device
node used with an ARM CLCD controller, note that the property already has
a description in display/panel/panel-dpi.txt documentation and it founds
practical usage while describing some panel devices connected to other
types of display controllers.

Signed-off-by: Vladimir Zapolskiy <vz@mleia.com>

So as you may have seen I already handle a RESET GPIO in the
Nomadik TPG110 panel subddriver in
drivers/video/fbdev/amba-clcd-nomadik.c

So is this all your panel needs?

I need "enable-gpios" property to define a GPIO, which literally enables
(powers up) a panel as a separate attached PCB.

To some extend "enable-gpios" property can be replaced by "power" property
with a phandle to a GPIO voltage regulator.

You may look at drivers/gpu/drm/panel/panel-simple.c, both "enable-gpios"
and "power" properties are defined for simple panels.

I guess it is OK for simple panels.

--
With best wishes,
Vladimir