From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 5/6] sunxi: video: Give hotplug-detect (hpd) signal some time to show up
Date: Sun, 28 Dec 2014 09:40:16 +0100 [thread overview]
Message-ID: <549FC1F0.9000108@redhat.com> (raw)
In-Reply-To: <549B05F4.4050200@openmailbox.org>
Hi,
On 24-12-14 19:29, B.R. Oake wrote:
> On 24/12/14 16:13, Hans de Goede wrote:> On 24-12-14 03:25, B.R. Oake wrote:
>>> Would it be acceptable to increase the wait from 300ms to about 1500ms?
>>
>> That means delaying the boot by another 1.2 seconds on all machines which
>> have hdmi without a cable plugged in. I'm not sure I'm a fan of that.
>
> Yes. How about if sunxi_hdmi_hpd_detect() was split into two, firstly the
> initialisation part that always needs to be run, and secondly the actual
> hpd detection, which would only be run if the hpd option to video-mode was
> selected?
That won't help since hpd=1 is the default, and we need to wait for the hpd
signal before doing edid, and we want to do edid by default.
> Another idea is that the hpd timeout value could be an additional option to
> video-mode.
That is probably the best solution, we may still need to tweak the default
delay a bit so that it will just work in most cases, but 1.5 seconds is
just too long IMHO. I've implemented this suggestion:
https://github.com/jwrdegoede/u-boot-sunxi/commit/d8f9051ca0562843435fa36c8dafc9f06ab657c2
Regards,
Hans
next prev parent reply other threads:[~2014-12-28 8:40 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-20 14:41 [U-Boot] [PATCH 0/6] sunxi: video: Add hdmi output fmt support + misc fixes Hans de Goede
2014-12-20 14:42 ` [U-Boot] [PATCH 1/6] edid: Add struct and defines for cea681 extension blocks Hans de Goede
2015-01-08 16:38 ` Anatolij Gustschin
2015-01-08 16:39 ` Anatolij Gustschin
2014-12-20 14:42 ` [U-Boot] [PATCH 2/6] sunxi: video: Add hdmi support Hans de Goede
2014-12-22 12:44 ` Ian Campbell
2015-01-08 16:51 ` Anatolij Gustschin
2014-12-20 14:42 ` [U-Boot] [PATCH 3/6] sunxi: video: Add sunxi_hdmi_edid_get_block helper function Hans de Goede
2014-12-22 12:46 ` Ian Campbell
2015-01-08 16:54 ` Anatolij Gustschin
2014-12-20 14:42 ` [U-Boot] [PATCH 4/6] sunxi: video: When using edid use CEA681 extension blocks to select hdmi output Hans de Goede
2014-12-22 12:47 ` Ian Campbell
2015-01-08 16:59 ` Anatolij Gustschin
2014-12-20 14:42 ` [U-Boot] [PATCH 5/6] sunxi: video: Give hotplug-detect (hpd) signal some time to show up Hans de Goede
2014-12-22 12:48 ` Ian Campbell
2014-12-24 2:25 ` B.R. Oake
2014-12-24 16:13 ` Hans de Goede
2014-12-24 18:29 ` B.R. Oake
2014-12-28 8:40 ` Hans de Goede [this message]
2015-01-03 16:43 ` B.R. Oake
2015-01-08 17:08 ` Anatolij Gustschin
2014-12-20 14:42 ` [U-Boot] [PATCH 6/6] sunxi: video: Set input sync enable Hans de Goede
2014-12-22 12:48 ` Ian Campbell
2015-01-08 17:11 ` Anatolij Gustschin
2014-12-22 12:49 ` [U-Boot] [PATCH 0/6] sunxi: video: Add hdmi output fmt support + misc fixes Ian Campbell
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=549FC1F0.9000108@redhat.com \
--to=hdegoede@redhat.com \
--cc=u-boot@lists.denx.de \
/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