Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2012-08-29 Thread Jean Delvare
Eddi, Thomas, On Wed, 14 Mar 2012 09:28:54 +0100, Eddi De Pieri wrote: Hi Thomas, With the updated patch and the modified sensors-detect I have been able to probe all ports and have not encountered the power issue again. If no further issues are identified I will seperate the patch

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2012-06-15 Thread Jean Delvare
Hi Eddi, On Wed, 13 Jun 2012 09:47:39 +0200, Jean Delvare wrote: On Wed, 13 Jun 2012 07:44:34 +0200, Eddi De Pieri wrote: It works for me too What about including this patch in the mainline? Yesterday I reviewed a patch set adding support for the auxiliary SMBus controller on the

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2012-06-13 Thread Jean Delvare
Hi Eddi, On Wed, 13 Jun 2012 07:44:34 +0200, Eddi De Pieri wrote: It works for me too What about including this patch in the mainline? Yesterday I reviewed a patch set adding support for the auxiliary SMBus controller on the SB600/SB700: http://marc.info/?t=13388612451r=1w=2

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2012-03-14 Thread Eddi De Pieri
Hi Thomas, With the updated patch and the modified sensors-detect I have been able to probe all ports and have not encountered the power issue again. If no further issues are identified I will seperate the patch into stages and submit for review. I confirm that your patch works well on my

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2012-02-24 Thread Thomas Brandon
Jean Delvare khali@... writes: You don't need a multiplexer device at all. Please just have i2c-piix4 driver register 4 i2c_adapter devices for the SB800. This would be a 3-step process: 1* Turn piix4_adapter to an array, so that it can hold up to 4 i2c_adapter structures. 2* Add a

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2011-12-07 Thread Eddi De Pieri
On Tue, Dec 6, 2011 at 9:39 AM, Jean Delvare kh...@linux-fr.org wrote: The SMBus multiplexing as it exists on the SB800 is very different from what was implemented for the Tyan S4882 and S4985 boards. Well... - the Tyan use PCA9556 to select the multiplexed bus... - SB800 (and SB700) seems to

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2011-12-06 Thread Jean Delvare
Hi Eddi, Please don't top-post, and only include as much context as necessary in your replies. On Mon, 5 Dec 2011 23:57:10 +0100, Eddi De Pieri wrote: I tried in rewriting the driver... but I can't understand the right way... since sb800 is a southbus, the mux driver must be defined as

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2011-12-05 Thread Eddi De Pieri
Hello, I tried in rewriting the driver... but I can't understand the right way... since sb800 is a southbus, the mux driver must be defined as i2c_device, platform_device, or pci_device? I tried using i2c_device structure, but I can't make sb800 to match the id_Table.. I tried usign pci_device

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2011-12-03 Thread Eddi De Pieri
This patch add support to multiplexed smbus for proliant microserver N36L and may be applicable to other configuration based on sb8xx southbus. root@proliant:/usr/src/lm-sensors/eddi# i2cdetect -l i2c-0 smbus SMBus piix4 adapter (SDA0) SMBus adapter i2c-1 smbus

Re: [lm-sensors] [PATCH] i2c multiplexer driver for Proliant microserver N36L

2011-12-03 Thread Guenter Roeck
On Sat, Dec 03, 2011 at 10:31:30AM -0500, Eddi De Pieri wrote: This patch add support to multiplexed smbus for proliant microserver N36L and may be applicable to other configuration based on sb8xx southbus. Did you read Documentation/SubmittingPatches ? The patch doesn't follow the

Re: [lm-sensors] [PATCH] i2c multiplexer driver for Proliant microserver N36L

2011-12-03 Thread Eddi De Pieri
Actually I don't have a development system with latest git. I build my patch on my debian with 2.6.32 kernel.. The patch is based on i2c-amd756-s4882.c and adapted to work with sb8xx... i2c-amd756-s4882.c is still present and don't use i2c-mux infrastructure. However if you are interested in

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2011-12-03 Thread Jean Delvare
Hi Eddi, On Sat, 3 Dec 2011 17:42:16 +0100, Eddi De Pieri wrote: Actually I don't have a development system with latest git. I build my patch on my debian with 2.6.32 kernel.. The patch is based on i2c-amd756-s4882.c and adapted to work with sb8xx... i2c-amd756-s4882.c is still present

Re: [PATCH] i2c multiplexer driver for Proliant microserver N36L

2011-11-27 Thread Ben Dooks
On Fri, Nov 25, 2011 at 11:07:21PM +0100, Eddi De Pieri wrote: This patch add support to multiplexed smbus for proliant microserver N36L and may be applicable to other configuration based on sb8xx southbus. root@proliant:/usr/src/lm-sensors/eddi# i2cdetect -l i2c-0   smbus   SMBus

[PATCH] i2c multiplexer driver for Proliant microserver N36L

2011-11-25 Thread Eddi De Pieri
This patch add support to multiplexed smbus for proliant microserver N36L and may be applicable to other configuration based on sb8xx southbus. root@proliant:/usr/src/lm-sensors/eddi# i2cdetect -l i2c-0   smbus   SMBus piix4 adapter (SDA0)  SMBus adapter i2c-1   smbus