From: Viresh Kumar <viresh.kumar@linaro.org>
Date: Tue, 24 Mar 2015 11:38:14 +0000 (+0530)
Subject: greybus: manifest: don't need to check for desc_size == 0 anymore
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=13fe6a9af3d86019a10bdbd3195434e5a795b3b3;p=openwrt%2Fstaging%2Fblogic.git

greybus: manifest: don't need to check for desc_size == 0 anymore

identify_descriptor() doesn't return 0 anymore and so we don't need to check the
returned value against 0.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
---

diff --git a/drivers/staging/greybus/manifest.c b/drivers/staging/greybus/manifest.c
index 545a3bd4e751..1b9edbcd08c5 100644
--- a/drivers/staging/greybus/manifest.c
+++ b/drivers/staging/greybus/manifest.c
@@ -383,9 +383,7 @@ bool gb_manifest_parse(struct gb_interface *intf, void *data, size_t size)
 		int desc_size;
 
 		desc_size = identify_descriptor(intf, desc, size);
-		if (desc_size <= 0) {
-			if (!desc_size)
-				pr_err("zero-sized manifest descriptor\n");
+		if (desc_size < 0) {
 			result = false;
 			goto out;
 		}