diff options
author | Trent Piepho <xyzzy@speakeasy.org> | 2007-10-10 04:37:43 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2007-10-10 12:35:29 -0400 |
commit | a991f44b79fa49b281eb078eed4a76a42101012a (patch) | |
tree | 5a6b7a4ce700ef8bcdf59f816c0c04b5e64d215b /drivers/media/video/tvmixer.c | |
parent | e77e2c2f2989eefff7e1c0fff9cb72afaedf6796 (diff) |
V4L/DVB (6316): Change list_for_each+list_entry to list_for_each_entry
The rest of V4L files.
There is one list_for_each+list_entry in cpia_pp.c that
wasn't changed because it expects the loop iterator to remain NULL if
the list is empty.
A bug in vivi is fixed; the 'safe' version needs to be used because the loop
deletes the list entries.
Simplify a second loop in vivi and get rid if an un-used variable in that loop.
Signed-off-by: Trent Piepho <xyzzy@speakeasy.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'drivers/media/video/tvmixer.c')
-rw-r--r-- | drivers/media/video/tvmixer.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/media/video/tvmixer.c b/drivers/media/video/tvmixer.c index 544a4ae7d2ed..9fa5b702e073 100644 --- a/drivers/media/video/tvmixer.c +++ b/drivers/media/video/tvmixer.c | |||
@@ -237,13 +237,10 @@ static const struct file_operations tvmixer_fops = { | |||
237 | 237 | ||
238 | static int tvmixer_adapters(struct i2c_adapter *adap) | 238 | static int tvmixer_adapters(struct i2c_adapter *adap) |
239 | { | 239 | { |
240 | struct list_head *item; | ||
241 | struct i2c_client *client; | 240 | struct i2c_client *client; |
242 | 241 | ||
243 | list_for_each(item,&adap->clients) { | 242 | list_for_each_entry(client, &adap->clients, list) |
244 | client = list_entry(item, struct i2c_client, list); | ||
245 | tvmixer_clients(client); | 243 | tvmixer_clients(client); |
246 | } | ||
247 | return 0; | 244 | return 0; |
248 | } | 245 | } |
249 | 246 | ||