drm/msm/dpu: fix DSC 1.2 enc subblock length
authorDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Wed, 2 Aug 2023 18:36:55 +0000 (21:36 +0300)
committerDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Thu, 3 Aug 2023 11:17:32 +0000 (14:17 +0300)
Both struct dpu_dsc_sub_blks instances declare enc subblock length to be
0x100, while the actual length is 0x9c (last register having offset 0x98).
Reduce subblock length to remove the empty register space from being
dumped.

Fixes: 0d1b10c63346 ("drm/msm/dpu: add DSC 1.2 hw blocks for relevant chipsets")
Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Patchwork: https://patchwork.freedesktop.org/patch/550999/
Link: https://lore.kernel.org/r/20230802183655.4188640-2-dmitry.baryshkov@linaro.org
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c

index ce369ee..713dfc0 100644 (file)
@@ -456,12 +456,12 @@ static const struct dpu_pingpong_sub_blks sc7280_pp_sblk = {
  * DSC sub blocks config
  *************************************************************/
 static const struct dpu_dsc_sub_blks dsc_sblk_0 = {
-       .enc = {.name = "enc", .base = 0x100, .len = 0x100},
+       .enc = {.name = "enc", .base = 0x100, .len = 0x9c},
        .ctl = {.name = "ctl", .base = 0xF00, .len = 0x10},
 };
 
 static const struct dpu_dsc_sub_blks dsc_sblk_1 = {
-       .enc = {.name = "enc", .base = 0x200, .len = 0x100},
+       .enc = {.name = "enc", .base = 0x200, .len = 0x9c},
        .ctl = {.name = "ctl", .base = 0xF80, .len = 0x10},
 };