From mboxrd@z Thu Jan 1 00:00:00 1970 From: Imre Deak Subject: Re: TSC2101/2102 Date: Sat, 31 Mar 2007 20:01:00 +0300 Message-ID: <20070331170100.GA2565@bitbox> References: <010f01c769fd$4ab24d30$c7a3580a@swcenter.sec.samsung.co.kr> <20070319103711.36ffe999.jarkko.nikula@nokia.com> <006e01c76a91$8ee234e0$c7a3580a@swcenter.sec.samsung.co.kr> <20070320132757.GB19913@atomide.com> <20070320155455.c1a3219b.jarkko.nikula@nokia.com> <20070320153812.GI19913@atomide.com> <20070322084830.1a91c99c.jarkko.nikula@nokia.com> <20070330184933.GB18375@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-omap-open-source-bounces@linux.omap.com Errors-To: linux-omap-open-source-bounces@linux.omap.com To: Eduardo Valentin Cc: Kyungmin Park , Linux OMAP ML List-Id: linux-omap@vger.kernel.org On Sat, Mar 31, 2007 at 08:52:17AM -0400, Eduardo Valentin wrote: > Hello Tony, > > > This is an updated patch to fix lcd for H2 board. This one is > applied against > current linux omap (commit ). > > The patch is a merge of what was discussed here and what is already > pushed. > > Signed-off-by: Eduardo Valentin > > BR, > > Eduardo Valentin > > [...] > > Index: linux-omap-2.6/arch/arm/mach-omap1/board-h2.c > =================================================================== > --- linux-omap-2.6.orig/arch/arm/mach-omap1/board-h2.c 2007-03-30 14:26:13.000000000 -0400 > +++ linux-omap-2.6/arch/arm/mach-omap1/board-h2.c 2007-03-30 14:36:53.000000000 -0400 > @@ -305,6 +305,19 @@ struct { > #define TSC2101_MUX_MCLK_ON R10_1610_MCLK_ON > #define TSC2101_MUX_MCLK_OFF R10_1610_MCLK_OFF > > +static void h2_lcd_dev_init(struct spi_device *tsc2101) > +{ > + /* The LCD is connected to the GPIO pins of the TSC2101, so > + * we have to tie them here. We can also register the LCD driver > + * first only here, where we know that the TSC driver is ready. > + */ > + > + device_initialize(&h2_lcd_device.dev); > + > + dev_set_drvdata(&h2_lcd_device.dev, tsc2101); Is this really necessary? Why isn't it ok to pass it through the platform_data? --Imre