Merge "disp: msm: sde: fix video mode prefill lines for RSCC"
This commit is contained in:

committed by
Gerrit - the friendly Code Review server

commit
b269dd05bc
@@ -1928,11 +1928,13 @@ static int _sde_encoder_update_rsc_client(
|
|||||||
struct drm_crtc *crtc;
|
struct drm_crtc *crtc;
|
||||||
enum sde_rsc_state rsc_state = SDE_RSC_IDLE_STATE;
|
enum sde_rsc_state rsc_state = SDE_RSC_IDLE_STATE;
|
||||||
struct sde_rsc_cmd_config *rsc_config;
|
struct sde_rsc_cmd_config *rsc_config;
|
||||||
int ret, prefill_lines;
|
int ret;
|
||||||
struct msm_display_info *disp_info;
|
struct msm_display_info *disp_info;
|
||||||
struct msm_mode_info *mode_info;
|
struct msm_mode_info *mode_info;
|
||||||
int wait_vblank_crtc_id = SDE_RSC_INVALID_CRTC_ID;
|
int wait_vblank_crtc_id = SDE_RSC_INVALID_CRTC_ID;
|
||||||
u32 qsync_mode = 0;
|
u32 qsync_mode = 0, v_front_porch;
|
||||||
|
struct drm_display_mode *mode;
|
||||||
|
bool is_vid_mode;
|
||||||
|
|
||||||
if (!drm_enc || !drm_enc->dev) {
|
if (!drm_enc || !drm_enc->dev) {
|
||||||
SDE_ERROR("invalid encoder arguments\n");
|
SDE_ERROR("invalid encoder arguments\n");
|
||||||
@@ -1977,17 +1979,31 @@ static int _sde_encoder_update_rsc_client(
|
|||||||
|
|
||||||
SDE_EVT32(rsc_state, qsync_mode);
|
SDE_EVT32(rsc_state, qsync_mode);
|
||||||
|
|
||||||
prefill_lines = mode_info->prefill_lines;
|
is_vid_mode = sde_encoder_check_curr_mode(&sde_enc->base,
|
||||||
|
MSM_DISPLAY_VIDEO_MODE);
|
||||||
|
mode = &sde_enc->crtc->state->mode;
|
||||||
|
v_front_porch = mode->vsync_start - mode->vdisplay;
|
||||||
|
|
||||||
/* compare specific items and reconfigure the rsc */
|
/* compare specific items and reconfigure the rsc */
|
||||||
if ((rsc_config->fps != mode_info->frame_rate) ||
|
if ((rsc_config->fps != mode_info->frame_rate) ||
|
||||||
(rsc_config->vtotal != mode_info->vtotal) ||
|
(rsc_config->vtotal != mode_info->vtotal) ||
|
||||||
(rsc_config->prefill_lines != prefill_lines) ||
|
(rsc_config->prefill_lines != mode_info->prefill_lines) ||
|
||||||
(rsc_config->jitter_numer != mode_info->jitter_numer) ||
|
(rsc_config->jitter_numer != mode_info->jitter_numer) ||
|
||||||
(rsc_config->jitter_denom != mode_info->jitter_denom)) {
|
(rsc_config->jitter_denom != mode_info->jitter_denom)) {
|
||||||
|
|
||||||
rsc_config->fps = mode_info->frame_rate;
|
rsc_config->fps = mode_info->frame_rate;
|
||||||
rsc_config->vtotal = mode_info->vtotal;
|
rsc_config->vtotal = mode_info->vtotal;
|
||||||
rsc_config->prefill_lines = prefill_lines;
|
/*
|
||||||
|
* for video mode, prefill lines should not go beyond vertical
|
||||||
|
* front porch for RSCC configuration. This will ensure bw
|
||||||
|
* downvotes are not sent within the active region. Additional
|
||||||
|
* -1 is to give one line time for rscc mode min_threshold.
|
||||||
|
*/
|
||||||
|
if (is_vid_mode && (mode_info->prefill_lines >= v_front_porch))
|
||||||
|
rsc_config->prefill_lines = v_front_porch - 1;
|
||||||
|
else
|
||||||
|
rsc_config->prefill_lines = mode_info->prefill_lines;
|
||||||
|
|
||||||
rsc_config->jitter_numer = mode_info->jitter_numer;
|
rsc_config->jitter_numer = mode_info->jitter_numer;
|
||||||
rsc_config->jitter_denom = mode_info->jitter_denom;
|
rsc_config->jitter_denom = mode_info->jitter_denom;
|
||||||
sde_enc->rsc_state_init = false;
|
sde_enc->rsc_state_init = false;
|
||||||
|
@@ -311,8 +311,7 @@ static u32 sde_rsc_timer_calculate(struct sde_rsc_priv *rsc,
|
|||||||
|
|
||||||
default_prefill_lines = (rsc->cmd_config.fps *
|
default_prefill_lines = (rsc->cmd_config.fps *
|
||||||
DEFAULT_PANEL_MIN_V_PREFILL) / DEFAULT_PANEL_FPS;
|
DEFAULT_PANEL_MIN_V_PREFILL) / DEFAULT_PANEL_FPS;
|
||||||
if ((state == SDE_RSC_CMD_STATE) ||
|
if ((state == SDE_RSC_CMD_STATE) || !rsc->cmd_config.prefill_lines)
|
||||||
(rsc->cmd_config.prefill_lines < default_prefill_lines))
|
|
||||||
rsc->cmd_config.prefill_lines = default_prefill_lines;
|
rsc->cmd_config.prefill_lines = default_prefill_lines;
|
||||||
|
|
||||||
pr_debug("frame fps:%d jitter_numer:%d jitter_denom:%d vtotal:%d prefill lines:%d\n",
|
pr_debug("frame fps:%d jitter_numer:%d jitter_denom:%d vtotal:%d prefill lines:%d\n",
|
||||||
|
Reference in New Issue
Block a user