From: Sakari Ailus <sakari.ailus@iki.fi>
To: "Hadli, Manjunath" <manjunath.hadli@ti.com>
Cc: "'Laurent Pinchart'" <laurent.pinchart@ideasonboard.com>,
LMML <linux-media@vger.kernel.org>,
dlos <davinci-linux-open-source@linux.davincidsp.com>
Subject: Re: [ RFC PATCH 0/8] RFC for Media Controller capture driver for DM365
Date: Mon, 04 Jul 2011 19:13:09 +0300 [thread overview]
Message-ID: <4E11E695.9090508@iki.fi> (raw)
In-Reply-To: <B85A65D85D7EB246BE421B3FB0FBB593024BCEF73A@dbde02.ent.ti.com>
Hadli, Manjunath wrote:
> Thank you Laurent.
Hi Manjunath,
> On Mon, Jul 04, 2011 at 18:52:37, Laurent Pinchart wrote:
>> Hi Manjunath,
>>
>> On Monday 04 July 2011 07:58:06 Hadli, Manjunath wrote:
>>> On Thu, Jun 30, 2011 at 19:27:36, Sakari Ailus wrote:
>>
>> [snip]
>>
>>>> I understand that not all the blocks are there. Are there any
>>>> major functional differences between those in Davinci and those
>>>> in OMAP 3? Could the OMAP 3 ISP driver made support Davinci ISP
>>>> as well?
>>>
>>> Yes, there are a lot of major differences between OMAP3 and
>>> Dm365/Dm355, both in terms of features, there IP, and the
>>> software interface, including all the registers which are
>>> entirely different. The closest omap3 would come to is only to
>>> DM6446. I do not think OMAP3 driver can be made to support Dm355
>>> and Dm365. It is good to keep the OMAP3 neat and clean to cater
>>> for OMAP4 and beyond, and keep the Davinci family separate. The
>>> names might look similar and hence confusing for you, but the
>>> names can as well be made the same as Dm365 blocks like ISIF and
>>> IPIPE and IPIPEIF which are different.
>>
>> The DM6446 ISP is very similar to the OMAP3 ISP, and thus quite
>> different from the DM355/365 ISPs. Should the DM6446 be supported
>> by the OMAP3 ISP driver, and the DM355/365 by this driver ?
>
> DM6446 capture IP is in some respects similar to OMAP3 for some
> features, but there are a large number of differences also (MMU,
> VRFB, a lot of display interfaces etc). Having a single driver
> catering to Since DM6446 and OMAP3 is going to be unwieldy. Also,
> DM6446 belongs to the Davinci family of chips, it should be clubbed
> with the other Davinci SoCs as it will simplify a lot of other things
> including directory subdirectory/file naming, organization of
> machine/platform code etc among other things. Other than Video a lot
> of other system registers and features which are common with the rest
> of Davinci SoCs which if treated together is a good thing, whereas
> OMAP3 can be modified and developed with those on the OMAP family
> (OMAP4 for ex).
Thanks for the clarifications.
What about the DM3730? As far as I understand, the ISP on that one is
supported by the OMAP 3 ISP driver. But it looks like that it's more
continuation for the OMAP family of the chips than the Davinci.
I glanced at the DM6446 documentation and at the register level the
interface looks somewhat different although some register names are the
same. I didn't found a proper TRM which would be as detailed as the OMAP
ones --- does TI have one available in public?
OMAP 4 has a quite different ISS --- which the ISP is a part of, and
which also is very different to the OMAP 3 one --- so it's unlikely
that the same driver would support OMAP 3 and OMAP 4 ISPs.
Kind regards,
--
Sakari Ailus
sakari.ailus@iki.fi
next prev parent reply other threads:[~2011-07-04 16:13 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-30 13:13 [ RFC PATCH 0/8] RFC for Media Controller capture driver for DM365 Manjunath Hadli
2011-06-30 13:13 ` [RFC PATCH 1/8] davinci: vpfe: add dm3xx IPIPEIF hardware support module Manjunath Hadli
2011-07-13 18:50 ` Sakari Ailus
2011-07-19 10:51 ` Hadli, Manjunath
2011-07-19 19:23 ` Sakari Ailus
2011-08-29 15:19 ` Hadli, Manjunath
2011-08-31 11:23 ` 'Sakari Ailus'
2011-10-23 18:30 ` new mbus formats Hadli, Manjunath
2011-11-02 14:20 ` Laurent Pinchart
2011-06-30 13:13 ` [RFC PATCH 2/8] davinci: vpfe: add IPIPE hardware layer support Manjunath Hadli
2011-08-07 23:03 ` Sakari Ailus
2011-08-08 16:25 ` Hadli, Manjunath
2011-06-30 13:13 ` [RFC PATCH 3/8] davinci: vpfe: add IPIPE support for media controller driver Manjunath Hadli
2011-06-30 13:13 ` [RFC PATCH 4/8] davinci: vpfe: add support for CCDC hardware for dm365 Manjunath Hadli
2011-06-30 13:13 ` [RFC PATCH 5/8] davinci: vpfe: add ccdc driver with media controller interface Manjunath Hadli
2011-06-30 13:13 ` [RFC PATCH 6/8] davinci: vpfe: add v4l2 video driver support Manjunath Hadli
2011-06-30 13:13 ` [RFC PATCH 7/8] davinci: vpfe: v4l2 capture driver with media interface Manjunath Hadli
2011-06-30 13:13 ` [RFC PATCH 8/8] davinci: vpfe: build infrastructure for dm365 Manjunath Hadli
2011-06-30 13:57 ` [ RFC PATCH 0/8] RFC for Media Controller capture driver for DM365 Sakari Ailus
2011-07-04 5:58 ` Hadli, Manjunath
2011-07-04 13:22 ` Laurent Pinchart
2011-07-04 14:55 ` Hadli, Manjunath
2011-07-04 16:13 ` Sakari Ailus [this message]
2011-07-06 5:40 ` Hadli, Manjunath
2011-07-12 12:01 ` Hadli, Manjunath
2011-07-12 21:22 ` 'Sakari Ailus'
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=4E11E695.9090508@iki.fi \
--to=sakari.ailus@iki.fi \
--cc=davinci-linux-open-source@linux.davincidsp.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=manjunath.hadli@ti.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