Skip to content

Commit

Permalink
video: mt6768: Don’t check for ovl status
Browse files Browse the repository at this point in the history
* Dumping ovl is just for debugging and causes small lag in UX

Signed-off-by: SakthivelNadar <[email protected]>
  • Loading branch information
erfanoabdi authored and SakthivelNadar committed Dec 3, 2022
1 parent e35cfae commit 4f1fbef
Showing 1 changed file with 0 additions and 31 deletions.
31 changes: 0 additions & 31 deletions drivers/misc/mediatek/video/mt6768/videox/primary_display.c
Original file line number Diff line number Diff line change
Expand Up @@ -3312,37 +3312,6 @@ static int _ovl_fence_release_callback(unsigned long userdata)
#endif
_primary_path_unlock(__func__);

/* check last ovl status: should be idle when config */
if (primary_display_is_video_mode() &&
!primary_display_is_decouple_mode()) {
unsigned int status;

cmdqBackupReadSlot(pgc->ovl_status_info, 0, &status);
#ifdef DEBUG_OVL_CONFIG_TIME
unsigned int time_event = 0;
unsigned int time_event1 = 0;
unsigned int time_event2 = 0;

cmdqBackupReadSlot(pgc->ovl_config_time, 0, &time_event);
cmdqBackupReadSlot(pgc->ovl_config_time, 1, &time_event1);
cmdqBackupReadSlot(pgc->ovl_config_time, 2, &time_event2);
DISPMSG(
"ovl config time_event %d time_event1 %d time_event2 %d time1_diff %d time2_diff %d\n",
time_event, time_event1, time_event2,
time_event1 - time_event, time_event2 - time_event1);
#endif
if (status & 0x1) {
/* ovl is not idle !! */
DISPERR("disp ovl status error!! stat=0x%x\n",
status);
/* disp_aee_print("ovl_stat 0x%x\n", status); */
mmprofile_log_ex(ddp_mmp_get_events()->primary_error,
MMPROFILE_FLAG_PULSE, status, 0);
primary_display_diagnose();
ret = -1;
}
}

for (i = 0; i < PRIMARY_SESSION_INPUT_LAYER_COUNT; i++) {
int fence_idx = 0;
int subtractor = 0;
Expand Down

0 comments on commit 4f1fbef

Please sign in to comment.