Hi Tomi,
-----Original Message-----
From: Tomi Valkeinen [mailto:tomi.valkei...@nokia.com]
Sent: Tuesday, August 17, 2010 4:31 PM
To: K, Mythri P
Cc: linux-omap@vger.kernel.org
Subject: RE: [PATCH] OMAP:DSS:Add support for Additional color modes supported
by OMAP4
Hi,
On Wed, 2010-08-11 at 11:22 +0200, ext K, Mythri P wrote:
> Hi Tomi,
> Can you please comment on the below patch . This is to add new color modes
> supported by OMAP4.
>
> Thanks and regards,
> Mythri.
> -----Original Message-----
> From: K, Mythri P
> Sent: Thursday, August 05, 2010 11:24 AM
> To: linux-omap@vger.kernel.org
> Cc: tomi.valkei...@nokia.com; Semwal, Sumit; K, Mythri P
> Subject: [PATCH] OMAP:DSS:Add support for Additional color modes supported by
> OMAP4
>
> From: Sumit semwal <sumit.sem...@ti.com>
>
> This patch adds support for new color modes that are supported by the
> video/graphics pipeline of OMAP4
>
> Signed-off-by: Mythri P K <mythr...@ti.com>
> ---
> arch/arm/plat-omap/include/plat/display.h | 16 ++++++++-
> drivers/video/omap2/dss/dispc.c | 53
> ++++++++++++++++++++++-------
> drivers/video/omap2/dss/overlay.c | 6 ++-
> 3 files changed, 59 insertions(+), 16 deletions(-)
>
> diff --git a/arch/arm/plat-omap/include/plat/display.h
> b/arch/arm/plat-omap/include/plat/display.h
> index 7a6eedd..ebf1020 100644
> --- a/arch/arm/plat-omap/include/plat/display.h
> +++ b/arch/arm/plat-omap/include/plat/display.h
> @@ -89,6 +89,12 @@ enum omap_color_mode {
> OMAP_DSS_COLOR_ARGB32 = 1 << 11, /* ARGB32 */
> OMAP_DSS_COLOR_RGBA32 = 1 << 12, /* RGBA32 */
> OMAP_DSS_COLOR_RGBX32 = 1 << 13, /* RGBx32 */
> + OMAP_DSS_COLOR_NV12 = 1 << 14, /* NV12 format: YUV 4:2:0 */
> + OMAP_DSS_COLOR_RGBA12 = 1 << 15, /* RGBA12 - 4444 */
Is this the same as ARGB16? Or is it the same, except alpha value is in
the other end? If so, I think the naming should be coherent, and either
this should be RGBA16, or ARGB16 should be ARGB12.
Then again, if TRM says this is RGBA12, and the other one is ARGB16, I
guess we should stick with TRM naming...
[Mythri] yes these are the TRM naming conventions. There are 2 formats in
ARGB16 , ARGB16-1555 and ARGB16-4444.
> + OMAP_DSS_COLOR_XRGB12 = 1 << 16, /* xRGB12, 16-bit container */
Is this RGB12U, or again otherwise same but the empty value is on the
other end?
[Mythri] again RGB12x-4444 and xRGB12-4444 are two different formats with empty
value or alignment otherwise.
> + OMAP_DSS_COLOR_ARGB16_1555 = 1 << 17, /* ARGB16-1555 */
> + OMAP_DSS_COLOR_RGBX24_32_ALGN = 1 << 18, /* 32-msb aligned 24bit */
Hmm, what is this?
[Mythri] This format is RGBx24-8888 (24-bit RGB aligned on MSB of the 32-bit
container). TRM DISPC- Table 10-199. DISPC_VID1_ATTRIBUTES , formats.
> + OMAP_DSS_COLOR_XRGB15 = 1 << 19, /* xRGB15: 1555*/
>
> OMAP_DSS_COLOR_GFX_OMAP2 =
> OMAP_DSS_COLOR_CLUT1 | OMAP_DSS_COLOR_CLUT2 |
> @@ -112,9 +118,17 @@ enum omap_color_mode {
> OMAP_DSS_COLOR_VID1_OMAP3 =
> OMAP_DSS_COLOR_RGB12U | OMAP_DSS_COLOR_RGB16 |
> OMAP_DSS_COLOR_RGB24U | OMAP_DSS_COLOR_RGB24P |
> - OMAP_DSS_COLOR_YUV2 | OMAP_DSS_COLOR_UYVY,
> + OMAP_DSS_COLOR_YUV2 | OMAP_DSS_COLOR_UYVY |
> + OMAP_DSS_COLOR_NV12 | OMAP_DSS_COLOR_RGBA12 |
> + OMAP_DSS_COLOR_XRGB12 | OMAP_DSS_COLOR_ARGB16_1555 |
> + OMAP_DSS_COLOR_RGBX24_32_ALGN | OMAP_DSS_COLOR_XRGB15 |
> + OMAP_DSS_COLOR_ARGB32 | OMAP_DSS_COLOR_RGBA32 |
> + OMAP_DSS_COLOR_RGBX32,
Why do you change OMAP3 modes, if this patch is about supported OMAP4
modes?
[Mythri] When the additional color formats supported for OMAP4 are added in the
OMAP3 modes, There is a check in the dispc color-modes for OMAP4 alone to
support , OMAP3 would return an error as before.
Do you suggest add a new OMAP_DSS_COLOR_VID1_OMAP4 instead?
Tomi
Thanks and regards,
Mythri.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
