> -----Original Message----- > From: Taneja, Archit > Sent: Friday, July 23, 2010 11:26 AM > To: Hiremath, Vaibhav; [email protected] > Cc: [email protected]; Janorkar, Mayuresh; Mittal, Mukund > Subject: RE: [PATCH 5/5] OMAP: DSS2: Context Save and Restore of Video3 > pipeline registers > > > > > -----Original Message----- > > > From: [email protected] [mailto:linux-omap- > > > [email protected]] On Behalf Of Taneja, Archit > > > Sent: Monday, July 19, 2010 5:26 PM > > > To: [email protected] > > > Cc: [email protected]; Janorkar, Mayuresh; Mittal, Mukund; > > > Taneja, Archit > > > Subject: [PATCH 5/5] OMAP: DSS2: Context Save and Restore of Video3 > > > pipeline registers > > > > > > From: Mayuresh Janorkar <[email protected]> > > > > > > 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
