All of lore.kernel.org
 help / color / mirror / Atom feed
From: peda@axentia.se (Peter Rosin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 0/7] tda998x: allow use with bridge based devices
Date: Mon, 12 Nov 2018 16:50:37 +0000	[thread overview]
Message-ID: <fd70023a-815b-697a-d01f-6ae9efb83653@axentia.se> (raw)
In-Reply-To: <1ad8d087-40a8-7a00-31cf-c4d34a9a12fb@axentia.se>

On 2018-08-02 08:06, Peter Rosin wrote:
> On 2018-08-01 11:35, Russell King - ARM Linux wrote:
>> On Wed, Aug 01, 2018 at 11:01:12AM +0200, Peter Rosin wrote:
>>> I don't think it's a problem with the atmel I2C driver. IIRC, the
>>> tda998x driver issues the command a initiate the EDID read, but that
>>> times out. So it appears to be the TDA19988 that fails to read the
>>> EDID over the DDC bus? Which brings me to the double problem with the
>>> scopes mentioned above...
>>
>> It sounds like it.
>>
>> It may be helpful to know that there are HDMI pass-through boards
>> available that give access to all the HDMI signals:
>>
>> https://elabbay.myshopify.com/collections/camera
>> https://elabbay.myshopify.com/collections/camera/products/hdmi-af-af-v1a-hdmi-type-a-female-to-hdmi-type-a-male-pass-through-adapter-breakout-board
>>
>> I've never bought from them, so please don't take this as a
>> recommendation - the fact that there seems to be no company details
>> on their site doesn't seem good, and as the whois for elabbay.com is
>> obscured also doesn't give me any confidence to buy from them.
> 
> I still will not be able to inspect the DDC bus between the TDA19988
> and the buffer circuit (IP4786), but the gadget seems useful enough and
> it's not a shitload of money. We'll see how long it takes for it to get
> here...
> 
> Thanks for the pointer! Maybe :-)

I got the pass-through board a while back, and that board works as expected
and there was no problem with ordering etc. What I could see with that was
that the TDA19988 was able to initiate a start condition (SDA -> low) but
then nothing more happened.

Then last week, someone noticed that even though the TDA19988 is driven by
1.8V, it still needs the high signals of the DDC bus to be above 3V, which
was unexpected and not catered for by the design. Changing VCC(SYS) of the
buffer circuit in place (IP4786, pin 27) to 3.3V fixed the issue and EDID
reading works, and this was confirmed earlier today.

So, the problem was that the TDA19988 only ever saw "low" DDC signals, and
probably aborted when the bus appeared busy. Or something.

If anyone cares...

Cheers,
Peter

WARNING: multiple messages have this Message-ID (diff)
From: Peter Rosin <peda@axentia.se>
To: Russell King - ARM Linux <linux@armlinux.org.uk>
Cc: David Airlie <airlied@linux.ie>,
	Liviu Dudau <Liviu.Dudau@arm.com>, Jyri Sarha <jsarha@ti.com>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v2 0/7] tda998x: allow use with bridge based devices
Date: Mon, 12 Nov 2018 16:50:37 +0000	[thread overview]
Message-ID: <fd70023a-815b-697a-d01f-6ae9efb83653@axentia.se> (raw)
In-Reply-To: <1ad8d087-40a8-7a00-31cf-c4d34a9a12fb@axentia.se>

On 2018-08-02 08:06, Peter Rosin wrote:
> On 2018-08-01 11:35, Russell King - ARM Linux wrote:
>> On Wed, Aug 01, 2018 at 11:01:12AM +0200, Peter Rosin wrote:
>>> I don't think it's a problem with the atmel I2C driver. IIRC, the
>>> tda998x driver issues the command a initiate the EDID read, but that
>>> times out. So it appears to be the TDA19988 that fails to read the
>>> EDID over the DDC bus? Which brings me to the double problem with the
>>> scopes mentioned above...
>>
>> It sounds like it.
>>
>> It may be helpful to know that there are HDMI pass-through boards
>> available that give access to all the HDMI signals:
>>
>> https://elabbay.myshopify.com/collections/camera
>> https://elabbay.myshopify.com/collections/camera/products/hdmi-af-af-v1a-hdmi-type-a-female-to-hdmi-type-a-male-pass-through-adapter-breakout-board
>>
>> I've never bought from them, so please don't take this as a
>> recommendation - the fact that there seems to be no company details
>> on their site doesn't seem good, and as the whois for elabbay.com is
>> obscured also doesn't give me any confidence to buy from them.
> 
> I still will not be able to inspect the DDC bus between the TDA19988
> and the buffer circuit (IP4786), but the gadget seems useful enough and
> it's not a shitload of money. We'll see how long it takes for it to get
> here...
> 
> Thanks for the pointer! Maybe :-)

I got the pass-through board a while back, and that board works as expected
and there was no problem with ordering etc. What I could see with that was
that the TDA19988 was able to initiate a start condition (SDA -> low) but
then nothing more happened.

Then last week, someone noticed that even though the TDA19988 is driven by
1.8V, it still needs the high signals of the DDC bus to be above 3V, which
was unexpected and not catered for by the design. Changing VCC(SYS) of the
buffer circuit in place (IP4786, pin 27) to 3.3V fixed the issue and EDID
reading works, and this was confirmed earlier today.

So, the problem was that the TDA19988 only ever saw "low" DDC signals, and
probably aborted when the bus appeared busy. Or something.

If anyone cares...

Cheers,
Peter

  reply	other threads:[~2018-11-12 16:50 UTC|newest]

