dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] radeon: don't take the stencil-specific codepath for buffers without stencil
@ 2012-09-29 13:12 Marek Olšák
  2012-10-03 15:48 ` Alex Deucher
  0 siblings, 1 reply; 2+ messages in thread
From: Marek Olšák @ 2012-09-29 13:12 UTC (permalink / raw)
  To: dri-devel

---
 radeon/radeon_surface.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/radeon/radeon_surface.c b/radeon/radeon_surface.c
index 80b1505..03b1c5d 100644
--- a/radeon/radeon_surface.c
+++ b/radeon/radeon_surface.c
@@ -911,7 +911,7 @@ static int eg_surface_best(struct radeon_surface_manager *surf_man,
      * fmask buffer has different optimal value figure them out once we
      * use it.
      */
-    if (surf->flags & (RADEON_SURF_ZBUFFER | RADEON_SURF_SBUFFER)) {
+    if (surf->flags & RADEON_SURF_SBUFFER) {
         /* assume 1 bytes for stencil, we optimize for stencil as stencil
          * and depth shares surface values
          */
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] radeon: don't take the stencil-specific codepath for buffers without stencil
  2012-09-29 13:12 [PATCH] radeon: don't take the stencil-specific codepath for buffers without stencil Marek Olšák
@ 2012-10-03 15:48 ` Alex Deucher
  0 siblings, 0 replies; 2+ messages in thread
From: Alex Deucher @ 2012-10-03 15:48 UTC (permalink / raw)
  To: Marek Olšák; +Cc: dri-devel

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

On Sat, Sep 29, 2012 at 9:12 AM, Marek Olšák <maraeo@gmail.com> wrote:
> ---
>  radeon/radeon_surface.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/radeon/radeon_surface.c b/radeon/radeon_surface.c
> index 80b1505..03b1c5d 100644
> --- a/radeon/radeon_surface.c
> +++ b/radeon/radeon_surface.c
> @@ -911,7 +911,7 @@ static int eg_surface_best(struct radeon_surface_manager *surf_man,
>       * fmask buffer has different optimal value figure them out once we
>       * use it.
>       */
> -    if (surf->flags & (RADEON_SURF_ZBUFFER | RADEON_SURF_SBUFFER)) {
> +    if (surf->flags & RADEON_SURF_SBUFFER) {
>          /* assume 1 bytes for stencil, we optimize for stencil as stencil
>           * and depth shares surface values
>           */
> --
> 1.7.9.5
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-10-03 15:48 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-29 13:12 [PATCH] radeon: don't take the stencil-specific codepath for buffers without stencil Marek Olšák
2012-10-03 15:48 ` Alex Deucher

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).