diff mbox

[RFC,6/9] media: entity: Remove useless WARN_ON()'s

Message ID 1441966152-28444-7-git-send-email-sakari.ailus@linux.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Sakari Ailus Sept. 11, 2015, 10:09 a.m. UTC 
The checks for entity ID not reaching too high value the framework could
not handle are now present in the entity registration. It's quite
far-fetched this condition could happen inside the framework, so remove
the WARN_ON()'s.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
 drivers/media/media-entity.c | 5 -----
 1 file changed, 5 deletions(-)
diff mbox

Patch

diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c
index a4d6e1b..e4f0f4a 100644
--- a/drivers/media/media-entity.c
+++ b/drivers/media/media-entity.c
@@ -334,9 +334,6 @@  void media_entity_graph_walk_start(struct media_entity_graph *graph,
 	graph->stack[graph->top].entity = NULL;
 	media_entity_enum_init(graph->entities);
 
-	if (WARN_ON(entity->low_id >= MEDIA_ENTITY_MAX_LOW_ID))
-		return;
-
 	media_entity_enum_set(graph->entities, entity);
 	stack_push(graph, entity);
 }
@@ -381,8 +378,6 @@  media_entity_graph_walk_next(struct media_entity_graph *graph)
 
 		/* Get the entity in the other end of the link . */
 		next = media_entity_other(entity, link);
-		if (WARN_ON(entity->low_id >= MEDIA_ENTITY_MAX_LOW_ID))
-			return NULL;
 
 		/* Has the entity already been visited? */
 		if (media_entity_enum_test_and_set(graph->entities, next)) {