drivers: video: sw_generator: Fix missing parameter

Loic Poulain · nashif · commit fd777c39ad1a · 2021-04-10T07:52:30.000-04:00
k_work_cancel_delayable_sync requires a second parameter for state
synchronization.

Signed-off-by: Loic Poulain &lt;loic.poulain@linaro.org&gt;
diff --git a/drivers/video/video_sw_generator.c b/drivers/video/video_sw_generator.c
@@ -16,6 +16,7 @@ struct video_sw_generator_data {
 	struct k_fifo fifo_in;
 	struct k_fifo fifo_out;
 	struct k_work_delayable buf_work;
+	struct k_work_sync work_sync;
 	int pattern;
 	bool ctrl_hflip;
 	bool ctrl_vflip;
@@ -63,7 +64,7 @@ static int video_sw_generator_stream_stop(const struct device *dev)
 {
 	struct video_sw_generator_data *data = dev->data;
 
-	k_work_cancel_delayable_sync(&data->buf_work);
+	k_work_cancel_delayable_sync(&data->buf_work, &data->work_sync);
 
 	return 0;
 }

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@ struct video_sw_generator_data {`
`16`	`16`	`struct k_fifo fifo_in;`
`17`	`17`	`struct k_fifo fifo_out;`
`18`	`18`	`struct k_work_delayable buf_work;`
	`19`	`+ struct k_work_sync work_sync;`
`19`	`20`	`int pattern;`
`20`	`21`	`bool ctrl_hflip;`
`21`	`22`	`bool ctrl_vflip;`
`@@ -63,7 +64,7 @@ static int video_sw_generator_stream_stop(const struct device *dev)`
`63`	`64`	`{`
`64`	`65`	`struct video_sw_generator_data *data = dev->data;`
`65`	`66`
`66`		`- k_work_cancel_delayable_sync(&data->buf_work);`
	`67`	`+ k_work_cancel_delayable_sync(&data->buf_work, &data->work_sync);`
`67`	`68`
`68`	`69`	`return 0;`
`69`	`70`	`}`