[lm-sensors] What about i2c_smbus_read_block_data() ?

[Jean Delvare]

Hi Mark, Hans-Jürgen,

On Wed, 4 Apr 2007 22:42:18 -0400, Mark M. Hoffman wrote:
> * Hans-Jürgen Koch <hjk at linutronix.de> [2007-04-04 23:32:34 +0200]:
> > Am Mittwoch, 4. April 2007 21:49 schrieb Hans-Jürgen Koch:
> > > I'm working on a driver for LM93. I've got some code by Mark Hoffman, (c)2004. 
> > > It doesn't compile on a recent kernel, because it heavily uses a function called
> > > i2c_smbus_read_block_data(). Googling for that, I found this old mail in the
> > > archives (October 2005):
> > > 
> > > http://lists.lm-sensors.org/pipermail/lm-sensors/2005-October/013971.html
> > > 
> > > Jean, what do you say to that today? Can we have that function back in
> > > mainline, or do I have to find another solution?
> > > If you agree to add it, I could prepare a patch for it, if you don't have
> > > a recent version handy.
> > 
> > Answering my own question: After looking at i2c-core.c et al., I find it quite
> > obvious that this function should be added, if a driver needs it. 

True.

> > I attached a patch that does this. Would that be acceptable? I can resend it
> > when I have the LM93 patch ready.
> 
> I think if you put it in the same patchset with lm93.c, that will be fine.

I have a different variant of this patch in my local tree, which takes
the i2c-core changes that happened in the meantime into account. I'm
attaching it to this post if someone want to take a look and/or
comment. In fact I think I'll schedule this patch for merge into 2.6.22
regardless of the state of the lm93 driver patch. I'm a bit tired to
have to maintain that patch off-tree and to see people waste their time
wondering where that function has gone, searching the mailing list for
explanations, finding the patch and resubmitting it. We all have better
things to do.

Anyway, the key point for Hans-Jürgen is: you can assume that function
i2c_smbus_read_block_data() will be available to you when your lm93
driver patch is merged upstream.

> Jean: I can do the initial review of this one for you when it's ready.

Great, thanks!

-- 
Jean Delvare
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: i2c-add-smbus-block-read.patch
Url: http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20070408/7bae90c8/attachment.pl