[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c138d0a859516d8a4176c4ddb32b83119d4ad51c.1468659580.git.bhaktipriya96@gmail.com>
Date: Sat, 16 Jul 2016 14:41:49 +0530
From: Bhaktipriya Shridhar <bhaktipriya96@...il.com>
To: Mauro Carvalho Chehab <mchehab@....samsung.com>,
Geunyoung Kim <nenggun.kim@...sung.com>,
Junghak Sung <jh1009.sung@...sung.com>,
Hans Verkuil <hans.verkuil@...co.com>,
Inki Dae <inki.dae@...sung.com>
Cc: linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
Tejun Heo <tj@...nel.org>
Subject: [PATCH 1/2] [media] cx25821: Drop Freeing of Workqueue
Workqueues shouldn't be freed. destroy_workqueue should be used instead.
destroy_workqueue safely destroys a workqueue and ensures that all pending
work items are done before destroying the workqueue.
Signed-off-by: Bhaktipriya Shridhar <bhaktipriya96@...il.com>
---
drivers/media/pci/cx25821/cx25821-audio-upstream.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/cx25821/cx25821-audio-upstream.c b/drivers/media/pci/cx25821/cx25821-audio-upstream.c
index 68dbc2d..05bd957 100644
--- a/drivers/media/pci/cx25821/cx25821-audio-upstream.c
+++ b/drivers/media/pci/cx25821/cx25821-audio-upstream.c
@@ -242,7 +242,7 @@ void cx25821_stop_upstream_audio(struct cx25821_dev *dev)
dev->_audioframe_count = 0;
dev->_audiofile_status = END_OF_FILE;
- kfree(dev->_irq_audio_queues);
+ destroy_workqueue(dev->_irq_audio_queues);
dev->_irq_audio_queues = NULL;
kfree(dev->_audiofilename);
--
2.1.4
Powered by blists - more mailing lists