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@xxxxxxxxxxxx] > > Sent: 2011年7月29日 14:39 > > To: Tang Yuantian-B29983 > > Cc: lm-sensors@xxxxxxxxxxxxxx; 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@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
