[Intel-gfx] [PATCH RESEND 1/2] drm/i915: workaround sparse warning on variable length arrays

Jani Nikula jani.nikula at intel.com
Tue Oct 4 10:09:19 UTC 2016


On Tue, 04 Oct 2016, Joonas Lahtinen <joonas.lahtinen at linux.intel.com> wrote:
> On ti, 2016-10-04 at 12:54 +0300, Jani Nikula wrote:
>> @@ -192,7 +192,7 @@ static void broadwell_sseu_info_init(struct drm_i915_private *dev_priv)
>>  	struct sseu_dev_info *sseu = &mkwrite_device_info(dev_priv)->sseu;
>>  	const int s_max = 3, ss_max = 3, eu_max = 8;
>
> Does sparse shut up if you split these into their own lines?
>
>>  	int s, ss;
>> -	u32 fuse2, eu_disable[s_max];
>> +	u32 fuse2, eu_disable[3]; /* s_max */
>
> If not;
>
> Reviewed-by: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>

No go, pushed this one. Thanks for the review.

How about patch 2/2?

BR,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center


More information about the Intel-gfx mailing list