Staging: intel_sst: cleanup naming a little
authorDan Carpenter <error27@gmail.com>
Fri, 15 Oct 2010 03:44:58 +0000 (05:44 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 15 Oct 2010 19:24:39 +0000 (12:24 -0700)
&sst_drv_ctx->streams[str_id].lock and &stream->lock are the same.
The mutex_lock() uses &stream->lock so this makes things consistent and
it's nicer to read as well.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/intel_sst/intel_sst_app_interface.c

index 82768fa5187462aa782aafb672a315a1e1cb2cb6..1429386a1c67392b74be2399368720577a90716d 100644 (file)
@@ -1056,20 +1056,18 @@ long intel_sst_ioctl(struct file *file_ptr, unsigned int cmd, unsigned long arg)
                                retval = sst_capture_frame(str_id);
                        else {
                                retval = -EINVAL;
-                               mutex_unlock(
-                                       &sst_drv_ctx->streams[str_id].lock);
+                               mutex_unlock(&stream->lock);
                                break;
                        }
                        if (retval < 0) {
                                stream->status = STREAM_INIT;
-                               mutex_unlock(
-                                       &sst_drv_ctx->streams[str_id].lock);
+                               mutex_unlock(&stream->lock);
                                break;
                        }
                } else {
                        retval = -EINVAL;
                }
-               mutex_unlock(&sst_drv_ctx->streams[str_id].lock);
+               mutex_unlock(&stream->lock);
                break;
        }