<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Feb 28, 2018 at 8:21 AM, Pohjolainen, Topi <span dir="ltr"><<a href="mailto:topi.pohjolainen@gmail.com" target="_blank">topi.pohjolainen@gmail.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 Tue, Feb 27, 2018 at 02:40:46PM +0200, Pohjolainen, Topi wrote:<br>
> On Thu, Feb 22, 2018 at 11:06:46PM -0800, Jason Ekstrand wrote:<br>
> > ---<br>
> >  src/intel/isl/isl.c | 7 +++++--<br>
> >  src/intel/isl/isl.h | 7 +++++++<br>
> >  2 files changed, 12 insertions(+), 2 deletions(-)<br>
> ><br>
> > diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c<br>
> > index 46aff46..95c1271 100644<br>
> > --- a/src/intel/isl/isl.c<br>
> > +++ b/src/intel/isl/isl.c<br>
> > @@ -154,9 +154,11 @@ isl_device_get_sample_counts(<wbr>struct isl_device *dev)<br>
> >  /**<br>
> >   * @param[out] info is written only on success<br>
> >   */<br>
> > -static void<br>
> > +void<br>
> >  isl_tiling_get_info(enum isl_tiling tiling,<br>
> > +                    enum isl_surf_dim dim,<br>
> >                      uint32_t format_bpb,<br>
> > +                    uint32_t samples,<br>
><br>
> Callers are changed to provide the extra arguments in next patch. Perhaps<br>
> easiest to just merge these two.<br>
<br>
</span>With that fixed either in this patch or by merging, 6 and 7 are also:<br></blockquote><div><br></div><div>I decided to be lazy and just squash them. :-)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Reviewed-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br><div class="HOEnZb"><div class="h5"></div></div></blockquote><div><br></div><div>Thanks!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
><br>
> >                      struct isl_tile_info *tile_info)<br>
> >  {<br>
> >     const uint32_t bs = format_bpb / 8;<br>
> > @@ -1585,7 +1587,8 @@ isl_surf_get_tile_info(const struct isl_surf *surf,<br>
> >                         struct isl_tile_info *tile_info)<br>
> >  {<br>
> >     const struct isl_format_layout *fmtl = isl_format_get_layout(surf-><wbr>format);<br>
> > -   isl_tiling_get_info(surf-><wbr>tiling, fmtl->bpb, tile_info);<br>
> > +   isl_tiling_get_info(surf-><wbr>tiling, surf->dim, fmtl->bpb,<br>
> > +                       surf->samples, tile_info);<br>
> >  }<br>
> ><br>
> >  bool<br>
> > diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h<br>
> > index cdc76fe..aa5bfed 100644<br>
> > --- a/src/intel/isl/isl.h<br>
> > +++ b/src/intel/isl/isl.h<br>
> > @@ -1572,6 +1572,13 @@ isl_tiling_is_std_y(enum isl_tiling tiling)<br>
> >     return (1u << tiling) & ISL_TILING_STD_Y_MASK;<br>
> >  }<br>
> ><br>
> > +void<br>
> > +isl_tiling_get_info(enum isl_tiling tiling,<br>
> > +                    enum isl_surf_dim dim,<br>
> > +                    uint32_t format_bpb,<br>
> > +                    uint32_t samples,<br>
> > +                    struct isl_tile_info *tile_info);<br>
> > +<br>
> >  uint32_t<br>
> >  isl_tiling_to_i915_tiling(enum isl_tiling tiling);<br>
> ><br>
> > --<br>
> > 2.5.0.400.gff86faf<br>
> ><br>
> > ______________________________<wbr>_________________<br>
> > mesa-dev mailing list<br>
> > <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> > <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div></div>