drm/i915: Fix wrong escape clock divisor init for GLK
authorStanislav Lisovskiy <stanislav.lisovskiy@intel.com>
Fri, 12 Jul 2019 08:19:38 +0000 (11:19 +0300)
committerJani Nikula <jani.nikula@intel.com>
Mon, 5 Aug 2019 08:30:54 +0000 (11:30 +0300)
commitce52ad5dd52cfaf3398058384e0ff94134bbd89c
treeccd4dd49cf70679ba90202a477cec8439ee7bcac
parentb40d73784ffc33f3c6431e7ceec3b20fffcd95c3
drm/i915: Fix wrong escape clock divisor init for GLK

According to Bspec clock divisor registers in GeminiLake
should be initialized by shifting 1(<<) to amount of correspondent
divisor. While i915 was writing all this time that value as is.

Surprisingly that it by accident worked, until we met some issues
with Microtech Etab.

v2: Added Fixes tag and cc
v3: Added stable to cc as well.

Signed-off-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>
Reviewed-by: Vandita Kulkarni <vandita.kulkarni@intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108826
Fixes: bcc657004841 ("drm/i915/glk: Program txesc clock divider for GLK")
Cc: Deepak M <m.deepak@intel.com>
Cc: Madhav Chauhan <madhav.chauhan@intel.com>
Cc: Jani Nikula <jani.nikula@intel.com>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Cc: stable@vger.kernel.org
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190712081938.14185-1-stanislav.lisovskiy@intel.com
drivers/gpu/drm/i915/display/vlv_dsi_pll.c