Commit a67f2cc6 authored by Dmitry Baryshkov's avatar Dmitry Baryshkov
Browse files

drm/msm/dpu: drop pe argument from _dpu_hw_sspp_setup_scaler3 



The _dpu_hw_sspp_setup_scaler3 (hw_sspp->setup_scaler) does not use pe
argument. Let's remove it while we are cleaning scaled configuration.

Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: default avatarAbhinav Kumar <quic_abhinavk@quicinc.com>
Link: https://lore.kernel.org/r/20211201225140.2481577-3-dmitry.baryshkov@linaro.org


Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
parent 6f4c23e7

drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c

+0 −2
Original line number Diff line number Diff line
@@ -413,13 +413,11 @@ static void dpu_hw_sspp_setup_pe_config(struct dpu_hw_pipe *ctx, 


static void _dpu_hw_sspp_setup_scaler3(struct dpu_hw_pipe *ctx,

		struct dpu_hw_pipe_cfg *sspp,

		struct dpu_hw_pixel_ext *pe,

		void *scaler_cfg)

{

	u32 idx;

	struct dpu_hw_scaler3_cfg *scaler3_cfg = scaler_cfg;



	(void)pe;

	if (_sspp_subblk_offset(ctx, DPU_SSPP_SCALER_QSEED3, &idx) || !sspp

		|| !scaler3_cfg)

		return;

drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h

+0 −2
Original line number Diff line number Diff line
@@ -334,12 +334,10 @@ struct dpu_hw_sspp_ops { 
	 * setup_scaler - setup scaler

	 * @ctx: Pointer to pipe context

	 * @pipe_cfg: Pointer to pipe configuration

	 * @pe_cfg: Pointer to pixel extension configuration

	 * @scaler_cfg: Pointer to scaler configuration

	 */

	void (*setup_scaler)(struct dpu_hw_pipe *ctx,

		struct dpu_hw_pipe_cfg *pipe_cfg,

		struct dpu_hw_pixel_ext *pe_cfg,

		void *scaler_cfg);



	/**

drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c

+1 −1
Original line number Diff line number Diff line
@@ -688,7 +688,7 @@ static void _dpu_plane_setup_scaler(struct dpu_plane *pdpu, 
	if (pdpu->pipe_hw->ops.setup_scaler &&

			pstate->multirect_index != DPU_SSPP_RECT_1)

		pdpu->pipe_hw->ops.setup_scaler(pdpu->pipe_hw,

				pipe_cfg, &pixel_ext,

				pipe_cfg,

				&scaler3_cfg);

}