linux-kernel - Re: [PATCH] Add driver for Dallas DS1682 elapsed time recorder

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20070515151437.3385449b@hyperion.delvare>
Date:	Tue, 15 May 2007 15:14:37 +0200
From:	Jean Delvare <khali@...ux-fr.org>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	Grant Likely <grant.likely@...retlab.ca>, i2c@...sensors.org,
	linux-kernel@...r.kernel.org,
	Alessandro Zummo <alessandro.zummo@...ertech.it>
Subject: Re: [PATCH] Add driver for Dallas DS1682 elapsed time recorder

Hi Andrew,

On Mon, 14 May 2007 23:34:01 -0700, Andrew Morton wrote:
> On Sun, 13 May 2007 01:43:42 -0600 Grant Likely <grant.likely@...retlab.ca> wrote:
> 
> > Signed-off-by: Grant Likely <grant.likely@...retlab.ca>
> > ---
> > 
> > Here is the 3rd iteration with the following changes:
> > - Modified to be an i2c "new style" driver based on David Brownell's work
> > - uses the new prototype for i2c_smbus_read_i2c_block_data() for block data
> >   reads instead of a single byte at a time.\
> 
> What new prototype for i2c_smbus_read_i2c_block_data()?
> 
> drivers/i2c/chips/ds1682.c: In function 'ds1682_show':
> drivers/i2c/chips/ds1682.c:75: warning: passing argument 3 of 'i2c_smbus_read_i2c_block_data' makes pointer from integer without a cast
> drivers/i2c/chips/ds1682.c:75: error: too many arguments to function 'i2c_smbus_read_i2c_block_data'
> drivers/i2c/chips/ds1682.c: In function 'ds1682_eeprom_read':
> drivers/i2c/chips/ds1682.c:165: warning: passing argument 3 of 'i2c_smbus_read_i2c_block_data' makes pointer from integer without a cast
> drivers/i2c/chips/ds1682.c:165: error: too many arguments to function 'i2c_smbus_read_i2c_block_data'

Please drop your version of the DS1682 patch, and refresh my
jdelvare-i2c quilt tree. You'll get the i2c_smbus_read_i2c_block_data
prototype update as well as the final version of Grant's DS1682 patch.

You would save yourself quite some work if you were waiting for i2c
patches to come to you through my tree. I am paying attention to the
dependencies, exactly in order to prevent problems like the one above.
Not to say I am not grateful when you collect important i2c fixes which
have been sent only to LKML and that I would otherwise have missed, but
in the case of the DS1682 patch, it was pretty visible that I was taking
care of it.

Thanks,
-- 
Jean Delvare
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/