From mboxrd@z Thu Jan  1 00:00:00 1970
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH v7 1/1] Documentation: drm: describing drm
	properties exposed by various drivers
Date: Tue, 11 Mar 2014 15:31:45 +0100
Message-ID: <2304224.zuX7RMNH2n@avalon>
References: <1690452.gh7mTdsdfJ@avalon>
	<1394547929-11931-1-git-send-email-sagar.a.kamble@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
In-Reply-To: <1394547929-11931-1-git-send-email-sagar.a.kamble@intel.com>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Sender: intel-gfx-bounces@lists.freedesktop.org
Errors-To: intel-gfx-bounces@lists.freedesktop.org
To: sagar.a.kamble@intel.com
Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>, linux-doc@vger.kernel.org, Daniel Vetter <daniel.vetter@ffwll.ch>, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Rob Landley <rob@landley.net>, David Herrmann <dh.herrmann@gmail.com>, Alex Deucher <alexander.deucher@amd.com>, Dave Airlie <airlied@redhat.com>
List-Id: intel-gfx@lists.freedesktop.org

Hi Sagar,

Thank you for the patch.

On Tuesday 11 March 2014 19:55:29 sagar.a.kamble@intel.com wrote:
> From: Sagar Kamble <sagar.a.kamble@intel.com>
> =

> Started documenting drm properties for drm drivers. This patch provides
> information about properties in drm, i915, psb and cdv/gma-500. Informati=
on
> about other properties can be added on top of these.
> =

> v2: Added description of drm properties in armada, exynos, i2c/ch7006,
> noveau, omap, qxl, radeon, rcar-du
> =

> v3: Removed "Property Object" column since it is implementation related.
> Property type column refined.[Ville's review comments]
> =

> v4: Removed whitespace warnings and minor nits. [Randy's review comments]
> =

> v5: Restructured output for ENUM properties
> =

> v6: Review comments on formatting the table. [Laurent's review comments]
> =

> v7: Minor restructuring. [Laurent's review comments]
> =

> Cc: Rob Landley <rob@landley.net>
> Cc: Dave Airlie <airlied@redhat.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> Cc: David Herrmann <dh.herrmann@gmail.com>
> Cc: Alex Deucher <alexander.deucher@amd.com>
> Cc: "Ville Syrj=E4l=E4" <ville.syrjala@linux.intel.com>
> Cc: Sagar Kamble <sagar.a.kamble@intel.com>
> Cc: "Purushothaman, Vijay A" <vijay.a.purushothaman@intel.com>
> Cc: linux-doc@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> =

> Signed-off-by: Sagar Kamble <sagar.a.kamble@intel.com>

As far as this patch is concerned,

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

However Daniel Vetter mentioned his preference for a simpler text-based =

format. As long as we can transform it to Docbook for inclusion in the main =

documentation I'd be fine with that as well.

-- =

Regards,

Laurent Pinchart