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.thie...@intel.com>
---
 drivers/gpu/drm/i915/intel_ringbuffer.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c 
b/drivers/gpu/drm/i915/intel_ringbuffer.c
index c277a26bbd99..7d57a5971f39 100644
--- a/drivers/gpu/drm/i915/intel_ringbuffer.c
+++ b/drivers/gpu/drm/i915/intel_ringbuffer.c
@@ -409,10 +409,12 @@ static void intel_ring_setup_status_page(struct 
intel_engine_cs *engine)
                        mmio = BLT_HWS_PGA_GEN7;
                        break;
                /*
-                * VCS2 actually doesn't exist on Gen7. Only shut up
+                * No more rings exist on Gen7. Only shut up
                 * gcc switch check warning
                 */
-               case VCS2:
+               default:
+                       MISSING_CASE(engine->id);
+                       /* fall through */
                case VCS:
                        mmio = BSD_HWS_PGA_GEN7;
                        break;
-- 
2.14.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to