Re: [PATCH v2 05/17] drm/msm/dpu: Remove duplicate register defines from INTF

From: Konrad Dybcio
Date: Tue Apr 18 2023 - 08:18:00 EST




On 17.04.2023 22:21, Marijn Suijten wrote:
> The INTF_FRAME_LINE_COUNT_EN, INTF_FRAME_COUNT and INTF_LINE_COUNT
> registers are already defined higher up, in the right place when sorted
> numerically.
>
> Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support")
> Signed-off-by: Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>

Konrad
> drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c
> index 84ee2efa9c66..b9dddf576c02 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c
> @@ -56,11 +56,6 @@
> #define INTF_TPG_RGB_MAPPING 0x11C
> #define INTF_PROG_FETCH_START 0x170
> #define INTF_PROG_ROT_START 0x174
> -
> -#define INTF_FRAME_LINE_COUNT_EN 0x0A8
> -#define INTF_FRAME_COUNT 0x0AC
> -#define INTF_LINE_COUNT 0x0B0
> -
> #define INTF_MUX 0x25C
> #define INTF_STATUS 0x26C
>
>