Hi Daniel, could you help me to assign best reviewers for these patches? also, please let me know if I should just remove patch 1 out from -collector. Thanks in advance, Rodrigo. On Mon, Nov 18, 2013 at 6:32 PM, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote: > > This is another drm-intel-collector updated notice: > http://cgit.freedesktop.org/~vivijim/drm-intel/log/?h=drm-intel-collector > > Here goes the update list in order for better reviewers assignment: > > Patch drm/i915: Asynchronously perform the set-base for a simple modeset - Reviewed by me. > Patch drm/i915: Fix gen3/4 vblank counter wraparound - Reviewer: > Patch drm/i915: Use frame counter for intel_wait_for_vblank() on CTG - Reviewer: > Patch drm/i915: Do hw quiescing first during unload - Reviewer: > Patch drm/i915: print object bindings in debugfs - Reviewer: > Patch drm/i915/vlv: enable HDMI audio for Valleyview2 - Reviewer: > Patch drm/i915: Hold pc8 lock around toggling pc8.gpu_idle - Reviewed by Paulo > Patch drm/i915: Do not enable package C8 on unsupported hardware - Reviewed by Paulo > Patch drm/i915: Enable pipe gamma for sprites - Reviewer: > > Overall Process: > > drm-intel-collector - review request > 1. Daniel pushs drm-intel-testing (every 2 weeks) > 2. I rebase drm-intel-collector onto drm-intel-testing > 3. Add Reviewer: tag with voluntered reviewers. If you don't believe you should be assigned on a particular patch please don't get mad just tell you wont review or volunteer someone else. > 4. I resubmit remaining patches for review without picking any new (drm-intel-collector - review request) > > drm-intel-collector - updated > 5. One week later I collect all simple (1-2) patches that wasn't yet reviewed and not queued by Daniel from one testing update until another. > 6. Request automated QA's PRTS automated i-g-t tests comparing drm-intel-testing x drm-intel-collector > 7. If tests are ok I send the update notification or the patches as a series to intel-gfx mailing list for better tracking and to be reviewed. (drm-intel-collector - updated) > 8. Let me know volunteers for review new patches and also let me know if I've picked any patch that I shouldn't. > > There are some reasons that some patches can be left behind: > 1. Your patch didn't applied cleanly and I couldn't easily solve the conflicts. > 2. Kernel didn't compiled with your patch. > 3. I simply missed it. If you believe this is the case please warn me. > > Please help me to get these patches reviewed and queued by Daniel. > > Also, please let me know if you have further ideas how to improve this process. > > Thanks in advance, > Rodrigo. > > > Chris Wilson (4): > drm/i915: Asynchronously perform the set-base for a simple modeset > drm/i915: Do hw quiescing first during unload > drm/i915: Hold pc8 lock around toggling pc8.gpu_idle > drm/i915: Do not enable package C8 on unsupported hardware > > Daniel Vetter (1): > drm/i915: print object bindings in debugfs > > Mengdong Lin (1): > drm/i915/vlv: enable HDMI audio for Valleyview2 > > Ville Syrjälä (3): > drm/i915: Fix gen3/4 vblank counter wraparound > drm/i915: Use frame counter for intel_wait_for_vblank() on CTG > drm/i915: Enable pipe gamma for sprites > > drivers/gpu/drm/i915/i915_debugfs.c | 6 ++++ > drivers/gpu/drm/i915/i915_dma.c | 10 +++--- > drivers/gpu/drm/i915/i915_drv.h | 1 + > drivers/gpu/drm/i915/i915_irq.c | 2 +- > drivers/gpu/drm/i915/i915_reg.h | 20 ++++++++++- > drivers/gpu/drm/i915/intel_display.c | 65 ++++++++++++++++++++++++++++-------- > drivers/gpu/drm/i915/intel_sprite.c | 18 ++++++++++ > 7 files changed, 103 insertions(+), 19 deletions(-) > > -- > 1.8.3.1 > -- Rodrigo Vivi Blog: http://blog.vivi.eng.br _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx