From: Damian <dhobsong@igel.co.jp>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: "linux-sh@vger.kernel.org" <linux-sh@vger.kernel.org>,
Magnus Damm <magnus.damm@gmail.com>,
linux-fbdev@vger.kernel.org, linux-omap@vger.kernel.org,
Tomi Valkeinen <tomi.valkeinen@nokia.com>
Subject: Re: [PATCH 1.5/4] sh: add a YUV422 output data format, that is also
Date: Wed, 19 May 2010 04:44:32 +0000 [thread overview]
Message-ID: <4BF36CB0.1070107@igel.co.jp> (raw)
In-Reply-To: <Pine.LNX.4.64.1005071246500.4777@axis700.grange>
(4:59), Guennadi Liakhovetski wrote:
> The LCDC block is allowed to use one of the two output data formats, when used
> with MIPI DSI: RGB24 and YUV422. YUV422 is not currently handled by the LCDC
> driver, but we have to add a define for it for MIPI. Fix indentation while
> at it.
>
> Signed-off-by: Guennadi Liakhovetski<g.liakhovetski@gmx.de>
> ---
>
> Hm, this one is actually worse, it actually has to go in before patch 2/4.
> Sorry again.
>
Console framebuffer tested on sh-2.6 tree and sh/dmaengine branch with
the necessary clock and intc patches applied.
Boot logo and framebuffer console displayed with no problems or unusual
effects.
Tested-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
> include/video/sh_mobile_lcdc.h | 39 +++++++++++++++++++++------------------
> 1 files changed, 21 insertions(+), 18 deletions(-)
>
> diff --git a/include/video/sh_mobile_lcdc.h b/include/video/sh_mobile_lcdc.h
> index 2cc893f..5eaea78 100644
> --- a/include/video/sh_mobile_lcdc.h
> +++ b/include/video/sh_mobile_lcdc.h
> @@ -3,24 +3,27 @@
>
> #include<linux/fb.h>
>
> -enum { RGB8, /* 24bpp, 8:8:8 */
> - RGB9, /* 18bpp, 9:9 */
> - RGB12A, /* 24bpp, 12:12 */
> - RGB12B, /* 12bpp */
> - RGB16, /* 16bpp */
> - RGB18, /* 18bpp */
> - RGB24, /* 24bpp */
> - SYS8A, /* 24bpp, 8:8:8 */
> - SYS8B, /* 18bpp, 8:8:2 */
> - SYS8C, /* 18bpp, 2:8:8 */
> - SYS8D, /* 16bpp, 8:8 */
> - SYS9, /* 18bpp, 9:9 */
> - SYS12, /* 24bpp, 12:12 */
> - SYS16A, /* 16bpp */
> - SYS16B, /* 18bpp, 16:2 */
> - SYS16C, /* 18bpp, 2:16 */
> - SYS18, /* 18bpp */
> - SYS24 };/* 24bpp */
> +enum {
> + RGB8, /* 24bpp, 8:8:8 */
> + RGB9, /* 18bpp, 9:9 */
> + RGB12A, /* 24bpp, 12:12 */
> + RGB12B, /* 12bpp */
> + RGB16, /* 16bpp */
> + RGB18, /* 18bpp */
> + RGB24, /* 24bpp */
> + YUV422, /* 16bpp */
> + SYS8A, /* 24bpp, 8:8:8 */
> + SYS8B, /* 18bpp, 8:8:2 */
> + SYS8C, /* 18bpp, 2:8:8 */
> + SYS8D, /* 16bpp, 8:8 */
> + SYS9, /* 18bpp, 9:9 */
> + SYS12, /* 24bpp, 12:12 */
> + SYS16A, /* 16bpp */
> + SYS16B, /* 18bpp, 16:2 */
> + SYS16C, /* 18bpp, 2:16 */
> + SYS18, /* 18bpp */
> + SYS24, /* 24bpp */
> +};
>
> enum { LCDC_CHAN_DISABLED = 0,
> LCDC_CHAN_MAINLCD,
--
Damian Hobson-Garcia
IGEL Co.,Ltd
http://www.igel.co.jp
WARNING: multiple messages have this Message-ID (diff)
From: Damian <dhobsong@igel.co.jp>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: "linux-sh@vger.kernel.org" <linux-sh@vger.kernel.org>,
Magnus Damm <magnus.damm@gmail.com>,
linux-fbdev@vger.kernel.org, linux-omap@vger.kernel.org,
Tomi Valkeinen <tomi.valkeinen@nokia.com>
Subject: Re: [PATCH 1.5/4] sh: add a YUV422 output data format, that is also supported by LCDC
Date: Wed, 19 May 2010 13:44:32 +0900 [thread overview]
Message-ID: <4BF36CB0.1070107@igel.co.jp> (raw)
In-Reply-To: <Pine.LNX.4.64.1005071246500.4777@axis700.grange>
(4:59), Guennadi Liakhovetski wrote:
> The LCDC block is allowed to use one of the two output data formats, when used
> with MIPI DSI: RGB24 and YUV422. YUV422 is not currently handled by the LCDC
> driver, but we have to add a define for it for MIPI. Fix indentation while
> at it.
>
> Signed-off-by: Guennadi Liakhovetski<g.liakhovetski@gmx.de>
> ---
>
> Hm, this one is actually worse, it actually has to go in before patch 2/4.
> Sorry again.
>
Console framebuffer tested on sh-2.6 tree and sh/dmaengine branch with
the necessary clock and intc patches applied.
Boot logo and framebuffer console displayed with no problems or unusual
effects.
Tested-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
> include/video/sh_mobile_lcdc.h | 39 +++++++++++++++++++++------------------
> 1 files changed, 21 insertions(+), 18 deletions(-)
>
> diff --git a/include/video/sh_mobile_lcdc.h b/include/video/sh_mobile_lcdc.h
> index 2cc893f..5eaea78 100644
> --- a/include/video/sh_mobile_lcdc.h
> +++ b/include/video/sh_mobile_lcdc.h
> @@ -3,24 +3,27 @@
>
> #include<linux/fb.h>
>
> -enum { RGB8, /* 24bpp, 8:8:8 */
> - RGB9, /* 18bpp, 9:9 */
> - RGB12A, /* 24bpp, 12:12 */
> - RGB12B, /* 12bpp */
> - RGB16, /* 16bpp */
> - RGB18, /* 18bpp */
> - RGB24, /* 24bpp */
> - SYS8A, /* 24bpp, 8:8:8 */
> - SYS8B, /* 18bpp, 8:8:2 */
> - SYS8C, /* 18bpp, 2:8:8 */
> - SYS8D, /* 16bpp, 8:8 */
> - SYS9, /* 18bpp, 9:9 */
> - SYS12, /* 24bpp, 12:12 */
> - SYS16A, /* 16bpp */
> - SYS16B, /* 18bpp, 16:2 */
> - SYS16C, /* 18bpp, 2:16 */
> - SYS18, /* 18bpp */
> - SYS24 };/* 24bpp */
> +enum {
> + RGB8, /* 24bpp, 8:8:8 */
> + RGB9, /* 18bpp, 9:9 */
> + RGB12A, /* 24bpp, 12:12 */
> + RGB12B, /* 12bpp */
> + RGB16, /* 16bpp */
> + RGB18, /* 18bpp */
> + RGB24, /* 24bpp */
> + YUV422, /* 16bpp */
> + SYS8A, /* 24bpp, 8:8:8 */
> + SYS8B, /* 18bpp, 8:8:2 */
> + SYS8C, /* 18bpp, 2:8:8 */
> + SYS8D, /* 16bpp, 8:8 */
> + SYS9, /* 18bpp, 9:9 */
> + SYS12, /* 24bpp, 12:12 */
> + SYS16A, /* 16bpp */
> + SYS16B, /* 18bpp, 16:2 */
> + SYS16C, /* 18bpp, 2:16 */
> + SYS18, /* 18bpp */
> + SYS24, /* 24bpp */
> +};
>
> enum { LCDC_CHAN_DISABLED = 0,
> LCDC_CHAN_MAINLCD,
--
Damian Hobson-Garcia
IGEL Co.,Ltd
http://www.igel.co.jp
next prev parent reply other threads:[~2010-05-19 4:44 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-07 9:07 [PATCH 0/4] MIPI DSI support for SH-mobile, common header, switch Guennadi Liakhovetski
2010-05-07 9:07 ` [PATCH 0/4] MIPI DSI support for SH-mobile, common header, switch OMAP to use it Guennadi Liakhovetski
2010-05-07 9:07 ` [PATCH 1/4] fbdev: add a MIPI DSI header Guennadi Liakhovetski
2010-05-07 9:07 ` Guennadi Liakhovetski
2010-05-19 4:42 ` Damian
2010-05-19 4:42 ` Damian
2010-05-19 7:49 ` Tomi Valkeinen
2010-05-19 7:49 ` Tomi Valkeinen
2010-05-19 8:08 ` Guennadi Liakhovetski
2010-05-19 8:08 ` Guennadi Liakhovetski
2010-05-19 8:21 ` Tomi Valkeinen
2010-05-19 8:21 ` Tomi Valkeinen
2010-05-19 14:27 ` Ville Syrjälä
2010-05-19 14:27 ` Ville Syrjälä
2010-05-19 15:00 ` Paul Mundt
2010-05-19 15:00 ` Paul Mundt
2010-05-19 15:39 ` Ville Syrjälä
2010-05-19 15:39 ` Ville Syrjälä
2010-05-20 8:07 ` Guennadi Liakhovetski
2010-05-20 8:07 ` Guennadi Liakhovetski
2010-05-20 8:32 ` Tomi Valkeinen
2010-05-20 8:32 ` Tomi Valkeinen
2010-05-20 8:54 ` Felipe Balbi
2010-05-20 8:54 ` Felipe Balbi
2010-05-20 9:02 ` Guennadi Liakhovetski
2010-05-20 9:02 ` Guennadi Liakhovetski
2010-05-20 11:03 ` Guennadi Liakhovetski
2010-05-20 11:03 ` Guennadi Liakhovetski
2010-05-20 14:20 ` Tomi Valkeinen
2010-05-20 14:20 ` Tomi Valkeinen
2010-05-07 9:07 ` [PATCH 2/4] sh-mobile: add support for displays, connected over the Guennadi Liakhovetski
2010-05-07 9:07 ` [PATCH 2/4] sh-mobile: add support for displays, connected over the MIPI bus Guennadi Liakhovetski
2010-05-19 4:43 ` [PATCH 2/4] sh-mobile: add support for displays, connected over Damian
2010-05-19 4:43 ` [PATCH 2/4] sh-mobile: add support for displays, connected over the MIPI bus Damian
2010-05-07 9:07 ` [PATCH 3/4] ARM: add framebuffer support for ap4evb Guennadi Liakhovetski
2010-05-07 9:07 ` Guennadi Liakhovetski
2010-05-07 14:45 ` [PATCH 3/4 v2] " Guennadi Liakhovetski
2010-05-07 14:45 ` Guennadi Liakhovetski
2010-05-10 0:11 ` Kuninori Morimoto
2010-05-10 0:11 ` Kuninori Morimoto
2010-05-10 6:12 ` Guennadi Liakhovetski
2010-05-10 6:12 ` Guennadi Liakhovetski
2010-05-10 9:37 ` Kuninori Morimoto
2010-05-10 9:37 ` Kuninori Morimoto
2010-05-19 4:36 ` Damian
2010-05-19 4:36 ` Damian
2010-05-19 4:47 ` [PATCH 3/4] " Damian
2010-05-19 4:47 ` Damian
2010-05-07 9:07 ` [PATCH 4/4] video: switch OMAP LCD MIPI driver to use the common Guennadi Liakhovetski
2010-05-07 9:07 ` [PATCH 4/4] video: switch OMAP LCD MIPI driver to use the common MIPI DSI header Guennadi Liakhovetski
2010-05-07 10:05 ` [PATCH 2.5/4] ARM: add LCDC and MIPI DSI-Tx clock definitions to Guennadi Liakhovetski
2010-05-07 10:05 ` [PATCH 2.5/4] ARM: add LCDC and MIPI DSI-Tx clock definitions to sh7372 Guennadi Liakhovetski
2010-05-07 10:49 ` [PATCH 1.5/4] sh: add a YUV422 output data format, that is also Guennadi Liakhovetski
2010-05-07 10:49 ` [PATCH 1.5/4] sh: add a YUV422 output data format, that is also supported by LCDC Guennadi Liakhovetski
2010-05-19 4:44 ` Damian [this message]
2010-05-19 4:44 ` Damian
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4BF36CB0.1070107@igel.co.jp \
--to=dhobsong@igel.co.jp \
--cc=g.liakhovetski@gmx.de \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-sh@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=tomi.valkeinen@nokia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.