The driver references CWB array from SM8650, but should use the Eliza
specific, which has different register space sizes.  This should not
have noticeable impact on function but is indeed confusing, since the
Eliza table is used for .cwb_count.

Fixes: 0eb707bbc7fc ("drm/msm/dpu: Add support for Eliza SoC")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

---

Fix for a patch in next.
---
 drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_12_4_eliza.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_12_4_eliza.h b/drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_12_4_eliza.h
index b482a7e4e6c0..b93d32888972 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_12_4_eliza.h
+++ b/drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_12_4_eliza.h
@@ -353,7 +353,7 @@ const struct dpu_mdss_cfg dpu_eliza_cfg = {
 	.wb_count = ARRAY_SIZE(eliza_wb),
 	.wb = eliza_wb,
 	.cwb_count = ARRAY_SIZE(eliza_cwb),
-	.cwb = sm8650_cwb,
+	.cwb = eliza_cwb,
 	.intf_count = ARRAY_SIZE(eliza_intf),
 	.intf = eliza_intf,
 	.vbif = &sm8650_vbif,
-- 
2.51.0

On Tue, Mar 31, 2026 at 06:11:57PM +0200, Krzysztof Kozlowski wrote:
> The driver references CWB array from SM8650, but should use the Eliza
> specific, which has different register space sizes.  This should not
> have noticeable impact on function but is indeed confusing, since the
> Eliza table is used for .cwb_count.
> 
> Fixes: 0eb707bbc7fc ("drm/msm/dpu: Add support for Eliza SoC")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> 
> ---
> 
> Fix for a patch in next.
> ---
>  drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_12_4_eliza.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>


-- 
With best wishes
Dmitry