SMBus regression (Re: cvs commit: src/sys/dev/bktr bktr_i2c.c bktr_i2c.h src/sys/dev/ichsmb ichsmb.c...

[Dmitry Pryanishnikov]

Hello!

On Fri, 22 Sep 2006, John Baldwin wrote:
>jhb         2006-09-22 19:19:16 UTC
>
>  FreeBSD src repository
>
>  Modified files:        (Branch: RELENG_6)
>    sys/dev/bktr         bktr_i2c.c bktr_i2c.h
>    sys/dev/ichsmb       ichsmb.c
>    sys/dev/iicbus       iicsmb.c
>    sys/dev/smbus        smb.c smb.h smbconf.c smbconf.h smbus.c
>                         smbus.h smbus_if.m
>    sys/pci              alpm.c amdpm.c amdsmb.c intpm.c nfsmb.c
>                         viapm.c
>  Log:
>  MFC: Minor overhaul of SMBus support including:

   This commit has broken SMBus attachment at least on ICH4; boot messages
difference (diff -u old new) gives only 1 line missing:

  ichsmb0: <Intel 82801DC (ICH4) SMBus controller> port 0xe000-0xe01f irq 17 at
    device 31.3 on pci0
  ichsmb0: [GIANT-LOCKED]
  smbus0: <System Management Bus> on ichsmb0
-smb0: <SMBus generic I/O> on smbus0

So with the new SMBus modules smb0 no longer attaches to the smbus0 despite
having all 3 modules loaded as before:

dmitry at homelynx$ kldstat|grep smb
21    3 0xc07b9000 29ac     smbus.ko
23    1 0xc07c0000 4ff8     ichsmb.ko
45    1 0xc6974000 3000     smb.ko

After reverting the contents of /sys/dev/{ichsmb,smbus} to the pre-commit 
state (date=2006.09.22.19.19.00) and recompiling the modules (smbus, ichsmb,
smb) I've got working SMBus subsystem again. Please look into this problem.

Sincerely, Dmitry
-- 
Atlantis ISP, System Administrator
e-mail:  dmitry at atlantis.dp.ua
nic-hdl: LYNX-RIPE