Thread overview: 80+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-30 16:41 [PATCH v2 0/7] tda998x: allow use with bridge based devices Russell King - ARM Linux
2018-07-30 16:41 ` Russell King - ARM Linux
2018-07-30 16:42 ` [PATCH v2 1/7] drm/i2c: tda998x: find the drm_device via the drm_connector Russell King
2018-07-30 16:42   ` Russell King
2018-07-30 16:42 ` [PATCH v2 2/7] drm/i2c: tda998x: split tda998x_encoder_dpms into enable/disable Russell King
2018-07-30 16:42   ` Russell King
2018-07-31  5:46   ` Peter Rosin
2018-07-31  5:46     ` Peter Rosin
2018-07-30 16:42 ` [PATCH v2 3/7] drm/i2c: tda998x: move tda998x_set_config() into tda998x_create() Russell King
2018-07-30 16:42   ` Russell King
2018-07-30 16:42 ` [PATCH v2 4/7] drm/i2c: tda998x: convert to bridge driver Russell King
2018-07-30 16:42   ` Russell King
2018-07-31  7:37   ` Peter Rosin
2018-07-31  7:37     ` Peter Rosin
2018-08-08 19:09   ` Sean Paul
2018-08-08 19:09     ` Sean Paul
2018-08-08 22:15     ` Russell King - ARM Linux
2018-08-08 22:15       ` Russell King - ARM Linux
2018-08-10 16:11       ` Sean Paul
2018-08-10 16:11         ` Sean Paul
2018-08-10 16:50         ` Russell King - ARM Linux
2018-08-10 16:50           ` Russell King - ARM Linux
2018-08-10 17:02           ` Sean Paul
2018-08-10 17:02             ` Sean Paul
2018-08-10 17:16             ` Russell King - ARM Linux
2018-08-10 17:16               ` Russell King - ARM Linux
2018-08-14 10:42               ` Daniel Vetter
2018-08-14 10:42                 ` Daniel Vetter
2018-08-14 10:48                 ` Russell King - ARM Linux
2018-08-14 10:48                   ` Russell King - ARM Linux
2018-08-14 11:11                   ` Daniel Vetter
2018-08-14 11:11                     ` Daniel Vetter
2018-08-27 16:15   ` Andrzej Hajda
2018-08-27 16:15     ` Andrzej Hajda
2018-08-27 17:59     ` Russell King - ARM Linux
2018-08-27 17:59       ` Russell King - ARM Linux
2018-08-28  7:31       ` Andrzej Hajda
2018-08-28  7:31         ` Andrzej Hajda
2018-07-30 16:42 ` [PATCH v2 5/7] drm/i2c: tda998x: allocate tda998x_priv inside tda998x_create() Russell King
2018-07-30 16:42   ` Russell King
2018-07-30 16:42 ` [PATCH v2 6/7] drm/i2c: tda998x: cleanup from previous changes Russell King
2018-07-30 16:42   ` Russell King
2018-07-30 16:42 ` [PATCH v2 7/7] drm/i2c: tda998x: register bridge outside of component helper Russell King
2018-07-30 16:42   ` Russell King
2018-08-27 16:19   ` Andrzej Hajda
2018-08-27 16:19     ` Andrzej Hajda
2018-07-31  5:44 ` [PATCH v2 0/7] tda998x: allow use with bridge based devices Peter Rosin
2018-07-31  5:44   ` Peter Rosin
2018-07-31  7:41   ` Russell King - ARM Linux
2018-07-31  7:41     ` Russell King - ARM Linux
2018-07-31  7:53     ` Peter Rosin
2018-07-31  7:53       ` Peter Rosin
2018-07-31  9:23       ` Russell King - ARM Linux
2018-07-31  9:23         ` Russell King - ARM Linux
2018-07-31  9:26         ` [PATCH 1/4] drm/i2c: tda998x: move mode_valid() to bridge Russell King
2018-07-31  9:26           ` Russell King
2018-08-27 16:24           ` Andrzej Hajda
2018-08-27 16:24             ` Andrzej Hajda
2018-07-31  9:26         ` [PATCH 2/4] drm/i2c: tda998x: get rid of private fill_modes function Russell King
2018-07-31  9:26           ` Russell King
2018-07-31  9:26         ` [PATCH 3/4] drm/i2c: tda998x: correct PLL divider calculation Russell King
2018-07-31  9:26           ` Russell King
2018-07-31  9:26         ` [PATCH 4/4] drm/i2c: tda998x: add support for pixel repeated modes Russell King
2018-07-31  9:26           ` Russell King
2018-07-31  9:42           ` Russell King - ARM Linux
2018-07-31  9:42             ` Russell King - ARM Linux
2018-07-31 10:43         ` [PATCH v2 0/7] tda998x: allow use with bridge based devices Peter Rosin
2018-07-31 10:43           ` Peter Rosin
2018-07-31 11:15           ` Russell King - ARM Linux
2018-07-31 11:15             ` Russell King - ARM Linux
2018-08-01  9:01             ` Peter Rosin
2018-08-01  9:01               ` Peter Rosin
2018-08-01  9:35               ` Russell King - ARM Linux
2018-08-01  9:35                 ` Russell King - ARM Linux
2018-08-02  6:06                 ` Peter Rosin
2018-08-02  6:06                   ` Peter Rosin
2018-11-12 16:50                   ` Peter Rosin [this message]
2018-11-12 16:50                     ` Peter Rosin
2018-11-12 17:00                     ` Russell King - ARM Linux
2018-11-12 17:00                       ` Russell King - ARM Linux

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=fd70023a-815b-697a-d01f-6ae9efb83653@axentia.se \
    --to=peda@axentia.se \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.