Re: [PATCH] hwmon: (zl6100) Add new chip support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxx]
> > Sent: 2011年7月29日 12:35
> > To: Tang Yuantian-B29983
> > Cc: lm-sensors@xxxxxxxxxxxxxx
> > Subject: Re: [PATCH] hwmon: (zl6100) Add new chip support
> > 
> > On Wed, Jun 22, 2011 at 09:49:21AM +0800, b29983@xxxxxxxxxxxxx wrote:
> > > From: Tang Yuantian <B29983@xxxxxxxxxxxxx>
> > >
> > > 	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@xxxxxxxxxxxxx>
> > 
> > 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@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors



[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux