On Thursday, 12 December 2024 02:52:34 EST Andy Yan wrote:

Hi Detlev,

At 2024-12-11 23:45:01, "Detlev Casanova" [off-list ref] 

wrote:

quoted

On Wednesday, 11 December 2024 01:34:34 EST Andy Yan wrote:

quoted

Hi Detlev,

At 2024-12-11 02:40:14, "Detlev Casanova" [off-list ref]

wrote:

quoted

quoted

Hi Andy,

On Monday, 9 December 2024 07:29:18 EST Andy Yan wrote:

quoted

From: Andy Yan <andy.yan@rock-chips.com>

There are two AXI bus in vop2, windows attached on the same bus must
have a unique channel YUV and RGB channel ID.

The default IDs will conflict with each other on the rk3588, so they
need to be reassigned.

Fixes: 5a028e8f062f ("drm/rockchip: vop2: Add support for rk3588")
Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
Tested-by: Derek Foreman <redacted>

---

Changes in v5:
- Added in V5

 drivers/gpu/drm/rockchip/rockchip_drm_vop2.c | 14 +++++++++++
 drivers/gpu/drm/rockchip/rockchip_drm_vop2.h |  9 +++++++
 drivers/gpu/drm/rockchip/rockchip_vop2_reg.c | 26
 +++++++++++++++++++-
 3 files changed, 48 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c

b/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c index
dc4edd65bc9e..8b9ca046eeeb 100644

--- a/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c
+++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c

@@ -1426,6 +1426,12 @@ static void vop2_plane_atomic_update(struct

drm_plane *plane, &fb->format->format,

 		afbc_en ? "AFBC" : "", &yrgb_mst);

+	if (vop2->data->soc_id > 3568) {

Shouldn't this be done only for rk3588, as specified in the comments
below
? The test we did before showed that it is failing on rk3576 and 3576 is

quoted

3588.

I think this is because you tested before with the patch I gave Derek
without axi id assigned for rk3576.
I assigned axi id for rk3576 in this version 18/18（The new ID assignment
can adapt to more application scenarios.）, can you test it with the
whole V5 version again?

Yes, I will do that. But patch 18 mentions:
+       /* Read only bit on rk3576*/
+       [VOP2_WIN_AXI_BUS_ID] = REG_FIELD(RK3568_CLUSTER_CTRL, 13, 13),

And the bit is being written here. If it is indeed writable, then I would
drop that comment.

The AXI_BUS_ID for two Cluster windows on RK3576 are fixed by hardware, that
means whether we write it or not, it won't change anything.
But the AXI_BUS_ID for Esmart windows on rk3576 and  AXI_YRGB/UV_R_ID for
all cluster/esmart windows on rk3576 are writeable.
I think we directly write it here as the current code can make things easy.

I see, saying it is read only may be a bit confusing then, but it makes sense. 
You could be more specific adding that writing has no effect.

Anyway, I tested this and it works as expected:

Tested-by: Detlev Casanova <detlev.casanova@collabora.com>

Regards,
Detlev.

quoted

quoted

quoted

I suggest

if (vop2->data->soc_id == 3588) {

Regards,
Detlev

quoted

+		vop2_win_write(win, VOP2_WIN_AXI_BUS_ID, win->data-

axi_bus_id);

+		vop2_win_write(win, VOP2_WIN_AXI_YRGB_R_ID, win->data-

axi_yrgb_r_id);

+		vop2_win_write(win, VOP2_WIN_AXI_UV_R_ID, win->data-

axi_uv_r_id);

+	}
+

 	if (vop2_cluster_window(win))
 	
 		vop2_win_write(win, VOP2_WIN_AFBC_HALF_BLOCK_EN,

half_block_en);

Detlev.



_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip