On Mon, Jan 09, 2023 at 07:58:08AM -0300, Maíra Canal wrote: > As the format validation is being dealt with exclusively > inside framebuffer_check(), there is no need to export the > drm_any_plane_has_format() symbol. > > Therefore, unexport the drm_any_plane_has_format() symbol, reinforcing > that format validation is being dealt with by the DRM API. > > Signed-off-by: Maíra Canal <mcanal@xxxxxxxxxx> Usually we also delete the kerneldoc at that point, since that's mainly for driver authors and they don't need this anymore. With that Also please move the function decl to the drm_crtc_internal.h since this is no longer used outside of drm core/helper code. Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > --- > drivers/gpu/drm/drm_plane.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/gpu/drm/drm_plane.c b/drivers/gpu/drm/drm_plane.c > index 24e7998d1731..67c0ab60c7b6 100644 > --- a/drivers/gpu/drm/drm_plane.c > +++ b/drivers/gpu/drm/drm_plane.c > @@ -837,7 +837,6 @@ bool drm_any_plane_has_format(struct drm_device *dev, > > return false; > } > -EXPORT_SYMBOL(drm_any_plane_has_format); > > /* > * __setplane_internal - setplane handler for internal callers > -- > 2.39.0 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
