Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9ba4378c authored by Gopikrishnaiah Anandan's avatar Gopikrishnaiah Anandan Committed by Gerrit - the friendly Code Review server
Browse files

drm/sde/msm: add support for AD properties 



If sde has assertive display block, driver will expose the properties
that user-space clients can set/get. Change adds the support for ad
properties in color processing driver.

Change-Id: I1a25d2def55e947c8adcb48ef5f2226b22c1ae5f
Signed-off-by: default avatarGopikrishnaiah Anandan <agopik@codeaurora.org>
parent 05688643

drivers/gpu/drm/msm/Makefile

+1 −0
Original line number Diff line number Diff line
@@ -51,6 +51,7 @@ msm_drm-y := \ 
	sde/sde_hw_color_proc_v4.o \

	sde_rsc.o \

	sde_rsc_hw.o \

	sde/sde_hw_ad4.o \



# use drm gpu driver only if qcom_kgsl driver not available

ifneq ($(CONFIG_QCOM_KGSL),y)

drivers/gpu/drm/msm/sde/sde_ad4.h

0 → 100644
+79 −0
Original line number Diff line number Diff line 
/* Copyright (c) 2017, The Linux Foundation. All rights reserved.

 *

 * This program is free software; you can redistribute it and/or modify

 * it under the terms of the GNU General Public License version 2 and

 * only version 2 as published by the Free Software Foundation.

 *

 * This program is distributed in the hope that it will be useful,

 * but WITHOUT ANY WARRANTY; without even the implied warranty of

 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the

 * GNU General Public License for more details.

 */

#ifndef _SDE_AD4_H_

#define _SDE_AD4_H_



#include <drm/drm_mode.h>

#include <drm/drm_property.h>

#include "sde_hw_dspp.h"



/**

 * enum ad4_modes - ad4 modes supported by driver

 */

enum ad4_modes {

	AD4_OFF,

	AD4_AUTO_STRENGTH,

	AD4_CALIBRATION,

	AD4_MANUAL,

};



/**

 * struct drm_prop_enum_list - drm structure for creating enum property and

 *                             enumerating values

 */

static const struct drm_prop_enum_list ad4_modes[] = {

	{AD4_OFF, "off"},

	{AD4_AUTO_STRENGTH, "auto_strength_mode"},

	{AD4_CALIBRATION, "calibration_mode"},

	{AD4_MANUAL, "manual_mode"},

};



/**

 * enum ad_property - properties that can be set for ad

 */

enum ad_property {

	AD_MODE,

	AD_INIT,

	AD_CFG,

	AD_INPUT,

	AD_SUSPEND,

	AD_ASSERTIVE,

	AD_BACKLIGHT,

	AD_PROPMAX,

};



/**

 * struct sde_ad_hw_cfg - structure for setting the ad properties

 * @prop: enum of ad property

 * @hw_cfg: payload for the prop being set.

 */

struct sde_ad_hw_cfg {

	enum ad_property prop;

	struct sde_hw_cp_cfg *hw_cfg;

};



/**

 * sde_validate_dspp_ad4() - api to validate if ad property is allowed for

 *                           the display with allocated dspp/mixers.

 * @dspp: pointer to dspp info structure.

 * @prop: pointer to u32 pointing to ad property

 */

int sde_validate_dspp_ad4(struct sde_hw_dspp *dspp, u32 *prop);



/**

 * sde_setup_dspp_ad4 - api to apply the ad property, sde_validate_dspp_ad4

 *                      should be called before call this function

 * @dspp: pointer to dspp info structure.

 * @cfg: pointer to struct sde_ad_hw_cfg

 */

void sde_setup_dspp_ad4(struct sde_hw_dspp *dspp, void *cfg);

#endif /* _SDE_AD4_H_ */

drivers/gpu/drm/msm/sde/sde_color_processing.c

+273 −22
Original line number Diff line number Diff line
@@ -21,6 +21,7 @@ 
#include "sde_crtc.h"

