diff options
| author | Axel Haslam <ahaslam@baylibre.com> | 2016-02-26 11:39:48 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@google.com> | 2016-02-29 14:51:38 -0800 |
| commit | 39c2787b120a6e69e065b255d4db97ee60d73bf8 (patch) | |
| tree | 20e88fd09c213fcd16617b23925322febdf5ab8d | |
| parent | 9b22f155beb4e5a71f86710034b478ec43af9c43 (diff) | |
greybus: Notify user space only when the test finished.
Currently, user space is notified for every message sent,
but this is not really needed and does not work in the async case
where all messages are sent from the start.
Instead, notify userspace only when all the transfers are complete.
This allows userspace to wait in a poll loop and wakeup only when
the test is finished.
Also, don't use the bundle kobj to send the notification it is
the loopback device that contains the loopback attributes.
Signed-off-by: Axel Haslam <ahaslam@baylibre.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
| -rw-r--r-- | drivers/staging/greybus/loopback.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/greybus/loopback.c b/drivers/staging/greybus/loopback.c index 9f0da577d842..9014c2b51ae0 100644 --- a/drivers/staging/greybus/loopback.c +++ b/drivers/staging/greybus/loopback.c @@ -986,14 +986,14 @@ static int gb_loopback_fn(void *data) break; mutex_lock(&gb->mutex); - sysfs_notify(&gb->connection->bundle->dev.kobj, - NULL, "iteration_count"); /* Optionally terminate */ if (send_count == gb->iteration_max) { if (gb->iteration_count == gb->iteration_max) { gb->type = 0; send_count = 0; + sysfs_notify(&gb->dev->kobj, NULL, + "iteration_count"); } mutex_unlock(&gb->mutex); continue; |