[PATCH] unhide smbus on Compaq Deskpro EN

[PATCH] unhide smbus on Compaq Deskpro EN

[Krzysztof Helt]

From: Krzysztof Helt <krzysztof.h1-5tc4TXWwyLM@public.gmane.org>

This patch unhides the smbus on Compaq Deskpro EN
SFF P667 with the Intel 815E chipset. 
The patch was tested on Fedora Core 9 with 2.6.25.4 kernel.

Signed-off-by: Krzysztof Helt <krzysztof.h1-5tc4TXWwyLM@public.gmane.org>
Tested-by: Rafał Haładuda <rh1985-5tc4TXWwyLM@public.gmane.org>

---

No side effects were observed. The ACPI controlled fan
correctly with the patch applied. Tested with 667 and
1000 MHz CPUs.
The patch allows using the THMC51 sensor chip on
the motherboard.
The patch was done against the kernel 2.6.26-rc2-mm1

diff -urp linux-2.6.25/drivers/pci/quirks.c linux-new/drivers/pci/quirks.c
--- linux-2.6.25/drivers/pci/quirks.c	2008-05-27 21:58:34.380144607 +0200
+++ linux-new/drivers/pci/quirks.c	2008-05-30 23:12:57.510219450 +0200
@@ -1054,6 +1054,14 @@ static void __init asus_hides_smbus_host
 				 * its on-board VGA controller */
 				asus_hides_smbus = 1;
 			}
+		else if (dev->device == PCI_DEVICE_ID_INTEL_82815_CGC)
+			switch (dev->subsystem_device) {
+			case 0x001A: /* Compaq Deskpro EN ENS/P667/e */
+				/* Motherboard doesn't have Host bridge
+				 * subvendor/subdevice IDs, therefore checking
+				 * its on-board VGA controller */
+				asus_hides_smbus = 1;
+			}
 	}
 }
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL,	PCI_DEVICE_ID_INTEL_82845_HB,	asus_hides_smbus_hostbridge);
@@ -1068,6 +1076,7 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_I
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL,	PCI_DEVICE_ID_INTEL_82915GM_HB, asus_hides_smbus_hostbridge);
 
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL,	PCI_DEVICE_ID_INTEL_82810_IG3,	asus_hides_smbus_hostbridge);
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL,	PCI_DEVICE_ID_INTEL_82815_CGC,	asus_hides_smbus_hostbridge);
 
 static void asus_hides_smbus_lpc(struct pci_dev *dev)
 {

_______________________________________________
i2c mailing list
i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org
http://lists.lm-sensors.org/mailman/listinfo/i2c

Re: [PATCH] unhide smbus on Compaq Deskpro EN

[Jean Delvare]

Hi Krzysztof,

On Fri, 30 May 2008 23:21:18 +0200, Krzysztof Helt wrote:
> From: Krzysztof Helt <krzysztof.h1@wp.pl>
> 
> This patch unhides the smbus on Compaq Deskpro EN
> SFF P667 with the Intel 815E chipset. 
> The patch was tested on Fedora Core 9 with 2.6.25.4 kernel.
> 
> Signed-off-by: Krzysztof Helt <krzysztof.h1@wp.pl>
> Tested-by: Rafał Haładuda <rh1985@wp.pl>
> 
> ---
> 
> No side effects were observed. The ACPI controlled fan
> correctly with the patch applied. Tested with 667 and
> 1000 MHz CPUs.
> The patch allows using the THMC51 sensor chip on
> the motherboard.
> The patch was done against the kernel 2.6.26-rc2-mm1

I would like to see the ACPI DSDT of this machine. If ACPI is
controlling the fan then there are good chances that unhiding the SMBus
is the wrong thing to do, even if you did not notice any problem yet.

Thanks,
-- 
Jean Delvare

_______________________________________________
i2c mailing list
i2c@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/i2c

Re: [PATCH] unhide smbus on Compaq Deskpro EN

[Jean Delvare]

On Fri, 30 May 2008 23:21:18 +0200, Krzysztof Helt wrote:
> From: Krzysztof Helt <krzysztof.h1@wp.pl>
> 
> This patch unhides the smbus on Compaq Deskpro EN
> SFF P667 with the Intel 815E chipset. 
> The patch was tested on Fedora Core 9 with 2.6.25.4 kernel.
> 
> Signed-off-by: Krzysztof Helt <krzysztof.h1@wp.pl>
> Tested-by: Rafał Haładuda <rh1985@wp.pl>
> 
> ---
> 
> No side effects were observed. The ACPI controlled fan
> correctly with the patch applied. Tested with 667 and
> 1000 MHz CPUs.
> The patch allows using the THMC51 sensor chip on
> the motherboard.
> The patch was done against the kernel 2.6.26-rc2-mm1
> 
> diff -urp linux-2.6.25/drivers/pci/quirks.c linux-new/drivers/pci/quirks.c
> --- linux-2.6.25/drivers/pci/quirks.c	2008-05-27 21:58:34.380144607 +0200
> +++ linux-new/drivers/pci/quirks.c	2008-05-30 23:12:57.510219450 +0200
> @@ -1054,6 +1054,14 @@ static void __init asus_hides_smbus_host
>  				 * its on-board VGA controller */
>  				asus_hides_smbus = 1;
>  			}
> +		else if (dev->device == PCI_DEVICE_ID_INTEL_82815_CGC)
> +			switch (dev->subsystem_device) {
> +			case 0x001A: /* Compaq Deskpro EN ENS/P667/e */
> +				/* Motherboard doesn't have Host bridge
> +				 * subvendor/subdevice IDs, therefore checking
> +				 * its on-board VGA controller */
> +				asus_hides_smbus = 1;
> +			}
>  	}
>  }
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL,	PCI_DEVICE_ID_INTEL_82845_HB,	asus_hides_smbus_hostbridge);
> @@ -1068,6 +1076,7 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_I
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL,	PCI_DEVICE_ID_INTEL_82915GM_HB, asus_hides_smbus_hostbridge);
>  
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL,	PCI_DEVICE_ID_INTEL_82810_IG3,	asus_hides_smbus_hostbridge);
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL,	PCI_DEVICE_ID_INTEL_82815_CGC,	asus_hides_smbus_hostbridge);
>  
>  static void asus_hides_smbus_lpc(struct pci_dev *dev)
>  {

Patch looks OK, but it should be sent to the pci subsystem maintainer,
not me.

But first you want to double check that nothing wrong will happen. ACPI
looks OK but there is still the possibility of SMM quirks. To make sure,
you need to get your hands on the THMC51 datasheet and/or test if the
fan is self-regulated. And check if the low/high limits change over
time, too.

-- 
Jean Delvare

_______________________________________________
i2c mailing list
i2c@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/i2c