mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-13 16:29:25 -04:00
drm/i915/lspcon: add intel_lspcon_active() and use it
Hide the direct lspcon->active use behind intel_lspcon_active(). Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/3eabe0cb07a84c63119ea893e149de410b4356b0.1738677489.git.jani.nikula@intel.com Signed-off-by: Jani Nikula <jani.nikula@intel.com>
This commit is contained in:
@@ -3025,7 +3025,7 @@ static void intel_ddi_pre_enable(struct intel_atomic_state *state,
|
||||
|
||||
/* FIXME precompute everything properly */
|
||||
/* FIXME how do we turn infoframes off again? */
|
||||
if (dig_port->lspcon.active && intel_dp_has_hdmi_sink(&dig_port->dp))
|
||||
if (intel_lspcon_active(dig_port) && intel_dp_has_hdmi_sink(&dig_port->dp))
|
||||
dig_port->set_infoframes(encoder,
|
||||
crtc_state->has_infoframe,
|
||||
crtc_state, conn_state);
|
||||
@@ -3381,7 +3381,7 @@ static void intel_ddi_enable_dp(struct intel_atomic_state *state,
|
||||
drm_connector_update_privacy_screen(conn_state);
|
||||
intel_edp_backlight_on(crtc_state, conn_state);
|
||||
|
||||
if (!dig_port->lspcon.active || intel_dp_has_hdmi_sink(&dig_port->dp))
|
||||
if (!intel_lspcon_active(dig_port) || intel_dp_has_hdmi_sink(&dig_port->dp))
|
||||
intel_dp_set_infoframes(encoder, true, crtc_state, conn_state);
|
||||
|
||||
trans_port_sync_stop_link_train(state, encoder, crtc_state);
|
||||
@@ -4082,7 +4082,7 @@ static void intel_ddi_read_func_ctl_dp_sst(struct intel_encoder *encoder,
|
||||
intel_de_read(display,
|
||||
dp_tp_ctl_reg(encoder, crtc_state)) & DP_TP_CTL_FEC_ENABLE;
|
||||
|
||||
if (dig_port->lspcon.active && intel_dp_has_hdmi_sink(&dig_port->dp))
|
||||
if (intel_lspcon_active(dig_port) && intel_dp_has_hdmi_sink(&dig_port->dp))
|
||||
crtc_state->infoframes.enable |=
|
||||
intel_lspcon_infoframes_enabled(encoder, crtc_state);
|
||||
else
|
||||
|
||||
@@ -3519,9 +3519,10 @@ void intel_dp_set_power(struct intel_dp *intel_dp, u8 mode)
|
||||
|
||||
ret = drm_dp_dpcd_writeb(&intel_dp->aux, DP_SET_POWER, mode);
|
||||
} else {
|
||||
struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
|
||||
struct intel_lspcon *lspcon = dp_to_lspcon(intel_dp);
|
||||
|
||||
lspcon_resume(dp_to_dig_port(intel_dp));
|
||||
lspcon_resume(dig_port);
|
||||
|
||||
/* Write the source OUI as early as possible */
|
||||
intel_dp_init_source_oui(intel_dp);
|
||||
@@ -3537,7 +3538,7 @@ void intel_dp_set_power(struct intel_dp *intel_dp, u8 mode)
|
||||
msleep(1);
|
||||
}
|
||||
|
||||
if (ret == 1 && lspcon->active)
|
||||
if (ret == 1 && intel_lspcon_active(dig_port))
|
||||
lspcon_wait_pcon_mode(lspcon);
|
||||
}
|
||||
|
||||
@@ -5588,7 +5589,7 @@ intel_dp_update_420(struct intel_dp *intel_dp)
|
||||
intel_dp->downstream_ports);
|
||||
/* on-board LSPCON always assumed to support 4:4:4->4:2:0 conversion */
|
||||
intel_dp->dfp.ycbcr_444_to_420 =
|
||||
dp_to_dig_port(intel_dp)->lspcon.active ||
|
||||
intel_lspcon_active(dp_to_dig_port(intel_dp)) ||
|
||||
drm_dp_downstream_444_to_420_conversion(intel_dp->dpcd,
|
||||
intel_dp->downstream_ports);
|
||||
intel_dp->dfp.rgb_to_ycbcr =
|
||||
|
||||
@@ -689,6 +689,13 @@ bool lspcon_init(struct intel_digital_port *dig_port)
|
||||
return true;
|
||||
}
|
||||
|
||||
bool intel_lspcon_active(struct intel_digital_port *dig_port)
|
||||
{
|
||||
struct intel_lspcon *lspcon = &dig_port->lspcon;
|
||||
|
||||
return lspcon->active;
|
||||
}
|
||||
|
||||
u32 intel_lspcon_infoframes_enabled(struct intel_encoder *encoder,
|
||||
const struct intel_crtc_state *pipe_config)
|
||||
{
|
||||
|
||||
@@ -16,6 +16,7 @@ struct intel_encoder;
|
||||
struct intel_lspcon;
|
||||
|
||||
bool lspcon_init(struct intel_digital_port *dig_port);
|
||||
bool intel_lspcon_active(struct intel_digital_port *dig_port);
|
||||
void lspcon_detect_hdr_capability(struct intel_lspcon *lspcon);
|
||||
void lspcon_resume(struct intel_digital_port *dig_port);
|
||||
void lspcon_wait_pcon_mode(struct intel_lspcon *lspcon);
|
||||
|
||||
Reference in New Issue
Block a user