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 <[email protected]>
> ---
>  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 <[email protected]>

>               case VCS:
>                       mmio = BSD_HWS_PGA_GEN7;
>                       break;

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to