On Fri, Oct 14, 2016 at 03:08:29PM +0100, Tvrtko Ursulin wrote: > > On 14/10/2016 14:31, Jani Nikula wrote: > >On Thu, 13 Oct 2016, Tvrtko Ursulin <tursulin@xxxxxxxxxxx> wrote: > >>From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > >> > >>unsigned long is too wide - use smaller types in > >>struct cxsr_latency to save 800-something bytes of .rodata. > >> > >>v2: All data even fits in u16 for even more saving. (Ville Syrjala) > >>v3: Move bitfields to the end of the struct. (Joonas Lahtinen) > >> > >>Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > >>Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > >Please learn how to run sparse, make it a habit to run it on your local > >branches before submitting patches, and make it a rule to run it before > >pushing patches. dim has helpers for this. > > Yeah I saw that you added dim sparse this week, however it only runs > if the dim tree is used for building it seems. > > I will try to figure out how to run in a separate build tree. apt-get install sparse make C=1 Start fuming. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
