[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <3a5b1be00611210734k79c81305q7b229139c2b17ef6@mail.gmail.com>
Date: Tue, 21 Nov 2006 17:34:54 +0200
From: "Komal Shah" <komal.shah802003@...il.com>
To: "James Simmons" <jsimmons@...radead.org>
Cc: Vladimir <vovan888@...il.com>, "Tony Lindgren" <tony@...mide.com>,
"Pavel Machek" <pavel@....cz>,
"kernel list" <linux-kernel@...r.kernel.org>,
"Linux Fbdev development list"
<linux-fbdev-devel@...ts.sourceforge.net>
Subject: Re: Siemens sx1: merge framebuffer support
On 11/21/06, James Simmons <jsimmons@...radead.org> wrote:
>
> Can you post the framebufer driver to the framebuffer list. We like to do
> peer review. Thank you :-)
>
> On Tue, 21 Nov 2006, Vladimir wrote:
>
> > 2006/11/20, Tony Lindgren <tony@...mide.com>:
> > > * Pavel Machek <pavel@....cz> [061118 18:16]:
> > > > From: Vladimir Ananiev <vovan888@...il.com>
> > > >
> > > > Framebuffer support for Siemens SX1; this is second big patch. (Third
> > > > one will be mixer/sound support). Support is simple / pretty minimal,
> > > > but seems to work okay (and is somehow important for a cell phone :-).
> > >
> > > Pushed to linux-omap. I guess you're planning to send the missing
> > > Kconfig + Makefile patch for this?
> > >
> > > Also, it would be better to use omap_mcbsp_xmit_word() or
> > > omap_mcsbsp_spi_master_xmit_word_poll() instead of OMAP_MCBSP_WRITE as
> > > it does not do any checking that it worked. The aic23 and tsc2101
> > > audio in linux-omap tree in general has the same problem.
> > >
> > > Regards,
> > >
> > > Tony
> > >
> >
> > Hmm. McBSP3 in SX1 is used in "GPIO mode". The only line used is CLKX,
> > so I think OMAP_MCBSP_WRITE would be enough. Am I wrong ?
> > -
Again, framebuffer support patch is based on the omap framebuffer
driver, which is not yet submitted to upstream/fbdevel list. sx1
framebuffer support just fill up the hooks required by -omap fb driver
framework.
--
---Komal Shah
http://komalshah.blogspot.com
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists