public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
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,
	"Purushothaman, Vijay A" <vijay.a.purushothaman@intel.com>,
	Rob Landley <rob@landley.net>,
	Alex Deucher <alexander.deucher@amd.com>,
	Dave Airlie <airlied@redhat.com>
Subject: Re: [PATCH v5 1/1] Documentation: drm: describing drm properties exposed by various drivers
Date: Mon, 10 Mar 2014 15:33:54 +0100	[thread overview]
Message-ID: <6398512.6Zeryzgla0@avalon> (raw)
In-Reply-To: <1394263723-10452-1-git-send-email-sagar.a.kamble@intel.com>

Hi Sagar,

Thank you for the patch, it's really appreciated.

On Saturday 08 March 2014 12:58:43 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. Information
> 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.
> 
> v5: Restructured output for ENUM properties

Some of my concerns have already been addressed. I just have two more small 
comments, please see below.

> 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älä" <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>
> ---
>  Documentation/DocBook/drm.tmpl | 846 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 846 insertions(+)
> 
> diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
> index ed1d6d2..5650d13 100644
> --- a/Documentation/DocBook/drm.tmpl
> +++ b/Documentation/DocBook/drm.tmpl
> @@ -2317,6 +2317,852 @@ void intel_crt_init(struct drm_device *dev)
>        pointer to the target object, a pointer to the previously created
> property and an initial instance value.
>      </para>

I would put the following in a <sect2> to make it more readable.

> +    <para>
> +     The following table gives description of drm properties exposed by
> various
> +     modules/drivers.
> +    </para>
> +<table border="1" cellpadding="0" cellspacing="0" >

Any particular reason for the space before the '>' ?

