kern/85106: The ICH7 smb interface is not recognised

[Oliver Fromme]

The following reply was made to PR kern/85106; it has been noted by GNATS.

From: Oliver Fromme <olli at lurza.secnetix.de>
To: bug-followup at FreeBSD.org, arthur.hartwig at nokia.com
Cc:  
Subject: Re: kern/85106: The ICH7 smb interface is not recognised
Date: Tue, 30 Aug 2005 17:01:56 +0200 (CEST)

    Hi,
 
    I've been able to verify that the suggested patch for ICH6
    does work.  I tested it on my Samsung X20 1600-V notebook
    which uses the i915GM chipset + 82801FB (ICH6).
 
    Without the patch (plain RELENG_6 == 6.0-BETA3), I got:
 
 pci0: <serial bus, SMBus> at device 31.3 (no driver attached)
 
    With the patch, I get:
 
 ichsmb0: <Intel 82801FB (ICH6) SMBus controller> port 0x18a0-0x18bf irq 19 at device 31.3 on pci0
 ichsmb0: [GIANT-LOCKED]
 smbus0: <System Management Bus> on ichsmb0
 smb0: <SMBus generic I/O> on smbus0
 
    Below is the complete patch.  It applies to all of HEAD,
    RELENG_6, RELENG_5 and RELENG_4.
 
    Best regards
       Oliver
 
 
 --- src/sys/dev/ichsmb/ichsmb_pci.c.orig        Fri Aug 19 20:38:55 2005
 +++ src/sys/dev/ichsmb/ichsmb_pci.c     Tue Aug 30 16:59:08 2005
 @@ -41,7 +41,8 @@
 
  /*
   * Support for the SMBus controller logical device which is part of the
 - * Intel 81801AA/AB/BA/CA/DC/EB (ICH/ICH[02345]) I/O controller hub chips.
 + * Intel 81801AA/AB/BA/CA/DC/EB/FB/GB (ICH/ICH[0234567])
 + * I/O controller hub chips.
   */
 
  #include <sys/param.h>
 @@ -73,6 +74,8 @@
  #define ID_82801CA			0x24838086
  #define ID_82801DC			0x24C38086
  #define ID_82801EB			0x24D38086
 +#define ID_82801FB			0x266a8086
 +#define ID_82801GB			0x27da8086
  #define ID_6300ESB			0x25a48086
 
  #define PCIS_SERIALBUS_SMBUS_PROGIF	0x00
 @@ -142,6 +145,12 @@
 		break;
 	case ID_82801EB:
 		device_set_desc(dev, "Intel 82801EB (ICH5) SMBus controller");
 +		break;
 +	case ID_82801FB:
 +		device_set_desc(dev, "Intel 82801FB (ICH6) SMBus controller");
 +		break;
 +	case ID_82801GB:
 +		device_set_desc(dev, "Intel 82801GB (ICH7) SMBus controller");
 		break;
 	case ID_6300ESB:
 		device_set_desc(dev, "Intel 6300ESB (ICH) SMBus controller");
 
 
 
 -- 
 Oliver Fromme,  secnetix GmbH & Co. KG, Marktplatz 29, 85567 Grafing
 Any opinions expressed in this message may be personal to the author
 and may not necessarily reflect the opinions of secnetix in any way.
 
 "I learned Java 3 years before Python.  It was my language of
 choice.  It took me two weekends with Python before I was more
 productive with it than with Java." -- Anthony Roberts