From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id F3A95C433EF for ; Sat, 29 Jan 2022 22:03:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Subject:Cc:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=amMuurKbzsbqI7+2dJwPMVgxU/JIfm4K8UTDmgL9AEA=; b=TTNiCqmWbMCzio rO81iHgu5sarxw1TIb2xS4V6AeSUp3h0WQWzKncES2U81HMH5kXs6zQdKuzNhTtSUY5yOZZCrjs5/ MU0dgiob4GLvNLEZHJ83Mlzv4q2Q/jcWpgr7ke2/soWf4m1lHrd2gChJFl1QPmmhIjbcl0mqv9wcA WSc4lUzyHDXqnkhpsGR3JtZ6EjxuRYqB/bEX9sJldAd0PYFuReC40yfPJ36uG1XefTAzfDDFHUaEN uMwD6BBvVn2Rf+zY7WJJhTk7TwcJpCb134rYETvI/5nipivtSMwhLMSP8bUovqym6S3hQPO2PRHtL GAHECbVt5X0t7l8Kw+Ig==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nDvnI-005Tf9-LM; Sat, 29 Jan 2022 22:02:04 +0000 Received: from mail-ej1-x62a.google.com ([2a00:1450:4864:20::62a]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nDvnE-005TeJ-4r for linux-arm-kernel@lists.infradead.org; Sat, 29 Jan 2022 22:02:01 +0000 Received: by mail-ej1-x62a.google.com with SMTP id d10so29178319eje.10 for ; Sat, 29 Jan 2022 14:01:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=date:from:to:cc:subject:message-id:mime-version:content-disposition :content-transfer-encoding; bh=yjUU9ER/vIEGLeysJCo+/yYTdTjRHWEWnfNYddtn3aA=; b=GasqK6RF2C1NLwNFKTrQaUjs2o8JvlWlPo88Gk33v2yX68jTVNTX8RuL9WeODxuYgI 0zVfdFLvDQ6Scm07F8G4M28CBxrFgmgzFCZN1YPFJOzIwEZtqfGnmpEunqvqASO6YJln Ync30uNDDbpnXWDLMpetV3oG0ExsqVaZqaLvT6dCXDjteTaouVkFwgpWOx4bV27QHNEy F7vqwf6ejDMbcH1ZrwJR177hpWXMzpm/Z3N3VNSo/1OHYAd9Bsz2df5sPio+OqhWcHZh LF/FInpELUZKeDEjeZyr1LsbNVfm1SYbnkZy0JFf+3/1ZopOuaeHe2zmd1Z9XxqiP/rV B77w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:mime-version :content-disposition:content-transfer-encoding; bh=yjUU9ER/vIEGLeysJCo+/yYTdTjRHWEWnfNYddtn3aA=; b=yvFO/syRuG/gM8D0r2iP+uxkL/pth5Oz2ofl/LBi3oIEWNqqnEFAQ+nxByMaMvg8xQ tg+mNE7TqbXLeEZexMbz8eGsp+KetPevVOANz1U1ok3hnaT3sZUa4jhLyybmKrWp1O/y f1gq1Yn3utav1xglbron2tvWTkJ98VEbiwznhPL86dS22+exJhtBLSfBbAXAIEPYGD2T cuU0tgGXzxYrjh4KGBEgn6+t5PZ+yGXiZRZxqDZIkHSBu/O8o3Sx/vrXIpPq84n2BB9Y CYr2d5gFsPCaViZx7sv+TY9foz2F3G6cnOaQudccyJwaCuoWhhYgyBDWo0NppYlo9oYW Rjtw== X-Gm-Message-State: AOAM533JvjxF/7KVuS4XrSSqxhsA5XnBzbMbzmaz5tmdd6chIWOi/SuD oSBqxgKWWNRkE+Ks0n458r4= X-Google-Smtp-Source: ABdhPJyn9Tnz9SCtDypxoXBmZOjSdAvG+KAgXi0wQd7Ix3xNYlAf5Zop3ncrx+pbyhT4baltFFW5EQ== X-Received: by 2002:a17:907:94d2:: with SMTP id dn18mr4507899ejc.304.1643493716705; Sat, 29 Jan 2022 14:01:56 -0800 (PST) Received: from adroid (027-177-184-091.ip-addr.vsenet.de. [91.184.177.27]) by smtp.gmail.com with ESMTPSA id ry6sm3338861ejc.45.2022.01.29.14.01.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 29 Jan 2022 14:01:56 -0800 (PST) Date: Sat, 29 Jan 2022 23:01:53 +0100 From: Martin =?iso-8859-1?Q?J=FCcker?= To: Inki Dae , Joonyoung Shim , Seung-Woo Kim , Kyungmin Park , David Airlie , Daniel Vetter , Krzysztof Kozlowski , dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Martin =?iso-8859-1?Q?J=FCcker?= Subject: [PATCH] drm/exynos: fimd: add BGR support for exynos4/5 Message-ID: <20220129220153.GA33165@adroid> MIME-Version: 1.0 Content-Disposition: inline X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220129_140200_226679_93497ABD X-CRM114-Status: GOOD ( 20.96 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org In the downstream kernels for exynos4 and exynos5 devices, there is an undocumented register that controls the order of the RGB output. It can be set to either normal order or reversed, which enables BGR support for those SoCs. This patch enables the BGR support for all the SoCs that were found to have at least one device with this logic in the corresponding downstream kernels. Signed-off-by: Martin J=FCcker --- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 42 ++++++++++++++++++++++-- include/video/samsung_fimd.h | 4 +++ 2 files changed, 44 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exy= nos/exynos_drm_fimd.c index c735e53939d8..cb632360c968 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c @@ -109,6 +109,7 @@ struct fimd_driver_data { unsigned int has_dp_clk:1; unsigned int has_hw_trigger:1; unsigned int has_trigger_per_te:1; + unsigned int has_bgr_support:1; }; = static struct fimd_driver_data s3c64xx_fimd_driver_data =3D { @@ -138,6 +139,7 @@ static struct fimd_driver_data exynos4_fimd_driver_data= =3D { .lcdblk_bypass_shift =3D 1, .has_shadowcon =3D 1, .has_vtsel =3D 1, + .has_bgr_support =3D 1, }; = static struct fimd_driver_data exynos5_fimd_driver_data =3D { @@ -149,6 +151,7 @@ static struct fimd_driver_data exynos5_fimd_driver_data= =3D { .has_vidoutcon =3D 1, .has_vtsel =3D 1, .has_dp_clk =3D 1, + .has_bgr_support =3D 1, }; = static struct fimd_driver_data exynos5420_fimd_driver_data =3D { @@ -162,6 +165,7 @@ static struct fimd_driver_data exynos5420_fimd_driver_d= ata =3D { .has_vtsel =3D 1, .has_mic_bypass =3D 1, .has_dp_clk =3D 1, + .has_bgr_support =3D 1, }; = struct fimd_context { @@ -226,6 +230,18 @@ static const uint32_t fimd_formats[] =3D { DRM_FORMAT_ARGB8888, }; = +static const uint32_t fimd_extended_formats[] =3D { + DRM_FORMAT_C8, + DRM_FORMAT_XRGB1555, + DRM_FORMAT_XBGR1555, + DRM_FORMAT_RGB565, + DRM_FORMAT_BGR565, + DRM_FORMAT_XRGB8888, + DRM_FORMAT_XBGR8888, + DRM_FORMAT_ARGB8888, + DRM_FORMAT_ABGR8888, +}; + static const unsigned int capabilities[WINDOWS_NR] =3D { 0, EXYNOS_DRM_PLANE_CAP_WIN_BLEND | EXYNOS_DRM_PLANE_CAP_PIX_BLEND, @@ -673,21 +689,25 @@ static void fimd_win_set_pixfmt(struct fimd_context *= ctx, unsigned int win, val |=3D WINCONx_BYTSWP; break; case DRM_FORMAT_XRGB1555: + case DRM_FORMAT_XBGR1555: val |=3D WINCON0_BPPMODE_16BPP_1555; val |=3D WINCONx_HAWSWP; val |=3D WINCONx_BURSTLEN_16WORD; break; case DRM_FORMAT_RGB565: + case DRM_FORMAT_BGR565: val |=3D WINCON0_BPPMODE_16BPP_565; val |=3D WINCONx_HAWSWP; val |=3D WINCONx_BURSTLEN_16WORD; break; case DRM_FORMAT_XRGB8888: + case DRM_FORMAT_XBGR8888: val |=3D WINCON0_BPPMODE_24BPP_888; val |=3D WINCONx_WSWP; val |=3D WINCONx_BURSTLEN_16WORD; break; case DRM_FORMAT_ARGB8888: + case DRM_FORMAT_ABGR8888: default: val |=3D WINCON1_BPPMODE_25BPP_A1888; val |=3D WINCONx_WSWP; @@ -695,6 +715,18 @@ static void fimd_win_set_pixfmt(struct fimd_context *c= tx, unsigned int win, break; } = + switch (pixel_format) { + case DRM_FORMAT_XBGR1555: + case DRM_FORMAT_XBGR8888: + case DRM_FORMAT_ABGR8888: + case DRM_FORMAT_BGR565: + writel(WIN_RGB_ORDER_REVERSE, ctx->regs + WIN_RGB_ORDER(win)); + break; + default: + writel(WIN_RGB_ORDER_FORWARD, ctx->regs + WIN_RGB_ORDER(win)); + break; + } + /* * Setting dma-burst to 16Word causes permanent tearing for very small * buffers, e.g. cursor buffer. Burst Mode switching which based on @@ -1074,8 +1106,14 @@ static int fimd_bind(struct device *dev, struct devi= ce *master, void *data) ctx->drm_dev =3D drm_dev; = for (i =3D 0; i < WINDOWS_NR; i++) { - ctx->configs[i].pixel_formats =3D fimd_formats; - ctx->configs[i].num_pixel_formats =3D ARRAY_SIZE(fimd_formats); + if (ctx->driver_data->has_bgr_support) { + ctx->configs[i].pixel_formats =3D fimd_extended_formats; + ctx->configs[i].num_pixel_formats =3D ARRAY_SIZE(fimd_extended_formats); + } else { + ctx->configs[i].pixel_formats =3D fimd_formats; + ctx->configs[i].num_pixel_formats =3D ARRAY_SIZE(fimd_formats); + } + ctx->configs[i].zpos =3D i; ctx->configs[i].type =3D fimd_win_types[i]; ctx->configs[i].capabilities =3D capabilities[i]; diff --git a/include/video/samsung_fimd.h b/include/video/samsung_fimd.h index c4a93ce1de48..e6966d187591 100644 --- a/include/video/samsung_fimd.h +++ b/include/video/samsung_fimd.h @@ -476,6 +476,10 @@ * 1111 -none- -none- -none- -none- -none- */ = +#define WIN_RGB_ORDER(_win) (0x2020 + ((_win) * 4)) +#define WIN_RGB_ORDER_FORWARD (0 << 11) +#define WIN_RGB_ORDER_REVERSE (1 << 11) + /* FIMD Version 8 register offset definitions */ #define FIMD_V8_VIDTCON0 0x20010 #define FIMD_V8_VIDTCON1 0x20014 -- = 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel