On ke, 2017-01-18 at 12:18 +0000, Chris Wilson wrote: > We commonly use an inheritance style approach to device parameters, > where later generations inherit the defaults from earlier generations > and then override settings that change. For example, in i915_pci.c > BDW_FEATURES pulls in HSW_FEATURES, makes a few changes for 48bit > contexts and then individual Broadwell stanzas make further adjustments > for different GT configs. > > This causes a lot of warnings with make W=1 from -Woverride-init. We > could use > #pragma GCC diagnostic push > #pragma GCC diagnostic ignored "-Woverride-init" > ... > #pragma GCC diagnostic pop > around the offenders, but the pattern is used frequently enough in the > driver to prefer just disabling the warning entirely. > > Signed-off-by: Chris Wilson <[email protected]> > Cc: Joonas Lahtinen <[email protected] > Cc: Jani Nikula <[email protected]> > Cc: Daniel Vetter <[email protected]> > Cc: Tomi Sarvela <[email protected]>
<SNIP> > -subdir-ccflags-$(CONFIG_DRM_I915_WERROR) := -Werror > +subdir-ccflags-y := -Wno-override-init # used frequently for "inheritance" Why always on, if somebody upper level decides to -Werror, this is kinda unexpected for them? Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
