[Intel-gfx] [PATCH 1/2] Fix pitchAlignMask on i915

Daniel Vetter daniel at ffwll.ch
Wed Mar 17 20:39:48 CET 2010


Acked-by: Daniel Vetter <daniel.vetter at ffwll.ch>

This won't break anything, it'll just waste a few bytes of memory due to
a too big alignment. So not really 2.11 material.

On Wed, Mar 17, 2010 at 01:26:03PM -0400, mattst88 at gmail.com wrote:
> From: Matt Turner <mattst88 at gmail.com>
> 
> Introduced in cb06aa32d433f54affe87da2cf964f0308d3c258
> 
> CC: Eric Anholt <eric at anholt.net>
> CC: Carl Worth <cworth at cworth.org>
> CC: Chris Wilson <chris at chris-wilson.co.uk>
> CC: Daniel Vetter <daniel.vetter at ffwll.ch>
> Signed-off-by: Matt Turner <mattst88 at gmail.com>
> ---
>  src/i830_video.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/i830_video.c b/src/i830_video.c
> index db72863..4e1b9ce 100644
> --- a/src/i830_video.c
> +++ b/src/i830_video.c
> @@ -1362,7 +1362,7 @@ i830_setup_dst_params(ScrnInfoPtr scrn, intel_adaptor_private *adaptor_priv, sho
>  		if (IS_I965G(intel))
>  			pitchAlignMask = 255;
>  		else
> -			pitchAlignMask = 255;
> +			pitchAlignMask = 63;
>  	}
>  
>  #if INTEL_XVMC
> -- 
> 1.6.4.4
> 

-- 
Daniel Vetter
Mail: daniel at ffwll.ch
Mobile: +41 (0)79 365 57 48



More information about the Intel-gfx mailing list