Re: [Intel-gfx] [PATCH 4/5] drm/i915:Add the VCS2 switch in Intel_ring_setup_status_page for Gen7 to remove the switch check warning

[Zhao Yakui]

On Wed, 2014-04-09 at 08:29 -0600, Daniel Vetter wrote:
> On Wed, Apr 09, 2014 at 09:59:55AM +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.
> > 
> > Signed-off-by: Zhao Yakui <yakui.z...@intel.com>
> > ---
> >  drivers/gpu/drm/i915/intel_ringbuffer.c |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c 
> > b/drivers/gpu/drm/i915/intel_ringbuffer.c
> > index 11d0687..43e0227 100644
> > --- a/drivers/gpu/drm/i915/intel_ringbuffer.c
> > +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c
> > @@ -984,6 +984,7 @@ void intel_ring_setup_status_page(struct 
> > intel_ring_buffer *ring)
> >             case BCS:
> >                     mmio = BLT_HWS_PGA_GEN7;
> >                     break;
> > +           case VCS2:
> 
> Maybe add a /* doesn't actually exist but shuts up gcc */ comment?

Make sense.

I will update it.

Thanks.
    Yakui
> -Daniel
> 
> >             case VCS:
> >                     mmio = BSD_HWS_PGA_GEN7;
> >                     break;
> > -- 
> > 1.7.10.1
> > 
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 


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