Re: [PATCH] drm/panel: simple: ensure Sharp lq123p1jx31 isn't turned off too soon

2017-02-06 Thread Doug Anderson
Hi, On Mon, Feb 6, 2017 at 4:03 AM, Thierry Reding wrote: > On Thu, Feb 02, 2017 at 03:38:53PM -0800, Douglas Anderson wrote: >> The Sharp lq123p1jx31 has a requirement that the VDD is on for at >> least 300 ms before being turned off. At the moment nothing anywhere >>

Re: [PATCH] drm/panel: simple: ensure Sharp lq123p1jx31 isn't turned off too soon

2017-02-06 Thread Thierry Reding
On Thu, Feb 02, 2017 at 03:38:53PM -0800, Douglas Anderson wrote: > The Sharp lq123p1jx31 has a requirement that the VDD is on for at > least 300 ms before being turned off. At the moment nothing anywhere > in the kernel is ensuring this. Looks to me like .delay.prepare would be a better fit.

Re: [PATCH] drm/panel: simple: ensure Sharp lq123p1jx31 isn't turned off too soon

2017-02-03 Thread Doug Anderson
Hi On Thu, Feb 2, 2017 at 3:38 PM, Douglas Anderson wrote: > The Sharp lq123p1jx31 has a requirement that the VDD is on for at > least 300 ms before being turned off. At the moment nothing anywhere > in the kernel is ensuring this. > > The simplest way to ensure this is

[PATCH] drm/panel: simple: ensure Sharp lq123p1jx31 isn't turned off too soon

2017-02-02 Thread Douglas Anderson
The Sharp lq123p1jx31 has a requirement that the VDD is on for at least 300 ms before being turned off. At the moment nothing anywhere in the kernel is ensuring this. The simplest way to ensure this is to add a "disable" timing of 150 ms. With this the we know that there will be at least 300 ms