summaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/i915/i915_drv.h
diff options
context:
space:
mode:
authorBenson Leung <bleung@chromium.org>2012-02-09 12:03:17 -0800
committerDaniel Vetter <daniel.vetter@ffwll.ch>2012-02-29 20:10:36 +0100
commitcaae745a352377f48f6eb43b5040214d803a885f (patch)
treec8d02a2a3ac5162e31ac2b8ee9b971256ad03e11 /drivers/gpu/drm/i915/i915_drv.h
parent5d1333fcce84e77ec865f6b09006401ed3f564b5 (diff)
downloadop-kernel-dev-caae745a352377f48f6eb43b5040214d803a885f.zip
op-kernel-dev-caae745a352377f48f6eb43b5040214d803a885f.tar.gz
drm/i915: Fix single msg gmbus_xfers writes
gmbus_xfer with a single message (particularly a single message write) would set Bus Cycle Select to 100b, the Gen Stop cycle, instead of 101b, No Index, Stop cycle. This would not start single message i2c transactions. Also, gmbus_xfer done: will disable the interface without checking if it is idle. In the case of writes, there will be no wait on status or delay to ensure the write starts and completes before the interface is turned off. Fixed the former issue by using the same cycle selection as used in the I2C_M_RD for the write case. GMBUS_CYCLE_WAIT | (i + 1 == num ? GMBUS_CYCLE_STOP : 0) Fixed the latter by waiting on GMBUS_ACTIVE to deassert before disable. Note from the grumpy d-i-n maintainer: The first hunk that changes the gmbus read path is just cosmetics to align the code with the write path. I.e. the commit message above is slightly lying because the first issue is _only_ with writes (and not simply "particularly"). Signed-off-by: Benson Leung <bleung@chromium.org> Reviewed-by: Daniel Kurtz <djkurtz@chromium.org> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/i915_drv.h')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud