From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Jyri Sarha <jsarha@ti.com>
Cc: dri-devel@lists.freedesktop.org,
Dan Carpenter <dan.carpenter@oracle.com>
Subject: Re: [bug report] drm/bridge: Add ti-tfp410 DVI transmitter driver
Date: Thu, 08 Dec 2016 00:29:43 +0200 [thread overview]
Message-ID: <2009764.EybUbF1gV2@avalon> (raw)
In-Reply-To: <6a550a96-0217-735c-925f-32c1fbd995d3@ti.com>
Hi Jyri,
On Wednesday 07 Dec 2016 22:28:56 Jyri Sarha wrote:
> On 12/07/16 16:46, Daniel Vetter wrote:
> >>> This implementation is a hack in the first place. The code will be
> >>> removed when DRM has a proper generic connector support.
> >
> > Hm, what's this generic drm connector thing you talk about? What's it for?
> > Asking since I haven't seen that yet nor heard in a discussion ...
>
> After looking up this comment from Laurent:
>
> On 11/10/16 14:15, Laurent Pinchart wrote [1]:
> >>> The implementation side is not so critical, because it more easily
> >>> changed, but should I create an independent generic platform-device
> >>> driver for such DVI/HDMI connector or just implement the connector side
> >>> within tfp410 driver?
> >
> > Longer term I'd like to go for connector drivers, but it might take a
> > bit of infrastructure work. If you can give it a try it would be great !
> > Otherwise I'm fine with handling that in the tfp410 driver for now.
>
> ... I think I miss understood it. He was probably referring to the
> infrastructure needed in the tfp410 driver. I'll try to get that done
> for 4.11 or 4.12.
I was referring to drivers for connectors, binding to the connectors DT nodes.
We shouldn't duplicate handling of connector-related features (ddc-i2c-bus
being the most important one) in every bridge driver, and we need to pave the
way for bridges chained with other bridges instead of being connected directly
to a connector.
> [1]
> https://lists.freedesktop.org/archives/dri-devel/2016-November/123337.html
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
prev parent reply other threads:[~2016-12-07 22:29 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-06 13:28 [bug report] drm/bridge: Add ti-tfp410 DVI transmitter driver Dan Carpenter
2016-12-07 14:20 ` Jyri Sarha
2016-12-07 14:46 ` Daniel Vetter
2016-12-07 20:28 ` Jyri Sarha
2016-12-07 22:29 ` Laurent Pinchart [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=2009764.EybUbF1gV2@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=dan.carpenter@oracle.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=jsarha@ti.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 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.