Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

On Wed, Jan 17, 2018 at 01:55:55PM +0000, Alexey Brodkin wrote:

Hi Daniel,

On Wed, 2018-01-17 at 14:43 +0100, Daniel Vetter wrote:

...

drm_encoder_slave is the old way to write bridge drivers, for i2c bridges only. It's deprecated, and definitely should not be used in new drivers.

What's even strange is that arcpgu doesn't even use any of this, it really only wants a plain normal drm_encoder. Nuke all the surplus real estate.

As of today we use either use adv7511 encoder or dw-hdmi. And I guess proposed change is OK for both bridge versions, rigth?

drm_encoder_slave is something completely different from drm_bridge. The latter will keep working. -Daniel