From: Sakari Ailus <sakari.ailus@maxwell.research.nokia.com>
To: "Hiremath, Vaibhav" <hvaibhav@ti.com>
Cc: "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
"Aguirre Rodriguez, Sergio Alberto" <saaguirre@ti.com>,
Toivonen Tuukka Olli Artturi <tuukka.o.toivonen@nokia.com>,
Hiroshi DOYU <Hiroshi.DOYU@nokia.com>,
DongSoo Kim <dongsoo.kim@gmail.com>
Subject: Re: [RFC 0/9] OMAP3 ISP and camera drivers
Date: Wed, 04 Mar 2009 17:38:25 +0200 [thread overview]
Message-ID: <49AEA071.9020900@maxwell.research.nokia.com> (raw)
In-Reply-To: <19F8576C6E063C45BE387C64729E73940427BCA193@dbde02.ent.ti.com>
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 <linux/hardirq.h> 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
next prev parent reply other threads:[~2009-03-04 15:38 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-03-03 10:06 [RFC 0/9] OMAP3 ISP and camera drivers Sakari Ailus
2009-03-03 10:06 ` [PATCH 1/9] omap3isp: Add ISP main driver and register definitions Sakari Ailus
2009-03-03 10:06 ` [PATCH 2/9] omap3isp: Add ISP MMU wrapper Sakari Ailus
2009-03-03 10:06 ` [PATCH 3/9] omap3isp: Add userspace header Sakari Ailus
2009-03-03 10:06 ` [PATCH 4/9] omap3isp: Add ISP frontend (CCDC) Sakari Ailus
2009-03-03 10:06 ` [PATCH 5/9] omap3isp: Add ISP backend (PRV and RSZ) Sakari Ailus
2009-03-03 10:06 ` [PATCH 6/9] omap3isp: Add statistics collection modules (H3A and HIST) Sakari Ailus
2009-03-03 10:06 ` [PATCH 7/9] omap3isp: Add CSI2 interface support Sakari Ailus
2009-03-03 10:06 ` [PATCH 8/9] omap3isp: Add ISP tables Sakari Ailus
2009-03-03 10:06 ` [PATCH 9/9] omap34xxcam: Add camera driver Sakari Ailus
2009-03-03 17:31 ` Alexey Klimov
2009-03-05 14:09 ` Sakari Ailus
2009-03-07 14:38 ` Alexey Klimov
2009-03-03 12:23 ` [PATCH 5/9] omap3isp: Add ISP backend (PRV and RSZ) Alexey Klimov
2009-03-03 12:07 ` [PATCH 4/9] omap3isp: Add ISP frontend (CCDC) Alexey Klimov
2009-03-06 12:51 ` [PATCH 3/9] omap3isp: Add userspace header Hans Verkuil
2009-03-03 11:56 ` [PATCH 1/9] omap3isp: Add ISP main driver and register definitions Alexey Klimov
2009-03-05 11:34 ` Sakari Ailus
2009-03-07 15:25 ` Alexey Klimov
2009-03-03 13:09 ` [RFC 0/9] OMAP3 ISP and camera drivers Sakari Ailus
2009-03-04 15:28 ` Hiremath, Vaibhav
2009-03-04 15:38 ` Sakari Ailus [this message]
2009-03-05 0:10 ` DongSoo(Nathaniel) Kim
2009-03-05 3:53 ` Hiremath, Vaibhav
2009-03-05 4:54 ` DongSoo(Nathaniel) Kim
2009-03-05 7:30 ` Sakari Ailus
2009-03-05 14:26 ` [PATCH] omap34xxcam: Don't use dev_err before we have a video device Sakari Ailus
2009-03-05 5:00 ` [RFC 0/9] OMAP3 ISP and camera drivers DongSoo(Nathaniel) Kim
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=49AEA071.9020900@maxwell.research.nokia.com \
--to=sakari.ailus@maxwell.research.nokia.com \
--cc=Hiroshi.DOYU@nokia.com \
--cc=dongsoo.kim@gmail.com \
--cc=hvaibhav@ti.com \
--cc=linux-media@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=saaguirre@ti.com \
--cc=tuukka.o.toivonen@nokia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox