Commit 42e67c3b authored by Roman Li's avatar Roman Li Committed by Alex Deucher

drm/amd/display: make FBC mem alloc dynamic

- FBC init reworked to alloc memory based on display mode.
- Removed asic-dependencies from dm
Signed-off-by: default avatarRoman Li <Roman.Li@amd.com>
Reviewed-by: default avatarTony Cheng <Tony.Cheng@amd.com>
Acked-by: default avatarHarry Wentland <harry.wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 6d5d346f
...@@ -345,25 +345,53 @@ static void hotplug_notify_work_func(struct work_struct *work) ...@@ -345,25 +345,53 @@ static void hotplug_notify_work_func(struct work_struct *work)
} }
#if defined(CONFIG_DRM_AMD_DC_FBC) #if defined(CONFIG_DRM_AMD_DC_FBC)
#include "dal_asic_id.h"
/* Allocate memory for FBC compressed data */ /* Allocate memory for FBC compressed data */
/* TODO: Dynamic allocation */ static void amdgpu_dm_fbc_init(struct amdgpu_device *adev)
#define AMDGPU_FBC_SIZE (3840 * 2160 * 4)
static void amdgpu_dm_initialize_fbc(struct amdgpu_device *adev)
{ {
int r;
struct dm_comressor_info *compressor = &adev->dm.compressor; struct dm_comressor_info *compressor = &adev->dm.compressor;
struct drm_connector *conn;
struct drm_device *dev = adev->ddev;
unsigned long max_size = 0;
if (adev->dm.dc->fbc_compressor == NULL)
return;
if (compressor->bo_ptr)
return;
drm_modeset_lock(&dev->mode_config.connection_mutex, NULL);
if (!compressor->bo_ptr) { /* For eDP connector find a mode requiring max size */
r = amdgpu_bo_create_kernel(adev, AMDGPU_FBC_SIZE, PAGE_SIZE, list_for_each_entry(conn,
AMDGPU_GEM_DOMAIN_VRAM, &compressor->bo_ptr, &dev->mode_config.connector_list, head) {
&compressor->gpu_addr, &compressor->cpu_addr); struct amdgpu_dm_connector *aconn;
aconn = to_amdgpu_dm_connector(conn);
if (aconn->dc_link->connector_signal == SIGNAL_TYPE_EDP) {
struct drm_display_mode *mode;
list_for_each_entry(mode, &conn->modes, head) {
if (max_size < mode->hdisplay * mode->vdisplay)
max_size = mode->htotal * mode->vtotal;
}
}
}
if (max_size) {
int r = amdgpu_bo_create_kernel(adev, max_size * 4, PAGE_SIZE,
AMDGPU_GEM_DOMAIN_VRAM, &compressor->bo_ptr,
&compressor->gpu_addr, &compressor->cpu_addr);
if (r) if (r)
DRM_ERROR("DM: Failed to initialize fbc\n"); DRM_ERROR("DM: Failed to initialize FBC\n");
else {
adev->dm.dc->ctx->fbc_gpu_addr = compressor->gpu_addr;
DRM_INFO("DM: FBC alloc %lu\n", max_size*4);
}
} }
drm_modeset_unlock(&dev->mode_config.connection_mutex);
} }
#endif #endif
...@@ -422,11 +450,6 @@ static int amdgpu_dm_init(struct amdgpu_device *adev) ...@@ -422,11 +450,6 @@ static int amdgpu_dm_init(struct amdgpu_device *adev)
else else
init_data.log_mask = DC_MIN_LOG_MASK; init_data.log_mask = DC_MIN_LOG_MASK;
#if defined(CONFIG_DRM_AMD_DC_FBC)
if (adev->family == FAMILY_CZ)
amdgpu_dm_initialize_fbc(adev);
init_data.fbc_gpu_addr = adev->dm.compressor.gpu_addr;
#endif
/* Display Core create. */ /* Display Core create. */
adev->dm.dc = dc_create(&init_data); adev->dm.dc = dc_create(&init_data);
...@@ -540,9 +563,12 @@ static int detect_mst_link_for_all_connectors(struct drm_device *dev) ...@@ -540,9 +563,12 @@ static int detect_mst_link_for_all_connectors(struct drm_device *dev)
static int dm_late_init(void *handle) static int dm_late_init(void *handle)
{ {
struct drm_device *dev = ((struct amdgpu_device *)handle)->ddev; struct amdgpu_device *adev = (struct amdgpu_device *)handle;
return detect_mst_link_for_all_connectors(dev); #if defined(CONFIG_DRM_AMD_DC_FBC)
amdgpu_dm_fbc_init(adev);
#endif
return detect_mst_link_for_all_connectors(adev->ddev);
} }
static void s3_handle_mst(struct drm_device *dev, bool suspend) static void s3_handle_mst(struct drm_device *dev, bool suspend)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment