Bug 7293

Summary:	w83781d modprobing failure
Product:	Drivers	Reporter:	Sylvain BERTRAND (sylvain.bertrand)
Component:	Hardware Monitoring	Assignee:	Jean Delvare (jdelvare)
Status:	CLOSED CODE_FIX
Severity:	blocking
Priority:	P2
Hardware:	i386
OS:	Linux
Kernel Version:	2.6.19-rc1	Subsystem:
Regression:	---	Bisected commit-id:
Attachments:	cat /proc/ioports Let w83781d and lm78 load again

Description Sylvain BERTRAND 2006-10-09 13:45:13 UTC

#modprobe w83781d fails with "FATAL: Error inserting w83781d
(/lib/modules/2.6.19-rc1/kernel/drivers/hwmon/w83781d.ko): Device or resource
busy" error message and the messages log with some debug info follows. Works
fine with 2.6.18.

/var/log/messages
Oct  9 22:51:23 zeus i2c-core: driver [w83781d] registered
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x20
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=40 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=40 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x21
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=42 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=42 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x22
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=44 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=44 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x23
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=46 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=46 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x24
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=48 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=48 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x25
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=4a DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=4a DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x26
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=4c DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=4c DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x27
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=4e DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=4e DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x28
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=50 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=50 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x29
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=52 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=52 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x2a
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=54 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=54 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x2b
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=56 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=56 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x2c
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=58 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=58 DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x2d
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=5a DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=5a DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=40
ADD=5b DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=40
ADD=5b DAT=01,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=4e
ADD=5b DAT=01,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=4e
ADD=5b DAT=80,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=4f
ADD=5b DAT=80,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=4f
ADD=5b DAT=12,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=4e
ADD=5b DAT=12,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=4e
ADD=5b DAT=80,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=4e
ADD=5a DAT=80,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=4e
ADD=5a DAT=80,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=4f
ADD=5b DAT=80,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=4f
ADD=5b DAT=12,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=58
ADD=5b DAT=12,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=58
ADD=5b DAT=31,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: client [as99127f] registered with bus id
0-002d
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=4a
ADD=5b DAT=31,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=4a
ADD=5b DAT=01,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: client [as99127f subclient] registered
with bus id 0-0049
Oct  9 22:51:23 zeus i2c_adapter i2c-0: client [as99127f subclient] registered
with bus id 0-0048
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=40
ADD=5b DAT=01,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=40
ADD=5b DAT=01,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=40
ADD=5a DAT=01,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=40
ADD=5a DAT=01,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=3b
ADD=5b DAT=01,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=3b
ADD=5b DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=3c
ADD=5b DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=3c
ADD=5b DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=3d
ADD=5b DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=08 CMD=3d
ADD=5b DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x2e
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=08 CMD=3d
ADD=5c DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=5c DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: found normal entry for adapter 0, addr 0x2f
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (pre): STS=40 CNT=00 CMD=3d
ADD=5e DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-0: Transaction (post): STS=00 CNT=00 CMD=3d
ADD=5e DAT=ff,00
Oct  9 22:51:23 zeus i2c_adapter i2c-9191: Driver w83781d-isa registered
Oct  9 22:51:23 zeus i2c_adapter i2c-9191: Request of region 0x290-0x297 for
w83781d failed
Oct  9 22:51:23 zeus i2c_adapter i2c-9191: Driver w83781d-isa failed to attach
adapter, unregistering
Oct  9 22:51:23 zeus i2c_adapter i2c-0: detaching client [as99127f] at 0x2d
Oct  9 22:51:23 zeus i2c_adapter i2c-0: detaching client [as99127f subclient] at
0x49
Oct  9 22:51:23 zeus i2c_adapter i2c-0: detaching client [as99127f subclient] at
0x48
Oct  9 22:51:23 zeus i2c-core: driver [w83781d] unregistered

Comment 1 Jean Delvare 2006-10-09 14:08:28 UTC

Must be caused by this patch:
http://www.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=8202632647278eba7223727dc442f49227c040d0
i2c-isa: Fail adding driver on attach_adapter error

This patch is nice for pure i2c-isa drivers, but has an unexpected effect on
"hybrid" hardware monitoring drivers (lm78, w83781d and it87).

Please attach the output of "cat /proc/ioports".

Comment 2 Sylvain BERTRAND 2006-10-09 14:30:03 UTC

Created attachment 9194 [details]
cat /proc/ioports

Comment 3 Jean Delvare 2006-10-09 23:02:29 UTC

Created attachment 9199 [details]
Let w83781d and lm78 load again

This should fix the bug, please report.

Comment 4 Sylvain BERTRAND 2006-10-11 04:58:11 UTC

Fixed with this patch.
Thx.

Comment 5 Jean Delvare 2006-10-19 02:25:29 UTC

Fix merged upstream.