DRM_FORMAT_MOD_NONE is in the list of vendors, which is pretty confusing. We already have DRM_FORMAT_MOD_VENDOR_NONE. Move it down in the list of format modifiers. DRM_FORMAT_MOD_NONE is an alias for DRM_FORMAT_MOD_LINEAR, however the name is confusing: NONE doesn't mean that the modifier is implicit, instead it means that the layout is linear. Deprecate it.
Showing the changes to include/uapi (1 of 1 file). Show all
Unified · Split
| @@ -350,7 +350,6 @@ extern "C" { · 349 unmodified lines | ||
| 350 | 350 | */ |
| 351 | 351 | |
| 352 | 352 | /* Vendor Ids: */ |
| 353 | −#define DRM_FORMAT_MOD_NONE 0 | |
| 354 | 353 | #define DRM_FORMAT_MOD_VENDOR_NONE 0 |
| 355 | 354 | #define DRM_FORMAT_MOD_VENDOR_INTEL 0x01 |
| 356 | 355 | #define DRM_FORMAT_MOD_VENDOR_AMD 0x02 |
| @@ -422,6 +421,16 @@ extern "C" { · 65 unmodified lines | ||
| 422 | 421 | */ |
| 423 | 422 | #define DRM_FORMAT_MOD_LINEAR fourcc_mod_code(NONE, 0) |
| 424 | 423 | |
| 424 | +/* | |
| 425 | + * Deprecated: use DRM_FORMAT_MOD_LINEAR instead | |
| 426 | + * | |
| 427 | + * The "none" format modifier doesn't actually mean that the modifier is | |
| 428 | + * implicit, instead it means that the layout is linear. Whether modifiers are | |
| 429 | + * used is out-of-band information carried in an API-specific way (e.g. in a | |
| 430 | + * flag for drm_mode_fb_cmd2). | |
| 431 | + */ | |
| 432 | +#define DRM_FORMAT_MOD_NONE 0 | |
| 433 | + | |
| 425 | 434 | /* Intel framebuffer modifiers */ |
| 426 | 435 | |
| 427 | 436 | /* |