From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hyungwon Hwang Subject: Re: [PATCH 07/13] tests/exynos: use XRGB8888 for framebuffer Date: Fri, 30 Oct 2015 15:41:10 +0900 Message-ID: <20151030154110.6e458145@hwh-ubuntu> References: <1442937302-8211-1-git-send-email-tjakobi@math.uni-bielefeld.de> <1442937302-8211-8-git-send-email-tjakobi@math.uni-bielefeld.de> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from mailout4.samsung.com ([203.254.224.34]:58034 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751485AbbJ3GlM (ORCPT ); Fri, 30 Oct 2015 02:41:12 -0400 Received: from epcpsbgr2.samsung.com (u142.gpu120.samsung.co.kr [203.254.230.142]) by mailout4.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0NX001GLYT8MRRA0@mailout4.samsung.com> for linux-samsung-soc@vger.kernel.org; Fri, 30 Oct 2015 15:41:10 +0900 (KST) In-reply-to: <1442937302-8211-8-git-send-email-tjakobi@math.uni-bielefeld.de> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Tobias Jakobi Cc: linux-samsung-soc@vger.kernel.org, dri-devel@lists.freedesktop.org, emil.l.velikov@gmail.com, jy0922.shim@samsung.com, gustavo.padovan@collabora.co.uk, inki.dae@samsung.com On Tue, 22 Sep 2015 17:54:56 +0200 Tobias Jakobi wrote: > This matches the G2D color mode that is used in the entire code. > The previous (incorrect) RGBA8888 would only work since the > Exynos mixer did its configuration based on the bpp, and not > based on the actual pixelformat. > > Signed-off-by: Tobias Jakobi > --- > tests/exynos/exynos_fimg2d_test.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/exynos/exynos_fimg2d_test.c > b/tests/exynos/exynos_fimg2d_test.c index 8794dac..dfb00a0 100644 > --- a/tests/exynos/exynos_fimg2d_test.c > +++ b/tests/exynos/exynos_fimg2d_test.c > @@ -675,7 +675,7 @@ int main(int argc, char **argv) > offsets[0] = 0; > > ret = drmModeAddFB2(dev->fd, screen_width, screen_height, > - DRM_FORMAT_RGBA8888, handles, > + DRM_FORMAT_XRGB8888, handles, > pitches, offsets, &fb_id, 0); Reviewed-by: Hyungwon Hwang Nice catch. It's right, if there was no previous setting for source image color mode. But I think it could be the source image color mode was set by another application before when this test runs. So I think the code which sets the source image color mode must be added. Best regards, Hyungwon Hwang > if (ret < 0) > goto err_destroy_buffer;