> +<tbody>
> +<tr style="font-weight: bold;">
> +<td valign="top" >Owner Module/Drivers</td>
> +<td valign="top" >Group</td>
> +<td valign="top" >Property Name</td>
> +<td valign="top" >Type</td>
> +<td valign="top" >Property Values</td>
> +<td valign="top" >Object attached</td>
> +<td valign="top" >Description/Restrictions</td>
> +</tr>
> +<tr>
> +<td rowspan="19" valign="top" >DRM</td>
> +<td rowspan="2" valign="top" >Generic</td>
> +<td valign="top" >“EDID”</td>
> +<td valign="top" >BLOB | IMMUTABLE</td>
> +<td valign="top" >0</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >Contains id of edid blob ptr object.</td>
> +</tr>
> +<tr>
> +<td valign="top" >“DPMS”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ “On”, “Standby”, “Suspend”, “Off” }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >Contains DPMS operation mode value.</td>
> +</tr>
> +<tr>
> +<td rowspan="2" valign="top" >DVI-I</td>
> +<td valign="top" >“subconnector”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ “Unknown”, “DVI-D”, “DVI-A” }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“select subconnector”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ “Automatic”, “DVI-D”, “DVI-A” }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="13" valign="top" >TV</td>
> +<td valign="top" >“subconnector”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "Unknown", "Composite", "SVIDEO", "Component", "SCART"
> }</td> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“select subconnector”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "Automatic", "Composite", "SVIDEO", "Component",
> "SCART" }</td> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“mode”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "NTSC_M", "NTSC_J", "NTSC_443", "PAL_B" } etc.</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“left margin”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“right margin”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“top margin”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“bottom margin”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“brightness”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“contrast”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“flicker reduction”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“overscan”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“saturation”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“hue”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="2" valign="top" >Optional</td>
> +<td valign="top" >“scaling mode”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "None", "Full", "Center", "Full aspect" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“dirty”</td>
> +<td valign="top" >ENUM | IMMUTABLE</td>
> +<td valign="top" >{ "Off", "On", "Annotate" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="21" valign="top" >i915</td>
> +<td rowspan="3" valign="top" >Generic</td>
> +<td valign="top" >"Broadcast RGB"</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "Automatic", "Full", "Limited 16:235" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“audio”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "force-dvi", "off", "auto", "on" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >Standard name as in DRM</td>
> +<td valign="top" >Standard type as in DRM</td>
> +<td valign="top" >Standard value as in DRM</td>
> +<td valign="top" >Standard Object as in DRM</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="17" valign="top" >SDVO-TV</td>
> +<td valign="top" >“mode”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "NTSC_M", "NTSC_J", "NTSC_443", "PAL_B" } etc.</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"left_margin"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"right_margin"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"top_margin"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"bottom_margin"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“hpos”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“vpos”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“contrast”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“saturation”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“hue”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“sharpness”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“flicker_filter”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“flicker_filter_adaptive”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“flicker_filter_2d”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“tv_chroma_filter”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“tv_luma_filter”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“dot_crawl”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=1</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >SDVO-TV/LVDS</td>
> +<td valign="top" >“brightness”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="3" valign="top" >CDV gma-500</td>
> +<td rowspan="3" valign="top" >Generic</td>
> +<td valign="top" >"Broadcast RGB"</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ “Full”, “Limited 16:235” }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"Broadcast RGB"</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ “off”, “auto”, “on” }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >Standard name as in DRM</td>
> +<td valign="top" >Standard type as in DRM</td>
> +<td valign="top" >Standard value as in DRM</td>
> +<td valign="top" >Standard Object as in DRM</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="20" valign="top" >Poulsbo</td>
> +<td rowspan="2" valign="top" >Generic</td>
> +<td valign="top" >“backlight”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=100</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >Standard name as in DRM</td>
> +<td valign="top" >Standard type as in DRM</td>
> +<td valign="top" >Standard value as in DRM</td>
> +<td valign="top" >Standard Object as in DRM</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="17" valign="top" >SDVO-TV</td>
> +<td valign="top" >“mode”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "NTSC_M", "NTSC_J", "NTSC_443", "PAL_B" } etc.</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"left_margin"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"right_margin"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"top_margin"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"bottom_margin"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“hpos”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“vpos”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“contrast”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“saturation”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“hue”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“sharpness”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“flicker_filter”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“flicker_filter_adaptive”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“flicker_filter_2d”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“tv_chroma_filter”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“tv_luma_filter”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“dot_crawl”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=1</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >SDVO-TV/LVDS</td>
> +<td valign="top" >“brightness”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max= SDVO dependent</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="11" valign="top" >armada</td>
> +<td rowspan="2" valign="top" >CRTC</td>
> +<td valign="top" >"CSC_YUV"</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "Auto" , "CCIR601", "CCIR709" }</td>
> +<td valign="top" >CRTC</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"CSC_RGB"</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "Auto", "Computer system", "Studio" }</td>
> +<td valign="top" >CRTC</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="9" valign="top" >Overlay</td>
> +<td valign="top" >"colorkey"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0xffffff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"colorkey_min"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0xffffff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"colorkey_max"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0xffffff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"colorkey_val"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0xffffff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"colorkey_alpha"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0xffffff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"colorkey_mode"</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "disabled", "Y component", "U component"
> +, "V component", "RGB", “R component", "G component", "B component" }</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"brightness"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=256 + 255</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"contrast"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0x7fff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"saturation"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0x7fff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="2" valign="top" >exynos</td>
> +<td valign="top" >CRTC</td>
> +<td valign="top" >“mode”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "normal", "blank" }</td>
> +<td valign="top" >CRTC</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >Overlay</td>
> +<td valign="top" >“zpos”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=MAX_PLANE-1</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="3" valign="top" >i2c/ch7006_drv</td>
> +<td valign="top" >Generic</td>
> +<td valign="top" >“scale”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=2</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="2" valign="top" >TV</td>
> +<td valign="top" >Standard names as in DRM</td>
> +<td valign="top" >Standard types as in DRM</td>
> +<td valign="top" >Standard Values as in DRM</td>
> +<td valign="top" >Standard object as in DRM</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“mode”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "PAL", "PAL-M","PAL-N"}, ”PAL-Nc"
> +, "PAL-60", "NTSC-M", "NTSC-J" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="16" valign="top" >noveau</td>
> +<td rowspan="6" valign="top" >NV10 Overlay</td>
> +<td valign="top" >"colorkey"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0x01ffffff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“contrast”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=8192-1</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“brightness”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=1024</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“hue”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=359</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“saturation”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=8192-1</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“iturbt_709”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=1</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="2" valign="top" >Nv04 Overlay</td>
> +<td valign="top" >“colorkey”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0x01ffffff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“brightness”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=1024</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="7" valign="top" >Display</td>
> +<td valign="top" >“dithering mode”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "auto", "off", "on" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“dithering depth”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "auto", "off", "on", "static 2x2", "dynamic 2x2",
> "temporal" }</td> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“underscan”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "auto", "6 bpc", "8 bpc" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“underscan hborder”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=128</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“underscan vborder”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=128</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“vibrant hue”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=180</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“color vibrance”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=200</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >Generic</td>
> +<td valign="top" >Standard name as in DRM</td>
> +<td valign="top" >Standard type as in DRM</td>
> +<td valign="top" >Standard value as in DRM</td>
> +<td valign="top" >Standard Object as in DRM</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="2" valign="top" >omap</td>
> +<td rowspan="2" valign="top" >Generic</td>
> +<td valign="top" >“rotation”</td>
> +<td valign="top" >BITMASK</td>
> +<td valign="top" >{ 0, "rotate-0" },
> +{ 1, "rotate-90" },
> +{ 2, "rotate-180" },
> +{ 3, "rotate-270" },
> +{ 4, "reflect-x" },
> +{ 5, "reflect-y" }</td>
> +<td valign="top" >CRTC, Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >“zorder”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=3</td>
> +<td valign="top" >CRTC, Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >qxl</td>
> +<td valign="top" >Generic</td>
> +<td valign="top" >“hotplug_mode_update"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=1</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="10" valign="top" >radeon</td>
> +<td valign="top" >DVI-I</td>
> +<td valign="top" >“coherent”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=1</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >DAC enable load detect</td>
> +<td valign="top" >“load detection”</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=1</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >TV Standard</td>
> +<td valign="top" >"tv standard"</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "ntsc", "pal", "pal-m", "pal-60", "ntsc-j"
> +, "scart-pal", "pal-cn", "secam" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >legacy TMDS PLL detect</td>
> +<td valign="top" >"tmds_pll"</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "driver", "bios" }</td>
> +<td valign="top" >-</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="3" valign="top" >Underscan</td>
> +<td valign="top" >"underscan"</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "off", "on", "auto" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"underscan hborder"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=128</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"underscan vborder"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=128</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >Audio</td>
> +<td valign="top" >“audio”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "off", "on", "auto" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >FMT Dithering</td>
> +<td valign="top" >“dither”</td>
> +<td valign="top" >ENUM</td>
> +<td valign="top" >{ "off", "on" }</td>
> +<td valign="top" >Connector</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >Generic</td>
> +<td valign="top" >Standard name as in DRM</td>
> +<td valign="top" >Standard type as in DRM</td>
> +<td valign="top" >Standard value as in DRM</td>
> +<td valign="top" >Standard Object as in DRM</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td rowspan="3" valign="top" >rcar-du</td>
> +<td rowspan="3" valign="top" >Generic</td>
> +<td valign="top" >"alpha"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=255</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"colorkey"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=0, Max=0x01ffffff</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +<tr>
> +<td valign="top" >"zpos"</td>
> +<td valign="top" >RANGE</td>
> +<td valign="top" >Min=1, Max=7</td>
> +<td valign="top" >Plane</td>
> +<td valign="top" >TBD</td>
> +</tr>
> +</tbody>
> +</table>
>    </sect1>
> 
>    <!-- Internals: vertical blanking -->

