Commit 8486de87 authored by Jani Nikula's avatar Jani Nikula
Browse files

drm/i915/dp: constify struct link_config_limits pointers 



The limits get passed around, but are only modified in a few
places. Constify the pointers elsewhere so it's easier to follow where
they can be modified.

Reviewed-by: default avatarImre Deak <imre.deak@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/6ab2f68eef7849aca18e82ad788e44e9f82b576e.1738161945.git.jani.nikula@intel.com


Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
parent e9bb15cd

drivers/gpu/drm/i915/display/intel_dp.c

+9 −9
Original line number Diff line number Diff line
@@ -1941,7 +1941,7 @@ static bool is_bw_sufficient_for_dsc_config(u16 compressed_bppx16, u32 link_cloc 


static int dsc_compute_link_config(struct intel_dp *intel_dp,

				   struct intel_crtc_state *pipe_config,

				   struct link_config_limits *limits,

				   const struct link_config_limits *limits,

				   u16 compressed_bppx16,

				   int timeslots)

{
@@ -2061,7 +2061,7 @@ static int dsc_src_max_compressed_bpp(struct intel_dp *intel_dp) 
static int

icl_dsc_compute_link_config(struct intel_dp *intel_dp,

			    struct intel_crtc_state *pipe_config,

			    struct link_config_limits *limits,

			    const struct link_config_limits *limits,

			    int dsc_max_bpp,

			    int dsc_min_bpp,

			    int pipe_bpp,
@@ -2104,7 +2104,7 @@ static int 
xelpd_dsc_compute_link_config(struct intel_dp *intel_dp,

			      const struct intel_connector *connector,

			      struct intel_crtc_state *pipe_config,

			      struct link_config_limits *limits,

			      const struct link_config_limits *limits,

			      int dsc_max_bpp,

			      int dsc_min_bpp,

			      int pipe_bpp,
@@ -2153,7 +2153,7 @@ xelpd_dsc_compute_link_config(struct intel_dp *intel_dp, 
static int dsc_compute_compressed_bpp(struct intel_dp *intel_dp,

				      const struct intel_connector *connector,

				      struct intel_crtc_state *pipe_config,

				      struct link_config_limits *limits,

				      const struct link_config_limits *limits,

				      int pipe_bpp,

				      int timeslots)

{
@@ -2185,7 +2185,7 @@ int intel_dp_dsc_min_src_input_bpc(void) 
}



static

bool is_dsc_pipe_bpp_sufficient(struct link_config_limits *limits,

bool is_dsc_pipe_bpp_sufficient(const struct link_config_limits *limits,

				int pipe_bpp)

{

	return pipe_bpp >= limits->pipe.min_bpp &&
@@ -2194,7 +2194,7 @@ bool is_dsc_pipe_bpp_sufficient(struct link_config_limits *limits, 


static

int intel_dp_force_dsc_pipe_bpp(struct intel_dp *intel_dp,

				struct link_config_limits *limits)

				const struct link_config_limits *limits)

{

	struct intel_display *display = to_intel_display(intel_dp);

	int forced_bpp;
@@ -2220,7 +2220,7 @@ int intel_dp_force_dsc_pipe_bpp(struct intel_dp *intel_dp, 
static int intel_dp_dsc_compute_pipe_bpp(struct intel_dp *intel_dp,

					 struct intel_crtc_state *pipe_config,

					 struct drm_connector_state *conn_state,

					 struct link_config_limits *limits,

					 const struct link_config_limits *limits,

					 int timeslots)

{

	const struct intel_connector *connector =
@@ -2270,7 +2270,7 @@ static int intel_dp_dsc_compute_pipe_bpp(struct intel_dp *intel_dp, 
static int intel_edp_dsc_compute_pipe_bpp(struct intel_dp *intel_dp,

					  struct intel_crtc_state *pipe_config,

					  struct drm_connector_state *conn_state,

					  struct link_config_limits *limits)

					  const struct link_config_limits *limits)

{

	struct intel_display *display = to_intel_display(intel_dp);

	struct intel_connector *connector =
@@ -2335,7 +2335,7 @@ static void intel_dp_fec_compute_config(struct intel_dp *intel_dp, 
int intel_dp_dsc_compute_config(struct intel_dp *intel_dp,

				struct intel_crtc_state *pipe_config,

				struct drm_connector_state *conn_state,

				struct link_config_limits *limits,

				const struct link_config_limits *limits,

				int timeslots,

				bool compute_pipe_bpp)

{

drivers/gpu/drm/i915/display/intel_dp.h

+1 −1
Original line number Diff line number Diff line
@@ -75,7 +75,7 @@ int intel_dp_compute_config(struct intel_encoder *encoder, 
int intel_dp_dsc_compute_config(struct intel_dp *intel_dp,

				struct intel_crtc_state *pipe_config,

				struct drm_connector_state *conn_state,

				struct link_config_limits *limits,

				const struct link_config_limits *limits,

				int timeslots,

				bool recompute_pipe_bpp);

void intel_dp_audio_compute_config(struct intel_encoder *encoder,

drivers/gpu/drm/i915/display/intel_dp_mst.c

+3 −3
Original line number Diff line number Diff line
@@ -363,7 +363,7 @@ int intel_dp_mtp_tu_compute_config(struct intel_dp *intel_dp, 
static int mst_stream_find_vcpi_slots_for_bpp(struct intel_dp *intel_dp,

					      struct intel_crtc_state *crtc_state,

					      int max_bpp, int min_bpp,

					      struct link_config_limits *limits,

					      const struct link_config_limits *limits,

					      struct drm_connector_state *conn_state,

					      int step, bool dsc)

{
@@ -388,7 +388,7 @@ static int mst_stream_find_vcpi_slots_for_bpp(struct intel_dp *intel_dp, 
static int mst_stream_compute_link_config(struct intel_dp *intel_dp,

					  struct intel_crtc_state *crtc_state,

					  struct drm_connector_state *conn_state,

					  struct link_config_limits *limits)

					  const struct link_config_limits *limits)

{

	/*

	 * FIXME: allocate the BW according to link_bpp, which in the case of
@@ -404,7 +404,7 @@ static int mst_stream_compute_link_config(struct intel_dp *intel_dp, 
static int mst_stream_dsc_compute_link_config(struct intel_dp *intel_dp,

					      struct intel_crtc_state *crtc_state,

					      struct drm_connector_state *conn_state,

					      struct link_config_limits *limits)

					      const struct link_config_limits *limits)

{

	struct intel_display *display = to_intel_display(intel_dp);

	struct intel_connector *connector = to_intel_connector(conn_state->connector);