<html> <head> <meta content="text/html; charset=utf-8" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> <div class="moz-cite-prefix">On Thursday 22 January 2015 12:18 PM, Daniel Vetter wrote: </div> <blockquote cite="mid:20150122064813.GT10113@phenom.ffwll.local" type="cite"> <pre wrap="">On Tue, Jan 13, 2015 at 05:27:01PM -0800, Rodrigo Vivi wrote: </pre> <blockquote type="cite"> <pre wrap="">I believe we could start this re-org by moving it out to intel_drrs.c renaming functions and adding entry docbook entry. But anyway this patch is right and doesn't seem to change anything that is already working so free free to use: Reviewed-by: Rodrigo Vivi <a class="moz-txt-link-rfc2396E" href="mailto:rodrigo.vivi@intel.com"><rodrigo.vivi@intel.com></a> I'll continue the reviews tomorrow and intend to finish by Thursday. I'll also resend 2 patches that had conflicts with latest -nightly... </pre> </blockquote> <pre wrap=""> I merged this, but it causes a compiler warning. Which means I need the revised patches to get this all into shape asap or need to drop the patch again.</pre> </blockquote> I have just shared a new patches for the changes suggested except for igt and related debugfs entry patches [ 9 and 10]. Working on the required changes for igt and the debugfs interface. But I hope the changes required for igt need not stop other patches [1 to 8 of 10 patches]. <blockquote cite="mid:20150122064813.GT10113@phenom.ffwll.local" type="cite"> <pre wrap=""> -Daniel </pre> </blockquote> --Ram </body> </html>