diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2023-04-04 14:16:46 +0200 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2024-01-08 14:01:48 +0100 |
| commit | cad7b07bc72490ff9a0446113b90206f3a86ef78 (patch) | |
| tree | 2392ce0dfdb369fedd21b38fc27e781090468bfc | |
| parent | 634253d3f306fd04ad28fb0f647b1b58bf974550 (diff) | |
drm/fb-helper: Rename module option to avoid drm 2x
It's a bit too much. This might break some things, but I also think
module options aren't too much stable uapi.
Also while at it mark the dangerous option as unsafe, it leaks
physical addresses and can't really be used in a safe way by
userspace.
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Maxime Ripard <mripard@kernel.org>
Cc: Thomas Zimmermann <tzimmermann@suse.de>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
| -rw-r--r-- | drivers/gpu/drm/drm_fb_helper.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c index d612133e2cf7..b05e5bb8d58f 100644 --- a/drivers/gpu/drm/drm_fb_helper.c +++ b/drivers/gpu/drm/drm_fb_helper.c @@ -51,7 +51,7 @@ MODULE_PARM_DESC(fbdev_emulation, "Enable legacy fbdev emulation [default=true]"); static int drm_fbdev_overalloc = CONFIG_DRM_FBDEV_OVERALLOC; -module_param(drm_fbdev_overalloc, int, 0444); +module_param_named(fbdev_overalloc, drm_fbdev_overalloc, int, 0444); MODULE_PARM_DESC(drm_fbdev_overalloc, "Overallocation of the fbdev buffer (%) [default=" __MODULE_STRING(CONFIG_DRM_FBDEV_OVERALLOC) "]"); @@ -71,7 +71,7 @@ MODULE_PARM_DESC(drm_fbdev_overalloc, */ static bool drm_leak_fbdev_smem; #if IS_ENABLED(CONFIG_DRM_FBDEV_LEAK_PHYS_SMEM) -module_param_unsafe(drm_leak_fbdev_smem, bool, 0600); +module_param_named_unsafe(leak_fbdev_smem, drm_leak_fbdev_smem, bool, 0600); MODULE_PARM_DESC(drm_leak_fbdev_smem, "Allow unsafe leaking fbdev physical smem address [default=false]"); #endif |