From mboxrd@z Thu Jan  1 00:00:00 1970
From: Guenter Roeck <guenter.roeck@ericsson.com>
Date: Fri, 29 Jul 2011 06:51:47 +0000
Subject: Re: [lm-sensors] [PATCH] hwmon: (zl6100) Add new chip support
Message-Id: <20110729065147.GA5911@ericsson.com>
List-Id: <lm-sensors.vger.kernel.org>
References: <1308707361-15558-1-git-send-email-b29983@freescale.com>
In-Reply-To: <1308707361-15558-1-git-send-email-b29983@freescale.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
To: lm-sensors@vger.kernel.org

On Fri, Jul 29, 2011 at 02:43:11AM -0400, Tang Yuantian-B29983 wrote:
> Great, waiting for good news.
> 
> Regards,
> Yuantian
> 
On a side note, top-posting is discouraged in this forum.

I'll let you know as I make progress.

Thanks,
Guenter

> > -----Original Message-----
> > From: Guenter Roeck [mailto:guenter.roeck@ericsson.com]
> > Sent: 2011年7月29日 14:39
> > To: Tang Yuantian-B29983
> > Cc: lm-sensors@lm-sensors.org; Tabi Timur-B04825; Huang Changming-R66093
> > Subject: Re: [PATCH] hwmon: (zl6100) Add new chip support
> > 
> > On Fri, Jul 29, 2011 at 12:56:25AM -0400, Tang Yuantian-B29983 wrote:
> > > Hi,
> > >
> > > I got this from Intersil Support.
> > > It is a zl6100 chip Errata and can't be found in zl6100 later chip.
> > > So, this chip maybe can't work on PMbus framework.
> > >
> > Hi,
> > 
> > excellent - I'll try to get the errata and information how to detect
> > affected chip revisions from Intersil.
> > 
> > Actually, turns out it will work quite nicely. All I'll have to do is to
> > write a little front-end driver which adds the delay for affected devices.
> > I am already working on it.
> > 
> > Thanks,
> > Guenter
> > 
> > > Good luck!
> > >
> > > Regards,
> > > Yuantian
> > >
> > >
> > > > -----Original Message-----
> > > > From: Guenter Roeck [mailto:guenter.roeck@ericsson.com]
> > > > Sent: 2011年7月29日 12:35
> > > > To: Tang Yuantian-B29983
> > > > Cc: lm-sensors@lm-sensors.org
> > > > Subject: Re: [PATCH] hwmon: (zl6100) Add new chip support
> > > >
> > > > On Wed, Jun 22, 2011 at 09:49:21AM +0800, b29983@freescale.com wrote:
> > > > > From: Tang Yuantian <B29983@freescale.com>
> > > > >
> > > > > 	The ZL6100 is a digital power controller with Power
> > Monitoring
> > > > support.
> > > > > 	It provides an I2C/SMBus digital interface that enables the
> > user to
> > > > > 	configure all aspects of the device operation as well as
> > monitor
> > > > the
> > > > > 	input and output parameters. It accepts most standard PMBus
> > > > commands.
> > > > > 	Four parameters are output here which are Output Current,
> > Output
> > > > Voltage,
> > > > > 	Local Temperature and Remote Temperature.
> > > > >
> > > > > 	This chip is currently used on some PowerPC platform.
> > > > >
> > > > > Signed-off-by: Tang Yuantian <b29983@freescale.com>
> > > >
> > > > Hi,
> > > >
> > > > I finally got an eval board for the ZL6100 and was able to do some
> > > > testing.
> > > >
> > > > The key problem with the chip and the generic PMBus driver is
> > > > reflected by the
> > > > 	msleep(2);
> > > > in your driver. Problem here is that this adds a delay of up to 20ms
> > > > to each and every I2C read command, which is really not desirable. I
> > > > found that udelay(500) does the trick as well, but is not much
> > > > better. I'll need to think about this a bit.
> > > >
> > > > Where do you have the 2ms value from ? I don't seem to be able to
> > > > find it in the datasheet.
> > > > Also, the ZL6105 does not seem to have the timing problem.
> > > >
> > > > Thanks,
> > > > Guenter
> > >
> 

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