lkml.org 
[lkml]   [2003]   [Apr]   [2]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateWed, 2 Apr 2003 19:08:52 +0100
FromChristoph Hellwig <>
SubjectRe: [patch] add i2c_clients_command()
On Wed, Apr 02, 2003 at 07:06:52PM +0200, Gerd Knorr wrote:
>   Hi,
> 
> This patch adds a function which loops over all i2c clients attached
> to some i2c adapter and calls the ->command function of the driver.
> 
> Currently the bttv and saa7134 drivers have simliar functions, but
> (currently) without sane locking and module handling.  Newer versions
> will switch to this function.  Updates for the two drivers which are
> actually using this new function are available from
> http://bytesex.org/patches/wip/
> 
>   Gerd
> 
> diff -u linux-2.5.66/drivers/i2c/i2c-core.c linux/drivers/i2c/i2c-core.c
> --- linux-2.5.66/drivers/i2c/i2c-core.c	2003-04-02 11:42:18.357220889 +0200
> +++ linux/drivers/i2c/i2c-core.c	2003-04-02 16:17:47.127702160 +0200
> @@ -494,6 +494,27 @@
>  	return 0;
>  }
> 
> +void i2c_clients_command(struct i2c_adapter *adap, unsigned int cmd, void *arg)
> +{
> +	int i;
> +
> +	down(&adap->list);
> +	for (i = 0; i < I2C_CLIENT_MAX; i++) {
> +		if (NULL == adap->clients[i])
> +			continue;
> +		if (!try_module_get(adap->clients[i]->driver->owner))
> +			continue;
> +		if (NULL == adap->clients[i]->driver->command)
> +			continue;
> +		
> +		up(&adap->list);
> +		adap->clients[i]->driver->command(adap->clients[i],cmd,arg);
> +		module_put(adap->clients[i]->driver->owner);
> +		down(&adap->list);
> +	}
> +	up(&adap->list);

This is a horrible algorithm!  Please introduce a per-adapter client
lists.

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

\
 
 \ /
  Last update: 2005-03-22 12:34    [W:0.109 / U:0.090 seconds]
©2003-2008 Jasper Spaans