On Tue, 04 Oct 2016, Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxx> No go, pushed this one. Thanks for the review. How about patch 2/2? BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx