* [PATCH v2] drm/rockchip: Disable blending for win0 @ 2018-04-18 17:31 Kristian H. Kristensen 2018-04-18 18:23 ` Sean Paul 2018-05-01 20:08 ` Sean Paul 0 siblings, 2 replies; 5+ messages in thread From: Kristian H. Kristensen @ 2018-04-18 17:31 UTC (permalink / raw) To: dri-devel; +Cc: Kristian H. Kristensen Blending win0 with the background color doesn't seem to work correctly. We only get the background color, no matter the contents of the win0 framebuffer. However, blending pre-multiplied color with the default opaque black default background color is a no-op, so we can just disable blending to get the correct result. Signed-off-by: Kristian H. Kristensen <hoegsberg@chromium.org> Cc: Sandy Huang <hjc@rock-chips.com> Cc: Sean Paul <seanpaul@chromium.org> --- v2: Drop CHROMIUM: prefix, rebase on Linus' master drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c index 53d4afe15278..753a7548da84 100644 --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c @@ -76,6 +76,9 @@ #define VOP_WIN_GET_YRGBADDR(vop, win) \ vop_readl(vop, win->base + win->phy->yrgb_mst.offset) +#define VOP_WIN_TO_INDEX(vop_win) \ + ((vop_win) - (vop_win)->vop->win) + #define to_vop(x) container_of(x, struct vop, crtc) #define to_vop_win(x) container_of(x, struct vop_win, base) @@ -708,6 +711,7 @@ static void vop_plane_atomic_update(struct drm_plane *plane, dma_addr_t dma_addr; uint32_t val; bool rb_swap; + int win_index = VOP_WIN_TO_INDEX(vop_win); int format; /* @@ -777,7 +781,14 @@ static void vop_plane_atomic_update(struct drm_plane *plane, rb_swap = has_rb_swapped(fb->format->format); VOP_WIN_SET(vop, win, rb_swap, rb_swap); - if (fb->format->has_alpha) { + /* + * Blending win0 with the background color doesn't seem to work + * correctly. We only get the background color, no matter the contents + * of the win0 framebuffer. However, blending pre-multiplied color + * with the default opaque black default background color is a no-op, + * so we can just disable blending to get the correct result. + */ + if (fb->format->has_alpha && win_index > 0) { VOP_WIN_SET(vop, win, dst_alpha_ctl, DST_FACTOR_M0(ALPHA_SRC_INVERSE)); val = SRC_ALPHA_EN(1) | SRC_COLOR_M0(ALPHA_SRC_PRE_MUL) | -- 2.17.0.484.g0c8726318c-goog _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] drm/rockchip: Disable blending for win0 2018-04-18 17:31 [PATCH v2] drm/rockchip: Disable blending for win0 Kristian H. Kristensen @ 2018-04-18 18:23 ` Sean Paul 2018-04-18 21:53 ` Kristian Kristensen 2018-05-01 20:08 ` Sean Paul 1 sibling, 1 reply; 5+ messages in thread From: Sean Paul @ 2018-04-18 18:23 UTC (permalink / raw) To: Kristian H. Kristensen; +Cc: Kristian H. Kristensen, dri-devel On Wed, Apr 18, 2018 at 10:31:52AM -0700, Kristian H. Kristensen wrote: > Blending win0 with the background color doesn't seem to work > correctly. Did you come to a conclusion about whether this applies across all Rk vops, or just 3399 big? Sean > We only get the background color, no matter the contents of > the win0 framebuffer. However, blending pre-multiplied color with the > default opaque black default background color is a no-op, so we can > just disable blending to get the correct result. > > Signed-off-by: Kristian H. Kristensen <hoegsberg@chromium.org> > Cc: Sandy Huang <hjc@rock-chips.com> > Cc: Sean Paul <seanpaul@chromium.org> > --- > > v2: Drop CHROMIUM: prefix, rebase on Linus' master > > drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > index 53d4afe15278..753a7548da84 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > @@ -76,6 +76,9 @@ > #define VOP_WIN_GET_YRGBADDR(vop, win) \ > vop_readl(vop, win->base + win->phy->yrgb_mst.offset) > > +#define VOP_WIN_TO_INDEX(vop_win) \ > + ((vop_win) - (vop_win)->vop->win) > + > #define to_vop(x) container_of(x, struct vop, crtc) > #define to_vop_win(x) container_of(x, struct vop_win, base) > > @@ -708,6 +711,7 @@ static void vop_plane_atomic_update(struct drm_plane *plane, > dma_addr_t dma_addr; > uint32_t val; > bool rb_swap; > + int win_index = VOP_WIN_TO_INDEX(vop_win); > int format; > > /* > @@ -777,7 +781,14 @@ static void vop_plane_atomic_update(struct drm_plane *plane, > rb_swap = has_rb_swapped(fb->format->format); > VOP_WIN_SET(vop, win, rb_swap, rb_swap); > > - if (fb->format->has_alpha) { > + /* > + * Blending win0 with the background color doesn't seem to work > + * correctly. We only get the background color, no matter the contents > + * of the win0 framebuffer. However, blending pre-multiplied color > + * with the default opaque black default background color is a no-op, > + * so we can just disable blending to get the correct result. > + */ > + if (fb->format->has_alpha && win_index > 0) { > VOP_WIN_SET(vop, win, dst_alpha_ctl, > DST_FACTOR_M0(ALPHA_SRC_INVERSE)); > val = SRC_ALPHA_EN(1) | SRC_COLOR_M0(ALPHA_SRC_PRE_MUL) | > -- > 2.17.0.484.g0c8726318c-goog > -- Sean Paul, Software Engineer, Google / Chromium OS _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] drm/rockchip: Disable blending for win0 2018-04-18 18:23 ` Sean Paul @ 2018-04-18 21:53 ` Kristian Kristensen 2018-04-19 15:46 ` Sean Paul 0 siblings, 1 reply; 5+ messages in thread From: Kristian Kristensen @ 2018-04-18 21:53 UTC (permalink / raw) To: Sean Paul; +Cc: Kristian H. Kristensen, Kristian H. Kristensen, DRI Development [-- Attachment #1.1: Type: text/plain, Size: 3101 bytes --] On Wed, Apr 18, 2018 at 11:23 AM, Sean Paul <seanpaul@chromium.org> wrote: > On Wed, Apr 18, 2018 at 10:31:52AM -0700, Kristian H. Kristensen wrote: > > Blending win0 with the background color doesn't seem to work > > correctly. > > Did you come to a conclusion about whether this applies across all Rk > vops, or > just 3399 big? > It didn't work on either big or little VOP on 3399, but oddly, 3288 seemed to work fine. Either way, what the patch is doing is correct and simpler, so I'm not sure it makes sense to try to be more specific about when we turn off blending. Kristian > Sean > > > We only get the background color, no matter the contents of > > the win0 framebuffer. However, blending pre-multiplied color with the > > default opaque black default background color is a no-op, so we can > > just disable blending to get the correct result. > > > > Signed-off-by: Kristian H. Kristensen <hoegsberg@chromium.org> > > Cc: Sandy Huang <hjc@rock-chips.com> > > Cc: Sean Paul <seanpaul@chromium.org> > > --- > > > > v2: Drop CHROMIUM: prefix, rebase on Linus' master > > > > drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 13 ++++++++++++- > > 1 file changed, 12 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > > index 53d4afe15278..753a7548da84 100644 > > --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > > @@ -76,6 +76,9 @@ > > #define VOP_WIN_GET_YRGBADDR(vop, win) \ > > vop_readl(vop, win->base + win->phy->yrgb_mst.offset) > > > > +#define VOP_WIN_TO_INDEX(vop_win) \ > > + ((vop_win) - (vop_win)->vop->win) > > + > > #define to_vop(x) container_of(x, struct vop, crtc) > > #define to_vop_win(x) container_of(x, struct vop_win, base) > > > > @@ -708,6 +711,7 @@ static void vop_plane_atomic_update(struct drm_plane > *plane, > > dma_addr_t dma_addr; > > uint32_t val; > > bool rb_swap; > > + int win_index = VOP_WIN_TO_INDEX(vop_win); > > int format; > > > > /* > > @@ -777,7 +781,14 @@ static void vop_plane_atomic_update(struct > drm_plane *plane, > > rb_swap = has_rb_swapped(fb->format->format); > > VOP_WIN_SET(vop, win, rb_swap, rb_swap); > > > > - if (fb->format->has_alpha) { > > + /* > > + * Blending win0 with the background color doesn't seem to work > > + * correctly. We only get the background color, no matter the > contents > > + * of the win0 framebuffer. However, blending pre-multiplied color > > + * with the default opaque black default background color is a > no-op, > > + * so we can just disable blending to get the correct result. > > + */ > > + if (fb->format->has_alpha && win_index > 0) { > > VOP_WIN_SET(vop, win, dst_alpha_ctl, > > DST_FACTOR_M0(ALPHA_SRC_INVERSE)); > > val = SRC_ALPHA_EN(1) | SRC_COLOR_M0(ALPHA_SRC_PRE_MUL) | > > -- > > 2.17.0.484.g0c8726318c-goog > > > > -- > Sean Paul, Software Engineer, Google / Chromium OS > [-- Attachment #1.2: Type: text/html, Size: 4433 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] drm/rockchip: Disable blending for win0 2018-04-18 21:53 ` Kristian Kristensen @ 2018-04-19 15:46 ` Sean Paul 0 siblings, 0 replies; 5+ messages in thread From: Sean Paul @ 2018-04-19 15:46 UTC (permalink / raw) To: Kristian Kristensen Cc: Kristian H. Kristensen, Kristian H. Kristensen, DRI Development On Wed, Apr 18, 2018 at 02:53:08PM -0700, Kristian Kristensen wrote: > On Wed, Apr 18, 2018 at 11:23 AM, Sean Paul <seanpaul@chromium.org> wrote: > > > On Wed, Apr 18, 2018 at 10:31:52AM -0700, Kristian H. Kristensen wrote: > > > Blending win0 with the background color doesn't seem to work > > > correctly. > > > > Did you come to a conclusion about whether this applies across all Rk > > vops, or > > just 3399 big? > > > > It didn't work on either big or little VOP on 3399, but oddly, 3288 seemed > to work fine. Either way, what the patch is doing is correct and simpler, > so I'm not sure it makes sense to try to be more specific about when we > turn off blending. Yeah, fair enough. We'll need to revisit if we ever support anything more than pre-mult/black bg anyways, so Reviewed-by: Sean Paul <seanpaul@chromium.org> Sean > > Kristian > > > > Sean > > > > > We only get the background color, no matter the contents of > > > the win0 framebuffer. However, blending pre-multiplied color with the > > > default opaque black default background color is a no-op, so we can > > > just disable blending to get the correct result. > > > > > > Signed-off-by: Kristian H. Kristensen <hoegsberg@chromium.org> > > > Cc: Sandy Huang <hjc@rock-chips.com> > > > Cc: Sean Paul <seanpaul@chromium.org> > > > --- > > > > > > v2: Drop CHROMIUM: prefix, rebase on Linus' master > > > > > > drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 13 ++++++++++++- > > > 1 file changed, 12 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > > b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > > > index 53d4afe15278..753a7548da84 100644 > > > --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > > > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > > > @@ -76,6 +76,9 @@ > > > #define VOP_WIN_GET_YRGBADDR(vop, win) \ > > > vop_readl(vop, win->base + win->phy->yrgb_mst.offset) > > > > > > +#define VOP_WIN_TO_INDEX(vop_win) \ > > > + ((vop_win) - (vop_win)->vop->win) > > > + > > > #define to_vop(x) container_of(x, struct vop, crtc) > > > #define to_vop_win(x) container_of(x, struct vop_win, base) > > > > > > @@ -708,6 +711,7 @@ static void vop_plane_atomic_update(struct drm_plane > > *plane, > > > dma_addr_t dma_addr; > > > uint32_t val; > > > bool rb_swap; > > > + int win_index = VOP_WIN_TO_INDEX(vop_win); > > > int format; > > > > > > /* > > > @@ -777,7 +781,14 @@ static void vop_plane_atomic_update(struct > > drm_plane *plane, > > > rb_swap = has_rb_swapped(fb->format->format); > > > VOP_WIN_SET(vop, win, rb_swap, rb_swap); > > > > > > - if (fb->format->has_alpha) { > > > + /* > > > + * Blending win0 with the background color doesn't seem to work > > > + * correctly. We only get the background color, no matter the > > contents > > > + * of the win0 framebuffer. However, blending pre-multiplied color > > > + * with the default opaque black default background color is a > > no-op, > > > + * so we can just disable blending to get the correct result. > > > + */ > > > + if (fb->format->has_alpha && win_index > 0) { > > > VOP_WIN_SET(vop, win, dst_alpha_ctl, > > > DST_FACTOR_M0(ALPHA_SRC_INVERSE)); > > > val = SRC_ALPHA_EN(1) | SRC_COLOR_M0(ALPHA_SRC_PRE_MUL) | > > > -- > > > 2.17.0.484.g0c8726318c-goog > > > > > > > -- > > Sean Paul, Software Engineer, Google / Chromium OS > > -- Sean Paul, Software Engineer, Google / Chromium OS _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] drm/rockchip: Disable blending for win0 2018-04-18 17:31 [PATCH v2] drm/rockchip: Disable blending for win0 Kristian H. Kristensen 2018-04-18 18:23 ` Sean Paul @ 2018-05-01 20:08 ` Sean Paul 1 sibling, 0 replies; 5+ messages in thread From: Sean Paul @ 2018-05-01 20:08 UTC (permalink / raw) To: Kristian H. Kristensen; +Cc: Kristian H. Kristensen, dri-devel On Wed, Apr 18, 2018 at 10:31:52AM -0700, Kristian H. Kristensen wrote: > Blending win0 with the background color doesn't seem to work > correctly. We only get the background color, no matter the contents of > the win0 framebuffer. However, blending pre-multiplied color with the > default opaque black default background color is a no-op, so we can > just disable blending to get the correct result. > > Signed-off-by: Kristian H. Kristensen <hoegsberg@chromium.org> Pushed to -misc-next Thank you for your patch! Sean > Cc: Sandy Huang <hjc@rock-chips.com> > Cc: Sean Paul <seanpaul@chromium.org> > --- > > v2: Drop CHROMIUM: prefix, rebase on Linus' master > > drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > index 53d4afe15278..753a7548da84 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > @@ -76,6 +76,9 @@ > #define VOP_WIN_GET_YRGBADDR(vop, win) \ > vop_readl(vop, win->base + win->phy->yrgb_mst.offset) > > +#define VOP_WIN_TO_INDEX(vop_win) \ > + ((vop_win) - (vop_win)->vop->win) > + > #define to_vop(x) container_of(x, struct vop, crtc) > #define to_vop_win(x) container_of(x, struct vop_win, base) > > @@ -708,6 +711,7 @@ static void vop_plane_atomic_update(struct drm_plane *plane, > dma_addr_t dma_addr; > uint32_t val; > bool rb_swap; > + int win_index = VOP_WIN_TO_INDEX(vop_win); > int format; > > /* > @@ -777,7 +781,14 @@ static void vop_plane_atomic_update(struct drm_plane *plane, > rb_swap = has_rb_swapped(fb->format->format); > VOP_WIN_SET(vop, win, rb_swap, rb_swap); > > - if (fb->format->has_alpha) { > + /* > + * Blending win0 with the background color doesn't seem to work > + * correctly. We only get the background color, no matter the contents > + * of the win0 framebuffer. However, blending pre-multiplied color > + * with the default opaque black default background color is a no-op, > + * so we can just disable blending to get the correct result. > + */ > + if (fb->format->has_alpha && win_index > 0) { > VOP_WIN_SET(vop, win, dst_alpha_ctl, > DST_FACTOR_M0(ALPHA_SRC_INVERSE)); > val = SRC_ALPHA_EN(1) | SRC_COLOR_M0(ALPHA_SRC_PRE_MUL) | > -- > 2.17.0.484.g0c8726318c-goog > -- Sean Paul, Software Engineer, Google / Chromium OS _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-05-01 20:08 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-04-18 17:31 [PATCH v2] drm/rockchip: Disable blending for win0 Kristian H. Kristensen 2018-04-18 18:23 ` Sean Paul 2018-04-18 21:53 ` Kristian Kristensen 2018-04-19 15:46 ` Sean Paul 2018-05-01 20:08 ` Sean Paul
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.