From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sakari Ailus Subject: Re: [RFC 0/9] OMAP3 ISP and camera drivers Date: Wed, 04 Mar 2009 17:38:25 +0200 Message-ID: <49AEA071.9020900@maxwell.research.nokia.com> References: <19F8576C6E063C45BE387C64729E73940427BCA193@dbde02.ent.ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from smtp.nokia.com ([192.100.105.134]:46085 "EHLO mgw-mx09.nokia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753388AbZCDPin (ORCPT ); Wed, 4 Mar 2009 10:38:43 -0500 In-Reply-To: <19F8576C6E063C45BE387C64729E73940427BCA193@dbde02.ent.ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Hiremath, Vaibhav" Cc: "linux-media@vger.kernel.org" , "linux-omap@vger.kernel.org" , "Aguirre Rodriguez, Sergio Alberto" , Toivonen Tuukka Olli Artturi , Hiroshi DOYU , DongSoo Kim Hiremath, Vaibhav wrote: > [Hiremath, Vaibhav] Sakari, Let me ask you basic question, have you > tested/verified these patch-sets? For the ISP and camera drivers, yes. That's actually the only thing that's contained in the patchset. > The reason I am asking this question is, for me it was not working. I > had to debug this and found that - > > - Changes missing in devices.c file, so isp_probe function will not > be called at all, keeping omap3isp = NULL. You will end up into > kernel crash in omap34xxcam_device_register. Anyway a crash shouldn't happen here. Could I see the kernel oops if there was such? > - The patches from Hiroshi DOYU doesn't build as is, you need to add > one include line #include in iovmmu.c (I am using > the patches submitted on 16th Jan 2009) Just pull the iommu branch, the Hiroshi's original patches are missing some hacks that you need to use them now. I'd expect Hiroshi to update the patchset when he comes back. > I have attached "git diff" output here with this mail for reference. Please pull also the "base" branch. -- Sakari Ailus sakari.ailus@maxwell.research.nokia.com