drm/komeda - At init write GCU control block to handle already on DPU
authorCarsten Haitzler <carsten.haitzler@arm.com>
Mon, 6 Jun 2022 11:47:13 +0000 (12:47 +0100)
committerLiviu Dudau <liviu.dudau@arm.com>
Fri, 22 Jul 2022 12:12:46 +0000 (13:12 +0100)
If something has already set up the DPU before the komeda driver comes
up, it will fail to init because it was just writing to the SRST bit in
the GCU control register and ignoring others. This resulted in TBU
bringup stalling and init failing. By writing completely we also  set the
mode back to 0 (inactive) too and thus TBU bringup works.

Signed-off-by: Carsten Haitzler <carsten.haitzler@arm.com>
Acked-by: Liviu Dudau <liviu.dudau@arm.com>
Signed-off-by: Liviu Dudau <liviu.dudau@arm.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220606114714.175499-2-carsten.haitzler@foss.arm.com
drivers/gpu/drm/arm/display/komeda/d71/d71_dev.c

index daa1fac..6c56f56 100644 (file)
@@ -310,8 +310,7 @@ static int d71_reset(struct d71_dev *d71)
        u32 __iomem *gcu = d71->gcu_addr;
        int ret;
 
-       malidp_write32_mask(gcu, BLK_CONTROL,
-                           GCU_CONTROL_SRST, GCU_CONTROL_SRST);
+       malidp_write32(gcu, BLK_CONTROL, GCU_CONTROL_SRST);
 
        ret = dp_wait_cond(!(malidp_read32(gcu, BLK_CONTROL) & GCU_CONTROL_SRST),
                           100, 1000, 10000);