Re: [PATCH] drm: deprecate DRM_FORMAT_MOD_NONE

2020-10-26 Thread Daniel Vetter
On Mon, Oct 26, 2020 at 2:17 PM Simon Ser wrote: > > 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 >

Re: [PATCH] drm: deprecate DRM_FORMAT_MOD_NONE

2020-10-26 Thread Pekka Paalanen
On Mon, 26 Oct 2020 13:17:02 + Simon Ser wrote: > 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

Re: [PATCH] drm: deprecate DRM_FORMAT_MOD_NONE

2020-10-26 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen On Mon, Oct 26, 2020 at 2:17 PM Simon Ser wrote: > > 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_

[PATCH] drm: deprecate DRM_FORMAT_MOD_NONE

2020-10-26 Thread Simon Ser
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 impli