Hi all,
This patchset supersedes [1]. Now the title is a bit misleading, but I left it this way to (hopefully) facilitate the maintainers' work.
A little context: Originally, I sent a patch adding modifiers to the VKMS driver and Simon Ser kindly reviewed it and pointed out that "format_mod_supported" was missing [2]. I asked if the docs were incorrect or if it was a bug in "create_in_format_blob".
In the first version of this series, Simon Ser and Dmitry Baryshkov agreed [1] that the code should behave as documented and "create_in_format_blob" should be changed.
The second version implemented the required changes and drops the "format_mod_supported" in the drivers that can use the default implementation. [3]
This third version fixes a compiler warning and adds the reviewed by tags.
Thanks, José Expósito
[1] https://lore.kernel.org/dri-devel/CAA8EJpqJ-tWmb5Ba6XSK59toCtLb3nRRmVH8da4Ud... [2] https://lore.kernel.org/dri-devel/20211216170532.GA16349@elementary/T/ [3] https://lore.kernel.org/dri-devel/20211222090552.25972-1-jose.exposito89@gma...
José Expósito (6): drm/plane: Make format_mod_supported truly optional drm/plane: Fix typo in format_mod_supported documentation drm/simple-kms: Drop format_mod_supported function drm/i915/display: Drop format_mod_supported function drm: mxsfb: Drop format_mod_supported function drm/stm: ltdc: Drop format_mod_supported function
drivers/gpu/drm/drm_plane.c | 9 ++------- drivers/gpu/drm/drm_simple_kms_helper.c | 8 -------- drivers/gpu/drm/i915/display/intel_cursor.c | 8 -------- drivers/gpu/drm/mxsfb/mxsfb_kms.c | 8 -------- drivers/gpu/drm/stm/ltdc.c | 11 ----------- include/drm/drm_plane.h | 2 +- 6 files changed, 3 insertions(+), 43 deletions(-)