On Sat 09 Jul 2016, Jason Ekstrand wrote:
> ---
>  src/intel/isl/isl.c                                  | 2 +-
>  src/intel/isl/isl_gen6.c                             | 2 +-
>  src/intel/isl/isl_gen7.c                             | 2 +-
>  src/intel/isl/isl_storage_image.c                    | 4 ++--
>  src/intel/vulkan/anv_formats.c                       | 4 ++--
>  src/mesa/drivers/dri/i965/brw_fs_surface_builder.cpp | 4 ++--
>  6 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c
> index a3a9427..796b4cc 100644
> --- a/src/intel/isl/isl.c
> +++ b/src/intel/isl/isl.c
> @@ -996,7 +996,7 @@ isl_apply_surface_padding(const struct isl_device *dev,
>      *      padding requirements.
>      */
>     if (isl_format_is_yuv(info->format) &&
> -       (fmtl->bs == 96 || fmtl->bs == 48|| fmtl->bs == 24)) {
> +       (fmtl->bpb == 96 || fmtl->bpb == 48|| fmtl->bpb == 24)) {
>        *total_h_el += 1;
>        *pad_bytes += 16;
>     }

This first hunk is actually a bugfix, not a refactor. It should be
split into a separate patch.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to