Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-31 Thread Alexey Brodkin
Hi Daniel, On Tue, 2018-01-30 at 10:15 +0100, Daniel Vetter wrote: > On Wed, Jan 17, 2018 at 03:17:55PM +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. This ha

Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-31 Thread Alexey Brodkin
Hi Daniel, On Tue, 2018-01-30 at 18:07 +0100, Daniel Vetter wrote: > On Tue, Jan 30, 2018 at 5:44 PM, Alexey Brodkin > wrote: > > Hi Daniel, > > > > On Tue, 2018-01-30 at 10:15 +0100, Daniel Vetter wrote: > > > On Wed, Jan 17, 2018 at 03:17:55PM +0100, Daniel Vetter wrote: > > > > drm_encoder_sl

Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-30 Thread Daniel Vetter
On Tue, Jan 30, 2018 at 5:44 PM, Alexey Brodkin wrote: > Hi Daniel, > > On Tue, 2018-01-30 at 10:15 +0100, Daniel Vetter wrote: >> On Wed, Jan 17, 2018 at 03:17:55PM +0100, Daniel Vetter wrote: >> > drm_encoder_slave is the old way to write bridge drivers, for i2c >> > bridges only. It's deprecate

Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-30 Thread Daniel Vetter
On Wed, Jan 17, 2018 at 03:17:55PM +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. This has absolutely nothing to do with the new bridge > driver infrastructure imple

Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-19 Thread kbuild test robot
Hi Daniel, I love your patch! Perhaps something to improve: [auto build test WARNING on drm/drm-next] [also build test WARNING on v4.15-rc8 next-20180119] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux

Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-19 Thread kbuild test robot
Hi Daniel, I love your patch! Yet something to improve: [auto build test ERROR on drm/drm-next] [also build test ERROR on v4.15-rc8 next-20180119] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits

Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-18 Thread Alexey Brodkin
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 > rea

[PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-17 Thread Daniel Vetter
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. This has absolutely nothing to do with the new bridge driver infrastructure implemented by drm_bridge. What's even strange is that arcpgu doesn't even

Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-17 Thread Daniel Vetter
On Wed, Jan 17, 2018 at 01:55:55PM +, 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. >

Re: [PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-17 Thread Laurent Pinchart
Hi Daniel, Thank you for the patch. On Wednesday, 17 January 2018 15:53:07 EET 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 does

[PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-17 Thread Daniel Vetter
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. v2

[PATCH] drm/arcpgu: remove drm_encoder_slave

2018-01-17 Thread Daniel Vetter
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. Cc