diff options
| author |   Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-12-11 11:25:23 -0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2016-01-11 12:19:05 -0200 |
| commit | 58f69ee9da3a2486450dac626c352f69faf964e9 (patch) | |
| tree | 84564787d4a65a07b9b1511d7e77cf2fa77fa50e /drivers/media | |
| parent | [media] media-entity: get rid of forward __media_entity_remove_link() declaration (diff) | |
| download | linux-dev-58f69ee9da3a2486450dac626c352f69faf964e9.tar.xz linux-dev-58f69ee9da3a2486450dac626c352f69faf964e9.zip | |
[media] media_entity: get rid of a unused var
> > > + if (rlink != link->reverse) {
> > > + r++;
> >
> > The variable is incremented here but otherwise never used, you can remove it.
Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media')
| -rw-r--r-- | drivers/media/media-entity.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c index 452af1d5a20d..57de11281af1 100644 --- a/drivers/media/media-entity.c +++ b/drivers/media/media-entity.c @@ -592,7 +592,6 @@ static void __media_entity_remove_link(struct media_entity *entity, { struct media_link *rlink, *tmp; struct media_entity *remote; - unsigned int r = 0; if (link->source->entity == entity) remote = link->sink->entity; @@ -600,10 +599,8 @@ static void __media_entity_remove_link(struct media_entity *entity, remote = link->source->entity; list_for_each_entry_safe(rlink, tmp, &remote->links, list) { - if (rlink != link->reverse) { - r++; + if (rlink != link->reverse) continue; - } if (link->source->entity == entity) remote->num_backlinks--; |