2017-06-28 09:31:56

by Laurentiu Palcu

[permalink] [raw]
Subject: [PATCH] drm/imx: fix typo in ipu_plane_formats[]

The BGRA8888 appears twice in the ipu_plane_formats[] list. The
duplicate should be BGRX8888.

The original commit is:

commit 59d6b7189a96 ("drm/imx: ipuv3-plane: enable support for RGBX8888
and RGBA8888 pixel formats")

Signed-off-by: Laurentiu Palcu <[email protected]>
---
drivers/gpu/drm/imx/ipuv3-plane.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c
index d63e853..7b1bb0f 100644
--- a/drivers/gpu/drm/imx/ipuv3-plane.c
+++ b/drivers/gpu/drm/imx/ipuv3-plane.c
@@ -54,7 +54,7 @@ static const uint32_t ipu_plane_formats[] = {
DRM_FORMAT_RGBA8888,
DRM_FORMAT_RGBX8888,
DRM_FORMAT_BGRA8888,
- DRM_FORMAT_BGRA8888,
+ DRM_FORMAT_BGRX8888,
DRM_FORMAT_UYVY,
DRM_FORMAT_VYUY,
DRM_FORMAT_YUYV,
--
2.7.4


2017-06-28 09:43:11

by Philipp Zabel

[permalink] [raw]
Subject: Re: [PATCH] drm/imx: fix typo in ipu_plane_formats[]

On Wed, 2017-06-28 at 12:31 +0300, Laurentiu Palcu wrote:
> The BGRA8888 appears twice in the ipu_plane_formats[] list. The
> duplicate should be BGRX8888.
>
> The original commit is:
>
> commit 59d6b7189a96 ("drm/imx: ipuv3-plane: enable support for RGBX8888
> and RGBA8888 pixel formats")
>
> Signed-off-by: Laurentiu Palcu <[email protected]>
> ---
> drivers/gpu/drm/imx/ipuv3-plane.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c
> index d63e853..7b1bb0f 100644
> --- a/drivers/gpu/drm/imx/ipuv3-plane.c
> +++ b/drivers/gpu/drm/imx/ipuv3-plane.c
> @@ -54,7 +54,7 @@ static const uint32_t ipu_plane_formats[] = {
> DRM_FORMAT_RGBA8888,
> DRM_FORMAT_RGBX8888,
> DRM_FORMAT_BGRA8888,
> - DRM_FORMAT_BGRA8888,
> + DRM_FORMAT_BGRX8888,
> DRM_FORMAT_UYVY,
> DRM_FORMAT_VYUY,
> DRM_FORMAT_YUYV,

Thank you, applied to the imx-drm/fixes branch.

regards
Philipp