| Message ID | 1397008796-3595-5-git-send-email-yakui.zhao@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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.zhao@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? -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
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.zhao@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 >
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: case VCS: mmio = BSD_HWS_PGA_GEN7; break;
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.zhao@intel.com> --- drivers/gpu/drm/i915/intel_ringbuffer.c | 1 + 1 file changed, 1 insertion(+)