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