On Thu, Mar 12, 2015 at 5:31 PM, Shobhit Kumar <shobhit.ku...@intel.com> wrote:

> Export PANEL_EN/DISABLE (offset 0x52) as additional GPIO. Needed
> by display driver to enable the DSI panel on BYT platform where
> the Panel EN/Disable control is routed thorugh CRC PMIC
>
> CC: Samuel Ortiz <sa...@linux.intel.com>
> Cc: Linus Walleij <linus.wall...@linaro.org>
> Cc: Alexandre Courbot <gnu...@gmail.com>
> Cc: Thierry Reding <thierry.red...@gmail.com>
> Signed-off-by: Shobhit Kumar <shobhit.ku...@intel.com>

NACK.

Spawn a separate MFD cell and write a fixed voltage
regulator driver for this.

> @@ -39,6 +39,7 @@
>  #define GPIO0P0CTLI            0x33
>  #define GPIO1P0CTLO            0x3b
>  #define GPIO1P0CTLI            0x43
> +#define GPIOPANELCTL           0x52

This is even a lie, you say in the commit message that the
register is indeed named PANEL_EN/DISABLE and is not
a GPIO.

Yours,
Linus Walleij
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to