All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Hadli, Manjunath" <manjunath.hadli@ti.com>
To: "'Laurent Pinchart'" <laurent.pinchart@ideasonboard.com>
Cc: "'Sakari Ailus'" <sakari.ailus@iki.fi>,
	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, 4 Jul 2011 20:25:34 +0530	[thread overview]
Message-ID: <B85A65D85D7EB246BE421B3FB0FBB593024BCEF73A@dbde02.ent.ti.com> (raw)
In-Reply-To: <201107041522.37437.laurent.pinchart@ideasonboard.com>

Thank you Laurent.

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).

> 
> > > There are number of possible improvements that still should be made 
> > > to the OMAP 3 ISP driver so this way both of the driver would easily 
> > > get them. To mention some:
> > > 
> > > - Multiple output pipeline
> > > - Switch to videobuf2
> > > - Switch to GENIRQ
> > 
> > Sure. There is definitely a design element convergence, and overtime I 
> > think some of these would get into the core v4l2 infrastructure.
> 
> --
> Regards,
> 
> Laurent Pinchart
> 


  reply	other threads:[~2011-07-04 14:55 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 [this message]
2011-07-04 16:13         ` Sakari Ailus
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=B85A65D85D7EB246BE421B3FB0FBB593024BCEF73A@dbde02.ent.ti.com \
    --to=manjunath.hadli@ti.com \
    --cc=davinci-linux-open-source@linux.davincidsp.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=sakari.ailus@iki.fi \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.