On Wed, 13 Jun 2018, Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx> wrote:
> On 13/06/18 12:49, Jani Nikula wrote:
>> No reason not to be const.
>>
>> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
>
> Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
Pushed to dinq, thanks for the review.
BR,
Jani.
>
>> ---
>> drivers/gpu/drm/i915/i915_perf.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm/i915/i915_perf.c
>> index 881a992305ec..447407fee3b8 100644
>> --- a/drivers/gpu/drm/i915/i915_perf.c
>> +++ b/drivers/gpu/drm/i915/i915_perf.c
>> @@ -315,7 +315,7 @@ static u32 i915_oa_max_sample_rate = 100000;
>> * code assumes all reports have a power-of-two size and ~(size - 1) can
>> * be used as a mask to align the OA tail pointer.
>> */
>> -static struct i915_oa_format hsw_oa_formats[I915_OA_FORMAT_MAX] = {
>> +static const struct i915_oa_format hsw_oa_formats[I915_OA_FORMAT_MAX] = {
>> [I915_OA_FORMAT_A13] = { 0, 64 },
>> [I915_OA_FORMAT_A29] = { 1, 128 },
>> [I915_OA_FORMAT_A13_B8_C8] = { 2, 128 },
>> @@ -326,7 +326,7 @@ static struct i915_oa_format hsw_oa_formats[I915_OA_FORMAT_MAX] = {
>> [I915_OA_FORMAT_C4_B8] = { 7, 64 },
>> };
>>
>> -static struct i915_oa_format gen8_plus_oa_formats[I915_OA_FORMAT_MAX] = {
>> +static const struct i915_oa_format gen8_plus_oa_formats[I915_OA_FORMAT_MAX] = {
>> [I915_OA_FORMAT_A12] = { 0, 64 },
>> [I915_OA_FORMAT_A12_B8_C8] = { 2, 128 },
>> [I915_OA_FORMAT_A32u40_A4u32_B8_C8] = { 5, 256 },
>
>
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