#include "sde_hw_dspp.h"

#include "sde_hw_lm.h"

#include "sde_ad4.h"



struct sde_cp_node {

	u32 property_id;
@@ -60,6 +61,12 @@ typedef void (*dspp_prop_install_func_t)(struct drm_crtc *crtc); 


static dspp_prop_install_func_t dspp_prop_install_func[SDE_DSPP_MAX];



static void sde_cp_update_list(struct sde_cp_node *prop_node,

		struct sde_crtc *crtc, bool dirty_list);



static int sde_cp_ad_validate_prop(struct sde_cp_node *prop_node,

		struct sde_crtc *crtc);



#define setup_dspp_prop_install_funcs(func) \

do { \

	func[SDE_DSPP_PCC] = dspp_pcc_install_property; \
@@ -96,6 +103,12 @@ enum { 
	SDE_CP_CRTC_DSPP_HIST,

	SDE_CP_CRTC_DSPP_AD,

	SDE_CP_CRTC_DSPP_VLUT,

	SDE_CP_CRTC_DSPP_AD_MODE,

	SDE_CP_CRTC_DSPP_AD_INIT,

	SDE_CP_CRTC_DSPP_AD_CFG,

	SDE_CP_CRTC_DSPP_AD_INPUT,

	SDE_CP_CRTC_DSPP_AD_ASSERTIVENESS,

	SDE_CP_CRTC_DSPP_AD_BACKLIGHT,

	SDE_CP_CRTC_DSPP_MAX,

	/* DSPP features end */
@@ -136,9 +149,10 @@ static void sde_cp_get_hw_payload(struct sde_cp_node *prop_node, 
	} else if (prop_node->prop_flags & DRM_MODE_PROP_RANGE) {

		/* Check if local blob is Set */

		if (!blob) {

			if (prop_node->prop_val) {

				hw_cfg->len = sizeof(prop_node->prop_val);

			if (prop_node->prop_val)

				hw_cfg->payload = &prop_node->prop_val;

			}

		} else {

			hw_cfg->len = (prop_node->prop_val) ? blob->length :

					0;
@@ -147,6 +161,10 @@ static void sde_cp_get_hw_payload(struct sde_cp_node *prop_node, 
		}

		if (prop_node->prop_val)

			*feature_enabled = true;

	} else if (prop_node->prop_flags & DRM_MODE_PROP_ENUM) {

		*feature_enabled = (prop_node->prop_val != 0);

		hw_cfg->len = sizeof(prop_node->prop_val);

		hw_cfg->payload = &prop_node->prop_val;

	} else {

		DRM_ERROR("property type is not supported\n");

	}
@@ -178,7 +196,7 @@ static int sde_cp_create_local_blob(struct drm_crtc *crtc, u32 feature, int len) 
		}

	}



	if (!found || prop_node->prop_flags & DRM_MODE_PROP_BLOB) {

	if (!found || !(prop_node->prop_flags & DRM_MODE_PROP_RANGE)) {

		DRM_ERROR("local blob create failed prop found %d flags %d\n",

		       found, prop_node->prop_flags);

		return ret;
@@ -232,10 +250,14 @@ static int sde_cp_disable_crtc_property(struct drm_crtc *crtc, 
{

	int ret = -EINVAL;



	if (property->flags & DRM_MODE_PROP_BLOB)

	if (property->flags & DRM_MODE_PROP_BLOB) {

		ret = sde_cp_disable_crtc_blob_property(prop_node);

	else if (property->flags & DRM_MODE_PROP_RANGE)

	} else if (property->flags & DRM_MODE_PROP_RANGE) {

		ret = sde_cp_handle_range_property(prop_node, 0);

	} else if (property->flags & DRM_MODE_PROP_ENUM) {

		ret = 0;

		prop_node->prop_val = 0;

	}

	return ret;

}
@@ -275,10 +297,14 @@ static int sde_cp_enable_crtc_property(struct drm_crtc *crtc, 
{

	int ret = -EINVAL;



	if (property->flags & DRM_MODE_PROP_BLOB)

	if (property->flags & DRM_MODE_PROP_BLOB) {

		ret = sde_cp_enable_crtc_blob_property(crtc, prop_node, val);

	else if (property->flags & DRM_MODE_PROP_RANGE)

	} else if (property->flags & DRM_MODE_PROP_RANGE) {

		ret = sde_cp_handle_range_property(prop_node, val);

	} else if (property->flags & DRM_MODE_PROP_ENUM) {

		ret = 0;

		prop_node->prop_val = val;

	}

	return ret;

}
@@ -331,6 +357,8 @@ void sde_cp_crtc_init(struct drm_crtc *crtc) 
	INIT_LIST_HEAD(&sde_crtc->active_list);

	INIT_LIST_HEAD(&sde_crtc->dirty_list);

	INIT_LIST_HEAD(&sde_crtc->feature_list);

	INIT_LIST_HEAD(&sde_crtc->ad_dirty);

	INIT_LIST_HEAD(&sde_crtc->ad_active);

}



static void sde_cp_crtc_install_immutable_property(struct drm_crtc *crtc,
@@ -357,8 +385,8 @@ static void sde_cp_crtc_install_immutable_property(struct drm_crtc *crtc, 
	prop = priv->cp_property[feature];



	if (!prop) {

		prop = drm_property_create(crtc->dev, DRM_MODE_PROP_IMMUTABLE,

					   name, 0);

		prop = drm_property_create_range(crtc->dev,

				DRM_MODE_PROP_IMMUTABLE, name, 0, 1);

		if (!prop) {

			DRM_ERROR("property create failed: %s\n", name);

			kfree(prop_node);
@@ -412,7 +440,7 @@ static void sde_cp_crtc_install_range_property(struct drm_crtc *crtc, 
	sde_cp_crtc_prop_attach(&prop_attach);

}



static void sde_cp_crtc_create_blob_property(struct drm_crtc *crtc, char *name,

static void sde_cp_crtc_install_blob_property(struct drm_crtc *crtc, char *name,

			u32 feature, u32 blob_sz)

{

	struct drm_property *prop;
@@ -452,6 +480,46 @@ static void sde_cp_crtc_create_blob_property(struct drm_crtc *crtc, char *name, 
	sde_cp_crtc_prop_attach(&prop_attach);

}



static void sde_cp_crtc_install_enum_property(struct drm_crtc *crtc,

	u32 feature, const struct drm_prop_enum_list *list, u32 enum_sz,

	char *name)

{

	struct drm_property *prop;

	struct sde_cp_node *prop_node = NULL;

	struct msm_drm_private *priv;

	uint64_t val = 0;

	struct sde_cp_prop_attach prop_attach;



	if (feature >=  SDE_CP_CRTC_MAX_FEATURES) {

		DRM_ERROR("invalid feature %d max %d\n", feature,

		       SDE_CP_CRTC_MAX_FEATURES);

		return;

	}



	prop_node = kzalloc(sizeof(*prop_node), GFP_KERNEL);

	if (!prop_node)

		return;



	priv = crtc->dev->dev_private;

	prop = priv->cp_property[feature];



	if (!prop) {

		prop = drm_property_create_enum(crtc->dev, 0, name,

			list, enum_sz);

		if (!prop) {

			DRM_ERROR("property create failed: %s\n", name);

			kfree(prop_node);

			return;

		}

		priv->cp_property[feature] = prop;

	}



	INIT_PROP_ATTACH(&prop_attach, crtc, prop, prop_node,

				feature, val);



	sde_cp_crtc_prop_attach(&prop_attach);

}



static void sde_cp_crtc_setfeature(struct sde_cp_node *prop_node,

				   struct sde_crtc *sde_crtc, u32 last_feature)

{
@@ -462,13 +530,18 @@ static void sde_cp_crtc_setfeature(struct sde_cp_node *prop_node, 
	int i = 0;

	bool feature_enabled = false;

	int ret = 0;

	struct sde_ad_hw_cfg ad_cfg;



	sde_cp_get_hw_payload(prop_node, &hw_cfg, &feature_enabled);

	hw_cfg.num_of_mixers = sde_crtc->num_mixers;

	hw_cfg.displayh = sde_crtc->base.mode.hdisplay;

	hw_cfg.displayv = sde_crtc->base.mode.vdisplay;



	for (i = 0; i < num_mixers && !ret; i++) {

		hw_lm = sde_crtc->mixers[i].hw_lm;

		hw_dspp = sde_crtc->mixers[i].hw_dspp;

		hw_cfg.ctl = sde_crtc->mixers[i].hw_ctl;

		hw_cfg.mixer_info = hw_lm;

		if (i == num_mixers - 1)

			hw_cfg.last_feature = last_feature;

		else
@@ -558,6 +631,60 @@ static void sde_cp_crtc_setfeature(struct sde_cp_node *prop_node, 
			}

			hw_lm->ops.setup_gc(hw_lm, &hw_cfg);

			break;

		case SDE_CP_CRTC_DSPP_AD_MODE:

			if (!hw_dspp || !hw_dspp->ops.setup_ad) {

				ret = -EINVAL;

				continue;

			}

			ad_cfg.prop = AD_MODE;

			ad_cfg.hw_cfg = &hw_cfg;

			hw_dspp->ops.setup_ad(hw_dspp, &ad_cfg);

			break;

		case SDE_CP_CRTC_DSPP_AD_INIT:

			if (!hw_dspp || !hw_dspp->ops.setup_ad) {

				ret = -EINVAL;

				continue;

			}

			ad_cfg.prop = AD_INIT;

			ad_cfg.hw_cfg = &hw_cfg;

			hw_dspp->ops.setup_ad(hw_dspp, &ad_cfg);

			break;

		case SDE_CP_CRTC_DSPP_AD_CFG:

			if (!hw_dspp || !hw_dspp->ops.setup_ad) {

				ret = -EINVAL;

				continue;

			}

			ad_cfg.prop = AD_CFG;

			ad_cfg.hw_cfg = &hw_cfg;

			hw_dspp->ops.setup_ad(hw_dspp, &ad_cfg);

			break;

		case SDE_CP_CRTC_DSPP_AD_INPUT:

			if (!hw_dspp || !hw_dspp->ops.setup_ad) {

				ret = -EINVAL;

				continue;

			}

			ad_cfg.prop = AD_INPUT;

			ad_cfg.hw_cfg = &hw_cfg;

			hw_dspp->ops.setup_ad(hw_dspp, &ad_cfg);

			break;

		case SDE_CP_CRTC_DSPP_AD_ASSERTIVENESS:

			if (!hw_dspp || !hw_dspp->ops.setup_ad) {

				ret = -EINVAL;

				continue;

			}

			ad_cfg.prop = AD_ASSERTIVE;

			ad_cfg.hw_cfg = &hw_cfg;

			hw_dspp->ops.setup_ad(hw_dspp, &ad_cfg);

			break;

		case SDE_CP_CRTC_DSPP_AD_BACKLIGHT:

			if (!hw_dspp || !hw_dspp->ops.setup_ad) {

				ret = -EINVAL;

				continue;

			}

			ad_cfg.prop = AD_BACKLIGHT;

			ad_cfg.hw_cfg = &hw_cfg;

			hw_dspp->ops.setup_ad(hw_dspp, &ad_cfg);

			break;

		default:

			ret = -EINVAL;

			break;
@@ -574,7 +701,7 @@ static void sde_cp_crtc_setfeature(struct sde_cp_node *prop_node, 
	if (feature_enabled) {

		DRM_DEBUG_DRIVER("Add feature to active list %d\n",

				 prop_node->property_id);

		list_add_tail(&prop_node->active_list, &sde_crtc->active_list);

		sde_cp_update_list(prop_node, sde_crtc, false);

	} else {

		DRM_DEBUG_DRIVER("remove feature from active list %d\n",

			 prop_node->property_id);
@@ -612,11 +739,18 @@ void sde_cp_crtc_apply_properties(struct drm_crtc *crtc) 
		return;

	}



	/* Check if dirty list is empty for early return */

	if (list_empty(&sde_crtc->dirty_list)) {

	/* Check if dirty lists are empty and ad features are disabled for

	 * early return. If ad properties are active then we need to issue

	 * dspp flush.

	 **/

	if (list_empty(&sde_crtc->dirty_list) &&

		list_empty(&sde_crtc->ad_dirty)) {

		if (list_empty(&sde_crtc->ad_active)) {

			DRM_DEBUG_DRIVER("Dirty list is empty\n");

			return;

		}

		set_dspp_flush = true;

	}



	num_of_features = 0;

	list_for_each_entry(prop_node, &sde_crtc->dirty_list, dirty_list)
@@ -634,6 +768,18 @@ void sde_cp_crtc_apply_properties(struct drm_crtc *crtc) 
			set_lm_flush = true;

	}



	num_of_features = 0;

	list_for_each_entry(prop_node, &sde_crtc->ad_dirty, dirty_list)

		num_of_features++;



	list_for_each_entry_safe(prop_node, n, &sde_crtc->ad_dirty,

				dirty_list) {

		num_of_features--;

		set_dspp_flush = true;

		sde_cp_crtc_setfeature(prop_node, sde_crtc,

				(num_of_features == 0));

	}



	for (i = 0; i < num_mixers; i++) {

		ctl = sde_crtc->mixers[i].hw_ctl;

		if (!ctl)
@@ -791,6 +937,13 @@ int sde_cp_crtc_set_property(struct drm_crtc *crtc, 
			sde_crtc->num_mixers);

		return -EINVAL;

	}



	ret = sde_cp_ad_validate_prop(prop_node, sde_crtc);

	if (ret) {

		DRM_ERROR("ad property validation failed ret %d\n", ret);

		return ret;

	}



	/* remove the property from dirty list */

	list_del_init(&prop_node->dirty_list);
@@ -804,7 +957,7 @@ int sde_cp_crtc_set_property(struct drm_crtc *crtc, 
		/* remove the property from active list */

		list_del_init(&prop_node->active_list);

		/* Mark the feature as dirty */

		list_add_tail(&prop_node->dirty_list, &sde_crtc->dirty_list);

		sde_cp_update_list(prop_node, sde_crtc, true);

	}

	return ret;

}
@@ -888,7 +1041,7 @@ void sde_cp_crtc_suspend(struct drm_crtc *crtc) 


	list_for_each_entry_safe(prop_node, n, &sde_crtc->active_list,

				 active_list) {

		list_add_tail(&prop_node->dirty_list, &sde_crtc->dirty_list);

		sde_cp_update_list(prop_node, sde_crtc, true);

		list_del_init(&prop_node->active_list);

	}

}
@@ -913,7 +1066,7 @@ static void dspp_pcc_install_property(struct drm_crtc *crtc) 
		"SDE_DSPP_PCC_V", version);

	switch (version) {

	case 1:

		sde_cp_crtc_create_blob_property(crtc, feature_name,

		sde_cp_crtc_install_blob_property(crtc, feature_name,

			SDE_CP_CRTC_DSPP_PCC, sizeof(struct drm_msm_pcc));

		break;

	default:
@@ -988,6 +1141,33 @@ static void dspp_ad_install_property(struct drm_crtc *crtc) 
		sde_cp_crtc_install_immutable_property(crtc,

			feature_name, SDE_CP_CRTC_DSPP_AD);

		break;

	case 4:

		sde_cp_crtc_install_immutable_property(crtc,

			feature_name, SDE_CP_CRTC_DSPP_AD);



		sde_cp_crtc_install_enum_property(crtc,

			SDE_CP_CRTC_DSPP_AD_MODE, ad4_modes,

			ARRAY_SIZE(ad4_modes), "SDE_DSPP_AD_V4_MODE");



		sde_cp_crtc_install_range_property(crtc, "SDE_DSPP_AD_V4_INIT",

			SDE_CP_CRTC_DSPP_AD_INIT, 0, U64_MAX, 0);

		sde_cp_create_local_blob(crtc, SDE_CP_CRTC_DSPP_AD_INIT,

			sizeof(struct drm_msm_ad4_init));



		sde_cp_crtc_install_range_property(crtc, "SDE_DSPP_AD_V4_CFG",

			SDE_CP_CRTC_DSPP_AD_CFG, 0, U64_MAX, 0);

		sde_cp_create_local_blob(crtc, SDE_CP_CRTC_DSPP_AD_CFG,

			sizeof(struct drm_msm_ad4_cfg));

		sde_cp_crtc_install_range_property(crtc,

			"SDE_DSPP_AD_V4_ASSERTIVNESS",

			SDE_CP_CRTC_DSPP_AD_ASSERTIVENESS, 0, (BIT(8) - 1), 0);

		sde_cp_crtc_install_range_property(crtc, "SDE_DSPP_AD_V4_INPUT",

			SDE_CP_CRTC_DSPP_AD_INPUT, 0, U16_MAX, 0);

		sde_cp_crtc_install_range_property(crtc,

				"SDE_DSPP_AD_V4_BACKLIGHT",

			SDE_CP_CRTC_DSPP_AD_BACKLIGHT, 0, (BIT(16) - 1),

			0);

		break;

	default:

		DRM_ERROR("version %d not supported\n", version);

		break;
@@ -1008,7 +1188,7 @@ static void lm_gc_install_property(struct drm_crtc *crtc) 
		 "SDE_LM_GC_V", version);

	switch (version) {

	case 1:

		sde_cp_crtc_create_blob_property(crtc, feature_name,

		sde_cp_crtc_install_blob_property(crtc, feature_name,

			SDE_CP_CRTC_LM_GC, sizeof(struct drm_msm_pgc_lut));

		break;

	default:
@@ -1032,7 +1212,7 @@ static void dspp_gamut_install_property(struct drm_crtc *crtc) 
		"SDE_DSPP_GAMUT_V", version);

	switch (version) {

	case 4:

		sde_cp_crtc_create_blob_property(crtc, feature_name,

		sde_cp_crtc_install_blob_property(crtc, feature_name,

			SDE_CP_CRTC_DSPP_GAMUT,

			sizeof(struct drm_msm_3d_gamut));

		break;
@@ -1057,7 +1237,7 @@ static void dspp_gc_install_property(struct drm_crtc *crtc) 
		"SDE_DSPP_GC_V", version);

	switch (version) {

	case 1:

		sde_cp_crtc_create_blob_property(crtc, feature_name,

		sde_cp_crtc_install_blob_property(crtc, feature_name,

			SDE_CP_CRTC_DSPP_GC, sizeof(struct drm_msm_pgc_lut));

		break;

	default:
@@ -1065,3 +1245,74 @@ static void dspp_gc_install_property(struct drm_crtc *crtc) 
		break;

	}

}



static void sde_cp_update_list(struct sde_cp_node *prop_node,

		struct sde_crtc *crtc, bool dirty_list)

{

	switch (prop_node->feature) {

	case SDE_CP_CRTC_DSPP_AD_MODE:

	case SDE_CP_CRTC_DSPP_AD_INIT:

	case SDE_CP_CRTC_DSPP_AD_CFG:

	case SDE_CP_CRTC_DSPP_AD_INPUT:

	case SDE_CP_CRTC_DSPP_AD_ASSERTIVENESS:

	case SDE_CP_CRTC_DSPP_AD_BACKLIGHT:

		if (dirty_list)

			list_add_tail(&prop_node->dirty_list, &crtc->ad_dirty);

		else

			list_add_tail(&prop_node->active_list,

					&crtc->ad_active);

		break;

	default:

		/* color processing properties handle here */

		if (dirty_list)

			list_add_tail(&prop_node->dirty_list,

					&crtc->dirty_list);

		else

			list_add_tail(&prop_node->active_list,

					&crtc->active_list);

		break;

	};

}



static int sde_cp_ad_validate_prop(struct sde_cp_node *prop_node,

		struct sde_crtc *crtc)

{

	int i = 0, ret = 0;

	u32 ad_prop;



	for (i = 0; i < crtc->num_mixers && !ret; i++) {

		if (!crtc->mixers[i].hw_dspp) {

			ret = -EINVAL;

			continue;

		}

		switch (prop_node->feature) {

		case SDE_CP_CRTC_DSPP_AD_MODE:

			ad_prop = AD_MODE;

			break;

		case SDE_CP_CRTC_DSPP_AD_INIT:

			ad_prop = AD_INIT;

			break;

		case SDE_CP_CRTC_DSPP_AD_CFG:

			ad_prop = AD_CFG;

			break;

		case SDE_CP_CRTC_DSPP_AD_INPUT:

			ad_prop = AD_INPUT;

			break;

		case SDE_CP_CRTC_DSPP_AD_ASSERTIVENESS:

			ad_prop = AD_ASSERTIVE;

			break;

		case SDE_CP_CRTC_DSPP_AD_BACKLIGHT:

			ad_prop = AD_BACKLIGHT;

			break;

		default:

			/* Not an AD property */

			return 0;

		}

		if (!crtc->mixers[i].hw_dspp->ops.validate_ad)

			ret = -EINVAL;

		else

			ret = crtc->mixers[i].hw_dspp->ops.validate_ad(

				crtc->mixers[i].hw_dspp, &ad_prop);

	}

	return ret;

}

drivers/gpu/drm/msm/sde/sde_crtc.c

+5 −6
Original line number Diff line number Diff line
@@ -1299,7 +1299,6 @@ static void sde_crtc_enable(struct drm_crtc *crtc) 
	struct sde_crtc_mixer *mixer;

	struct sde_hw_mixer *lm;

	struct drm_display_mode *mode;

	struct sde_hw_mixer_cfg cfg;

	struct drm_encoder *encoder;

	int i;
@@ -1330,11 +1329,11 @@ static void sde_crtc_enable(struct drm_crtc *crtc) 


	for (i = 0; i < sde_crtc->num_mixers; i++) {

		lm = mixer[i].hw_lm;

		cfg.out_width = sde_crtc_mixer_width(sde_crtc, mode);

		cfg.out_height = mode->vdisplay;

		cfg.right_mixer = (i == 0) ? false : true;

		cfg.flags = 0;

		lm->ops.setup_mixer_out(lm, &cfg);

		lm->cfg.out_width = sde_crtc_mixer_width(sde_crtc, mode);

		lm->cfg.out_height = mode->vdisplay;

		lm->cfg.right_mixer = (i == 0) ? false : true;

		lm->cfg.flags = 0;

		lm->ops.setup_mixer_out(lm, &lm->cfg);

	}

}

drivers/gpu/drm/msm/sde/sde_crtc.h

+4 −0
Original line number Diff line number Diff line
@@ -124,6 +124,8 @@ struct sde_crtc_event { 
 * @feature_list  : list of color processing features supported on a crtc

 * @active_list   : list of color processing features are active

 * @dirty_list    : list of color processing features are dirty

 * @ad_dirty: list containing ad properties that are dirty

 * @ad_active: list containing ad properties that are active

 * @crtc_lock     : crtc lock around create, destroy and access.

 * @frame_pending : Whether or not an update is pending

 * @frame_events  : static allocation of in-flight frame events
@@ -165,6 +167,8 @@ struct sde_crtc { 
	struct list_head feature_list;

	struct list_head active_list;

	struct list_head dirty_list;

	struct list_head ad_dirty;

	struct list_head ad_active;



	struct mutex crtc_lock;