On Jul-15-2014 12:18 PM, Daniel Vetter wrote:
[...]
>
> I've pulled all 4 patches. Please double-check that I've picked up the
> right ones since the series is a bit spread out.
>
> Thanks, Daniel
>
Hi Daniel,
I checked the 4 patches in -next-queued. They are the correct version.
Thanks,
: Thierry's review comments.
> - Fixed indentation
>
> v4: Thierry's review comments.
> - Return ENOMEM when property creation fails
>
> Signed-off-by: Vandana Kannan
> Cc: Thierry Reding
> ---
> drivers/gpu/drm/drm_crtc.c | 33 +
Updated drm documentation to include desscription of aspect ratio property.
v2: Updated aspect ratio specific documentation on top of the HTML table
created.
Signed-off-by: Vandana Kannan
Cc: Sagar Kamble
Cc: Daniel Vetter
---
Documentation/DocBook/drm.tmpl | 13 +++--
1 file
aspect ratio enum changes
v3: Modified the patch according the change in the earlier patch to return
errno in case property creation fails. With this change, property will be
attached only if creation is successful
Signed-off-by: Vandana Kannan
Cc: Thierry Reding
Cc: Daniel Vetter
---
drivers
variable
v3: Thierry's review comments.
- Fixed indentation
v4: Thierry's review comments.
- Return ENOMEM when property creation fails
Signed-off-by: Vandana Kannan
Cc: Thierry Reding
---
drivers/gpu/drm/drm_crtc.c | 33 +
include/drm/drm_crtc.h
On Jun-05-2014 2:58 PM, Thierry Reding wrote:
> On Thu, Jun 05, 2014 at 02:40:08PM +0530, Vandana Kannan wrote:
> [...]
>> diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
> [...]
>> /**
>> + * drm_mode_create_aspect_ratio_property - create aspect
w comments.
- Modified the comment to block comment format.
Signed-off-by: Vandana Kannan
Cc: Thierry Reding
---
drivers/gpu/drm/drm_edid.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 7a4fd2
variable
v3: Thierry's review comments.
- Fixed indentation drm_mode_create_aspect_ratio_property()
Signed-off-by: Vandana Kannan
Cc: Thierry Reding
---
drivers/gpu/drm/drm_crtc.c | 27 +++
include/drm/drm_crtc.h | 2 ++
include/uapi/drm/drm_mode.h | 5 +
3
Updated drm documentation to include description of aspect ratio property.
HTML table structure created by Sagar Kamble.
Signed-off-by: Vandana Kannan
Cc: Sagar Kamble
Cc: Daniel Vetter
Cc: Thierry Reding
---
Documentation/DocBook/drm.tmpl | 31 +++
1 file changed
on changes to aspect ratio enum list.
Signed-off-by: Vandana Kannan
Cc: Thierry Reding
Cc: Daniel Vetter
---
drivers/gpu/drm/i915/intel_drv.h | 1 +
drivers/gpu/drm/i915/intel_hdmi.c | 31 +++
2 files changed, 32 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_drv.h
In case user has specified an input for aspect ratio through the property,
then the user space value for PAR would take preference over the value from
CEA mode list.
v2: Thierry's review comments.
- Modified the comment "Populate..." as per review comments
Signed-off-by: Vand
variable
Signed-off-by: Vandana Kannan
Cc: Thierry Reding
Cc: Daniel Vetter
---
drivers/gpu/drm/drm_crtc.c | 27 +++
include/drm/drm_crtc.h | 2 ++
include/uapi/drm/drm_mode.h | 5 +
3 files changed, 34 insertions(+)
diff --git a/drivers/gpu/drm/drm_crtc.c b
On May-23-2014 4:18 PM, Vandana Kannan wrote:
> On May-22-2014 5:46 PM, Daniel Vetter wrote:
>> On Thu, May 22, 2014 at 04:50:48PM +0530, Vandana Kannan wrote:
>>> Added a property to enable user space to set aspect ratio.
>>> This patch contains declaration of the
On May-22-2014 5:46 PM, Daniel Vetter wrote:
> On Thu, May 22, 2014 at 04:50:48PM +0530, Vandana Kannan wrote:
>> Added a property to enable user space to set aspect ratio.
>> This patch contains declaration of the property and code to create the
>> property.
>>
>&
On May-22-2014 5:12 PM, Thierry Reding wrote:
> On Thu, May 22, 2014 at 04:50:49PM +0530, Vandana Kannan wrote:
>> In case user has specified an input for aspect ratio through the property,
>> then the user space value for PAR would take preference over the value from
&
On May-22-2014 5:08 PM, Thierry Reding wrote:
> On Thu, May 22, 2014 at 04:50:48PM +0530, Vandana Kannan wrote:
>> Added a property to enable user space to set aspect ratio.
>> This patch contains declaration of the property and code to create the
>> property.
>>
>&
ser would
> come into effect with a mode set.
>
> Signed-off-by: Vandana Kannan
> ---
> drivers/gpu/drm/i915/i915_drv.h | 5 +
> drivers/gpu/drm/i915/intel_drv.h | 1 +
> drivers/gpu/drm/i915/intel_hdmi.c | 31 +++
> 3 files changed,
In case user has specified an input for aspect ratio through the property,
then the user space value for PAR would take preference over the value from
CEA mode list.
Signed-off-by: Vandana Kannan
Cc: dri-devel at lists.freedesktop.org
---
drivers/gpu/drm/drm_edid.c | 9 +++--
1 file changed
Added a property to enable user space to set aspect ratio.
This patch contains declaration of the property and code to create the
property.
Signed-off-by: Vandana Kannan
Cc: dri-devel at lists.freedesktop.org
---
drivers/gpu/drm/drm_crtc.c | 31 +++
include/drm
On Apr-07-2014 3:24 PM, Vandana Kannan wrote:
> On Apr-07-2014 3:33 PM, Kannan, Vandana wrote:
>> Added a property to enable user space to set aspect ratio.
>> This patch contains declaration of the property and code to create the
>> property.
>>
>> Signed-off-by:
In case user has specified an input for aspect ratio through the property,
then the user space value for PAR would take preference over the value from
CEA mode list.
Signed-off-by: Vandana Kannan
Cc: dri-devel at lists.freedesktop.org
---
drivers/gpu/drm/drm_edid.c | 9 +++--
1 file changed
Added a property to enable user space to set aspect ratio.
This patch contains declaration of the property and code to create the
property.
Signed-off-by: Vandana Kannan
Cc: dri-devel at lists.freedesktop.org
---
drivers/gpu/drm/drm_crtc.c | 31 +++
include/drm
In case user has specified an input for aspect ratio through the property,
then the user space value for PAR would take preference over the value from
CEA mode list.
Signed-off-by: Vandana Kannan
Cc: dri-devel at lists.freedesktop.org
---
drivers/gpu/drm/drm_edid.c | 9 +++--
1 file changed
Added a property to enable user space to set aspect ratio.
This patch contains declaration of the property and code to create the
property.
Signed-off-by: Vandana Kannan
Cc: dri-devel at lists.freedesktop.org
---
drivers/gpu/drm/drm_crtc.c | 31 +++
include/drm
On Apr-07-2014 3:33 PM, Kannan, Vandana wrote:
> Added a property to enable user space to set aspect ratio.
> This patch contains declaration of the property and code to create the
> property.
>
> Signed-off-by: Vandana Kannan
> Cc: dri-devel at lists.freedesktop.org
>
pect ratio for AVI infoframe.
Signed-off-by: Vandana Kannan
Cc: Jesse Barnes
Cc: Vijay Purushothaman
Cc: Ville Syrj?l?
Cc: intel-gfx at lists.freedesktop.org
Cc: Wu, Fengguang
Reviewed-by: Jesse Barnes
---
drivers/gpu/drm/drm_edid.c | 22 ++
include/drm/drm_crtc.h | 1
-by: Vandana Kannan
Cc: Jesse Barnes
Cc: Vijay Purushothaman
Cc: Ville Syrj?l?
Cc: intel-gfx at lists.freedesktop.org
Reviewed-by: Jesse Barnes
---
drivers/gpu/drm/drm_edid.c | 21 +
include/drm/drm_crtc.h | 1 +
2 files changed, 22 insertions(+)
diff --git a/drivers/gpu
On Apr-01-2014 5:04 PM, Ville Syrj?l? wrote:
> On Tue, Apr 01, 2014 at 04:26:59PM +0530, Vandana Kannan wrote:
>> Populate PAR in infoframe structure. If there is a user setting for PAR, then
>> that value is set. Else, value is taken from CEA mode list if VIC is found.
>> El
which sets PAR according to user input, based on
Daniel's review comments.
A separate patch will be submitted to create a property that would enable a
user space app to set aspect ratio for AVI infoframe.
Signed-off-by: Vandana Kannan
Cc: Jesse Barnes
Cc: Vijay Purushothaman
Cc: Ville Syrj?l?
Cc
On Apr-01-2014 12:57 PM, Daniel Vetter wrote:
> On Tue, Apr 01, 2014 at 08:06:04AM +0530, Vandana Kannan wrote:
>> On Apr-01-2014 12:35 AM, Daniel Vetter wrote:
>>> On Fri, Mar 21, 2014 at 08:31:29AM +0530, Vandana Kannan wrote:
>>>> Populate PAR in infoframe structu
On Apr-01-2014 12:35 AM, Daniel Vetter wrote:
> On Fri, Mar 21, 2014 at 08:31:29AM +0530, Vandana Kannan wrote:
>> Populate PAR in infoframe structure. If there is a user setting for PAR, then
>> that value is set. Else, value is taken from CEA mode list if VIC is found.
>> El
a property that would enable a user space app to set
aspect ratio. (will be pushed as a separate patch)
Signed-off-by: Vandana Kannan
Cc: Jesse Barnes
Cc: Vijay Purushothaman
Cc: Ville Syrj?l?
Cc: intel-gfx at lists.freedesktop.org
---
drivers/gpu/drm/drm_edid.c | 34
Adding picture aspect ratio for CEA modes based on CEA-861D Table 3 or
CEA-861E Table 4. This is useful for filling up the detail in AVI
infoframe.
v2: Ville's inputs incorporated. Added picture aspect ratio as part of
edid_cea_modes instead of DRM_MODE
Signed-off-by: Vandana Kannan
Reviewed
Adding picture aspect ratio for CEA modes based on CEA-861D Table 3 or
CEA-861E Table 4. This is useful for filling up the detail in AVI
infoframe.
v2: Ville's review comments incorporated
Added picture aspect ratio as part of edid_cea_modes instead of DRM_MODE
Signed-off-by: Vandana Kannan
ic picture aspect ratio. Adding this field
>> as part of drm_display_mode. This is required to be sent as part of AVI
>> infoframe for HDMI compliance.
>
>>
>> Signed-off-by: Vandana Kannan
>> ---
>> drivers/gpu/drm/drm_edid.c | 374
>>
based on aspect
ratio.
Priority of PAR value: 1. custom value, 2. based on CEA mode list,
3. calculate from resolution
Signed-off-by: Vandana Kannan vandana.kan...@intel.com
---
drivers/gpu/drm/drm_edid.c|7 +++
drivers/gpu/drm/i915/intel_hdmi.c | 25
From: vkannan vandana.kan...@intel.com
Populate bar information, colorimetry, IT content, quantization range fields
of AVI infoframe based on CEA 861-D spec.
Signed-off-by: Vandana Kannan vandana.kan...@intel.com
---
drivers/gpu/drm/i915/intel_drv.h |4
drivers/gpu/drm/i915
37 matches
Mail list logo
