From mboxrd@z Thu Jan 1 00:00:00 1970 From: Imre Deak Subject: Re: [PATCH 00/66] runtime pm for DPMS Date: Wed, 07 May 2014 16:49:54 +0300 Message-ID: <1399470594.11648.2.camel@intelbox> References: <1398376542-27825-1-git-send-email-daniel.vetter@ffwll.ch> Reply-To: imre.deak@intel.com Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1719287060==" Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id DF69F6ED05 for ; Wed, 7 May 2014 06:49:58 -0700 (PDT) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: Intel Graphics Development , "akash.goel" List-Id: intel-gfx@lists.freedesktop.org --===============1719287060== Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-EP6ltf0GdR/igt/rqmrP" --=-EP6ltf0GdR/igt/rqmrP Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Fri, 2014-04-25 at 10:45 +0200, Daniel Vetter wrote: > Ok, review assignements. Please complain if you don't have the > bandwidth so that I can find someone else. >=20 > On Thu, Apr 24, 2014 at 11:54 PM, Daniel Vetter = wrote: > > > > Daniel Vetter (66): > > drm/i915: Make encoder->mode_set callbacks optional > > drm/i915/dvo: Remove ->mode_set callback > > drm/i915/tv: extract set_tv_mode_timings > > drm/i915/tv: extract set_color_conversion > > drm/i915/tv: De-magic device check > > drm/i915/tv: Rip out pipe-disabling nonsense from ->mode_set > > drm/i915/tv: Remove ->mode_set callback > > drm/i915/crt: Remove ->mode_set callback > > drm/i915/sdvo: Remove ->mode_set callback > > > > Removal of encoder->mode_set callbacks, part 1 >=20 > Reviewer: Imre >=20 > > drm/i915/hdmi: Enable hdmi mode on g4x, too > > drm/i915: Track hdmi mode in the pipe config > > drm/i915/sdvo: Use pipe_config->limited_color_range consistently > > drm/i915: state readout and cross checking for limited_color_range > > drm/i915/sdvo: use config->has_hdmi_sink > > drm/i915: Simplify audio handling on DDI ports > > drm/i915: Track has_audio in the pipe config > > drm/i915/dp: Move port A pll setup to g4x_pre_enable_dp > > drm/i915/dp: Remove ->mode_set callback > > drm/i915/hdmi: Remove redundant IS_VLV checks > > drm/i915/hdmi: Remove ->mode_set callback > > > > Removal of the encoder->mode_set callbacks for hdmi/sdvo/dp with small > > interludes to move a bit of the hdmi/audio state into the pipe config. >=20 > Reviewer: Naresh Kumar >=20 >=20 > > drm/i915/lvds: Remove ->mode_set callback > > drm/i915/ddi: Remove ->mode_set callback > > drm/i915/dsi: Remove ->mode_set callback > > drm/i915: Stop calling encoder->mode_set > > > > Final removals of encoder->mode_set callbacks >=20 > Reviewer: Imre On patches 21-24: Reviewed-by: Imre Deak >=20 > > drm/i915: Make ->update_primary_plane infallible > > drm/i915: More cargo-culted locking for intel_update_fbc > > drm/i915: Sprinkle intel_edp_psr_update over crtc_enable/disable > > drm/i915: Inline set_base into crtc_mode_set > > drm/i915: Move fb pinning into __intel_set_mode > > > > Some shuffling to get the primary->fb handling out of crtc mode_set cal= lbacks >=20 > Reviewer: Akash Goel >=20 > > drm/i915: Shovel hw setup code out of i9xx_crtc_mode_set > > drm/i915: Move lowfreq_avail around a bit in ilk/hsw_crtc_mode_set > > drm/i915: Shovel hw setup code out of ilk_crtc_mode_set > > drm/i915: Shovel hw setup code out of hsw_crtc_mode_set > > drm/i915: Extract i9xx_set_pll_dividers > > drm/i915: Extract vlv_prepare_pll > > > > gmch pll moved out of crtc mode_set callbacks into ->enable hooks >=20 > Reviewer: Shobhit Kumar >=20 > > drm/i915: Only update shared dpll state when needed > > drm/i915: Extract intel_prepare_shared_dpll > > drm/i915: s/ironlake_/intel_ for the enable_share_dpll function > > > > Prep polish on the existing shared_dpll code >=20 > Reviewer: Damien (same comment as below) >=20 > > drm/i915: Check hw state in assert_can_disable_lcpll > > drm/i915: Remove spll_refcount for hsw > > drm/i915: Clean up WRPLL/SPLL #defines > > drm/i915: Make intel_wait_for_pipe_off static > > drm/i915: Disable pipe before ports on ilk > > drm/i915: Pass port explicitly to intel_ddi_get_hw_state > > drm/i915: Unexport intel_ddi_connector_get_hw_state > > drm/i915: Move hsw_fdi_link_train into intel_crt.c > > drm/i915: Move pch fifo underrun report enabling to hsw_crt_pre_enabl= e > > drm/i915: Move the SPLL enabling into hsw_crt_pre_enable > > drm/i915: Move lpt_pch_enable int hsw_crt_enable > > drm/i915: Move the pch fifo underrun handling into hsw_crt_disable > > drm/i915: Move lpt_disable_pch_transcoder into the hsw crt encoder > > drm/i915: Move pch fifo underrun report re-enabling into > > hsw_crt_post_disable > > drm/i915: Move the hsw fdi disabling into hsw_crt_post_disable > > drm/i915: Move SPLL disabling into hsw_crt_post_disable > > > > Create a new hsw-specific crt encoder which subsumes the entire fdi/pch= handling > > on haswell. This has the nice upshot to make SPLL logically a port-priv= ate clock > > and so removes it from further considerations. >=20 > Reviewer: Paulo >=20 > > drm/i915: Add a debugfs file for the shared dpll state > > drm/i915: Move ddi_pll_sel into the pipe config > > drm/i915: State readout and cross-checking for ddi_pll_sel > > drm/i915: Precompute static ddi_pll_sel values in encoders > > drm/i915: Basic shared dpll support for WRPLLs > > drm/i915: Document that the pll->mode_set hook is optional > > drm/i915: State readout support for WRPLLs > > drm/i915: ->disable hook for WRPLLs > > drm/i915: ->enable hook for WRPLLs > > drm/i915: Switch to common shared dpll framework for WRPLLs > > drm/i915: Only touch WRPLL hw state in enable/disable hooks > > > > Convert wrpll handling to the common shared_dpll framework. We need thi= s since > > runtime pm for dpms requires us to separately track pll refernces from = crtcs and > > active usage by crtcs >=20 > Reviewer: Damien (maybe find someone from the vpg guys who do the pll > stuff for your project instead) >=20 > > drm/i915: runtime PM support for DPMS >=20 > Reviewer: Paulo/Imre --=-EP6ltf0GdR/igt/rqmrP Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.14 (GNU/Linux) iQEcBAABAgAGBQJTajoCAAoJEORIIAnNuWDFwuUIAKIoei+LFAYjc545/79UR+SX LpoBUxlwKDyjr1Q0tc7+EkufhNfNv6nZKcBNFI93bigwOOomyiqbna+HBSpG2wsV EqW7WjSWfNa8m7/ZC02D7XSRDXbeHfWUfETVlsFk4/n0phvHxWtdYYNdvuzpT4Vy wbntPoYqSrILneWyqETgQ3vgqK5/lHKQaP+h4L295Iro0F2AmQEttl6em4IY73nC tSDyh6cb27mPu01TbnhteV7XovxyADQ/tH0sXe8goe9GjkMP19I/8+UCNtwr+RjM 6rQUVrcLlzce4kbl+gy1Ce5uibI/793MTaNIWL/RS23mcy5xgF57Zr2ev9MKulg= =JBnV -----END PGP SIGNATURE----- --=-EP6ltf0GdR/igt/rqmrP-- --===============1719287060== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx --===============1719287060==--