The driver allocates the spinlock but not initialize it.
Use spin_lock_init() on it to initialize it correctly.

This is detected by Coccinelle semantic patch.

Fixes: d2b4387f3bdf ("media: platform: Add Aspeed Video Engine driver")
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
 drivers/media/platform/aspeed-video.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/media/platform/aspeed-video.c 
b/drivers/media/platform/aspeed-video.c
index dfec813..692e08e 100644
--- a/drivers/media/platform/aspeed-video.c
+++ b/drivers/media/platform/aspeed-video.c
@@ -1661,6 +1661,7 @@ static int aspeed_video_probe(struct platform_device 
*pdev)
 
        video->frame_rate = 30;
        video->dev = &pdev->dev;
+       spin_lock_init(&video->lock);
        mutex_init(&video->video_lock);
        init_waitqueue_head(&video->wait);
        INIT_DELAYED_WORK(&video->res_work, aspeed_video_resolution_work);



Reply via email to