From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ramalingam C Subject: Re: [PATCH 1/10] drm/i915: Modifying structures related to DRRS Date: Thu, 22 Jan 2015 17:05:37 +0530 Message-ID: <54C0E089.4070604@intel.com> References: <1420836965-10068-1-git-send-email-vandana.kannan@intel.com> <1420836965-10068-2-git-send-email-vandana.kannan@intel.com> <20150122064813.GT10113@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0179730004==" Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTP id A3F7989CB2 for ; Thu, 22 Jan 2015 03:40:02 -0800 (PST) In-Reply-To: <20150122064813.GT10113@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter , Rodrigo Vivi Cc: intel-gfx , Paulo Zanoni , "Vivi, Rodrigo" List-Id: intel-gfx@lists.freedesktop.org This is a multi-part message in MIME format. --===============0179730004== Content-Type: multipart/alternative; boundary="------------060702050000060704040206" This is a multi-part message in MIME format. --------------060702050000060704040206 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit On Thursday 22 January 2015 12:18 PM, Daniel Vetter wrote: > On Tue, Jan 13, 2015 at 05:27:01PM -0800, Rodrigo Vivi wrote: >> 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 >> >> I'll continue the reviews tomorrow and intend to finish by Thursday. >> I'll also resend 2 patches that had conflicts with latest -nightly... > 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. 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]. > -Daniel --Ram --------------060702050000060704040206 Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: 7bit
On Thursday 22 January 2015 12:18 PM, Daniel Vetter wrote:
On Tue, Jan 13, 2015 at 05:27:01PM -0800, Rodrigo Vivi wrote:

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 <rodrigo.vivi@intel.com>

I'll continue the reviews tomorrow and intend to finish by Thursday.
I'll also resend 2 patches that had conflicts with latest -nightly...

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.
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].
-Daniel
--Ram
--------------060702050000060704040206-- --===============0179730004== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4 IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK --===============0179730004==--