greybus: control printing message

There's a message that gets printed in gb_manifest_parse() if excess
descriptors are found in the manifest.  This should only be printed
if the parse was successful.  If it was not successful it's not
really very helpful.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>

diff --git a/drivers/staging/greybus/manifest.c b/drivers/staging/greybus/manifest.c
index 2e22f545e15b59fb1767c8f2495b60c64c2f7ac2..563463955cb085a382905fa6176d71a60242e7cc 100644 (file)
--- a/drivers/staging/greybus/manifest.c
+++ b/drivers/staging/greybus/manifest.c
@@ -391,7 +391,7 @@ bool gb_manifest_parse(struct gb_module *gmod, void *data, size_t size)
         * We really should have no remaining descriptors, but we
         * don't know what newer format manifests might leave.
         */
-       if (!list_empty(&manifest_descs))
+       if (result && !list_empty(&manifest_descs))
                pr_info("excess descriptors in module manifest\n");
 out:
        release_manifest_descriptors();