All of lore.kernel.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:32 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.