Re: [PATCH] drm/arcpgu: Get rid of "encoder-slave" property

2017-03-29 Thread liviu.du...@arm.com
On Wed, Mar 29, 2017 at 01:34:00PM +, Alexey Brodkin wrote: > Hi Liviu, Rob, Hi Alexey, > > On Fri, 2017-03-03 at 18:21 +, liviu.du...@arm.com wrote: > > On Fri, Mar 03, 2017 at 05:48:19PM +, Alexey Brodkin wrote: > > > > > > Hi Liviu, > > > > > > On Fri, 2017-03-03 at 16:28 +,

Re: [PATCH] drm/arcpgu: Get rid of "encoder-slave" property

2017-03-29 Thread Alexey Brodkin
Hi Liviu, Rob, On Fri, 2017-03-03 at 18:21 +, liviu.du...@arm.com wrote: > On Fri, Mar 03, 2017 at 05:48:19PM +, Alexey Brodkin wrote: > > > > Hi Liviu, > > > > On Fri, 2017-03-03 at 16:28 +, Liviu Dudau wrote: > > > > > > On Fri, Mar 03, 2017 at 06:19:24PM +0300, Alexey Brodkin wro

Re: [PATCH] drm/arcpgu: Get rid of "encoder-slave" property

2017-03-03 Thread liviu.du...@arm.com
On Fri, Mar 03, 2017 at 05:48:19PM +, Alexey Brodkin wrote: > Hi Liviu, > > On Fri, 2017-03-03 at 16:28 +, Liviu Dudau wrote: > > On Fri, Mar 03, 2017 at 06:19:24PM +0300, Alexey Brodkin wrote: > > > > > > - /* find the encoder node and initialize it */ > > > - encoder_node = of_parse_pha

Re: [PATCH] drm/arcpgu: Get rid of "encoder-slave" property

2017-03-03 Thread Alexey Brodkin
Hi Liviu, On Fri, 2017-03-03 at 16:28 +, Liviu Dudau wrote: > On Fri, Mar 03, 2017 at 06:19:24PM +0300, Alexey Brodkin wrote: > > > > - /* find the encoder node and initialize it */ > > - encoder_node = of_parse_phandle(drm->dev->of_node, "encoder-slave", 0); > > - if (encoder_node) { >

Re: [PATCH] drm/arcpgu: Get rid of "encoder-slave" property

2017-03-03 Thread Liviu Dudau
On Fri, Mar 03, 2017 at 06:19:24PM +0300, Alexey Brodkin wrote: > We used to use "encoder-slave" property in PGU's > Device Tree node to refer to the encoder, but since there's > a way to find it with some code smarts we get rid of > obviously extra complication in PGU node. > > Again inspired by