Hi Chris, On 2023-10-26 22:02, Christopher Obbard wrote: > Hi Jonas, > > On Thu, 2023-10-26 at 19:14 +0000, Jonas Karlman wrote: >> Use of DRM_FORMAT_RGB888 and DRM_FORMAT_BGR888 on e.g. RK3288, RK3328 >> and RK3399 result in wrong colors being displayed. >> >> The issue can be observed using modetest: >> >> modetest -s <connector_id>@<crtc_id>:1920x1080-60@RG24 >> modetest -s <connector_id>@<crtc_id>:1920x1080-60@BG24 >> >> Vendor 4.4 kernel apply an inverted rb swap for these formats on VOP >> full framework (IP version 3.x) compared to VOP little framework (2.x). >> >> Fix colors by applying different rb swap for VOP full framework (3.x) >> and VOP little framework (2.x) similar to vendor 4.4 kernel. >> >> Fixes: 85a359f25388 ("drm/rockchip: Add BGR formats to VOP") >> Signed-off-by: Jonas Karlman <jonas@xxxxxxxxx> > > Reviewed-by: Christopher Obbard <chris.obbard@xxxxxxxxxxxxx> > Tested-by: Christopher Obbard <chris.obbard@xxxxxxxxxxxxx> > > Since you missed adding my *-by tags in v2. > Thanks, and sorry about that ;-) Regards, Jonas