From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sean Paul Subject: Re: [PATCH v2 3/6] drm/panel: simple: Add "no-hpd" delay for Innolux TV123WAM Date: Fri, 26 Oct 2018 10:41:39 -0400 Message-ID: <20181026144139.GR154160@art_vandelay> References: <20181025222134.174583-1-dianders@chromium.org> <20181025222134.174583-3-dianders@chromium.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20181025222134.174583-3-dianders@chromium.org> Sender: linux-kernel-owner@vger.kernel.org To: Douglas Anderson Cc: Sean Paul , Thierry Reding , Sandeep Panda , linux-arm-msm@vger.kernel.org, Laurent Pinchart , jsanka@codeaurora.org, ryandcase@chromium.org, David Airlie , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org On Thu, Oct 25, 2018 at 03:21:31PM -0700, Douglas Anderson wrote: > If the HPD signal isn't hooked up to this panel we need a 200 ms > delay. In the datasheet this is shown as the maximum time that HPD > will take to be asserted after power is given to the panel. > > Signed-off-by: Douglas Anderson Reviewed-by: Sean Paul > --- > > Changes in v2: > - Use "hpd_absent_delay" property instead of a bool + prepare delay > > drivers/gpu/drm/panel/panel-simple.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > index 687fd087b9fc..88592f9a0018 100644 > --- a/drivers/gpu/drm/panel/panel-simple.c > +++ b/drivers/gpu/drm/panel/panel-simple.c > @@ -1396,6 +1396,7 @@ static const struct panel_desc innolux_tv123wam = { > .height = 173, > }, > .delay = { > + .hpd_absent_delay = 200, > .unprepare = 500, > }, > }; > -- > 2.19.1.568.g152ad8e336-goog > -- Sean Paul, Software Engineer, Google / Chromium OS