[Intel-gfx] [PATCH 07/13] drm/i915: Add register whitelist for DRM master

Daniel Vetter daniel at ffwll.ch
Thu Jan 30 21:41:38 CET 2014


Re-adding intel-gfx, this seems generally useful.

On Thu, Jan 30, 2014 at 6:22 PM, Volkin, Bradley D
<bradley.d.volkin at intel.com> 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



More information about the Intel-gfx mailing list