Thomas Zimmermann <tzimmermann@xxxxxxx> writes: > Consolidate all handling of CONFIG_DRM_FBDEV_LEAK_PHYS_SMEM by > making the module parameter optional in drm_fb_helper.c. > > Without the config option, modules can set smem_start in struct > fb_info for internal usage, but not export if to userspace. The > address can only be exported by enabling the option and setting > the module parameter. Also update the comment. > I was going to ask at what point we could just get rid of this Kconfig symbol since it already depends on EXPERT anyways so most distros will not enable it. But I looked then and noticed that it was added just a few years ago in commit 4be9bd10e22d "(drm/fb_helper: Allow leaking fbdev smem_start"), so it seems that people still need that :( > Signed-off-by: Thomas Zimmermann <tzimmermann@xxxxxxx> > --- The patch looks good to me. Reviewed-by: Javier Martinez Canillas <javierm@xxxxxxxxxx> -- Best regards, Javier Martinez Canillas Core Platforms Red Hat
