diff options
| author | Zack Rusin <zackr@vmware.com> | 2022-10-18 22:43:50 -0400 |
|---|---|---|
| committer | Zack Rusin <zackr@vmware.com> | 2022-10-19 21:46:16 -0400 |
| commit | 7c99616e3fe7f35fe25bf6f5797267da29b4751e (patch) | |
| tree | c26ae66177a60d9c9c4a2824122cfaeeb8080179 /drivers/gpu/drm/radeon/radeon_display.c | |
| parent | b389286d0234e1edbaf62ed8bc0892a568c33662 (diff) | |
| download | lwn-7c99616e3fe7f35fe25bf6f5797267da29b4751e.tar.gz lwn-7c99616e3fe7f35fe25bf6f5797267da29b4751e.zip | |
drm: Remove drm_mode_config::fb_base
The fb_base in struct drm_mode_config has been unused for a long time.
Some drivers set it and some don't leading to a very confusing state
where the variable can't be relied upon, because there's no indication
as to which driver sets it and which doesn't.
The only usage of fb_base is internal to two drivers so instead of trying
to force it into all the drivers to get it into a coherent state
completely remove it.
Signed-off-by: Zack Rusin <zackr@vmware.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Thomas Zimmermann <tzimemrmann@suse.de>
Acked-by: Christian König <christian.koenig@amd.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20221019024401.394617-1-zack@kde.org
Diffstat (limited to 'drivers/gpu/drm/radeon/radeon_display.c')
| -rw-r--r-- | drivers/gpu/drm/radeon/radeon_display.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c index ca5598ae8bfc..9bed1a6cb163 100644 --- a/drivers/gpu/drm/radeon/radeon_display.c +++ b/drivers/gpu/drm/radeon/radeon_display.c @@ -1604,8 +1604,6 @@ int radeon_modeset_init(struct radeon_device *rdev) rdev->ddev->mode_config.fb_modifiers_not_supported = true; - rdev->ddev->mode_config.fb_base = rdev->mc.aper_base; - ret = radeon_modeset_create_props(rdev); if (ret) { return ret; |