[lm-sensors] [PATCH] hwmon: (zl6100) Add new chip support

[Tang Yuantian-B29983]

Great, waiting for good news.

Regards,
Yuantian

> -----Original Message-----
> From: Guenter Roeck [mailto:guenter.roeck at ericsson.com]
> Sent: 2011年7月29日 14:39
> To: Tang Yuantian-B29983
> Cc: lm-sensors at 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 at ericsson.com]
> > > Sent: 2011年7月29日 12:35
> > > To: Tang Yuantian-B29983
> > > Cc: lm-sensors at lm-sensors.org
> > > Subject: Re: [PATCH] hwmon: (zl6100) Add new chip support
> > >
> > > On Wed, Jun 22, 2011 at 09:49:21AM +0800, b29983 at freescale.com wrote:
> > > > From: Tang Yuantian <B29983 at 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 at 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
> >