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 ce369eeffa7dcd70f2f02e434b7cb0f75974554c..713dfc0797181de3d39b0930aed4ef8a2c508c60 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},
 };