Re: [PATCH v4 07/14] drm/mediatek: add the comment about color format setting for OVL

From: CK Hu
Date: Wed Aug 08 2018 - 22:37:50 EST


Hi, Stu:

On Thu, 2018-08-09 at 10:15 +0800, Stu Hsieh wrote:
> This patch add the comment about color format setting for OVL
>
> Signed-off-by: Stu Hsieh <stu.hsieh@xxxxxxxxxxxx>

Reviewed-by: CK Hu <ck.hu@xxxxxxxxxxxx>

> ---
> drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> index 978782a77629..0facd823c552 100644
> --- a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> +++ b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> @@ -157,6 +157,11 @@ static void mtk_ovl_layer_off(struct mtk_ddp_comp *comp, unsigned int idx)
>
> static unsigned int ovl_fmt_convert(struct mtk_disp_ovl *ovl, unsigned int fmt)
> {
> + /* The return value in switch "MEM_MODE_INPUT_FORMAT_XXX"
> + * is defined in mediatek HW data sheet.
> + * The alphabet order in XXX is no relation to data
> + * arrangement in memory.
> + */
> switch (fmt) {
> default:
> case DRM_FORMAT_RGB565: