On 2017-06-06 03:43 AM, Petri Latvala wrote: > On Mon, Jun 05, 2017 at 03:59:37PM -0400, Harry Wentland wrote: >> On 2017-06-05 03:50 PM, Alex Deucher wrote: >>> On Mon, Jun 5, 2017 at 2:43 PM, Harry Wentland <harry.wentland at amd.com> wrote: >>>> Create crtc/connector combinations based on actual adapter >>>> information obtained from drmModeRes. >>>> >>>> Also set MAX_CRTCs to 6 for AMD GPUs. >>>> >>>> Signed-off-by: Harry Wentland <harry.wentland at amd.com> >>> >>> The code is kind of hard to follow, but it looks good to me: >>> Acked-by: Alex Deucher <alexander.deucher at amd.com> >>> >> >> I agree, that's why I only tried to limit the scope of this patch to the >> size of the combinations struct. >> >> Thanks for the Ack. Is it okay to merge into drm/igt-gpu-tools? Do you know >> what the rules are on that? >> https://01.org/linuxgraphics/gfx-docs/maintainer-tools/drm-intel.html is >> quite vague on IGT patches. >> > > > Thanks, pushed this. > > Getting commits to IGT is described in igt/CONTRIBUTING. That page > could use a short description of IGT patch flow indeed. > Thanks, Petri. I believe I have push access but didn't want to push anything prematurely. In the future I'll push after a-b or r-b if there are no objections. Thanks, Harry >