]> git.baikalelectronics.ru Git - kernel.git/commit
drm/i915: Fix 400 MHz FSB readout on elk
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 14 May 2020 12:38:36 +0000 (15:38 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 19 May 2020 09:06:55 +0000 (12:06 +0300)
commit5c209d36cbf68051b66b5196c1ea4fa38271e102
tree16aef80b55dd7815aa5ad04e6d9f9196ed9c7f23
parent1dc7990a46208f436eb705e8e34bd729a8b7e1ba
drm/i915: Fix 400 MHz FSB readout on elk

Looks like elk redefines some of the CLKCFG FSB values to
make room for 400 MHz FSB. The setting overlaps with one of
the 266MHz settings (which is even documented in the ctg docs,
and cofirmed to be correct on my ctg). So we limit the special
case to elk only.

Though it might also be that we have some kind of desktop vs.
mobile difference going on here as eg. both g35 and elk
use 0x0 for the 266 MHz setting, vs. 0x6 used by ctg). The
g35 doesn't let me select 400MHz for the FSB strap so can't
confirm which way it would go here. But anyways as it seems
only elk has the 400MHz option we shouldn't lose anything
by limiting the special case to it alone.

My earlier experiments on this appear to have been nonsense as
the comment I added claims that FSB strap of 400MHz results in
a value of 0x4, but I've now retested it and I definitely get a
value of 0x6 instead. So let's remove that bogus comment.

v2: s/_ELK/_ALT/ in the define in anticipation of a full
    mobile vs. desktop CLKCFG split

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200514123838.3017-1-ville.syrjala@linux.intel.com
Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
drivers/gpu/drm/i915/display/intel_cdclk.c
drivers/gpu/drm/i915/i915_reg.h