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