<div dir="ltr">I don't really like the lack of isl_ prefix.  We could call it isl_tiling_from_i915_tiling().  I don't care all that much though.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 19, 2017 at 12:48 PM, Topi Pohjolainen <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">Suggested-by: Daniel Stone <<a href="mailto:daniels@collabora.com">daniels@collabora.com</a>><br>
Signed-off-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br>
---<br>
 src/intel/isl/isl.h     |  3 +++<br>
 src/intel/isl/isl_drm.c | 17 +++++++++++++++++<br>
 2 files changed, 20 insertions(+)<br>
<br>
diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h<br>
index d81df31a37..af46b6bf29 100644<br>
--- a/src/intel/isl/isl.h<br>
+++ b/src/intel/isl/isl.h<br>
@@ -1516,6 +1516,9 @@ isl_tiling_is_std_y(enum isl_tiling tiling)<br>
 uint32_t<br>
 isl_tiling_to_i915_tiling(enum isl_tiling tiling);<br>
<br>
+enum isl_tiling<br>
+i915_tiling_to_isl_tiling(<wbr>uint32_t tiling);<br>
+<br>
 const struct isl_drm_modifier_info * ATTRIBUTE_CONST<br>
 isl_drm_modifier_get_info(<wbr>uint64_t modifier);<br>
<br>
diff --git a/src/intel/isl/isl_drm.c b/src/intel/isl/isl_drm.c<br>
index b7a4997d2d..b8f8e1214e 100644<br>
--- a/src/intel/isl/isl_drm.c<br>
+++ b/src/intel/isl/isl_drm.c<br>
@@ -54,6 +54,23 @@ isl_tiling_to_i915_tiling(enum isl_tiling tiling)<br>
    unreachable("Invalid ISL tiling");<br>
 }<br>
<br>
+enum isl_tiling<br>
+i915_tiling_to_isl_tiling(<wbr>uint32_t tiling)<br>
+{<br>
+   switch (tiling) {<br>
+   case I915_TILING_NONE:<br>
+      return ISL_TILING_LINEAR;<br>
+<br>
+   case I915_TILING_X:<br>
+      return ISL_TILING_X;<br>
+<br>
+   case I915_TILING_Y:<br>
+      return ISL_TILING_Y0;<br>
+   }<br>
+<br>
+   unreachable("Invalid i915 tiling");<br>
+}<br>
+<br>
 struct isl_drm_modifier_info modifier_info[] = {<br>
    {<br>
       .modifier = DRM_FORMAT_MOD_NONE,<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.11.0<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>
</font></span></blockquote></div><br></div>