Re-adding intel-gfx, this seems generally useful. On Thu, Jan 30, 2014 at 6:22 PM, Volkin, Bradley D <[email protected]> wrote: > Ok. There's still a couple ways I could see doing it. > - one big patch with all the new #defines, up front > - split the #defines from their current patches, all up front > - split the #defines from their current patches, just before the patch that > uses them > > I guess I would tend towards the 3rd option, but maybe that's overkill. Which > do you suggest?
Whatever you like better, and tbh I'd wait for review and only split things if you need to. In essence this was just a suggestion for the future for smoother merging. For the patches themselves the riskiest thing from my maintainer pov is breaking old userspace. So I'd like to get the command streamer merged as soon as possible, and enabled in enforcing mode. We can then plug in all the missing pieces while getting test coverage in the wild (lots of people tend to use -nightly). I've sigend up Jani to review the current series in detail, including the tests. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
