<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 12, 2016 at 2:01 PM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat 09 Jul 2016, Jason Ekstrand wrote:<br>
> ---<br>
>  src/intel/isl/isl.c                                  | 2 +-<br>
>  src/intel/isl/isl_gen6.c                             | 2 +-<br>
>  src/intel/isl/isl_gen7.c                             | 2 +-<br>
>  src/intel/isl/isl_storage_image.c                    | 4 ++--<br>
>  src/intel/vulkan/anv_formats.c                       | 4 ++--<br>
>  src/mesa/drivers/dri/i965/brw_fs_surface_builder.cpp | 4 ++--<br>
>  6 files changed, 9 insertions(+), 9 deletions(-)<br>
><br>
> diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c<br>
> index a3a9427..796b4cc 100644<br>
> --- a/src/intel/isl/isl.c<br>
> +++ b/src/intel/isl/isl.c<br>
> @@ -996,7 +996,7 @@ isl_apply_surface_padding(const struct isl_device *dev,<br>
>      *      padding requirements.<br>
>      */<br>
>     if (isl_format_is_yuv(info->format) &&<br>
> -       (fmtl->bs == 96 || fmtl->bs == 48|| fmtl->bs == 24)) {<br>
> +       (fmtl->bpb == 96 || fmtl->bpb == 48|| fmtl->bpb == 24)) {<br>
>        *total_h_el += 1;<br>
>        *pad_bytes += 16;<br>
>     }<br>
<br>
</span>This first hunk is actually a bugfix, not a refactor. It should be<br>
split into a separate patch.<br>
</blockquote></div><br></div><div class="gmail_extra">Done<br></div></div>