Thanks for the review comments, Emil.
Regards
Shashank
On 10/10/2015 3:53 AM, Emil Velikov wrote:
> Hi Shashank,
>
> On 9 October 2015 at 20:28, Shashank Sharma <shashank.sharma at intel.com>
> wrote:
>> This patch adds new variables in CRTC state, to hold respective color
>> correction blobs. These blobs will be required during the atomic commit
>> for writing the color correction values in correction registers.
>>
>> Signed-off-by: Shashank Sharma <shashank.sharma at intel.com>
>> Signed-off-by: Kausal Malladi <kausalmalladi at gmail.com>
>> ---
>> drivers/gpu/drm/drm_atomic_helper.c | 12 ++++++++++++
>> include/drm/drm_crtc.h | 7 ++++++-
>> 2 files changed, 18 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/drm_atomic_helper.c
>> b/drivers/gpu/drm/drm_atomic_helper.c
>> index 87a2a44..d73ca9b9 100644
>> --- a/drivers/gpu/drm/drm_atomic_helper.c
>> +++ b/drivers/gpu/drm/drm_atomic_helper.c
>> @@ -2193,6 +2193,12 @@ void __drm_atomic_helper_crtc_duplicate_state(struct
>> drm_crtc *crtc,
>>
>> if (state->mode_blob)
>> drm_property_reference_blob(state->mode_blob);
>> + if (state->ctm_blob)
>> + drm_property_reference_blob(state->ctm_blob);
>> + if (state->palette_after_ctm_blob)
>> + drm_property_reference_blob(state->palette_after_ctm_blob);
>> + if (state->palette_before_ctm_blob)
>> + drm_property_reference_blob(state->palette_before_ctm_blob);
>> state->mode_changed = false;
>> state->active_changed = false;
>> state->planes_changed = false;
>> @@ -2238,6 +2244,12 @@ void __drm_atomic_helper_crtc_destroy_state(struct
>> drm_crtc *crtc,
>> {
>> if (state->mode_blob)
>> drm_property_unreference_blob(state->mode_blob);
>> + if (state->ctm_blob)
>> + drm_property_unreference_blob(state->ctm_blob);
>> + if (state->palette_after_ctm_blob)
>> + drm_property_unreference_blob(state->palette_after_ctm_blob);
>> + if (state->palette_before_ctm_blob)
>> +
>> drm_property_unreference_blob(state->palette_before_ctm_blob);
>> }
>> EXPORT_SYMBOL(__drm_atomic_helper_crtc_destroy_state);
>>
>> diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
>> index 6e0f177..9cd4123 100644
>> --- a/include/drm/drm_crtc.h
>> +++ b/include/drm/drm_crtc.h
>> @@ -302,6 +302,11 @@ struct drm_crtc_state {
>> /* blob property to expose current mode to atomic userspace */
>> struct drm_property_blob *mode_blob;
>>
>> + /* blob properties to hold the color properties' blobs */
>> + struct drm_property_blob *palette_before_ctm_blob;
>> + struct drm_property_blob *palette_after_ctm_blob;
>> + struct drm_property_blob *ctm_blob;
>> +
>> struct drm_pending_vblank_event *event;
>>
>> struct drm_atomic_state *state;
>> @@ -1151,7 +1156,7 @@ struct drm_mode_config {
>> struct drm_property *cm_palette_after_ctm_property;
>> struct drm_property *cm_ctm_property;
>>
>> - /* Coor management capabilities query */
>> + /* Color management capabilities query */
> This should be part of the previous patch.
>
Agree, will do it.
> Regards,
> Emil
>
