drm/i915: add ICP support to cnp_rawclk() and kill icp_rawclk()
authorPaulo Zanoni <paulo.r.zanoni@intel.com>
Mon, 12 Nov 2018 23:23:13 +0000 (15:23 -0800)
committerPaulo Zanoni <paulo.r.zanoni@intel.com>
Wed, 14 Nov 2018 00:37:16 +0000 (16:37 -0800)
commit704e504bd6167084dd0841c02d1d2bb58889e967
tree2c28b6ca263f3f8a4d4b4e95a75aea233787787a
parent228a5cf381f761ca16ef6b31b24c4533594468bd
drm/i915: add ICP support to cnp_rawclk() and kill icp_rawclk()

I think I'm probably the one who argued in favor of having separate
implementations for both PCHs, but the calculations are actually the
same, the clocks are the same and the only difference is that on ICP
we write the numerator to the register.

I have previously suggested to kill cnp_rawclk() and keep the
icp_rawclk() style, but Ville gave some good arguments that what's in
this patch may be the better choice.

v2: Switch numerator to 1 from 1000 and adjust calculations
    accordingly (Ville).

Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20181112232313.26373-3-paulo.r.zanoni@intel.com
drivers/gpu/drm/i915/intel_cdclk.c