public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
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

  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