Skip to content

Commit 655c6c1

Browse files
Hsin-Te YuanChun-Kuang Hu
authored andcommitted
drm/mediatek: Fix color format MACROs in OVL
In commit 9f428b9 ("drm/mediatek: Add new color format MACROs in OVL"), some new color formats are defined in the MACROs to make the switch statement more concise. That commit was intended to be a no-op cleanup. However, there are typos in these formats MACROs, which cause the return value to be incorrect. Fix the typos to ensure the return value remains unchanged. Fixes: 9f428b9 ("drm/mediatek: Add new color format MACROs in OVL") Reviewed-by: Douglas Anderson <dianders@chromium.org> Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com> Signed-off-by: Hsin-Te Yuan <yuanhsinte@chromium.org> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> Reviewed-by: CK Hu <ck.hu@mediatek.com> Link: https://patchwork.kernel.org/project/dri-devel/patch/20241016-color-v3-1-e0f5f44a72d8@chromium.org/ Signed-off-by: Chun-Kuang Hu <chunkuang.hu@kernel.org>
1 parent e6411bf commit 655c6c1

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

drivers/gpu/drm/mediatek/mtk_disp_ovl.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,8 @@
6565
#define OVL_CON_CLRFMT_RGB (1 << 12)
6666
#define OVL_CON_CLRFMT_ARGB8888 (2 << 12)
6767
#define OVL_CON_CLRFMT_RGBA8888 (3 << 12)
68-
#define OVL_CON_CLRFMT_ABGR8888 (OVL_CON_CLRFMT_RGBA8888 | OVL_CON_BYTE_SWAP)
69-
#define OVL_CON_CLRFMT_BGRA8888 (OVL_CON_CLRFMT_ARGB8888 | OVL_CON_BYTE_SWAP)
68+
#define OVL_CON_CLRFMT_ABGR8888 (OVL_CON_CLRFMT_ARGB8888 | OVL_CON_BYTE_SWAP)
69+
#define OVL_CON_CLRFMT_BGRA8888 (OVL_CON_CLRFMT_RGBA8888 | OVL_CON_BYTE_SWAP)
7070
#define OVL_CON_CLRFMT_UYVY (4 << 12)
7171
#define OVL_CON_CLRFMT_YUYV (5 << 12)
7272
#define OVL_CON_MTX_YUV_TO_RGB (6 << 16)

0 commit comments

Comments
 (0)