greybus: always drop reference in gb_operation_work()

Currently we issue a warning in gb_operation_work() if an operation
has no callback function defined.  But we return without dropping
the reference to the operation as we should.

Stop warning if there's no callback, call it only if it's defined,
and always drop the operation reference before returning.

This means we're now treating a NULL callback pointer as a normal
condition.

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

diff --git a/drivers/staging/greybus/operation.c b/drivers/staging/greybus/operation.c
index 75900d3129be222106b7119dcd7c9d95131e1a14..aaac03733fd45b065f5b8f824b8ae0e985125d05 100644 (file)
--- a/drivers/staging/greybus/operation.c
+++ b/drivers/staging/greybus/operation.c
@@ -265,11 +265,10 @@ static void gb_operation_work(struct work_struct *work)
        struct gb_operation *operation;
 
        operation = container_of(work, struct gb_operation, work);
-       if (WARN_ON(!operation->callback))
-               return;
-
-       operation->callback(operation);
-       operation->callback = NULL;
+       if (operation->callback) {
+               operation->callback(operation);
+               operation->callback = NULL;
+       }
        gb_operation_put(operation);
 }