Staging: intel_sst: cleanup naming a little (81a2fff6) · Commits · 戴 / test

&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

+3 −5

Original line number	Diff line number	Diff line
		@@ -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;
		}

Admin message