diff options
| author | Johan Hovold <johan@hovoldconsulting.com> | 2015-12-07 15:05:40 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@google.com> | 2015-12-08 15:56:38 -0500 |
| commit | 1db1b24304a064b9b4213ba1812450ac054c2c58 (patch) | |
| tree | 08ac8c616852c704b07442c81ce7d4156ddf295e /drivers/staging/greybus/bundle.h | |
| parent | 1e6fb9a1451f01f8ea092e49f27338ef9d3c7811 (diff) | |
| download | linux-stable-1db1b24304a064b9b4213ba1812450ac054c2c58.tar.gz linux-stable-1db1b24304a064b9b4213ba1812450ac054c2c58.tar.bz2 linux-stable-1db1b24304a064b9b4213ba1812450ac054c2c58.zip | |
greybus: bundle: kill the bundle lock
Kill the bundle lock, which looked like it protected the interface
bundle lists but really did not as lock-less manipulations were still
made.
No locking for the interface bundle list is in fact needed as bundles
are created along with the interface, and the list is only used to check
for duplicate bundle ids when parsing the manifest and to destroy the
bundles when removing the interface itself.
Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/bundle.h')
| -rw-r--r-- | drivers/staging/greybus/bundle.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/staging/greybus/bundle.h b/drivers/staging/greybus/bundle.h index 70d7b9d897da..eae375c12100 100644 --- a/drivers/staging/greybus/bundle.h +++ b/drivers/staging/greybus/bundle.h @@ -33,6 +33,4 @@ struct gb_bundle *gb_bundle_create(struct gb_interface *intf, u8 bundle_id, u8 class); void gb_bundle_destroy(struct gb_bundle *bundle); -struct gb_bundle *gb_bundle_find(struct gb_interface *intf, u8 bundle_id); - #endif /* __BUNDLE_H */ |