From: Sam Ravnborg <sam@ravnborg.org>
To: Douglas Anderson <dianders@chromium.org>
Cc: robdclark@chromium.org, David Airlie <airlied@linux.ie>,
linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
steev@kali.org, bjorn.andersson@linaro.org,
Thierry Reding <thierry.reding@gmail.com>,
dri-devel@lists.freedesktop.org
Subject: Re: [PATCH] drm: panel: simple: Delay HPD checking on boe_nv133fhm_n61 for 15 ms
Date: Sun, 26 Jul 2020 19:00:50 +0200 [thread overview]
Message-ID: <20200726170050.GB3275923@ravnborg.org> (raw)
In-Reply-To: <20200716132120.1.I01e738cd469b61fc9b28b3ef1c6541a4f48b11bf@changeid>
On Thu, Jul 16, 2020 at 01:21:22PM -0700, Douglas Anderson wrote:
> On boe_nv133fhm_n62 (and presumably on boe_nv133fhm_n61) a scope shows
> a small spike on the HPD line right when you power the panel on. The
> picture looks something like this:
>
> +--------------------------------------
> |
> |
> |
> Power ---+
> +---
> |
> ++ |
> +----+| |
> HPD -----+ +---------------------------+
>
> So right when power is applied there's a little bump in HPD and then
> there's small spike right before it goes low. The total time of the
> little bump plus the spike was measured on one panel as being 8 ms
> long. The total time for the HPD to go high on the same panel was
> 51.2 ms, though the datasheet only promises it is < 200 ms.
>
> When asked about this glitch, BOE indicated that it was expected and
> persisted until the TCON has been initialized.
>
> If this was a real hotpluggable DP panel then this wouldn't matter a
> whole lot. We'd debounce the HPD signal for a really long time and so
> the little blip wouldn't hurt. However, this is not a hotpluggable DP
> panel and the the debouncing logic isn't needed and just shows down
> the time needed to get the display working. This is why the code in
> panel_simple_prepare() doesn't do debouncing and just waits for HPD to
> go high once. Unfortunately if we get unlucky and happen to poll the
> HPD line right at the spike we can try talking to the panel before
> it's ready.
>
> Let's handle this situation by putting in a 15 ms prepare delay and
> decreasing the "hpd absent delay" by 15 ms. That means:
> * If you don't have HPD hooked up at all you've still got the
> hardcoded 200 ms delay.
> * If you've got HPD hooked up you will always wait at least 15 ms
> before checking HPD. The only case where this could be bad is if
> the panel is sharing a voltage rail with something else in the
> system and was already turned on long before the panel came up. In
> such a case we'll be delaying 15 ms for no reason, but it's not a
> huge delay and I don't see any other good solution to handle that
> case.
>
> Even though the delay was measured as 8 ms, 15 ms was chosen to give a
> bit of margin.
>
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
Nice drawing and good explanation.
Applied to drm-misc-fixes.
Sam
> ---
> I don't actually have a device in front of me that is exhibiting these
> problems. I believe that it is only some devices and some of the
> time. Still, this patch seems safe and seems likely to fix the issue
> given the scope shots.
>
> drivers/gpu/drm/panel/panel-simple.c | 16 +++++++++++++++-
> 1 file changed, 15 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> index 88493538a147..046a06b55800 100644
> --- a/drivers/gpu/drm/panel/panel-simple.c
> +++ b/drivers/gpu/drm/panel/panel-simple.c
> @@ -1260,7 +1260,21 @@ static const struct panel_desc boe_nv133fhm_n61 = {
> .height = 165,
> },
> .delay = {
> - .hpd_absent_delay = 200,
> + /*
> + * When power is first given to the panel there's a short
> + * spike on the HPD line. It was explained that this spike
> + * was until the TCON data download was complete. On
> + * one system this was measured at 8 ms. We'll put 15 ms
> + * in the prepare delay just to be safe and take it away
> + * from the hpd_absent_delay (which would otherwise be 200 ms)
> + * to handle this. That means:
> + * - If HPD isn't hooked up you still have 200 ms delay.
> + * - If HPD is hooked up we won't try to look at it for the
> + * first 15 ms.
> + */
> + .prepare = 15,
> + .hpd_absent_delay = 185,
> +
> .unprepare = 500,
> },
> .bus_format = MEDIA_BUS_FMT_RGB888_1X24,
> --
> 2.28.0.rc0.105.gf9edc3c819-goog
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
prev parent reply other threads:[~2020-07-26 17:00 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-16 20:21 [PATCH] drm: panel: simple: Delay HPD checking on boe_nv133fhm_n61 for 15 ms Douglas Anderson
2020-07-16 23:39 ` Bjorn Andersson
2020-07-20 20:24 ` Doug Anderson
2020-07-26 17:00 ` Sam Ravnborg [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200726170050.GB3275923@ravnborg.org \
--to=sam@ravnborg.org \
--cc=airlied@linux.ie \
--cc=bjorn.andersson@linaro.org \
--cc=dianders@chromium.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robdclark@chromium.org \
--cc=steev@kali.org \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).