drm/msm: dpu: Handle crtc pm_runtime_resume() directly
Instead of registering through dpu_power_handle just to get a call on runtime_resume, call the crtc function directly. Changes in v2: - None Reviewed-by: Jeykumar Sankaran <jsanka@codeaurora.org> Signed-off-by: Sean Paul <seanpaul@chromium.org> Signed-off-by: Rob Clark <robdclark@gmail.com>hifive-unleashed-5.1
parent
c24b633003
commit
3cf63cd5f8
|
@ -33,7 +33,6 @@
|
||||||
#include "dpu_plane.h"
|
#include "dpu_plane.h"
|
||||||
#include "dpu_encoder.h"
|
#include "dpu_encoder.h"
|
||||||
#include "dpu_vbif.h"
|
#include "dpu_vbif.h"
|
||||||
#include "dpu_power_handle.h"
|
|
||||||
#include "dpu_core_perf.h"
|
#include "dpu_core_perf.h"
|
||||||
#include "dpu_trace.h"
|
#include "dpu_trace.h"
|
||||||
|
|
||||||
|
@ -69,8 +68,6 @@ static void dpu_crtc_destroy(struct drm_crtc *crtc)
|
||||||
if (!crtc)
|
if (!crtc)
|
||||||
return;
|
return;
|
||||||
|
|
||||||
dpu_crtc->phandle = NULL;
|
|
||||||
|
|
||||||
drm_crtc_cleanup(crtc);
|
drm_crtc_cleanup(crtc);
|
||||||
mutex_destroy(&dpu_crtc->crtc_lock);
|
mutex_destroy(&dpu_crtc->crtc_lock);
|
||||||
kfree(dpu_crtc);
|
kfree(dpu_crtc);
|
||||||
|
@ -860,15 +857,17 @@ static struct drm_crtc_state *dpu_crtc_duplicate_state(struct drm_crtc *crtc)
|
||||||
return &cstate->base;
|
return &cstate->base;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void dpu_crtc_handle_power_event(u32 event_type, void *arg)
|
void dpu_crtc_runtime_resume(struct drm_crtc *crtc)
|
||||||
{
|
{
|
||||||
struct drm_crtc *crtc = arg;
|
|
||||||
struct dpu_crtc *dpu_crtc = to_dpu_crtc(crtc);
|
struct dpu_crtc *dpu_crtc = to_dpu_crtc(crtc);
|
||||||
struct drm_encoder *encoder;
|
struct drm_encoder *encoder;
|
||||||
|
|
||||||
mutex_lock(&dpu_crtc->crtc_lock);
|
mutex_lock(&dpu_crtc->crtc_lock);
|
||||||
|
|
||||||
trace_dpu_crtc_handle_power_event(DRMID(crtc), event_type);
|
if (!dpu_crtc->enabled)
|
||||||
|
goto end;
|
||||||
|
|
||||||
|
trace_dpu_crtc_runtime_resume(DRMID(crtc));
|
||||||
|
|
||||||
/* restore encoder; crtc will be programmed during commit */
|
/* restore encoder; crtc will be programmed during commit */
|
||||||
drm_for_each_encoder(encoder, crtc->dev) {
|
drm_for_each_encoder(encoder, crtc->dev) {
|
||||||
|
@ -878,6 +877,7 @@ static void dpu_crtc_handle_power_event(u32 event_type, void *arg)
|
||||||
dpu_encoder_virt_restore(encoder);
|
dpu_encoder_virt_restore(encoder);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
end:
|
||||||
mutex_unlock(&dpu_crtc->crtc_lock);
|
mutex_unlock(&dpu_crtc->crtc_lock);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -933,10 +933,6 @@ static void dpu_crtc_disable(struct drm_crtc *crtc)
|
||||||
dpu_encoder_register_frame_event_callback(encoder, NULL, NULL);
|
dpu_encoder_register_frame_event_callback(encoder, NULL, NULL);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (dpu_crtc->power_event)
|
|
||||||
dpu_power_handle_unregister_event(dpu_crtc->phandle,
|
|
||||||
dpu_crtc->power_event);
|
|
||||||
|
|
||||||
memset(cstate->mixers, 0, sizeof(cstate->mixers));
|
memset(cstate->mixers, 0, sizeof(cstate->mixers));
|
||||||
cstate->num_mixers = 0;
|
cstate->num_mixers = 0;
|
||||||
|
|
||||||
|
@ -988,11 +984,6 @@ static void dpu_crtc_enable(struct drm_crtc *crtc,
|
||||||
|
|
||||||
/* Enable/restore vblank irq handling */
|
/* Enable/restore vblank irq handling */
|
||||||
drm_crtc_vblank_on(crtc);
|
drm_crtc_vblank_on(crtc);
|
||||||
|
|
||||||
dpu_crtc->power_event = dpu_power_handle_register_event(
|
|
||||||
dpu_crtc->phandle, DPU_POWER_EVENT_ENABLE,
|
|
||||||
dpu_crtc_handle_power_event, crtc, dpu_crtc->name);
|
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
struct plane_state {
|
struct plane_state {
|
||||||
|
@ -1539,8 +1530,6 @@ struct drm_crtc *dpu_crtc_init(struct drm_device *dev, struct drm_plane *plane,
|
||||||
/* initialize event handling */
|
/* initialize event handling */
|
||||||
spin_lock_init(&dpu_crtc->event_lock);
|
spin_lock_init(&dpu_crtc->event_lock);
|
||||||
|
|
||||||
dpu_crtc->phandle = &kms->phandle;
|
|
||||||
|
|
||||||
DPU_DEBUG("%s: successfully initialized crtc\n", dpu_crtc->name);
|
DPU_DEBUG("%s: successfully initialized crtc\n", dpu_crtc->name);
|
||||||
return crtc;
|
return crtc;
|
||||||
}
|
}
|
||||||
|
|
|
@ -151,7 +151,6 @@ struct dpu_crtc_frame_event {
|
||||||
* @event_worker : Event worker queue
|
* @event_worker : Event worker queue
|
||||||
* @event_lock : Spinlock around event handling code
|
* @event_lock : Spinlock around event handling code
|
||||||
* @phandle: Pointer to power handler
|
* @phandle: Pointer to power handler
|
||||||
* @power_event : registered power event handle
|
|
||||||
* @cur_perf : current performance committed to clock/bandwidth driver
|
* @cur_perf : current performance committed to clock/bandwidth driver
|
||||||
*/
|
*/
|
||||||
struct dpu_crtc {
|
struct dpu_crtc {
|
||||||
|
@ -187,9 +186,6 @@ struct dpu_crtc {
|
||||||
/* for handling internal event thread */
|
/* for handling internal event thread */
|
||||||
spinlock_t event_lock;
|
spinlock_t event_lock;
|
||||||
|
|
||||||
struct dpu_power_handle *phandle;
|
|
||||||
struct dpu_power_event *power_event;
|
|
||||||
|
|
||||||
struct dpu_core_perf_params cur_perf;
|
struct dpu_core_perf_params cur_perf;
|
||||||
|
|
||||||
struct dpu_crtc_smmu_state_data smmu_state;
|
struct dpu_crtc_smmu_state_data smmu_state;
|
||||||
|
@ -334,4 +330,10 @@ static inline bool dpu_crtc_is_enabled(struct drm_crtc *crtc)
|
||||||
return crtc ? crtc->enabled : false;
|
return crtc ? crtc->enabled : false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* dpu_crtc_runtime_resume - called by the top-level on pm_runtime_resume
|
||||||
|
* @crtc: CRTC to resume
|
||||||
|
*/
|
||||||
|
void dpu_crtc_runtime_resume(struct drm_crtc *crtc);
|
||||||
|
|
||||||
#endif /* _DPU_CRTC_H_ */
|
#endif /* _DPU_CRTC_H_ */
|
||||||
|
|
|
@ -1141,6 +1141,7 @@ static int __maybe_unused dpu_runtime_resume(struct device *dev)
|
||||||
int rc = -1;
|
int rc = -1;
|
||||||
struct platform_device *pdev = to_platform_device(dev);
|
struct platform_device *pdev = to_platform_device(dev);
|
||||||
struct dpu_kms *dpu_kms = platform_get_drvdata(pdev);
|
struct dpu_kms *dpu_kms = platform_get_drvdata(pdev);
|
||||||
|
struct drm_crtc *crtc;
|
||||||
struct drm_device *ddev;
|
struct drm_device *ddev;
|
||||||
struct dss_module_power *mp = &dpu_kms->mp;
|
struct dss_module_power *mp = &dpu_kms->mp;
|
||||||
|
|
||||||
|
@ -1158,6 +1159,9 @@ static int __maybe_unused dpu_runtime_resume(struct device *dev)
|
||||||
|
|
||||||
dpu_vbif_init_memtypes(dpu_kms);
|
dpu_vbif_init_memtypes(dpu_kms);
|
||||||
|
|
||||||
|
drm_for_each_crtc(crtc, ddev)
|
||||||
|
dpu_crtc_runtime_resume(crtc);
|
||||||
|
|
||||||
rc = dpu_power_resource_enable(&dpu_kms->phandle, true);
|
rc = dpu_power_resource_enable(&dpu_kms->phandle, true);
|
||||||
if (rc)
|
if (rc)
|
||||||
DPU_ERROR("resource enable failed: %d\n", rc);
|
DPU_ERROR("resource enable failed: %d\n", rc);
|
||||||
|
|
|
@ -298,6 +298,10 @@ DEFINE_EVENT(dpu_drm_obj_template, dpu_kms_wait_for_commit_done,
|
||||||
TP_PROTO(uint32_t drm_id),
|
TP_PROTO(uint32_t drm_id),
|
||||||
TP_ARGS(drm_id)
|
TP_ARGS(drm_id)
|
||||||
);
|
);
|
||||||
|
DEFINE_EVENT(dpu_drm_obj_template, dpu_crtc_runtime_resume,
|
||||||
|
TP_PROTO(uint32_t drm_id),
|
||||||
|
TP_ARGS(drm_id)
|
||||||
|
);
|
||||||
|
|
||||||
TRACE_EVENT(dpu_enc_enable,
|
TRACE_EVENT(dpu_enc_enable,
|
||||||
TP_PROTO(uint32_t drm_id, int hdisplay, int vdisplay),
|
TP_PROTO(uint32_t drm_id, int hdisplay, int vdisplay),
|
||||||
|
@ -518,10 +522,6 @@ DEFINE_EVENT(dpu_id_event_template, dpu_crtc_frame_event_cb,
|
||||||
TP_PROTO(uint32_t drm_id, u32 event),
|
TP_PROTO(uint32_t drm_id, u32 event),
|
||||||
TP_ARGS(drm_id, event)
|
TP_ARGS(drm_id, event)
|
||||||
);
|
);
|
||||||
DEFINE_EVENT(dpu_id_event_template, dpu_crtc_handle_power_event,
|
|
||||||
TP_PROTO(uint32_t drm_id, u32 event),
|
|
||||||
TP_ARGS(drm_id, event)
|
|
||||||
);
|
|
||||||
DEFINE_EVENT(dpu_id_event_template, dpu_crtc_frame_event_done,
|
DEFINE_EVENT(dpu_id_event_template, dpu_crtc_frame_event_done,
|
||||||
TP_PROTO(uint32_t drm_id, u32 event),
|
TP_PROTO(uint32_t drm_id, u32 event),
|
||||||
TP_ARGS(drm_id, event)
|
TP_ARGS(drm_id, event)
|
||||||
|
|
Loading…
Reference in New Issue