[Intel-gfx] [PATCH] drm/i915: Add a default case in gen7 hwsp switch-case

Michel Thierry michel.thierry at intel.com
Tue Aug 29 19:21:12 UTC 2017


On 29/08/17 12:18, Chris Wilson wrote:
> Quoting Michel Thierry (2017-08-29 19:55:45)
>> Gen7 won't get any new engines, and we already added VCS2 there to just
>> silence gcc's not-handled-in-switch warnings.
>>
>> Use a default case instead, otherwise we will need to keep adding extra
>> cases if changes happen in the future.
>>
>> Signed-off-by: Michel Thierry <michel.thierry at intel.com>
> 
> Left spray paint behind, so
> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
> 
> (Not really a case of MISSING_CASE, as it is completely impossible, so
> GEM_BUG?)

True, GEM_BUG makes more sense here.


More information about the Intel-gfx mailing list