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

Commit 89628139 authored by Clarence Ip's avatar Clarence Ip
Browse files

drm/msm/sde: select shorter idle pc timeout for lp modes 



Setting the LP property on a connector indicates a request
for more aggressive power savings, so reduce the idle
power collapse timeout to a shorter one for those modes.

CRs-Fixed: 2073016
Change-Id: I9a992629ecadf052064bb5694933c75586b4e519
Signed-off-by: default avatarClarence Ip <cip@codeaurora.org>
parent 1996ec83

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

+14 −0
Original line number Diff line number Diff line
@@ -380,6 +380,20 @@ static inline uint64_t sde_connector_get_topology_name( 
			CONNECTOR_PROP_TOPOLOGY_NAME);

}



/**

 * sde_connector_get_lp - helper accessor to retrieve LP state

 * @connector: pointer to drm connector

 * Returns: value of the CONNECTOR_PROP_LP property or 0

 */

static inline uint64_t sde_connector_get_lp(

		struct drm_connector *connector)

{

	if (!connector || !connector->state)

		return 0;

	return sde_connector_get_property(connector->state,

			CONNECTOR_PROP_LP);

}



/**

 * sde_connector_init - create drm connector object for a given display

 * @dev: Pointer to drm device struct

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

+17 −2
Original line number Diff line number Diff line
@@ -72,6 +72,7 @@ 
#define MISR_BUFF_SIZE			256



#define IDLE_TIMEOUT	64

#define IDLE_SHORT_TIMEOUT	1



/**

 * enum sde_enc_rc_events - events for resource control state machine
@@ -1346,6 +1347,7 @@ static int sde_encoder_resource_control(struct drm_encoder *drm_enc, 
{

	bool schedule_off = false;

	bool autorefresh_enabled = false;

	unsigned int lp, idle_timeout;

	struct sde_encoder_virt *sde_enc;

	struct msm_drm_private *priv;

	struct msm_drm_thread *disp_thread;
@@ -1435,13 +1437,26 @@ static int sde_encoder_resource_control(struct drm_encoder *drm_enc, 
				sde_enc->cur_master->ops.is_autorefresh_enabled(

							sde_enc->cur_master);



		/* set idle timeout based on master connector's lp value */

		if (sde_enc->cur_master)

			lp = sde_connector_get_lp(

					sde_enc->cur_master->connector);

		else

			lp = SDE_MODE_DPMS_ON;



		if (lp == SDE_MODE_DPMS_LP2)

			idle_timeout = IDLE_SHORT_TIMEOUT;

		else

			idle_timeout = IDLE_TIMEOUT;



		if (!autorefresh_enabled)

			kthread_queue_delayed_work(

				&disp_thread->worker,

				&sde_enc->delayed_off_work,

				msecs_to_jiffies(IDLE_TIMEOUT));

				msecs_to_jiffies(idle_timeout));

		SDE_EVT32(DRMID(drm_enc), sw_event, sde_enc->rc_state,

				SDE_EVTLOG_FUNC_CASE2);

				autorefresh_enabled,

				idle_timeout, SDE_EVTLOG_FUNC_CASE2);

		SDE_DEBUG_ENC(sde_enc, "sw_event:%d, work scheduled\n",

				sw_event);

		break;