[DPU PATCH v2] drm/msm/dpu: Change RGB565 and BGR565 format map.

[DPU PATCH v2] drm/msm/dpu: Change RGB565 and BGR565 format map.

[Tanmay Shah]

Red and Blue colors will be interchanged on display with
current format maps for RGB565 and BGR565.

Change both format maps to display correct colors.

Signed-off-by: Tanmay Shah <tanmay@codeaurora.org>
---
 drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c
index d53abc8ce670..f59fe1a9f4b9 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c
@@ -263,13 +263,13 @@ static const struct dpu_format dpu_format_map[] = {
 
 	INTERLEAVED_RGB_FMT(RGB565,
 		0, COLOR_5BIT, COLOR_6BIT, COLOR_5BIT,
-		C1_B_Cb, C0_G_Y, C2_R_Cr, 0, 3,
+		C2_R_Cr, C0_G_Y, C1_B_Cb, 0, 3,
 		false, 2, 0,
 		DPU_FETCH_LINEAR, 1),
 
 	INTERLEAVED_RGB_FMT(BGR565,
 		0, COLOR_5BIT, COLOR_6BIT, COLOR_5BIT,
-		C2_R_Cr, C0_G_Y, C1_B_Cb, 0, 3,
+		C1_B_Cb, C0_G_Y, C2_R_Cr, 0, 3,
 		false, 2, 0,
 		DPU_FETCH_LINEAR, 1),
 
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

Re: [DPU PATCH v2] drm/msm/dpu: Change RGB565 and BGR565 format map.

[Jeykumar Sankaran]

On 2018-12-14 11:46, Tanmay Shah wrote:
> Red and Blue colors will be interchanged on display with
> current format maps for RGB565 and BGR565.
> 
> Change both format maps to display correct colors.
> 
You can drop "DPU PATCH" prefix in the patches.
Can you also provide history on what has changed since v1.

Thanks,
Jeykumar S.
> Signed-off-by: Tanmay Shah <tanmay@codeaurora.org>
> ---
>  drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c
> index d53abc8ce670..f59fe1a9f4b9 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c
> @@ -263,13 +263,13 @@ static const struct dpu_format dpu_format_map[] = 
> {
> 
>  	INTERLEAVED_RGB_FMT(RGB565,
>  		0, COLOR_5BIT, COLOR_6BIT, COLOR_5BIT,
> -		C1_B_Cb, C0_G_Y, C2_R_Cr, 0, 3,
> +		C2_R_Cr, C0_G_Y, C1_B_Cb, 0, 3,
>  		false, 2, 0,
>  		DPU_FETCH_LINEAR, 1),
> 
>  	INTERLEAVED_RGB_FMT(BGR565,
>  		0, COLOR_5BIT, COLOR_6BIT, COLOR_5BIT,
> -		C2_R_Cr, C0_G_Y, C1_B_Cb, 0, 3,
> +		C1_B_Cb, C0_G_Y, C2_R_Cr, 0, 3,
>  		false, 2, 0,
>  		DPU_FETCH_LINEAR, 1),

-- 
Jeykumar S
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno