[PATCH 02/22] drm/amd/display: Refactor to move color helper into module

Harry Wentland harry.wentland at amd.com
Fri Dec 23 14:58:39 UTC 2016


Hi Andreas,

On 2016-12-21 12:21 PM, Andreas Boll wrote:
> Hi,
>
> 2016-12-21 17:29 GMT+01:00 Harry Wentland <harry.wentland at amd.com>:
>> From: Anthony Koo <Anthony.Koo at amd.com>
>>
>> Refactor part 2 - Moving color helper into color module
>>
>> The color module will need table defined in the helper in order to calculate
>> the logical regamma curve, and also to fully handle gamut remapping.
>>
>> Change-Id: I826335cd42a9d5d0a872f9206fae83daeb1c45a6
>> Signed-off-by: Anthony Koo <anthony.koo at amd.com>
>> Reviewed-by: Aric Cyr <Aric.Cyr at amd.com>
>> Acked-by: Harry Wentland <Harry.Wentland at amd.com>
>> ---
>>  .../drm/amd/display/modules/color/color_helper.c   | 199 +++++++++++++++++++++
>>  .../drm/amd/display/modules/color/color_helper.h   |  50 ++++++
>>  .../gpu/drm/amd/display/modules/inc/mod_color.h    |   4 +-
>>  3 files changed, 251 insertions(+), 2 deletions(-)
>>  create mode 100644 drivers/gpu/drm/amd/display/modules/color/color_helper.c
>>  create mode 100644 drivers/gpu/drm/amd/display/modules/color/color_helper.h
>>
>> diff --git a/drivers/gpu/drm/amd/display/modules/color/color_helper.c b/drivers/gpu/drm/amd/display/modules/color/color_helper.c
>> new file mode 100644
>> index 000000000000..c47d2962c6d4
>> --- /dev/null
>> +++ b/drivers/gpu/drm/amd/display/modules/color/color_helper.c
>> @@ -0,0 +1,199 @@
>> +
>> +/**
>> + *  @file
>> + *  @date Copyright (c) 2008 Advanced Micro Devices, Inc. (unpublished)
>> + *
>> + *  @brief Helper functions for color gamut calculation
>> + *
>> + *  @internal
>> + *  All rights reserved.  This notice is intended as a precaution against
>> + *  inadvertent publication and does not imply publication or any waiver
>> + *  of confidentiality.  The year included in the foregoing notice is the
>> + *  year of creation of the work.
>> + */
>
> This copyright header needs to be fixed.

Thanks for spotting this.

>
>> diff --git a/drivers/gpu/drm/amd/display/modules/color/color_helper.h b/drivers/gpu/drm/amd/display/modules/color/color_helper.h
>> new file mode 100644
>> index 000000000000..c0e6334b947e
>> --- /dev/null
>> +++ b/drivers/gpu/drm/amd/display/modules/color/color_helper.h
>> @@ -0,0 +1,50 @@
>> +/**
>> + *  @file
>> + *  @date Copyright (c) 2008 Advanced Micro Devices, Inc. (unpublished)
>> + *
>> + *  @brief Helper functions for color gamut calculation
>> + *
>> + *  @internal
>> + *  All rights reserved.  This notice is intended as a precaution against
>> + *  inadvertent publication and does not imply publication or any waiver
>> + *  of confidentiality.  The year included in the foregoing notice is the
>> + *  year of creation of the work.
>> + */
>
> same here

All fixed in attached patch.

Harry

>
> Thanks,
> Andreas
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0010-drm-amd-display-Fix-licensing-header.patch
Type: text/x-patch
Size: 4543 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161223/c2ca9264/attachment.bin>


More information about the amd-gfx mailing list