Re: [PATCH v3 01/13] drm/msm/dpu: remove scalar config definitions

From: Sean Paul <seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
To: Jeykumar Sankaran <jsanka-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
Cc: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	robdclark-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org,
	hoegsberg-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org,
	freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: [PATCH v3 01/13] drm/msm/dpu: remove scalar config definitions
Date: Mon, 13 Aug 2018 09:50:13 -0400	[thread overview]
Message-ID: <20180813135013.GW20303@art_vandelay> (raw)
In-Reply-To: <1533697956-29686-2-git-send-email-jsanka-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>

On Tue, Aug 07, 2018 at 08:12:28PM -0700, Jeykumar Sankaran wrote:
> cleans up left out scalar config definitions from headers
> 
> changes in v2:
> 	- none
> changes in v3:
> 	- none
> 
> Change-Id: Id824dd5075c666f97b964573c97215bb786eac75

Please strip Change-Id before sending patches.

Reviewed-by: Sean Paul <seanpaul@chromium.org>

> Signed-off-by: Jeykumar Sankaran <jsanka@codeaurora.org>
> ---
>  drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h    |  2 --
>  drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h | 10 ----------
>  2 files changed, 12 deletions(-)
> 
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h
> index e87109e..0e9aafa 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h
> @@ -164,7 +164,6 @@ struct dpu_crtc_frame_event {
>   * @cur_perf      : current performance committed to clock/bandwidth driver
>   * @rp_lock       : serialization lock for resource pool
>   * @rp_head       : list of active resource pool
> - * @scl3_cfg_lut  : qseed3 lut config
>   */
>  struct dpu_crtc {
>  	struct drm_crtc base;
> @@ -175,7 +174,6 @@ struct dpu_crtc {
>  	u32 num_mixers;
>  	bool mixers_swapped;
>  	struct dpu_crtc_mixer mixers[CRTC_DUAL_MIXERS];
> -	struct dpu_hw_scaler3_lut_cfg *scl3_lut_cfg;
>  
>  	struct drm_pending_vblank_event *event;
>  	u32 vsync_count;
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h
> index 1240f50..c5c8f60 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h
> @@ -148,16 +148,6 @@ struct dpu_hw_scaler3_cfg {
>  	struct dpu_hw_scaler3_de_cfg de;
>  };
>  
> -struct dpu_hw_scaler3_lut_cfg {
> -	bool is_configured;
> -	u32 *dir_lut;
> -	size_t dir_len;
> -	u32 *cir_lut;
> -	size_t cir_len;
> -	u32 *sep_lut;
> -	size_t sep_len;
> -};
> -
>  /**
>   * struct dpu_drm_pix_ext_v1 - version 1 of pixel ext structure
>   * @num_ext_pxls_lr: Number of total horizontal pixels
> -- 
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
> 

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno