> -----Original Message-----
> From: Taneja, Archit
> Sent: Friday, July 23, 2010 11:26 AM
> To: Hiremath, Vaibhav; tomi.valkei...@nokia.com
> Cc: linux-omap@vger.kernel.org; Janorkar, Mayuresh; Mittal, Mukund
> Subject: RE: [PATCH 5/5] OMAP: DSS2: Context Save and Restore of Video3
> pipeline registers
> 
> 
> > > -----Original Message-----
> > > From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-
> > > ow...@vger.kernel.org] On Behalf Of Taneja, Archit
> > > Sent: Monday, July 19, 2010 5:26 PM
> > > To: tomi.valkei...@nokia.com
> > > Cc: linux-omap@vger.kernel.org; Janorkar, Mayuresh; Mittal, Mukund;
> > > Taneja, Archit
> > > Subject: [PATCH 5/5] OMAP: DSS2: Context Save and Restore of Video3
> > > pipeline registers
> > >
> > > From: Mayuresh Janorkar <ma...@ti.com>
> > >
> > > Context Save and Restore of Video3 pipeline registers.
> > [Hiremath, Vaibhav] I still fill, we are un-necessary
> > dividing the patch here. This patch can very well be part of
> > another patch,
> >
> > [PATCH 2/5] OMAP: DSS2: Add Video 3 pipeline functionality in DISPC
> >
> > In-fact, for me this should be consolidated single patch,
> > something like,
> >
> > OMAP: DSS2: Add Video-3 pipeline functionality
> >
> >
> > I don't see any reason why we are fragmenting this into
> > multiple patches.
> >
> > Thanks,
> > Vaibhav
> >
> > >
> [Archit] Okay, Does the following split sound good?:
> -OMAP: DSS2: [1/2]Add Video-3 pipeline functionality in dispc
>       - This patch will have all dispc related changes
> -OMAP: DSS2: [2/2]Add Video-3 pipeline overlay object
>       - This patch will add the new overlay and changes
>       required in manager, irqs etc
> 
> Or does a single patch make more sense?
> 
[Hiremath, Vaibhav] I still feel single patch makes more sense here. 

Thanks,
Vaibhav

> Thanks,
> 
> Archit
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to