Commit 01a526f3 authored by Harry Wentland's avatar Harry Wentland Committed by Alex Deucher

drm/amd/display: Pass log_mask from DM

Linux and Windows often desire different log levels.
Signed-off-by: default avatarHarry Wentland <harry.wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 02e749dc
...@@ -374,6 +374,8 @@ int amdgpu_dm_init(struct amdgpu_device *adev) ...@@ -374,6 +374,8 @@ int amdgpu_dm_init(struct amdgpu_device *adev)
init_data.dce_environment = DCE_ENV_PRODUCTION_DRV; init_data.dce_environment = DCE_ENV_PRODUCTION_DRV;
init_data.log_mask = DC_DEFAULT_LOG_MASK;
#ifdef ENABLE_FBC #ifdef ENABLE_FBC
if (adev->family == FAMILY_CZ) if (adev->family == FAMILY_CZ)
amdgpu_dm_initialize_fbc(adev); amdgpu_dm_initialize_fbc(adev);
......
...@@ -64,40 +64,9 @@ static const struct dc_log_type_info log_type_info_tbl[] = { ...@@ -64,40 +64,9 @@ static const struct dc_log_type_info log_type_info_tbl[] = {
}; };
#define DC_DEFAULT_LOG_MASK ((1 << LOG_ERROR) | \
(1 << LOG_WARNING) | \
(1 << LOG_EVENT_MODE_SET) | \
(1 << LOG_EVENT_DETECTION) | \
(1 << LOG_EVENT_LINK_TRAINING) | \
(1 << LOG_EVENT_LINK_LOSS) | \
(1 << LOG_EVENT_UNDERFLOW) | \
(1 << LOG_RESOURCE) | \
(1 << LOG_FEATURE_OVERRIDE) | \
(1 << LOG_DETECTION_EDID_PARSER) | \
(1 << LOG_DC) | \
(1 << LOG_HW_HOTPLUG) | \
(1 << LOG_HW_SET_MODE) | \
(1 << LOG_HW_RESUME_S3) | \
(1 << LOG_HW_HPD_IRQ) | \
(1 << LOG_SYNC) | \
(1 << LOG_BANDWIDTH_VALIDATION) | \
(1 << LOG_MST) | \
(1 << LOG_DETECTION_DP_CAPS) | \
(1 << LOG_BACKLIGHT)) | \
(1 << LOG_I2C_AUX) | \
(1 << LOG_IF_TRACE) | \
(1 << LOG_DTN) /* | \
(1 << LOG_DEBUG) | \
(1 << LOG_BIOS) | \
(1 << LOG_SURFACE) | \
(1 << LOG_SCALER) | \
(1 << LOG_DML) | \
(1 << LOG_HW_LINK_TRAINING) | \
(1 << LOG_HW_AUDIO)| \
(1 << LOG_BANDWIDTH_CALCS)*/
/* ----------- Object init and destruction ----------- */ /* ----------- Object init and destruction ----------- */
static bool construct(struct dc_context *ctx, struct dal_logger *logger) static bool construct(struct dc_context *ctx, struct dal_logger *logger,
uint32_t log_mask)
{ {
/* malloc buffer and init offsets */ /* malloc buffer and init offsets */
logger->log_buffer_size = DAL_LOGGER_BUFFER_MAX_SIZE; logger->log_buffer_size = DAL_LOGGER_BUFFER_MAX_SIZE;
...@@ -120,7 +89,7 @@ static bool construct(struct dc_context *ctx, struct dal_logger *logger) ...@@ -120,7 +89,7 @@ static bool construct(struct dc_context *ctx, struct dal_logger *logger)
logger->ctx = ctx; logger->ctx = ctx;
logger->mask = DC_DEFAULT_LOG_MASK; logger->mask = log_mask;
return true; return true;
} }
...@@ -133,14 +102,14 @@ static void destruct(struct dal_logger *logger) ...@@ -133,14 +102,14 @@ static void destruct(struct dal_logger *logger)
} }
} }
struct dal_logger *dal_logger_create(struct dc_context *ctx) struct dal_logger *dal_logger_create(struct dc_context *ctx, uint32_t log_mask)
{ {
/* malloc struct */ /* malloc struct */
struct dal_logger *logger = dm_alloc(sizeof(struct dal_logger)); struct dal_logger *logger = dm_alloc(sizeof(struct dal_logger));
if (!logger) if (!logger)
return NULL; return NULL;
if (!construct(ctx, logger)) { if (!construct(ctx, logger, log_mask)) {
dm_free(logger); dm_free(logger);
return NULL; return NULL;
} }
......
...@@ -493,7 +493,7 @@ static bool construct(struct dc *dc, ...@@ -493,7 +493,7 @@ static bool construct(struct dc *dc,
dc_ctx->asic_id = init_params->asic_id; dc_ctx->asic_id = init_params->asic_id;
/* Create logger */ /* Create logger */
logger = dal_logger_create(dc_ctx); logger = dal_logger_create(dc_ctx, init_params->log_mask);
if (!logger) { if (!logger) {
/* can *not* call logger. call base driver 'print error' */ /* can *not* call logger. call base driver 'print error' */
......
...@@ -267,6 +267,7 @@ struct dc_init_data { ...@@ -267,6 +267,7 @@ struct dc_init_data {
enum dce_environment dce_environment; enum dce_environment dce_environment;
struct dc_config flags; struct dc_config flags;
uint32_t log_mask;
#ifdef ENABLE_FBC #ifdef ENABLE_FBC
uint64_t fbc_gpu_addr; uint64_t fbc_gpu_addr;
#endif #endif
......
...@@ -40,7 +40,7 @@ struct dc_state; ...@@ -40,7 +40,7 @@ struct dc_state;
* *
*/ */
struct dal_logger *dal_logger_create(struct dc_context *ctx); struct dal_logger *dal_logger_create(struct dc_context *ctx, uint32_t log_mask);
uint32_t dal_logger_destroy(struct dal_logger **logger); uint32_t dal_logger_destroy(struct dal_logger **logger);
......
...@@ -70,6 +70,38 @@ enum dc_log_type { ...@@ -70,6 +70,38 @@ enum dc_log_type {
LOG_SECTION_TOTAL_COUNT LOG_SECTION_TOTAL_COUNT
}; };
#define DC_DEFAULT_LOG_MASK ((1 << LOG_ERROR) | \
(1 << LOG_WARNING) | \
(1 << LOG_EVENT_MODE_SET) | \
(1 << LOG_EVENT_DETECTION) | \
(1 << LOG_EVENT_LINK_TRAINING) | \
(1 << LOG_EVENT_LINK_LOSS) | \
(1 << LOG_EVENT_UNDERFLOW) | \
(1 << LOG_RESOURCE) | \
(1 << LOG_FEATURE_OVERRIDE) | \
(1 << LOG_DETECTION_EDID_PARSER) | \
(1 << LOG_DC) | \
(1 << LOG_HW_HOTPLUG) | \
(1 << LOG_HW_SET_MODE) | \
(1 << LOG_HW_RESUME_S3) | \
(1 << LOG_HW_HPD_IRQ) | \
(1 << LOG_SYNC) | \
(1 << LOG_BANDWIDTH_VALIDATION) | \
(1 << LOG_MST) | \
(1 << LOG_DETECTION_DP_CAPS) | \
(1 << LOG_BACKLIGHT)) | \
(1 << LOG_I2C_AUX) | \
(1 << LOG_IF_TRACE) | \
(1 << LOG_DTN) /* | \
(1 << LOG_DEBUG) | \
(1 << LOG_BIOS) | \
(1 << LOG_SURFACE) | \
(1 << LOG_SCALER) | \
(1 << LOG_DML) | \
(1 << LOG_HW_LINK_TRAINING) | \
(1 << LOG_HW_AUDIO)| \
(1 << LOG_BANDWIDTH_CALCS)*/
union logger_flags { union logger_flags {
struct { struct {
uint32_t ENABLE_CONSOLE:1; /* Print to console */ uint32_t ENABLE_CONSOLE:1; /* Print to console */
......
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