CHROMIUM: drm: rockchip/dw_hdmi-rockchip: Fixup the clock to be what we expect
authorYakir Yang <ykk@rock-chips.com>
Sat, 16 Jul 2016 08:44:26 +0000 (16:44 +0800)
committerHuang, Tao <huangtao@rock-chips.com>
Tue, 19 Jul 2016 07:23:31 +0000 (15:23 +0800)
commit329f1b1ff70cca751c88d04e7aa3d4d2c9830e22
tree265daf92f8f4b7c88ced76d27a3a29a0347f5bc1
parent8974d5e9bdfa350d563ff9805966466084da4a3b
CHROMIUM: drm: rockchip/dw_hdmi-rockchip: Fixup the clock to be what we expect

We allow some amount of slop in dw_hdmi_rockchip_mode_valid().  That's
a good thing since allowing a little bit of slop lets us support a
bunch of extra resolutions.

Originally, we also made a change to the VOP code to add the concept
of slop in there.  That was reasonable, but there was a problem: it
would tend to request clock rates that weren't _exactly_ clock rates
that we thought about.  It's possible that the common clock framework
would map these to PLL rates that we haven't thought about and we
haven't tested for jitter.

Instead of changing VOP, we should probably adjust the clock ourselves
in the mode_fixup function.  That way we'll request the exact clock we
tested and we'll know how the common clock framework will map it.

Change-Id: I56c2b046f76d554aab5eaed7a6b171ea074d6a62
Signed-off-by: Douglas Anderson <dianders@chromium.org>
Signed-off-by: Yakir Yang <ykk@rock-chips.com>
Reviewed-on: https://chromium-review.googlesource.com/284376
Reviewed-by: Alexandru Stan <amstan@chromium.org>
drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c