i2c-viapro and via686a don't mix with linux-2.6.0-test1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> With kernel 2.6.0-test1 (likely with lm-sensors 2.8.0 too), it is
> possible to load both via686a and i2c-viapro, but only one of them
> will work, depending on which was loaded first. 
> 
> The reason for this is, that both drivers feel responsible for
> the VIA_82C686_4 and once the probe was successful, it is registered
> only to the driver which was loaded first. (So, you can load another
> driver for the device but the probe function will never get called). 
> 
> Sorry for not providing a patch, I have on clue how to fix this
> properly.

I tested this case on my own system and I could load both modules, and
both seemed to work (lm_sensors-2.8.0). I think I remember seeing
something about that on the mailing list recently:

Modified Files:
	i2c-voodoo3.c i2c-via.c i2c-viapro.c i2c-i810.c i2c-savage4.c 
Log Message:
(mds) don't use pci_module_init() for video i2c drivers so we don't
      conflict with video drivers

It was for a different conflict (it seems). However, the changes made to
i2c-viapro.c may also prevent the conflict you observed. This is a very
recent change (occured on July 2nd.). I attached the patch, could you
please check wether it is already applied to the 2.6.0 kernel tree? If
not, could you try to apply it and tell us if it solves your problem?

I admit I'm a bit confused, because i2c-viapro is a bus driver while
via686a is a chip driver. Thus I wonder how they can conflict...
However, the driver registering mechanisms of the Linux kernel is beyond
me, so it may just be me ;)

-- 
Jean Delvare
http://www.ensicaen.ismra.fr/~delvare/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: i2c-viapro-fix.diff
Url: http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20030715/b52e4bbb/attachment.pl 


[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux