vk/formats: Document new meaning of anv_format::cpp
authorChad Versace <chad.versace@intel.com>
Tue, 30 Jun 2015 16:20:07 +0000 (09:20 -0700)
committerChad Versace <chad.versace@intel.com>
Tue, 30 Jun 2015 16:41:41 +0000 (09:41 -0700)
The way the code currently works is that anv_format::cpp is the cpp of
anv_format::surface_format.

Me and Kristian disagree about how the code *should* work. Despite that,
I think it's in our discussion's best interest to document how the code
*currently* works. That should eliminate confusion.

If and when the code begins to work differently, then we'll update the
anv_format comments.

src/vulkan/private.h

index efb97391c56748e558eb8696383e72bfabf80a9a..0f5986b8b8ab0951c80c2b8bfbc30b044874f6d6 100644 (file)
@@ -775,7 +775,7 @@ void anv_compiler_free(struct anv_pipeline *pipeline);
 struct anv_format {
    const char *name;
    uint16_t surface_format; /**< RENDER_SURFACE_STATE.SurfaceFormat */
-   uint8_t cpp;
+   uint8_t cpp; /**< Bytes-per-pixel of anv_format::surface_format. */
    uint8_t num_channels;
    uint8_t depth_format; /**< 3DSTATE_DEPTH_BUFFER.SurfaceFormat */
    bool has_stencil;