<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Nov 9, 2016 at 4:36 AM, Daniel Vetter <span dir="ltr"><<a href="mailto:daniel.vetter@ffwll.ch" target="_blank">daniel.vetter@ffwll.ch</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Not setting the fb modifiers flag is something different from setting<br>
the fb modifiers to 0 (which means explicitly linear). We kinda failed<br>
to document that properly. Spotted by Kristian.<br>
<br>
Cc: <a href="mailto:hoegsberg@google.com">hoegsberg@google.com</a><br>
Signed-off-by: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>><br>
</blockquote><div><br></div><div>Thanks, looks good.</div><div><br></div><div>Reviewed-by: Kristian H. Kristensen <<a href="mailto:hoegsberg@google.com">hoegsberg@google.com</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">---<br>
 include/uapi/drm/drm_fourcc.h | 10 ++++++++++<br>
 1 file changed, 10 insertions(+)<br>
<br>
diff --git a/include/uapi/drm/drm_fourcc.<wbr>h b/include/uapi/drm/drm_fourcc.<wbr>h<br>
index da49f69e4d7e..2fca7e1f6aab 100644<br>
--- a/include/uapi/drm/drm_fourcc.<wbr>h<br>
+++ b/include/uapi/drm/drm_fourcc.<wbr>h<br>
@@ -172,6 +172,16 @@ extern "C" {<br>
  * authoritative source for all of these.<br>
  */<br>
<br>
+/*<br>
+ * Linear Layout<br>
+ *<br>
+ * Just plain linear layout. Note that this is different from no specifying any<br>
+ * modifier (e.g. not setting DRM_MODE_FB_MODIFIERS in the DRM_ADDFB2 ioctl),<br>
+ * which tells the driver to also take driver-internal information into account<br>
+ * and so might actually result in a tiled framebuffer.<br>
+ */<br>
+#define DRM_FORMAT_MOD_LINEAR  fourcc_mod_code(NONE, 0)<br>
+<br>
 /* Intel framebuffer modifiers */<br>
<br>
 /*<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.7.4<br>
<br>
</font></span></blockquote></div><br></div></div>