[v3] drm/msm/dpu: Change definition of RGB565 and BGR565

Submitted by Tanmay Shah on Dec. 17, 2018, 11:34 p.m.

Details

Message ID 20181217233409.22955-1-tanmay@codeaurora.org
State New
Headers show
Series "drm/msm/dpu: Change definition of RGB565 and BGR565" ( rev: 1 ) in Freedreno

Not browsing as part of any series.

Commit Message

Tanmay Shah Dec. 17, 2018, 11:34 p.m.
Correct definition of both formats by swapping red
and blue channels

v3: update commit message

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(-)

Patch hide | download patch | download mbox

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),
 

Comments

On Mon, Dec 17, 2018 at 03:34:09PM -0800, Tanmay Shah wrote:
> Correct definition of both formats by swapping red
> and blue channels
> 
> v3: update commit message
> 
> Signed-off-by: Tanmay Shah <tanmay@codeaurora.org>

Applied to dpu-staging, thanks!

Sean
> ---
>  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