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 <[email protected]> Reviewed-by: Matthias Brugger <[email protected]> Signed-off-by: Hsin-Te Yuan <[email protected]> Reviewed-by: AngeloGioacchino Del Regno <[email protected]> Reviewed-by: CK Hu <[email protected]> Link: https://patchwork.kernel.org/project/dri-devel/patch/[email protected]/ Signed-off-by: Chun-Kuang Hu <[email protected]>
1 parent e6411bf commit 655c6c1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

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)