[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Daniel Vetter
On Mon, Oct 19, 2015 at 02:26:38PM +0100, Russell King - ARM Linux wrote: > On Mon, Oct 19, 2015 at 02:02:58PM +0100, Liviu Dudau wrote: > > On Mon, Oct 19, 2015 at 01:25:37PM +0100, Russell King - ARM Linux wrote: > > > Please don't move this into here, it's completely inappropriate. Just > > > b

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Russell King - ARM Linux
On Mon, Oct 19, 2015 at 04:04:27PM +0100, Emil Velikov wrote: > Unlike Daniel, I carry little to no weight here, yet bashing on people > if they don't get things correct the first time is inconsiderable. > We are people - we can misread/misinterpret things, have a headache > and/or just a bad day.

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Liviu Dudau
On Mon, Oct 19, 2015 at 03:50:27PM +0100, Russell King - ARM Linux wrote: > On Mon, Oct 19, 2015 at 04:42:25PM +0200, Daniel Vetter wrote: > > On Mon, Oct 19, 2015 at 02:26:38PM +0100, Russell King - ARM Linux wrote: > > > On Mon, Oct 19, 2015 at 02:02:58PM +0100, Liviu Dudau wrote: > > > > On Mon,

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Emil Velikov
On 19 October 2015 at 15:50, Russell King - ARM Linux wrote: > On Mon, Oct 19, 2015 at 04:42:25PM +0200, Daniel Vetter wrote: >> On Mon, Oct 19, 2015 at 02:26:38PM +0100, Russell King - ARM Linux wrote: >> > On Mon, Oct 19, 2015 at 02:02:58PM +0100, Liviu Dudau wrote: >> > > On Mon, Oct 19, 2015 a

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Russell King - ARM Linux
On Mon, Oct 19, 2015 at 04:42:25PM +0200, Daniel Vetter wrote: > On Mon, Oct 19, 2015 at 02:26:38PM +0100, Russell King - ARM Linux wrote: > > On Mon, Oct 19, 2015 at 02:02:58PM +0100, Liviu Dudau wrote: > > > On Mon, Oct 19, 2015 at 01:25:37PM +0100, Russell King - ARM Linux wrote: > > > > Please

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Russell King - ARM Linux
On Mon, Oct 19, 2015 at 02:32:55PM +0100, Liviu Dudau wrote: > On Mon, Oct 19, 2015 at 02:26:38PM +0100, Russell King - ARM Linux wrote: > > On Mon, Oct 19, 2015 at 02:02:58PM +0100, Liviu Dudau wrote: > > > On Mon, Oct 19, 2015 at 01:25:37PM +0100, Russell King - ARM Linux wrote: > > > > Please do

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Liviu Dudau
On Mon, Oct 19, 2015 at 02:26:38PM +0100, Russell King - ARM Linux wrote: > On Mon, Oct 19, 2015 at 02:02:58PM +0100, Liviu Dudau wrote: > > On Mon, Oct 19, 2015 at 01:25:37PM +0100, Russell King - ARM Linux wrote: > > > Please don't move this into here, it's completely inappropriate. Just > > > b

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Russell King - ARM Linux
On Mon, Oct 19, 2015 at 02:02:58PM +0100, Liviu Dudau wrote: > On Mon, Oct 19, 2015 at 01:25:37PM +0100, Russell King - ARM Linux wrote: > > Please don't move this into here, it's completely inappropriate. Just > > because something makes use of this does not mean they only support > > 32-bit DMA.

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Liviu Dudau
On Mon, Oct 19, 2015 at 01:25:37PM +0100, Russell King - ARM Linux wrote: > On Mon, Oct 19, 2015 at 01:21:50PM +0100, Liviu Dudau wrote: > > A lot of component based DRM drivers use a variant of the same code > > as the probe function. They bind the crtc ports in the first iteration > > and then sc

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Russell King - ARM Linux
On Mon, Oct 19, 2015 at 01:21:50PM +0100, Liviu Dudau wrote: > A lot of component based DRM drivers use a variant of the same code > as the probe function. They bind the crtc ports in the first iteration > and then scan through the child nodes and bind the encoders attached > to the remote endpoint

[RFC PATCH v2 1/4] drm: Introduce generic probe function for component based masters.

2015-10-19 Thread Liviu Dudau
A lot of component based DRM drivers use a variant of the same code as the probe function. They bind the crtc ports in the first iteration and then scan through the child nodes and bind the encoders attached to the remote endpoints. Factor the common code into a separate function called drm_of_comp