All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 4/4] sunxi: video: Force h/vsync active high when using ext. vga dac
Date: Wed, 28 Jan 2015 10:37:14 +0100	[thread overview]
Message-ID: <54C8ADCA.6080001@redhat.com> (raw)
In-Reply-To: <1422436425.29309.31.camel@hellion.org.uk>

Hi,

On 28-01-15 10:13, Ian Campbell wrote:
> On Sun, 2015-01-25 at 15:35 +0100, Hans de Goede wrote:
>> On both my A13-OLinuxIno and my A13-OLinuxIno-Micro, the vga output gives an
>> unstable image when active low v or hsync is used, so when an external vga dac
>> is used force v and hsync active high independent of what the modeline says.
>
> Does this not just mean the modeline is wrong and should be fixed or an
> alternative provided which is correct for these systems?

I would rather not, the modeline uses is a vesa standard timing, so it
certainly is correct.

The problem seems to be specific with the OLinuxIno A13 (normal & micro)
boards. I've just looked up the schematics and they use an opendrain driver
for the vga sync lines, and with sync pulses it is the logical high->low
edge of the pulse which counts for the timing, which with an active low
sync is being driven by the pull-up, and that simply seems to not drive
it hard enough to get a stable image.

> In the code comment you say "some dacs", what about ones which don't
> have this issue or, worse, ones which have the inverse issue and don't
> work with active high?
>
> At the least I'd say this aught to be controllable via Kconfig.

I agree this should be a Kconfig and only turned on on the Olinuxino
13 boards. I'll do a v2.

Regards,

Hans

  reply	other threads:[~2015-01-28  9:37 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-25 14:35 [U-Boot] [PATCH 1/4] sunxi: Convert sun5i boards to use auto dram configuration Hans de Goede
2015-01-25 14:35 ` [U-Boot] [PATCH 2/4] sunxi: Add a GMAC Transmit Clock Delay Chain Kconfig option Hans de Goede
2015-01-28  9:10   ` Ian Campbell
2015-01-25 14:35 ` [U-Boot] [PATCH 3/4] sunxi: Remove CONFIG_TARGET_FOO for sun5i and sun7i boards Hans de Goede
2015-01-28  9:10   ` Ian Campbell
2015-01-25 14:35 ` [U-Boot] [PATCH 4/4] sunxi: video: Force h/vsync active high when using ext. vga dac Hans de Goede
2015-01-28  9:13   ` Ian Campbell
2015-01-28  9:37     ` Hans de Goede [this message]
2015-01-28  9:08 ` [U-Boot] [PATCH 1/4] sunxi: Convert sun5i boards to use auto dram configuration 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=54C8ADCA.6080001@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.