From mboxrd@z Thu Jan 1 00:00:00 1970 From: Damian Date: Wed, 19 May 2010 04:44:32 +0000 Subject: Re: [PATCH 1.5/4] sh: add a YUV422 output data format, that is also Message-Id: <4BF36CB0.1070107@igel.co.jp> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Guennadi Liakhovetski Cc: "linux-sh@vger.kernel.org" , Magnus Damm , linux-fbdev@vger.kernel.org, linux-omap@vger.kernel.org, Tomi Valkeinen (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 > --- > > 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 > 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 > > -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