Commit f0f2c2b5 authored by Dan Carpenter's avatar Dan Carpenter Committed by Linus Torvalds

dca: remove unneeded NULL check

The return here doesn't release the locks or re-enable IRQs.  But as
Andrew Morton points out, domain is never NULL.  list_first_entry()
essentially never returns NULL and also we already verified that the list
is not empty.
Signed-off-by: default avatarDan Carpenter <error27@gmail.com>
Acked-by: default avatarMaciej Sosnowski <maciej.sosnowski@intel.com>
Cc: David Miller <davem@davemloft.net>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 455cd5ab
...@@ -110,8 +110,6 @@ static void unregister_dca_providers(void) ...@@ -110,8 +110,6 @@ static void unregister_dca_providers(void)
/* at this point only one domain in the list is expected */ /* at this point only one domain in the list is expected */
domain = list_first_entry(&dca_domains, struct dca_domain, node); domain = list_first_entry(&dca_domains, struct dca_domain, node);
if (!domain)
return;
list_for_each_entry_safe(dca, _dca, &domain->dca_providers, node) { list_for_each_entry_safe(dca, _dca, &domain->dca_providers, node) {
list_del(&dca->node); list_del(&dca->node);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment