Bug 6623 - Fix I2C bus Mach64 initialisation failure
Summary: Fix I2C bus Mach64 initialisation failure
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/mach64 (show other bugs)
Version: git
Hardware: All All
: high normal
Assignee: George -
QA Contact:
Keywords: patch
Depends on:
Reported: 2006-04-17 02:42 UTC by Samuel Thibault
Modified: 2006-04-22 22:50 UTC (History)
3 users (show)

See Also:
i915 platform:
i915 features:

split ATII2CAddress into ATII2CStart and ATII2CAddress (1.38 KB, patch)
2006-04-17 02:43 UTC, Samuel Thibault
no flags Details | Splinter Review

Description Samuel Thibault 2006-04-17 02:42:44 UTC

The logs show

(WW) ATI(0): I2C bus Mach64 initialisation failure.
(II) ATI(0): I2C bus "Mach64" removed.

That's because atii2c.c hasn't splitted I2CAddress into I2CStart and I2CAddress. Here is a patch.

Comment 1 Samuel Thibault 2006-04-17 02:43:13 UTC
Created attachment 5334 [details] [review]
split ATII2CAddress into ATII2CStart and ATII2CAddress
Comment 2 Erik Andren 2006-04-17 18:11:02 UTC
Changing the topic a little bit.
Comment 3 Erik Andren 2006-04-18 04:58:21 UTC
Adjusting topic and add the patch keyword
Comment 4 Luc Verhaegen 2006-04-20 07:17:51 UTC
Will be included in xf86-video-mach64, which is post 7.1 material.

This probably only hurts gatos functionality, as EDID is gathered through VBE.
That will be fixed too in xf86-video-mach64.
Comment 5 George - 2006-04-21 03:42:22 UTC
Hi Samuel,

I'll try your patch and probably try some other stuff over I2C now that it
works. However, I am new to I2C and will appreciate if you elaborate a little
bit on what was wrong, how it is fixed and other stuff you found useful fixing
this. Also, why are you declaring ATII2CStart as:
ATII2CStart(I2CDevPtr pI2CDev)
while xf86i2c.h has the following prototype:
Bool (*I2CStart)  (I2CBusPtr b, int timeout);

Comment 6 Samuel Thibault 2006-04-21 04:21:13 UTC

I don't know much about i2c in xorg, but it looks like ati's i2c was
not updated to the current convention of I2CStart/I2CAddress/I2CStop
(while the old one probably was only I2CAddress/I2CStop). So I just
split I2CAddress into I2CStart/I2CAddress. I don't have multimedia
devices for actually testing the I2C driver, I mostly submitted the
bug for fixing a warning in my logs :)

The prototype that I set for ATII2CStart is indeed bogus.

Comment 7 Luc Verhaegen 2006-04-22 07:38:43 UTC
Warning has gone in xf86-video-mach64. Samuel, please switch over to
xf86-video-mach64, you'll get George's EXA to boot.
Comment 8 Samuel Thibault 2006-04-22 21:00:24 UTC
Why not fixing it in xf86-driver-ati too? Some distributions (like debian) don't package xf86-driver-mach64 yet.
Comment 9 Luc Verhaegen 2006-04-23 15:50:22 UTC
xf86-driver-ati is way beyond fixing, but i'm not stopping you from attempting
to do so.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.