[Mesa-dev] [PATCH 07/14] isl: Use bpb in a few places where it's more natural than bs

[Jason Ekstrand]

On Tue, Jul 12, 2016 at 2:01 PM, Chad Versace <chad.versace at intel.com>
wrote:

> 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.
>

Done
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160712/5f5f953d/attachment.html>