All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sam Ravnborg <sam@ravnborg.org>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Jernej Skrabec <jernej.skrabec@siol.net>,
	Neil Armstrong <narmstrong@baylibre.com>,
	Jonas Karlman <jonas@kwiboo.se>,
	dri-devel@lists.freedesktop.org,
	Andrzej Hajda <a.hajda@samsung.com>,
	Thierry Reding <thierry.reding@gmail.com>
Subject: Re: [PATCH v1 4/4] drm/bridge: tc358764: make connector creation optional
Date: Tue, 14 Apr 2020 15:36:26 +0200	[thread overview]
Message-ID: <20200414133626.GA14282@ravnborg.org> (raw)
In-Reply-To: <20200414093816.GD19819@pendragon.ideasonboard.com>

On Tue, Apr 14, 2020 at 12:38:16PM +0300, Laurent Pinchart wrote:
> Hi Sam,
> 
> Thank you for the patch.
> 
> On Tue, Apr 14, 2020 at 10:47:27AM +0200, Sam Ravnborg wrote:
> > Make the connector creation optional to enable usage of the
> > tc358764 bridge with the DRM bridge connector helper.
> > 
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Andrzej Hajda <a.hajda@samsung.com>
> > Cc: Neil Armstrong <narmstrong@baylibre.com>
> > Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
> > Cc: Jonas Karlman <jonas@kwiboo.se>
> > Cc: Jernej Skrabec <jernej.skrabec@siol.net>
> 
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Thanks. I will wait a few days for more feedback before applying.
 
> Thank you for the conversion. "Just a few" more bridge drivers to go,
> and then there will be no more excuse not to use the DRM bridge
> connector helper :-)

A quick grep for ATTACH and drm_bridge_attach_flags gave me following list:

adv7511				ongoing, Laurent
analogix/analogix-anx6345	todo
analogix/analogix-anx78xx	todo
analogix/analogix_dp_core	todo
cdns-dsi			done?
display-connector.c		done
lvds-codec			done?
megachips-stdpxxxx-ge-b850v3-fw	todo
nwl-dsi				(ongoing), Guido
nxp-ptn3460			todo
panel.c				done
parade-ps8622			todo
parade-ps8640			done?
sii902x				todo
sil-sii8620			done?
simple-bridge			ongoing, Laurent
synopsys/dw-hdmi		todo
synopsys/dw-mipi-dsi		done?
tc358764			ongoing, Sam
tc358767			todo
tc358768			done?
thc63lvd1024			done?
ti-sn65dsi86			todo
ti-tfp410			done
ti-tpd12s015			todo

3	done
7	done?
4	ongoing
11	todo

Maybe a little simplistic - but gives some kind of overview.

Drivers marked with "Done?" do not have any hits for ATTACH,
dunno if they need more attantion.

I will try to take a look at a few of the trivial conversions
later this week.

	Sam
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2020-04-14 13:36 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-14  8:47 [PATCH v1 0/4] drm: tc358764: support drm bridge connector helper Sam Ravnborg
2020-04-14  8:47 ` [PATCH v1 1/4] drm/panel: add connector type to boe, hv070wsa-100 panel Sam Ravnborg
2020-04-14  9:32   ` Laurent Pinchart
2020-04-14  8:47 ` [PATCH v1 2/4] drm/bridge: tc358764: drop drm_connector_(un)register Sam Ravnborg
2020-04-14  9:33   ` Laurent Pinchart
2020-04-14  8:47 ` [PATCH v1 3/4] drm/bridge: tc358764: add drm_panel_bridge support Sam Ravnborg
2020-04-14  9:35   ` Laurent Pinchart
2020-04-14  8:47 ` [PATCH v1 4/4] drm/bridge: tc358764: make connector creation optional Sam Ravnborg
2020-04-14  9:38   ` Laurent Pinchart
2020-04-14 13:36     ` Sam Ravnborg [this message]
2020-04-14 13:41       ` Laurent Pinchart

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=20200414133626.GA14282@ravnborg.org \
    --to=sam@ravnborg.org \
    --cc=a.hajda@samsung.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jernej.skrabec@siol.net \
    --cc=jonas@kwiboo.se \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=narmstrong@baylibre.com \
    --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 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.