Re: [PATCH] drm/radeon/r200: fix typo in packet checker for tracking textures

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jun 9, 2010 at 10:17 AM,  <rscheidegger_lists@xxxxxxxxxx> wrote:
> From: Roland Scheidegger <sroland@xxxxxxxxxx>
>
> using R100_TRACK_COMP_DXT1 instead of R100_TRACK_COMP_DXT35 for dxt3/5
> textures leads to size calculations only half as big as they should be.
>
> Signed-off-by: sroland@xxxxxxxxxx
> ---
>  drivers/gpu/drm/radeon/r200.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/r200.c b/drivers/gpu/drm/radeon/r200.c
> index 85617c3..aac4b99 100644
> --- a/drivers/gpu/drm/radeon/r200.c
> +++ b/drivers/gpu/drm/radeon/r200.c
> @@ -476,7 +476,7 @@ int r200_packet0_check(struct radeon_cs_parser *p,
>                case R200_TXFORMAT_DXT23:
>                case R200_TXFORMAT_DXT45:
>                        track->textures[i].cpp = 1;
> -                       track->textures[i].compress_format = R100_TRACK_COMP_DXT1;
> +                       track->textures[i].compress_format = R100_TRACK_COMP_DXT35;
>                        break;
>                }
>                track->textures[i].cube_info[4].width = 1 << ((idx_value >> 16) & 0xf);
> --
> 1.5.2.4

Blatant typo, as discussed on IRC. Very nice catch.

Reviewed-by: Corbin Simpson <MostAwesomeDude@xxxxxxxxx>

~ C.

-- 
When the facts change, I change my mind. What do you do, sir? ~ Keynes

Corbin Simpson
<MostAwesomeDude@xxxxxxxxx>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux