On 08/09/16 17:44, Laurent Pinchart wrote: > The format helpers have historically treated unsupported formats as part > of the default case, returning values that are likely wrong. We can't > change this behaviour now without risking breaking drivers in difficult > to detect ways, but we can WARN on unsupported formats to catch faulty > callers. > > The only exception is the framebuffer_check() function that calls > drm_format_info() to validate the format passed from userspace. This is > a valid use case that shouldn't generate a warning. > > Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com> > --- > drivers/gpu/drm/drm_fourcc.c | 32 ++++++++++++++++++++++++-------- > drivers/gpu/drm/drm_framebuffer.c | 2 +- > include/drm/drm_fourcc.h | 1 + > 3 files changed, 26 insertions(+), 9 deletions(-)
Reviewed-by: Tomi Valkeinen <tomi.valkeinen at ti.com> Tomi -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160915/083050a3/attachment.sig>