On Wed, 2014-04-16 at 10:41 +0800, Zhao Yakui wrote: > The Gen7 doesn't have the second BSD ring. But it will complain the switch check > warning message during compilation. So just add it to remove the > switch check warning. > > V1->V2: Follow Daniel's comment to update the comment > > Signed-off-by: Zhao Yakui <yakui.zhao@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_ringbuffer.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c > index 8b9b89080..2c89525 100644 > --- a/drivers/gpu/drm/i915/intel_ringbuffer.c > +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c > @@ -988,6 +988,11 @@ void intel_ring_setup_status_page(struct intel_ring_buffer *ring) > case BCS: > mmio = BLT_HWS_PGA_GEN7; > break; > + /* > + * VCS2 actually doesn't exist on Gen7. Only shut up > + * gcc switch check warning > + */ > + case VCS2: A WARN would've been better here, but in any case: Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx> > case VCS: > mmio = BSD_HWS_PGA_GEN7; > break;
Attachment:
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
