From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: Re: new style i2c client problem: probe () is not called Date: Tue, 13 May 2008 14:56:41 +0200 Message-ID: <20080513145641.14522f5c@hyperion.delvare> References: <20080513104921.GA4485@pengutronix.de> <20080513141337.20068339@hyperion.delvare> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: i2c-bounces-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org Errors-To: i2c-bounces-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org To: Darius Cc: i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org List-Id: linux-i2c@vger.kernel.org On Tue, 13 May 2008 15:45:29 +0300, Darius wrote: > > This code looks OK to me. But which driver handles i2c bus 0? Does your > > platform code create i2c bus 0? i2c-code will instantiate the ov7670 > > i2c device when i2c bus 0 is created by i2c_add_numbered_adapter(). So > > if you don't see it created, I suspect that i2c bus 0 itself is never > > registered. > > > > Thanks, there was the problem. I have modified i2c adapter driver to use > i2c_add_numbered_adapter() instead i2c_add_adapter() and pass to the > adapter->nr = pdev->id. now probe is called. > > Jean, what do you think about adding my i2c driver to mainline kernel? Sorry, which driver? > I've added this to Russel patch tracking system. Now it is a bit > ugraded. Should I make new patch and send it here? Sure, sending an updated patch can't hurt. -- Jean Delvare _______________________________________________ i2c mailing list i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org http://lists.lm-sensors.org/mailman/listinfo/i2c