-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2014-03-10 14:33 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-05 10:56 [PATCH 1/1] Documentation: drm: describing drm properties exposed by various drivers sagar.a.kamble
2014-03-06  7:15 ` [PATCH v2 " sagar.a.kamble
2014-03-06 12:09   ` Ville Syrjälä
2014-03-06 14:01     ` Sagar Arun Kamble
2014-03-06 14:27       ` [PATCH v3 " sagar.a.kamble
2014-03-07 18:44         ` Randy Dunlap
2014-03-08  6:33         ` [PATCH v4 " sagar.a.kamble
2014-03-08  7:28           ` [PATCH v5 " sagar.a.kamble
2014-03-10 14:33             ` Laurent Pinchart [this message]
2014-03-11 10:37               ` [PATCH v6 " sagar.a.kamble
2014-03-11 11:22                 ` Laurent Pinchart
2014-03-11 14:25                   ` [PATCH v7 " sagar.a.kamble
2014-03-11 14:31                     ` Laurent Pinchart
2014-03-11 13:13                 ` [PATCH v6 " Deucher, Alexander
2014-03-11 14:07                   ` Sagar Arun Kamble
2014-03-06 14:41       ` [PATCH v3 " sagar.a.kamble
2014-03-10  5:21 ` [PATCH " Daniel Vetter
2014-03-10 14:36   ` Laurent Pinchart
2014-03-12 11:16     ` Sagar Arun Kamble
2014-03-12 11:25       ` Laurent Pinchart
2014-05-10 10:39         ` Ville Syrjälä
2014-05-10 10:56           ` Rob Clark
2014-05-12  6:07             ` Sagar Arun Kamble
2014-05-12  8:03               ` Daniel Vetter
2014-05-13  7:17                 ` Thierry Reding
2014-05-13  7:34                   ` Daniel Vetter
2014-05-13  9:05                     ` Thierry Reding
2014-05-13 11:02                     ` Laurent Pinchart
2014-05-13 11:51                       ` Daniel Vetter
2014-05-12  8:24           ` Dave Airlie
2014-05-12  8:58             ` Daniel Vetter
2014-05-12 15:23               ` Randy Dunlap
2014-05-12 15:54                 ` Daniel Vetter
2014-05-12 18:04                   ` Randy Dunlap
2014-07-31 22:16                     ` Randy Dunlap
2014-08-01 12:58                       ` Laurent Pinchart
2014-08-01 22:21                         ` Randy Dunlap
2014-08-04  7:30                         ` Daniel Vetter
2014-08-04 13:58                           ` Laurent Pinchart

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=6398512.6Zeryzgla0@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=airlied@redhat.com \
    --cc=alexander.deucher@amd.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=laurent.pinchart+renesas@ideasonboard.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=rob@landley.net \
    --cc=sagar.a.kamble@intel.com \
    --cc=vijay.a.purushothaman@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox