From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Widawsky Subject: Re: [PATCH 00/58] modeset-rework, the basic conversion Date: Mon, 20 Aug 2012 20:27:13 -0700 Message-ID: <20120820202713.0000082c@unknown> References: <1345403595-9678-1-git-send-email-daniel.vetter@ffwll.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from shiva.chad-versace.us (209-20-75-48.static.cloud-ips.com [209.20.75.48]) by gabe.freedesktop.org (Postfix) with ESMTP id 1A7DD9EFEA for ; Mon, 20 Aug 2012 20:27:29 -0700 (PDT) In-Reply-To: <1345403595-9678-1-git-send-email-daniel.vetter@ffwll.ch> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Daniel Vetter Cc: Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Sun, 19 Aug 2012 21:12:17 +0200 Daniel Vetter wrote: > Hi all, > > Changes since last time around: > - The prep patches are all merged now. > - I've left out the actual DP fixes/cleanups, I think we should merge > those in a separte step. > - A few bugfixes (thanks to Paulo, Jani and Chris). > - I've also applied a few bikesheds for naming that Paulo suggested > (but I'm not sure whether I've sent those out already in a previous > patchbomb). > > Essentially this is just the core rework, which addes the new > get_hw_state code, refactors all the encoders to use the new > functions and finally reworks the modeset logic to disable/enable > entire pipes, always (and with a deterministic order). > > For merging to -next, I plan to pull in everything with a real merge > commit. For that reason I've put up a modeset-rework-base branch onto > my private fdo repo[1]. That way I can put a short documentation for > the new modeset design into the merge commit (stichted together from > the previous patchbomb cover letters), documenting my folly > assumptions for eternity. > > I'll also plan to put tags for the entire series in the merge commit, > so if you have tested this on a few machines, read through and agree > with the new designs, please reply with your > tested-by/acked-by/reviewed-by tags. > > Flames, comments and test reports highly welcome. > > Cheers, Daniel Tested-by: Ben Widawsky (before the rebase) > > [1] http://cgit.freedesktop.org/~danvet/drm/log/?h=modeset-rework-base > > Daniel Vetter (58): > drm/i915: add crtc->enable/disable vfuncs insted of dpms > drm/i915: rip out crtc prepare/commit indirection > drm/i915: add direct encoder disable/enable infrastructure > drm/i915/hdmi: convert to encoder->disable/enable > drm/i915/tv: convert to encoder enable/disable > drm/i915/lvds: convert to encoder disable/enable > drm/i915/dp: convert to encoder disable/enable > drm/i915/crt: convert to encoder disable/enable > drm/i915/sdvo: convert to encoder disable/enable > drm/i915/dvo: convert to encoder disable/enable > drm/i915: convert dpms functions of dvo/sdvo/crt > drm/i915: rip out encoder->disable/enable checks > drm/i915: clean up encoder_prepare/commit > drm/i915: copy&paste drm_crtc_helper_set_config > drm/i915: call set_base directly > drm/i915: inline intel_best_encoder > drm/i915: copy&paste drm_crtc_helper_set_mode > drm/i915: simplify intel_crtc_prepare_encoders > drm/i915: rip out encoder->prepare/commit > drm/i915: call crtc functions directly > drm/i915: WARN when trying to enabled an unused crtc > drm/i915: Add interfaces to read out encoder/connector hw state > drm/i915/dp: implement get_hw_state > drm/i915/hdmi: implement get_hw_state > drm/i915/tv: implement get_hw_state > drm/i915/lvds: implement get_hw_state > drm/i915/crt: implement get_hw_state > drm/i915/sdvo: implement get_hw_state > drm/i915/dvo: implement get_hw_state > drm/i915: read out the modeset hw state at load and resume time > drm/i915: check connector hw/sw state > drm/i915: rip out intel_crtc->dpms_mode > drm/i915: rip out intel_dp->dpms_mode > drm/i915: ensure the force pipe A quirk is actually followed > drm/i915: introduce struct intel_set_config > drm/i915: extract modeset config save/restore code > drm/i915: extract intel_set_config_compute_mode_changes > drm/i915: extract intel_set_config_update_output_state > drm/i915: implement crtc helper semantics relied upon by the fb > helper > drm/i915: don't update the fb base if there is no fb > drm/i915: convert pointless error checks in set_config to BUGs > drm/i915: don't save all the encoder/crtc state in set_config > drm/i915: stage modeset output changes > drm/i915: push crtc->fb update into pipe_set_base > drm/i915: remove crtc disabling special case > drm/i915: move output commit and crtc disabling into set_mode > drm/i915: extract adjusted mode computation > drm/i915: use staged outuput config in tv->mode_fixup > drm/i915: use staged outuput config in lvds->mode_fixup > drm/i915: compute masks of crtcs affected in set_mode > drm/i915: implement new set_mode code flow > drm/i915: push commit_output_state past crtc disabling > drm/i915: s/intel_encoder_disable/intel_encoder_noop > drm/i915: WARN if the pipe won't turn off > drm/i915: switch the load detect code to the staged modeset config > drm/i915: push commit_output_state past the crtc/encoder preparing > drm/i915: disable all crtcs at suspend time > drm/i915: add tons of modeset state checks > > drivers/gpu/drm/i915/dvo.h | 6 + > drivers/gpu/drm/i915/dvo_ch7017.c | 13 + > drivers/gpu/drm/i915/dvo_ch7xxx.c | 13 + > drivers/gpu/drm/i915/dvo_ivch.c | 15 + > drivers/gpu/drm/i915/dvo_ns2501.c | 15 + > drivers/gpu/drm/i915/dvo_sil164.c | 16 + > drivers/gpu/drm/i915/dvo_tfp410.c | 14 + > drivers/gpu/drm/i915/i915_drv.c | 4 + > drivers/gpu/drm/i915/i915_drv.h | 4 +- > drivers/gpu/drm/i915/i915_reg.h | 2 + > drivers/gpu/drm/i915/intel_crt.c | 136 ++- > drivers/gpu/drm/i915/intel_ddi.c | 59 +- > drivers/gpu/drm/i915/intel_display.c | 1499 > ++++++++++++++++++++++++++++------ > drivers/gpu/drm/i915/intel_dp.c | 144 ++-- > drivers/gpu/drm/i915/intel_drv.h | 64 +- > drivers/gpu/drm/i915/intel_dvo.c | 94 ++- > drivers/gpu/drm/i915/intel_hdmi.c | 151 +++- > drivers/gpu/drm/i915/intel_lvds.c | 95 +-- > drivers/gpu/drm/i915/intel_sdvo.c | 164 +++- > drivers/gpu/drm/i915/intel_tv.c | 61 +- 20 files changed, 2044 > insertions(+), 525 deletions(-) >