[PATCH RESEND] drm/mediatek: Add valid modifier check

Daniel Stone daniel at fooishbar.org
Tue Jul 25 10:28:51 UTC 2023


On Mon, 24 Jul 2023 at 18:58, Justin Green <greenjustin at chromium.org> wrote:

> +       if (cmd->modifier[0] &&
>

This is not DRM_FORMAT_MOD_INVALID. Please either explicitly compare
against INVALID if that's what you meant, or against LINEAR if that's what
you meant, or both.

Cheers,
Daniel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20230725/b8b5678f/attachment.htm>


More information about the dri-devel mailing list